add initial tox.ini

.github/workflows/ci.yml

@@ -276,10 +276,15 @@ jobs:
           python-version: ${{ fromJSON(format('["{0}", "{1}"]', format('{0}.0-alpha - {0}.X', matrix.python), matrix.python))[startsWith(matrix.python, 'pypy')] }}
           cache: pip
           cache-dependency-path: test-requirements.txt
+      - name: Check Formatting
+        if: matrix.check_formatting == '1'
+        run:
+          python -m pip install tox &&
+          tox -m check
       - name: Run tests
+        if: matrix.check_formatting == '0'
         run: ./ci.sh
         env:
-          CHECK_FORMATTING: '${{ matrix.check_formatting }}'
           NO_TEST_REQUIREMENTS: '${{ matrix.no_test_requirements }}'
       - if: >-
           always()

.gitignore

@@ -1,3 +1,6 @@
+# generated by cythonize
+tests/cython/test_cython.c
+
 # In case somebody wants to restore the directory for local testing
 notes-to-self/
 

ci.sh

@@ -47,115 +47,109 @@ python -m build
 wheel_package=$(ls dist/*.whl)
 python -m uv pip install "trio @ $wheel_package" -c test-requirements.txt
 
-if [ "$CHECK_FORMATTING" = "1" ]; then
-    python -m uv pip install -r test-requirements.txt exceptiongroup
-    echo "::endgroup::"
-    source check.sh
+# Actual tests
+# expands to 0 != 1 if NO_TEST_REQUIREMENTS is not set, if set the `-0` has no effect
+# https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_02
+if [ "${NO_TEST_REQUIREMENTS-0}" == 1 ]; then
+    python -m uv pip install pytest coverage -c test-requirements.txt
+    flags="--skip-optional-imports"
 else
-    # Actual tests
-    # expands to 0 != 1 if NO_TEST_REQUIREMENTS is not set, if set the `-0` has no effect
-    # https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_02
-    if [ "${NO_TEST_REQUIREMENTS-0}" == 1 ]; then
-        python -m uv pip install pytest coverage -c test-requirements.txt
-        flags="--skip-optional-imports"
-    else
-        python -m uv pip install -r test-requirements.txt
-        flags=""
-    fi
+    python -m uv pip install -r test-requirements.txt
+    flags=""
+fi
 
-    # So we can run the test for our apport/excepthook interaction working
-    if [ -e /etc/lsb-release ] && grep -q Ubuntu /etc/lsb-release; then
-        sudo apt install -q python3-apport
-    fi
+# So we can run the test for our apport/excepthook interaction working
+if [ -e /etc/lsb-release ] && grep -q Ubuntu /etc/lsb-release; then
+    sudo apt install -q python3-apport
+fi
 
-    # If we're testing with a LSP installed, then it might break network
-    # stuff, so wait until after we've finished setting everything else
-    # up.
-    if [ "$LSP" != "" ]; then
-        echo "Installing LSP from ${LSP}"
-        # We use --insecure because one of the LSP's has been observed to give
-        # cert verification errors:
-        #
-        #   https://github.com/python-trio/trio/issues/1478
-        #
-        # *Normally*, you should never ever use --insecure, especially when
-        # fetching an executable! But *in this case*, we're intentionally
-        # installing some untrustworthy quasi-malware onto into a sandboxed
-        # machine for testing. So MITM attacks are really the least of our
-        # worries.
-        if [ "$LSP_EXTRACT_FILE" != "" ]; then
-            # We host the Astrill VPN installer ourselves, and encrypt it
-            # so as to decrease the chances of becoming an inadvertent
-            # public redistributor.
-            curl-harder -o lsp-installer.zip "$LSP"
-            unzip -P "not very secret trio ci key" lsp-installer.zip "$LSP_EXTRACT_FILE"
-            mv "$LSP_EXTRACT_FILE" lsp-installer.exe
-        else
-            curl-harder --insecure -o lsp-installer.exe "$LSP"
-        fi
-        # This is only needed for the Astrill LSP, but there's no harm in
-        # doing it all the time. The cert was manually extracted by installing
-        # the package in a VM, clicking "Always trust from this publisher"
-        # when installing, and then running 'certmgr.msc' and exporting the
-        # certificate. See:
-        #    http://www.migee.com/2010/09/24/solution-for-unattendedsilent-installs-and-would-you-like-to-install-this-device-software/
-        certutil -addstore "TrustedPublisher" src/trio/_tests/astrill-codesigning-cert.cer
-        # Double-slashes are how you tell windows-bash that you want a single
-        # slash, and don't treat this as a unix-style filename that needs to
-        # be replaced by a windows-style filename.
-        # http://www.mingw.org/wiki/Posix_path_conversion
-        ./lsp-installer.exe //silent //norestart
-        echo "Waiting for LSP to appear in Winsock catalog"
-        while ! netsh winsock show catalog | grep "Layered Chain Entry"; do
-            sleep 1
-        done
-        netsh winsock show catalog
+# If we're testing with a LSP installed, then it might break network
+# stuff, so wait until after we've finished setting everything else
+# up.
+if [ "$LSP" != "" ]; then
+    echo "Installing LSP from ${LSP}"
+    # We use --insecure because one of the LSP's has been observed to give
+    # cert verification errors:
+    #
+    #   https://github.com/python-trio/trio/issues/1478
+    #
+    # *Normally*, you should never ever use --insecure, especially when
+    # fetching an executable! But *in this case*, we're intentionally
+    # installing some untrustworthy quasi-malware onto into a sandboxed
+    # machine for testing. So MITM attacks are really the least of our
+    # worries.
+    if [ "$LSP_EXTRACT_FILE" != "" ]; then
+        # We host the Astrill VPN installer ourselves, and encrypt it
+        # so as to decrease the chances of becoming an inadvertent
+        # public redistributor.
+        curl-harder -o lsp-installer.zip "$LSP"
+        unzip -P "not very secret trio ci key" lsp-installer.zip "$LSP_EXTRACT_FILE"
+        mv "$LSP_EXTRACT_FILE" lsp-installer.exe
+    else
+        curl-harder --insecure -o lsp-installer.exe "$LSP"
     fi
-    echo "::endgroup::"
-
-    echo "::group::Setup for tests"
+    # This is only needed for the Astrill LSP, but there's no harm in
+    # doing it all the time. The cert was manually extracted by installing
+    # the package in a VM, clicking "Always trust from this publisher"
+    # when installing, and then running 'certmgr.msc' and exporting the
+    # certificate. See:
+    #    http://www.migee.com/2010/09/24/solution-for-unattendedsilent-installs-and-would-you-like-to-install-this-device-software/
+    certutil -addstore "TrustedPublisher" src/trio/_tests/astrill-codesigning-cert.cer
+    # Double-slashes are how you tell windows-bash that you want a single
+    # slash, and don't treat this as a unix-style filename that needs to
+    # be replaced by a windows-style filename.
+    # http://www.mingw.org/wiki/Posix_path_conversion
+    ./lsp-installer.exe //silent //norestart
+    echo "Waiting for LSP to appear in Winsock catalog"
+    while ! netsh winsock show catalog | grep "Layered Chain Entry"; do
+        sleep 1
+    done
+    netsh winsock show catalog
+fi
+echo "::endgroup::"
 
-    # We run the tests from inside an empty directory, to make sure Python
-    # doesn't pick up any .py files from our working dir. Might have already
-    # been created by a previous run.
-    mkdir empty || true
-    cd empty
+echo "::group::Setup for tests"
 
-    INSTALLDIR=$(python -c "import os, trio; print(os.path.dirname(trio.__file__))")
-    cp ../pyproject.toml "$INSTALLDIR"  # TODO: remove this
+# We run the tests from inside an empty directory, to make sure Python
+# doesn't pick up any .py files from our working dir. Might have already
+# been created by a previous run.
+mkdir empty || true
+cd empty
 
-    # get mypy tests a nice cache
-    MYPYPATH=".." mypy --config-file= --cache-dir=./.mypy_cache -c "import trio" >/dev/null 2>/dev/null || true
+INSTALLDIR=$(python -c "import os, trio; print(os.path.dirname(trio.__file__))")
+cp ../pyproject.toml "$INSTALLDIR"  # TODO: remove this
 
-    # support subprocess spawning with coverage.py
-    echo "import coverage; coverage.process_startup()" | tee -a "$INSTALLDIR/../sitecustomize.py"
+# get mypy tests a nice cache
+MYPYPATH=".." mypy --config-file= --cache-dir=./.mypy_cache -c "import trio" >/dev/null 2>/dev/null || true
 
-    perl -i -pe 's/-p trio\._tests\.pytest_plugin//' "$INSTALLDIR/pyproject.toml"
+# support subprocess spawning with coverage.py
+echo "import coverage; coverage.process_startup()" | tee -a "$INSTALLDIR/../sitecustomize.py"
 
-    echo "::endgroup::"
-    echo "::group:: Run Tests"
-    if PYTHONPATH=../tests COVERAGE_PROCESS_START=$(pwd)/../pyproject.toml \
-            coverage run --rcfile=../pyproject.toml -m \
-            pytest -ra --junitxml=../test-results.xml \
-            -p _trio_check_attrs_aliases --verbose --durations=10 \
-            -p trio._tests.pytest_plugin --run-slow $flags "${INSTALLDIR}"; then
-        PASSED=true
-    else
-        PASSED=false
-    fi
-    echo "::endgroup::"
-    echo "::group::Coverage"
+perl -i -pe 's/-p trio\._tests\.pytest_plugin//' "$INSTALLDIR/pyproject.toml"
 
-    coverage combine --rcfile ../pyproject.toml
-    coverage report -m --rcfile ../pyproject.toml
-    coverage xml --rcfile ../pyproject.toml
+echo "::endgroup::"
+echo "::group:: Run Tests"
+if PYTHONPATH=../tests COVERAGE_PROCESS_START=$(pwd)/../pyproject.toml \
+        coverage run --rcfile=../pyproject.toml -m \
+        pytest -ra --junitxml=../test-results.xml \
+        -p _trio_check_attrs_aliases --verbose --durations=10 \
+        -p trio._tests.pytest_plugin --run-slow $flags "${INSTALLDIR}"; then
+    PASSED=true
+else
+    PASSED=false
+fi
+echo "::endgroup::"
+echo "::group::Coverage"
 
-    # Remove the LSP again; again we want to do this ASAP to avoid
-    # accidentally breaking other stuff.
-    if [ "$LSP" != "" ]; then
-        netsh winsock reset
-    fi
+coverage combine --rcfile ../pyproject.toml
+coverage report -m --rcfile ../pyproject.toml
+coverage xml --rcfile ../pyproject.toml
 
-    echo "::endgroup::"
-    $PASSED
+# Remove the LSP again; again we want to do this ASAP to avoid
+# accidentally breaking other stuff.
+if [ "$LSP" != "" ]; then
+    netsh winsock reset
 fi
+
+echo "::endgroup::"
+$PASSED