deprecate start (#553)

pradeepgv-db · web-flow · commit 64c370f9333c · 2022-09-01T13:34:16.000+02:00
diff --git a/databricks_cli/sdk/service.py b/databricks_cli/sdk/service.py
@@ -1495,7 +1495,7 @@ def start_update(self, pipeline_id=None, full_refresh=None, headers=None):
         _data['cause'] = 'USER_ACTION'
         return self.client.perform_query(
             'POST',
-            '/pipelines/{pipeline_id}/start'.format(pipeline_id=pipeline_id),
+            '/pipelines/{pipeline_id}/updates'.format(pipeline_id=pipeline_id),
             data=_data,
             headers=headers,
         )
diff --git a/tests/pipelines/test_api.py b/tests/pipelines/test_api.py
@@ -233,7 +233,7 @@ def test_start_without_refresh(pipelines_api):
     expected_data = {
         "cause": "USER_ACTION"
     }
-    client_mock.assert_called_with('POST', '/pipelines/{}/start'.format(PIPELINE_ID),
+    client_mock.assert_called_with('POST', '/pipelines/{}/updates'.format(PIPELINE_ID),
                                    data=expected_data, headers=None)
 
 
@@ -245,7 +245,7 @@ def test_start_with_refresh(pipelines_api):
         "cause": "USER_ACTION",
         "full_refresh": True,
     }
-    client_mock.assert_called_with('POST', '/pipelines/{}/start'.format(PIPELINE_ID),
+    client_mock.assert_called_with('POST', '/pipelines/{}/updates'.format(PIPELINE_ID),
                                    data=expected_data, headers=None)
 
 

Original file line number	Diff line number	Diff line change
`@@ -1495,7 +1495,7 @@ def start_update(self, pipeline_id=None, full_refresh=None, headers=None):`
`1495`	`1495`	`_data['cause'] = 'USER_ACTION'`
`1496`	`1496`	`return self.client.perform_query(`
`1497`	`1497`	`'POST',`
`1498`		`- '/pipelines/{pipeline_id}/start'.format(pipeline_id=pipeline_id),`
	`1498`	`+ '/pipelines/{pipeline_id}/updates'.format(pipeline_id=pipeline_id),`
`1499`	`1499`	`data=_data,`
`1500`	`1500`	`headers=headers,`
`1501`	`1501`	`)`
Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ def test_start_without_refresh(pipelines_api):`
`233`	`233`	`expected_data = {`
`234`	`234`	`"cause": "USER_ACTION"`
`235`	`235`	`}`
`236`		`- client_mock.assert_called_with('POST', '/pipelines/{}/start'.format(PIPELINE_ID),`
	`236`	`+ client_mock.assert_called_with('POST', '/pipelines/{}/updates'.format(PIPELINE_ID),`
`237`	`237`	`data=expected_data, headers=None)`
`238`	`238`
`239`	`239`
`@@ -245,7 +245,7 @@ def test_start_with_refresh(pipelines_api):`
`245`	`245`	`"cause": "USER_ACTION",`
`246`	`246`	`"full_refresh": True,`
`247`	`247`	`}`
`248`		`- client_mock.assert_called_with('POST', '/pipelines/{}/start'.format(PIPELINE_ID),`
	`248`	`+ client_mock.assert_called_with('POST', '/pipelines/{}/updates'.format(PIPELINE_ID),`
`249`	`249`	`data=expected_data, headers=None)`
`250`	`250`
`251`	`251`