Skip to content

Harness: honor PHPT --SKIPIF-- like php-src (#24888) - #24897

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-24888-skipif-harness
Jul 29, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/issue-24888-skipif-harness

Conversation

@PurHur

@PurHur PurHur commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Evaluate PHPT --SKIPIF-- under host PHP before running FILE (php-src run-tests.php semantics: output starting with skip → markTestSkipped).
  • Prepend vendor/autoload.php so SKIPIF can call CompilerVersion / extension policies; cwd is the PHPT directory; ENV/INI are applied.
  • Non-skip SKIPIF failures bork with a clear exception (do not silently run).

Closes #24888

Test plan

  • vendor/bin/phpunit --filter PhptSkipIfTest → OK (5)
  • Docker VMTest: lazy_ghost_basic, socket_atmark, mysqli_prepare_select, randomizer_getfloat_83, intl_error_functions, pgsql_basic, odbc_prepare_fetch_roundtrip, inotify_enum_operand, get_defined_functions_phantom_forward_profile, msgfmt_create_unmatched_braces, lazy_object_ghost, mysqli_real_connect → Skipped (not failed)
  • PHP_COMPILER_PROFILE=8.4 + randomizer_getfloat_83 → runs and passes (env gate honored)

Trust gates (pre-claim)

release-readiness --json: user_release_ready=yes (inventory 6635/6635, helper-runtime-prelink ok, north-star5-fast ok).

Not covered

Full VMTest/JITTest baseline regenerations (#24726) — SKIPIF alone removes false fails from future collects; committed .failing lists are unchanged in this PR.

Made with Cursor

Evaluate SKIPIF under host PHP before FILE so environment/profile
guards skip instead of polluting the compliance failing baseline.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit d140a80 into master Jul 29, 2026
@PurHur
PurHur deleted the agent/issue-24888-skipif-harness branch July 29, 2026 16:20
PurHur added a commit that referenced this pull request Jul 29, 2026
Docs correction for #24697 scoping: SKIPIF was dead (#24888/#24897), so intl/gmp failure rates counted inapplicable cases. Verified diff is RELEASE-PLAN.md only.
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.

Harness: --SKIPIF-- is silently ignored — 671 cases carry one, and >=19 baseline failures are only failing because of it

1 participant