From 2b1c535c128984cc0ee2a097ecaa3ab638ae2a5b Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Thu, 29 Jul 2021 17:44:16 -0700
Subject: [PATCH] pytest.mark.serial for any test using isolated_filesystem(),
 refs #1406

---
 tests/test_package.py          | 3 ++-
 tests/test_publish_cloudrun.py | 7 +++++++
 tests/test_publish_heroku.py   | 5 +++++
 3 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/tests/test_package.py b/tests/test_package.py
index bb939643..76693d2f 100644
--- a/tests/test_package.py
+++ b/tests/test_package.py
@@ -2,7 +2,7 @@ from click.testing import CliRunner
 from datasette import cli
 from unittest import mock
 import pathlib
-import json
+import pytest
 
 
 class CaptureDockerfile:
@@ -24,6 +24,7 @@ CMD datasette serve --host 0.0.0.0 -i test.db --cors --inspect-file inspect-data
 """.strip()
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.cli.call")
 def test_package(mock_call, mock_which):
diff --git a/tests/test_publish_cloudrun.py b/tests/test_publish_cloudrun.py
index 7881ebae..826860d7 100644
--- a/tests/test_publish_cloudrun.py
+++ b/tests/test_publish_cloudrun.py
@@ -6,6 +6,7 @@ import pytest
 import textwrap
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 def test_publish_cloudrun_requires_gcloud(mock_which):
     mock_which.return_value = False
@@ -27,6 +28,7 @@ def test_publish_cloudrun_invalid_database(mock_which):
     assert "Path 'woop.db' does not exist" in result.output
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
@@ -75,6 +77,7 @@ Service name: input-service
         )
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
@@ -103,6 +106,7 @@ def test_publish_cloudrun(mock_call, mock_output, mock_which):
         )
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
@@ -147,6 +151,7 @@ def test_publish_cloudrun_memory(
         )
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
@@ -225,6 +230,7 @@ def test_publish_cloudrun_plugin_secrets(mock_call, mock_output, mock_which):
         } == json.loads(metadata)
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
@@ -280,6 +286,7 @@ def test_publish_cloudrun_apt_get_install(mock_call, mock_output, mock_which):
         assert expected == dockerfile
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.cloudrun.check_output")
 @mock.patch("datasette.publish.cloudrun.check_call")
diff --git a/tests/test_publish_heroku.py b/tests/test_publish_heroku.py
index c011ab43..acbdafeb 100644
--- a/tests/test_publish_heroku.py
+++ b/tests/test_publish_heroku.py
@@ -1,8 +1,10 @@
 from click.testing import CliRunner
 from datasette import cli
 from unittest import mock
+import pytest
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 def test_publish_heroku_requires_heroku(mock_which):
     mock_which.return_value = False
@@ -15,6 +17,7 @@ def test_publish_heroku_requires_heroku(mock_which):
         assert "Publishing to Heroku requires heroku" in result.output
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.heroku.check_output")
 @mock.patch("datasette.publish.heroku.call")
@@ -44,6 +47,7 @@ def test_publish_heroku_invalid_database(mock_which):
     assert "Path 'woop.db' does not exist" in result.output
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.heroku.check_output")
 @mock.patch("datasette.publish.heroku.call")
@@ -79,6 +83,7 @@ def test_publish_heroku(mock_call, mock_check_output, mock_which):
         )
 
 
+@pytest.mark.serial
 @mock.patch("shutil.which")
 @mock.patch("datasette.publish.heroku.check_output")
 @mock.patch("datasette.publish.heroku.call")