Fix MacOS Pipeline: Include tests not in subdirectories (#797)

* Fix glob expansion by adding quotes and add diagnostics

* Remove workaround no longer needed with Gymnasium

* Bugfix in sys.modules restore

* Try avoiding echo to prevent one big test filename

Author: AdamGleave

.circleci/config.yml

@@ -358,7 +358,10 @@ jobs:
           name: run tests
           command: |
             source ~/venv/bin/activate
-            TESTFILES=$(circleci tests glob tests/**/test*.py | circleci tests split --split-by=timings)
+            ALL_TESTFILES=$(circleci tests glob "tests/**/test*.py")
+            echo "All testfiles: ${ALL_TESTFILES}"
+            TESTFILES=$(circleci tests glob "tests/**/test*.py" | circleci tests split --split-by=timings)
+            echo "This shard testing: ${TESTFILES}"
             pytest -n auto --junitxml=/tmp/test-reports/junit.xml -vv $TESTFILES
 
       - save-pytest-cache

docs/requirements.txt

@@ -12,8 +12,6 @@ build:
 
 python:
   install:
-    # TODO(GH#707): remove docs/requirements.txt once Gym upgraded
-    - requirements: docs/requirements.txt
     - method: pip
       path: .
       extra_requirements:

readthedocs.yml

@@ -122,4 +122,4 @@ def test_wandb_module_import_error():
         with pytest.raises(ModuleNotFoundError, match=r"Trying to log data.*"):
             logger.configure(format_strs=["wandb"])
     finally:
-        sys.modules[wandb] = wandb_module
+        sys.modules["wandb"] = wandb_module