Skip to content

Fix PHPUnit tests. - #254

Merged
jay-oswald merged 3 commits into
composerfrom
unit-test-fixes
Oct 9, 2026
Merged

jay-oswald merged 3 commits into
composerfrom
unit-test-fixes

Conversation

@abhinavgandham

@abhinavgandham abhinavgandham commented Oct 9, 2026 •

Copy link
Copy Markdown

This PR resolves the following:

1st Commit:

  • PHPUnit data providers missing static key word.
  • Moodle core has removed the moodlenetprofile column, so removed the field from user_lookup_step.php.
  • Fixed Call to undefined function error in process_workflows_test.php by removing the getActualOutput call as this was removed in PHPUnit 10 and is no longer used in modern versions of PHPUnit.

2nd Commit

  • Fix several code checker errors and warnings.

Comment thread tests/process_workflows_test.php Outdated
@abhinavgandham
abhinavgandham force-pushed the unit-test-fixes branch 2 times, most recently from 0c66c7b to c892567 Compare October 9, 2026 04:46
@jay-oswald
jay-oswald merged commit b708651 into composer Oct 9, 2026
@jay-oswald
jay-oswald deleted the unit-test-fixes branch October 9, 2026 05:59
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