Merge pull request #697 from ecalifornica/remove-test-network-calls-http

Remove test HTTP network calls
biocommons · Sep 18, 2023 · c69e03f · c69e03f
2 parents 34a4c88 + 48b3cdc
commit c69e03f
Show file tree

Hide file tree

Showing 37 changed files with 5,988 additions and 2 deletions.
diff --git a/docs/contributing.rst b/docs/contributing.rst
@@ -97,12 +97,25 @@ A Quick Contribution Example
 
 * Code, code, code!
 
-  You probably want to test code with::
+.. spacer
+
+* Test, test, test
+
+  You can run the tests using make::
 
     $ make test
 
   See `Local UTA`_ and `make`_ for tips on accellerating testing.
 
+  If your tests make network calls you can capture those calls
+  with vcrpy and the pytest-recording plugin::
+
+    $ pytest --record-mode=once tests
+
+  To confirm that network calls are mocked::
+
+    $ pytest --block-network tests
+
 .. spacer
 
 * Reformat code

diff --git a/setup.cfg b/setup.cfg
@@ -41,7 +41,7 @@ dev =
     jupyter
     pytest >= 5.3
     pytest-cov >= 2.8
-    pytest-vcr
+    pytest-recording
     restview
     setuptools
     sphinx