Update gql version to official prerelease (anvilco#51)

aalmazan · web-flow · commit e88b6aa8d0a6 · 2023-02-22T10:45:47.000-08:00
* Update deps, use new dev-dependencies poetry format, maybe tox update

* Fix weld --list getting wrong gql field

* Force local schema switch

* Fix lint warnings

* Remove field
diff --git a/.pylint.ini b/.pylint.ini
@@ -74,6 +74,7 @@ disable=
     too-many-nested-blocks,
     protected-access,
     wrong-import-order,
+    use-dict-literal,
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option
diff --git a/Makefile b/Makefile
@@ -115,7 +115,7 @@ endif
 # Export PACKAGES so tox doesn't have to be reconfigured if these change
 tox: export TESTS = $(PACKAGE) tests
 tox: install
-	poetry run tox
+	poetry run tox -p 2
 
 .PHONY: read-coverage
 read-coverage:
diff --git a/poetry.lock b/poetry.lock
diff --git a/pyproject.toml b/pyproject.toml
@@ -39,36 +39,35 @@ requests = "^2.25.0"
 ratelimit = "^2.2.1"
 tabulate = "^0.9.0"
 pydantic = "^1.8.2"
-# TODO: Update to an actual release version when out.
-gql = {git = "https://github.com/graphql-python/gql.git", rev = "930fca579220fd4810053491878366fa69862e58", extras = ["requests"]}
+gql = {version = "3.5.0b1", extras = ["requests"]}
 
-[tool.poetry.dev-dependencies]
+[tool.poetry.group.dev.dependencies]
 
 # Formatters
 black = "=22.12.0"
 isort = "^5.11.4"
 
 # Linters
 mypy = "*"
-pydocstyle = "6.2.3"
-pylint = "2.15.10"
+pydocstyle = "^6.3.0"
+pylint = "^2.16.2"
 
 # Testing
-pytest = "^7.2.0"
-pytest-cov = "*"
+pytest = "^7.2.1"
+pytest-cov = "^4.0.0"
 pytest-describe = "^2.0"
-pytest-random = "*"
+pytest-random = "^0.2"
 freezegun = "*"
 
 # Reports
 coveragespace = "*"
 
 # Documentation
-mkdocs = "1.4.2"
-pygments = "^2.5.2"
+mkdocs = "^1.4.2"
+pygments = "^2.14.0"
 
 # Tooling
-pyinstaller = "^5.7.0"
+pyinstaller = "^5.8.0"
 sniffer = "*"
 MacFSEvents = { version = "*", platform = "darwin" }
 pync = { version = "*", platform = "darwin" }
diff --git a/python_anvil/api.py b/python_anvil/api.py
@@ -307,7 +307,7 @@ def get_welds(self, **kwargs) -> Union[List, Tuple[List, Dict]]:
                   welds {
                     eid
                     slug
-                    title
+                    name
                     forges {
                       eid
                       name
diff --git a/python_anvil/api_resources/mutations/forge_submit.py b/python_anvil/api_resources/mutations/forge_submit.py
@@ -47,7 +47,6 @@
     $timezone: String,
     $groupArrayId: String,
     $groupArrayIndex: Int,
-    $errorType: String,
     $webhookURL: String,
 ) {{
     forgeSubmit (
@@ -61,7 +60,6 @@
         timezone: $timezone,
         groupArrayId: $groupArrayId,
         groupArrayIndex: $groupArrayIndex,
-        errorType: $errorType,
         webhookURL: $webhookURL
     ) {query}
 }}
diff --git a/python_anvil/api_resources/requests.py b/python_anvil/api_resources/requests.py
@@ -34,6 +34,7 @@ def handle_error(self, response, status_code, headers):
         else:
             message = f"Error: {status_code}: {response} {extra}"
 
+        # pylint: disable=broad-exception-raised
         raise Exception(message)
 
     def process_response(self, response, status_code, headers, **kwargs):
diff --git a/python_anvil/cli.py b/python_anvil/cli.py
@@ -102,11 +102,12 @@ def weld(ctx, eid, list_all):
             if debug:
                 click.echo(headers)
 
-        data = [(w["eid"], w.get("slug"), w.get("title"), w.get("forges")) for w in res]
+        data = [(w["eid"], w.get("slug"), w.get("name"), w.get("forges")) for w in res]
         click.echo(tabulate(data, tablefmt="pretty", headers=["eid", "slug", "title"]))
         return
 
     if not eid:
+        # pylint: disable=broad-exception-raised
         raise Exception("You need to pass in a weld eid")
 
     res = anvil.get_weld(eid)
diff --git a/python_anvil/http.py b/python_anvil/http.py
@@ -62,6 +62,7 @@ def get_client(
         environment: str = "dev",  # pylint: disable=unused-argument
         endpoint_url: Optional[str] = None,
         fetch_schema_from_transport: bool = False,
+        force_local_schema: bool = False,
     ) -> Client:
         auth = HTTPBasicAuth(username=api_key, password="")
         endpoint_url = endpoint_url or GRAPHQL_ENDPOINT
@@ -72,7 +73,9 @@ def get_client(
             verify=True,
         )
 
-        schema = get_local_schema(raise_on_error=False)
+        schema = None
+        if force_local_schema or not fetch_schema_from_transport:
+            schema = get_local_schema(raise_on_error=False)
 
         return Client(
             schema=schema,
diff --git a/tox.ini b/tox.ini
@@ -8,3 +8,13 @@ passenv = TESTS
 commands =
     poetry install -v
     poetry run pytest {env:TESTS}
+
+[testenv:package]
+description = check sdist and wheel
+skip_install = true
+deps =
+    poetry>=0.12
+    twine
+commands =
+    poetry build
+    twine check dist/*

-Original file line number
+Diff line change
                   welds {
                     eid
                     slug
 -                    title
 +                    name
                     forges {
                       eid
                       name