update publish message with studio.mapbox.com link (#132)

* update publish message with studio.mapbox.com link

* fix other pre-commit errors

* update message with job command

* update and pin black versions

* update flake8 repo

* pre-commit run --all-files

* update language to "job received"

* 1.7.0

Author: mapsam

.pre-commit-config.yaml

@@ -2,13 +2,13 @@ repos:
     -
         repo: 'https://github.com/ambv/black'
         # 18.6b1
-        rev: stable
+        rev: 20.8b1
         hooks:
             - id: black
               args: ['--safe']
     -
-        repo: 'https://github.com/pre-commit/pre-commit-hooks'
-        rev: v2.0.0
+        repo: 'https://gitlab.com/pycqa/flake8'
+        rev: 3.9.0
         hooks:
             - id: flake8
               args: [

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Unreleased
 
+# 1.7.0 (2021-04-02)
+- Update `tilesets publish` success message to include link to studio.mapbox.com/tilesets/* endpoint and include `tilesets job` command to view the status.
+
 # 1.6.0 (2021-02-16)
 - Fix problem that prevented estimate-area from working for MultiPolygons
 - Improve documentation for delete-source and upload-source
@@ -8,10 +11,10 @@
 - Update README for estimate-area
 
 # 1.5.0 (2020-10-19)
-- Create estimate-area command 
+- Create estimate-area command
 
 # 1.4.3 (2020-10-08)
-- Update Click version to 7.1.2 and fix assertions to pass all tests 
+- Update Click version to 7.1.2 and fix assertions to pass all tests
 
 # 1.4.2 (2020-08-18)
 - Check if the token matches the username before uploading sources

mapbox_tilesets/__init__.py

@@ -1,3 +1,3 @@
 """mapbox_tilesets package"""
 
-__version__ = "1.6.0"
+__version__ = "1.7.0"

mapbox_tilesets/scripts/cli.py

@@ -119,10 +119,19 @@ def publish(tileset, token=None, indent=None):
     )
     r = s.post(url)
     if r.status_code == 200:
-        click.echo(json.dumps(r.json(), indent=indent))
+        response_msg = r.json()
+        click.echo(json.dumps(response_msg, indent=indent))
+
+        studio_url = click.style(
+            f"https://studio.mapbox.com/tilesets/{tileset}", bold=True
+        )
+        job_id = response_msg["jobId"]
+        job_cmd = click.style(f"tilesets job {tileset} {job_id}", bold=True)
+        message = f"\n✔ Tileset job received. Visit {studio_url} or run {job_cmd} to view the status of your tileset."
+        # print(message)
         click.echo(
-            f"You can view the status of your tileset with the `tilesets status {tileset}` command.",
-            err=True,
+            message,
+            err=True,  # print to stderr so the JSON output can be parsed separately from the success message
         )
     else:
         raise errors.TilesetsError(r.text)

setup.py

@@ -40,7 +40,14 @@ def read(fname):
     include_package_data=True,
     zip_safe=False,
     extras_require={
-        "test": ["pytest==4.6.11", "pytest-cov", "pre-commit", "black", "pep8"]
+        "test": [
+            "pytest==4.6.11",
+            "pytest-cov",
+            "pre-commit",
+            "black==20.8b1",
+            "pep8",
+            "toml==0.10.2",
+        ]
     },
     entry_points="""
       [console_scripts]

tests/test_cli_publish.py

@@ -26,14 +26,17 @@ def test_cli_publish(mock_request_post):
     runner = CliRunner()
 
     # sends expected request
-    mock_request_post.return_value = MockResponse({"message": "mock message"}, 200)
+    mock_request_post.return_value = MockResponse(
+        {"message": "mock message", "jobId": "1234fakejob"}, 200
+    )
     result = runner.invoke(publish, ["test.id"])
     mock_request_post.assert_called_with(
         "https://api.mapbox.com/tilesets/v1/test.id/publish?access_token=pk.eyJ1IjoidGVzdC11c2VyIn0K"
     )
     assert result.exit_code == 0
+    print(result.output)
     assert (
-        "You can view the status of your tileset with the `tilesets status test.id` command."
+        '{"message": "mock message", "jobId": "1234fakejob"}\n\n✔ Tileset job received. Visit https://studio.mapbox.com/tilesets/test.id or run tilesets job test.id 1234fakejob to view the status of your tileset.\n'
         in result.output
     )
 
@@ -42,14 +45,16 @@ def test_cli_publish(mock_request_post):
 @mock.patch("requests.Session.post")
 def test_cli_publish_use_token_flag(mock_request_post):
     runner = CliRunner()
-    mock_request_post.return_value = MockResponse({"message": "mock message"}, 200)
+    mock_request_post.return_value = MockResponse(
+        {"message": "mock message", "jobId": "1234fakejob"}, 200
+    )
     # Provides the flag --token
     result = runner.invoke(publish, ["test.id", "--token", "flag-token"])
     mock_request_post.assert_called_with(
         "https://api.mapbox.com/tilesets/v1/test.id/publish?access_token=flag-token"
     )
     assert result.exit_code == 0
     assert (
-        "You can view the status of your tileset with the `tilesets status test.id` command."
+        '{"message": "mock message", "jobId": "1234fakejob"}\n\n✔ Tileset job received. Visit https://studio.mapbox.com/tilesets/test.id or run tilesets job test.id 1234fakejob to view the status of your tileset.\n'
         in result.output
     )