Skip to content

patch: handle submitted complilation error#97

Closed
neenjaw wants to merge 1 commit intomainfrom
patch-error-case
Closed

patch: handle submitted complilation error#97
neenjaw wants to merge 1 commit intomainfrom
patch-error-case

Conversation

@neenjaw
Copy link
Copy Markdown
Contributor

@neenjaw neenjaw commented Jan 27, 2024

Presently a compilation error was classified as a test failure, however
this causes the website to show all of the dependent test cases which
appear to be passing (despite actually not being run), because the
j-unit output doesn't have a success child element to indicate success,
only the absense of a skipped/failure element indicates success.

Presently a compilation error was classified as a test failure, however
this causes the website to show all of the dependent test cases which
appear to be passing (despite actually not being run), because the
j-unit output doesn't have a success child element to indicate success,
only the absense of a skipped/failure element indicates success.
@neenjaw
Copy link
Copy Markdown
Contributor Author

neenjaw commented Jan 27, 2024

Following the CI run here: Will need to more closely look at how to consider errors in test cases as distinct from errors in the test file.

@mk-mxp
Copy link
Copy Markdown
Contributor

mk-mxp commented Jan 30, 2024

Maybe you can run php -l $solution_file to separately lint the students file for errors and not run PHPUnit at all when that fails?

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.

2 participants