Merge pull request #4 from scrapinghub/fix-deploy-tests

Minor test fixes
scrapinghub · May 3, 2016 · 154d932 · 154d932
2 parents 55a3624 + d243e9e
commit 154d932
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/tests/test_deploy.py b/tests/test_deploy.py
@@ -66,7 +66,7 @@ def test_prepare_deploy_params(self, mocked):
             add_fake_setup_py(tmpdir)
             assert _prepare_deploy_params(
                 123, 'test-vers', 'registry/user/project',
-                None, None, None, False) == {
+                None, None, None) == {
                     'image_url': 'registry/user/project',
                     'project': 123,
                     'pull_insecure_registry': True,
@@ -83,11 +83,10 @@ def test_prepare_deploy_params_more_params(self, mocked):
                              ' "pass", "username": "user"}')
             assert _prepare_deploy_params(
                 123, 'test-vers', 'registry/user/project',
-                'user', 'pass', 'email@mail', True) == {
+                'user', 'pass', 'email@mail') == {
                     'image_url': 'registry/user/project',
                     'project': 123,
                     'pull_auth_config': expected_auth,
-                    'sync': True,
                     'scripts': 'scriptA.py,scriptB.py',
                     'spiders': 'a1f,abc,spi-der',
                     'version': 'test-vers'}
diff --git a/tests/test_upload.py b/tests/test_upload.py
@@ -17,7 +17,7 @@ def test_cli(self, build, push, deploy):
         result = runner.invoke(
             cli, ["dev", "-d", "--version", "test",
                   "--username", "user", "--password", "pass",
-                  "--email", "mail", "--sync"])
+                  "--email", "mail", "--async"])
         assert result.exit_code == 0
         build.assert_called_with('dev', True, 'test')
         push.assert_called_with('dev', True, 'test', 'user', 'pass', 'mail')