Merge pull request #61 from scaleapi/da/export_autotags

Simplify error handling + Export Autotags

Author: ardila

README.md

@@ -162,10 +162,9 @@ pip3 install poetry
 poetry install
 ```
 
-Please install the pre-commit hooks by running the following commands:
+Please install the pre-commit hooks by running the following command:
 ```python
-pip install pre-commit
-pre-commit install
+poetry run pre-commit install
 ```
 
 **Best practices for testing:**

conftest.py

@@ -29,36 +29,8 @@
 
 
 @pytest.fixture(scope="session")
-def monkeypatch_session(request):
-    """This workaround is needed to allow monkeypatching in session-scoped fixtures.
-
-    See https://github.com/pytest-dev/pytest/issues/363
-    """
-    from _pytest.monkeypatch import MonkeyPatch
-
-    mpatch = MonkeyPatch()
-    yield mpatch
-    mpatch.undo()
-
-
-@pytest.fixture(scope="session")
-def CLIENT(monkeypatch_session):
+def CLIENT():
     client = nucleus.NucleusClient(API_KEY)
-
-    # Change _make_request to raise AsssertionErrors when the
-    # HTTP status code is not successful, so that tests fail if
-    # the request was unsuccessful.
-    def _make_request_patch(
-        payload: dict, route: str, requests_command=requests.post
-    ) -> dict:
-        response = client._make_request_raw(payload, route, requests_command)
-        assert response.status_code in SUCCESS_STATUS_CODES, (
-            f"HTTP response had status code: {response.status_code}. "
-            f"Full JSON: {response.json()}"
-        )
-        return response.json()
-
-    monkeypatch_session.setattr(client, "_make_request", _make_request_patch)
     return client