Skip to content

chore: remove gpt-oss-120b-free#64

Merged
sachaservan merged 1 commit intomainfrom
chore/model-rename
Mar 11, 2026
Merged

chore: remove gpt-oss-120b-free#64
sachaservan merged 1 commit intomainfrom
chore/model-rename

Conversation

@sachaservan
Copy link
Copy Markdown
Member

@sachaservan sachaservan commented Mar 9, 2026

Summary by cubic

Removed hardcoded references to gpt-oss-120b-free and made integration tests require explicit env vars. This decouples tests from a single model and prevents misconfigured runs.

  • Refactors

    • tests/test_tdx_attestation_flow.py: require TINFOIL_TEST_REPO and TINFOIL_TEST_ENCLAVE; skip module if missing; updated examples to <org/repo> and <enclave-hostname>.
    • tests/test_verification_failures_integration.py: require TINFOIL_TEST_CORRECT_REPO and TINFOIL_TEST_WRONG_REPO; skip module if missing; generalized wrong-repo messaging.
  • Migration

    • Set TINFOIL_TEST_REPO and TINFOIL_TEST_ENCLAVE for the TDX flow test.
    • Set TINFOIL_TEST_CORRECT_REPO and TINFOIL_TEST_WRONG_REPO for the verification failures test.

Written for commit 66c3cc3. Summary will update on new commits.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="tests/test_verification_failures_integration.py">

<violation number="1" location="tests/test_verification_failures_integration.py:30">
P2: Validate that `TINFOIL_TEST_CORRECT_REPO` and `TINFOIL_TEST_WRONG_REPO` are distinct; otherwise misconfigured CI/dev envs can make mismatch tests fail for the wrong reason.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@sachaservan sachaservan merged commit 9c95879 into main Mar 11, 2026
2 checks passed
@sachaservan sachaservan deleted the chore/model-rename branch March 11, 2026 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant