Molecule
Molecule copied to clipboard
OpenSmock
Integration of new visualization options
@Eliott-Guevel I found a problem in this panel:
From the state of the screenshot (two selected elements): 1 - I click on "Browse" the class Equipment is browsed. 2 - I click on Receiver in the required list, and I click on "Browse" the class Equipment is always browsed
I don't know if GitHub also shows you this message, but I fixed my code according to your comments so you can merge this pull request since the interfaces function as intended and are named correctly. Thanks
Hi @Eliott-Guevel, the PR #177 is now merged. This PR contains new components start/stop announcement system. You can test your dev with it. Some tests for P12 are not passing, please check that (ignore P13 tests). Thanks!
There are no problems with my interfaces using the new main branch with #177 merged.
However, I don't understand why some of MolComponentManager's tests fail sometimes. When executing all of MolComponentManager's tests at the same time, 3 (or less) tests can fail (always the same ones, see Pharo 12 CI as well as testGarbageCollect (don't know why testFlushComponent is executed 2 times), which is not the case when executing them individually.
Putting relevant variables and going line by line in the debugger isn't useful to identify the fail (doesn't lead to the same result).
My guess is that since the order of tests is random, there's sometimes a bug with cleanUp and not all components are cleaned or something of the sort (and ONLY in Pharo 12, so could abso be a problem with the VM which apparently it is not, see https://github.com/pharo-project/pharo/issues/16808#issuecomment-2191257456).
I have no problem using the visualization option (which isn't currently intergrated to Molecule) with #177, it could be merged in the near future