Reduce scope of pip warnings as errors

The CI has been failing too often recently because of changes in pip and
dependencies. Those failures were mostly irrelevant warning that we turned into
errors for better visibility. However, so far there was never any action we
needed to take other than muting that warning on or the other way.

Therefore, this commit changes it so that all pip installation calls are run
without treating warning as errors with the exception of the installation of
the driver itself. We want to catch if the installation of our own packages
causes warning (e.g., `DepercationWarning`s).

Author: robsdedude

testkit/_common.py

@@ -13,5 +13,9 @@ def run(args, env=None):
     )
 
 
-def run_python(args, env=None):
-    run([TEST_BACKEND_VERSION, "-u", "-W", "error", *args], env=env)
+def run_python(args, env=None, warning_as_error=True):
+    cmd = [TEST_BACKEND_VERSION, "-u"]
+    if warning_as_error:
+        cmd += ["-W", "error"]
+    cmd += list(args)
+    run(cmd, env=env)

testkit/build.py

@@ -28,6 +28,13 @@
 
 
 if __name__ == "__main__":
-    run_python(["-m", "pip", "install", "-U", "pip"])
-    run_python(["-m", "pip", "install", "--use-pep517", "-Ur",
-                "requirements-dev.txt"])
+    run_python(["-m", "pip", "install", "-U", "pip"],
+               warning_as_error=False)
+    run_python(["-m", "pip", "install", "-Ur", "requirements-dev.txt"],
+               warning_as_error=False)
+
+    # uninstall the driver and re-install it with warnings as errors to make
+    # sure that we don't cause any warnings
+    run_python(["-m", "pip", "uninstall", "-y", "neo4j"],
+               warning_as_error=False)
+    run_python(["-m", "pip", "install", "-U", "-e", "."])