Skip to content

[IMPROVEMENT] Replace PHPUnit MockBuilder#11329

Open
chfsx wants to merge 1 commit intoILIAS-eLearning:trunkfrom
srsolutionsag:proposel/12/bootstrap-mocking
Open

[IMPROVEMENT] Replace PHPUnit MockBuilder#11329
chfsx wants to merge 1 commit intoILIAS-eLearning:trunkfrom
srsolutionsag:proposel/12/bootstrap-mocking

Conversation

@chfsx
Copy link
Copy Markdown
Member

@chfsx chfsx commented Mar 24, 2026

@thibsy as discussed, I propose to replace the Object-Mocking in bootstraping with an own implementation. therefore we no longer need PHPUnits-mocking and PHPUnits as prod-dependency. beside this, bootstrapping is twice as fast as before...

I'd like to change the following things before merging:

  • get rid of the static calls
  • inject the mock-builder
  • add some unit tests

but you can have a first look at the implementation.

@chfsx chfsx requested a review from thibsy March 24, 2026 15:06
@chfsx chfsx force-pushed the proposel/12/bootstrap-mocking branch from 17eeda1 to 0378399 Compare April 14, 2026 11:56
@chfsx chfsx changed the title WIP: [IMPROVEMENT] Replace PHPUnit MockBuilder [IMPROVEMENT] Replace PHPUnit MockBuilder Apr 14, 2026
@chfsx
Copy link
Copy Markdown
Member Author

chfsx commented Apr 14, 2026

@thibsy I pushed the branch with changes mentioned above:

  • get rid of the static calls
  • introduced interface MockBuilder and 3 implementations (PHPUnit, Eval based and File-Based)
  • we Use Eval-Based now for Bootstrapping
  • Added Unittests to test the new MockBuilder

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants