test(extgen): tighten parser tests by asserting warnings and dropping misleading conditionals

[alexandre-daubois]

No description provided.

[Copilot]

Pull request overview

This PR tightens internal/extgen parser tests by replacing conditional assertions with per-test assert callbacks, adding explicit coverage for invalid/orphan directives, and ensuring warnings are asserted when parsers intentionally emit them. It also makes integration tests skip (instead of fail) when external dependencies (PHP sources, xcaddy, php-config) are not available.

Changes:

• Refactor parser table tests (func/const/class) to use expect + optional assert callbacks, removing name-based conditional assertions.
• Add func parser tests for invalid signatures being dropped and orphan //export_php:function directives surfacing as errors.
• Capture and assert parser warnings in tests, and switch integration prereq failures to t.Skip.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
internal/extgen/integration_test.go	Skip integration tests when required external tools/files are missing (instead of failing).
internal/extgen/helpers_test.go	Add captureStdout helper for warning assertions (needs robustness fixes).
internal/extgen/funcparser_test.go	Tighten func parser assertions; add new directive/signature tests; assert warning output.
internal/extgen/constparser_test.go	Tighten constant parser assertions via callbacks; simplify error test expectations.
internal/extgen/classparser_test.go	Tighten class parser assertions via callbacks; remove misleading conditionals.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.