Merge pull request #1683 from alphagov/billing-endpoints

Remove commented out line

app/notify_client/service_api_client.py

@@ -20,7 +20,6 @@ class ServiceAPIClient(NotifyAdminAPIClient):
         self,
         service_name,
         organisation_type,
-        free_sms_fragment_limit,
         message_limit,
         restricted,
         user_id,
@@ -32,7 +31,6 @@ class ServiceAPIClient(NotifyAdminAPIClient):
         data = {
             "name": service_name,
             "organisation_type": organisation_type,
-            "free_sms_fragment_limit": free_sms_fragment_limit,
             "active": True,
             "message_limit": message_limit,
             "user_id": user_id,

tests/app/main/views/test_dashboard.py

@@ -631,7 +631,6 @@ def test_usage_page_with_year_argument(
     mock_get_usage,
     mock_get_billable_units,
     mock_get_free_sms_fragment_limit,
-    # mock_create_or_update_free_sms_fragment_limit
 ):
     assert logged_in_client.get(url_for('main.usage', service_id=SERVICE_ONE_ID, year=2000)).status_code == 200
     mock_get_billable_units.assert_called_once_with(SERVICE_ONE_ID, 2000)

tests/app/notify_client/test_service_api_client.py

@@ -57,7 +57,6 @@ def test_client_creates_service_with_correct_data(
     client.create_service(
         service_name='My first service',
         organisation_type='central_government',
-        free_sms_fragment_limit=2,
         message_limit=1,
         restricted=True,
         user_id=fake_uuid,
@@ -73,7 +72,6 @@ def test_client_creates_service_with_correct_data(
             name='My first service',
             # The rest pass through with the same names
             organisation_type='central_government',
-            free_sms_fragment_limit=2,
             message_limit=1,
             restricted=True,
             user_id=fake_uuid,