fix skipif check on tests involving gated HF models (#1553)

brian-dellabetta · web-flow · commit fa0f79388d46 · 2025-06-16T16:28:49.000Z
SUMMARY: Community user PRs are still failing tracing test checks. This loosens the skip check so it resolves correctly After merge and rebase, this should also resolve issue for #1521 TEST PLAN: Confirmed with @dbarbuzzi that this resolved the issue for #1445 . Signed-off-by: Brian Dellabetta <bdellabe@redhat.com>
diff --git a/tests/llmcompressor/transformers/tracing/test_models.py b/tests/llmcompressor/transformers/tracing/test_models.py
@@ -20,7 +20,7 @@
 
 
 @pytest.mark.skipif(
-    os.getenv("HF_TOKEN") is None,
+    (not os.getenv("HF_TOKEN")),
     reason="Skipping tracing tests requiring gated model access",
 )
 @pytest.mark.parametrize(

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`
`21`	`21`
`22`	`22`	`@pytest.mark.skipif(`
`23`		`- os.getenv("HF_TOKEN") is None,`
	`23`	`+ (not os.getenv("HF_TOKEN")),`
`24`	`24`	`reason="Skipping tracing tests requiring gated model access",`
`25`	`25`	`)`
`26`	`26`	`@pytest.mark.parametrize(`