Bump GitHub Actions to latest versions

Author: oleavr

.github/workflows/code-style.yml

@@ -4,13 +4,12 @@ jobs:
   black:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v7
       - uses: psf/black@stable
   isort:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-python@v4
-        with:
-          python-version: 3.8
-      - uses: jamescurtin/isort-action@master
+      - uses: actions/checkout@v7
+      - uses: actions/setup-python@v6
+      - run: pip install isort
+      - run: isort --check-only --diff .

.github/workflows/linting.yml

@@ -4,13 +4,13 @@ jobs:
   pyflakes:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v7
       - run: pip install pyflakes
       - run: pyflakes $(git ls-files '*.py' | grep -v '^frida/frida_bindgen/assets/')
   mypy:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v7
       - uses: jpetrucciani/mypy-check@master
         with:
           mypy_flags: '--exclude examples --exclude setup --exclude frida/frida_bindgen/assets'

.github/workflows/tests.yml

@@ -3,6 +3,7 @@ on: [push, pull_request]
 jobs:
   test:
     runs-on: ubuntu-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@v7
         with:
@@ -15,4 +16,4 @@ jobs:
       - name: Build the extension
         run: make
       - name: Run the test suite
-        run: python3 -m unittest tests.test_bindgen -v
+        run: python3 tests/run_isolated.py

tests/run_isolated.py

@@ -0,0 +1,43 @@
+import subprocess
+import sys
+import unittest
+
+
+def test_ids(suite):
+    for item in suite:
+        if isinstance(item, unittest.TestSuite):
+            yield from test_ids(item)
+        else:
+            yield item.id()
+
+
+def main():
+    suite = unittest.TestLoader().discover("tests", "test_bindgen.py")
+    hung = []
+    failed = []
+    for test_id in test_ids(suite):
+        try:
+            proc = subprocess.run(
+                [sys.executable, "-m", "unittest", test_id],
+                capture_output=True,
+                text=True,
+                timeout=45,
+            )
+        except subprocess.TimeoutExpired:
+            hung.append(test_id)
+            print(f"HANG {test_id}", flush=True)
+            continue
+        if proc.returncode == 0:
+            print(f"pass {test_id}", flush=True)
+        else:
+            failed.append(test_id)
+            print(f"FAIL {test_id}\n{proc.stderr[-3000:]}", flush=True)
+
+    print("\n===== SUMMARY =====", flush=True)
+    print("HUNG:", hung, flush=True)
+    print("FAILED:", failed, flush=True)
+    sys.exit(1 if (hung or failed) else 0)
+
+
+if __name__ == "__main__":
+    main()

tests/test_bindgen.py

@@ -608,12 +608,15 @@ def test_device_get_process_by_name(self):
         with self.assertRaises(self.frida.ProcessNotFoundError):
             device.get_process("frida-nonexistent-process-zzz")
 
-    def test_attach_accepts_pid_and_name(self):
-        import os
-
+    @unittest.skipIf(sys.platform == "win32", "requires a POSIX shell")
+    def test_attach_accepts_pid(self):
         device = self.frida.get_local_device()
-        session = device.attach(os.getpid())
-        session.detach()
+        pid = device.spawn(["/bin/sh", "-c", "sleep 30"])
+        try:
+            session = device.attach(pid)
+            session.detach()
+        finally:
+            device.kill(pid)
 
     def test_facade_repr(self):
         device = self.frida.get_local_device()