Skip to content

Make non-vlt e2e legs fail when every test soft-skips #1054

Description

Follow-up from #1046 (audit B69/test hygiene).

The generalized passed > 0 guard in the ci.yml e2e step (ci.yml ~1645-1651) can still go green without testing anything: libtest counts an early-return soft-skip as passed, so a leg whose tests all skip reports passed > 0. Only per-suite *_REQUIRED env gates catch this, and not every row sets one (e.g. e2e_redirect_rush_sim).

Options:

  • have the non-vlt e2e step grep a standard skip marker, as check-vlt-legs.py does for the vlt step; or
  • give every e2e row a *_REQUIRED gate.

🤖 Generated with Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions