test_sign: disable more staging tests (#997)

* test_sign: disable more staging tests

Signed-off-by: William Woodruff <william@trailofbits.com>

* ci: add --skip-staging to interactive tests too

Not sure why we have this split.

Signed-off-by: William Woodruff <william@trailofbits.com>

* Revert "test_sign: disable more staging tests"

This reverts commit a105caa.

* conftest: another skipping strategy

Signed-off-by: William Woodruff <william@trailofbits.com>

* disable skip

Signed-off-by: William Woodruff <william@trailofbits.com>

* test: actually delete old skip stuff

Signed-off-by: William Woodruff <william@trailofbits.com>

---------

Signed-off-by: William Woodruff <william@trailofbits.com>

Author: woodruffw

.github/workflows/ci.yml

@@ -55,7 +55,7 @@ jobs:
 
       - name: test (interactive)
         if: (github.event_name != 'pull_request') || !github.event.pull_request.head.repo.fork
-        run: make test-interactive TEST_ARGS="-vv --showlocals"
+        run: make test-interactive TEST_ARGS="-vv --showlocals --skip-staging"
 
       - uses: ./.github/actions/upload-coverage
         # only aggregate test coverage over linux-based tests to avoid any OS-specific filesystem information stored in

test/unit/conftest.py

@@ -262,6 +262,7 @@ def tuf_dirs(monkeypatch, tmp_path):
 
 @pytest.fixture
 def sign_ctx_and_ident_for_env(
+    pytestconfig,
     env: str,
 ) -> tuple[type[SigningContext], type[IdentityToken]]:
     if env == "staging":