Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

assertModulesLoaded not matching all loaded Modules #25

Closed
nusphere opened this issue Apr 13, 2021 · 1 comment · Fixed by #84
Closed

assertModulesLoaded not matching all loaded Modules #25

nusphere opened this issue Apr 13, 2021 · 1 comment · Fixed by #84
Assignees
@visto9259
Labels
Bug Something isn't working
Milestone
4.10.1

Comments

@nusphere
Copy link
Contributor

Bug Report

Q A
Version(s) 3.4.2

Summary

$modulesLoaded = array_keys($moduleManager->getLoadedModules());

Current behavior

assertModulesLoaded not matching all loaded Modules

How to reproduce

  1. Add an internal dependency in a module (like 'Laminas\Mvc\I18n') with InitProviderInterface
  2. assert for this Module with assertModulesLoaded()

Expected behavior

this will be true
@nusphere nusphere added the Bug Something isn't working label Apr 13, 2021
@visto9259
Copy link
Contributor

Was able to reproduce with latest version.
Assign to me and I will fix it

@visto9259 visto9259 mentioned this issue Sep 12, 2024
Merged
@Xerkus Xerkus added this to the 4.10.1 milestone Sep 12, 2024
Xerkus added a commit that referenced this issue Sep 12, 2024
@Xerkus
Merge pull request #84 from visto9259/fix-25
a8de7f8 
Fix #25  assertModulesLoaded not matching all loaded Modules
@Xerkus Xerkus linked a pull request Sep 12, 2024 that will close this issue
Fix #25 assertModulesLoaded not matching all loaded Modules #84
Merged
@Xerkus Xerkus closed this as completed Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@visto9259 visto9259

Labels
Bug Something isn't working
Projects
None yet
Milestone
4.10.1
Development

Successfully merging a pull request may close this issue.

Fix #25 assertModulesLoaded not matching all loaded Modules visto9259/laminas-test
3 participants
@Xerkus @nusphere @visto9259