Jules - fix test_story() for latest mupdf, new testing .yml files that use windows-2022.

.github/workflows/test.yml

@@ -26,7 +26,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest, windows-2019, macos-13, macos-14]
+        os: [ubuntu-latest, windows-2022, macos-13, macos-14]
 
       # Avoid cancelling of all runs after a single failure.
       fail-fast: false

.github/workflows/test_multiple.yml

@@ -0,0 +1,39 @@
+# Run scripts/test.py on multiple OS's (Windows, Linux, MacOS x64, MacOS arm64)
+# and with multiple specifications of MuPDF (PyMuPDF's hard-coded default,
+# master branch, release branch).
+
+name: multiple
+
+on:
+  workflow_dispatch:
+    inputs:
+      args:
+        type: string
+        default: ''
+        description: 'Arguments to pass to scripts/test.py'
+  schedule:
+    - cron: '13 6 * * *'
+
+jobs:
+
+  multiple:
+    name: multiple
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-2022, macos-13, macos-14]
+        args: [
+            '',
+            '-m "git:--branch master https://github.com/ArtifexSoftware/mupdf"', 
+            '-m "git:--branch 1.26.x https://github.com/ArtifexSoftware/mupdf"',
+            ]
+      fail-fast: false
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+      - name: multiple
+        env:
+          PYMUPDF_test_args: ${{inputs.args}}
+        run:
+          python scripts/test.py ${{matrix.args}} wheel test -a PYMUPDF_test_args

scripts/test.py

@@ -1128,10 +1128,14 @@ def venv_run(args, path, recreate=True):
     if recreate or not os.path.isdir(path):
         run(f'{sys.executable} -m venv {path}')
     if platform.system() == 'Windows':
-        command = f'{path}\\Scripts\\activate'
+        command = f'{path}\\Scripts\\activate && python'
+        # shlex not reliable on Windows.
+        # Use crude quoting with "...". Seems to work.
+        for arg in args:
+            assert '"' not in arg
+            command += f' "{arg}"'
     else:
-        command = f'. {path}/bin/activate'
-    command += f' && python {shlex.join(args)}'
+        command = f'. {path}/bin/activate && python {shlex.join(args)}'
     e = run(command, check=0)
     return e
 

tests/test_story.py

@@ -38,7 +38,7 @@ def test_story():
 def test_2753():
 
     def rectfn(rect_num, filled):
-        return pymupdf.Rect(0, 0, 200, 200), pymupdf.Rect(50, 50, 100, 100), None
+        return pymupdf.Rect(0, 0, 200, 200), pymupdf.Rect(50, 50, 100, 150), None
 
     def make_pdf(html, path_out):
         story = pymupdf.Story(html=html)
@@ -65,6 +65,9 @@ def make_pdf(html, path_out):
             os.path.abspath(f'{__file__}/../../tests/test_2753-out-after.pdf'),
             )
 
+    path = os.path.normpath(f'{__file__}/../../tests/test_2753_out')
+    doc_before.save(f'{path}_before.pdf')
+    doc_after.save(f'{path}_after.pdf')
     assert len(doc_before) == 2
     assert len(doc_after) == 2