OpenSmock / Pyramid

Pyramid is a Graphical User-Interface (GUI) builder and editor for Bloc and Toplo.
MIT License
29 stars 5 forks source link

PyramidPluginManagerTest>>testIsPluginInstalled with non installed plugin #127

Open labordep opened 5 months ago

labordep commented 5 months ago

This test cannot past because of a test plugin (PyramidPluginTestMode) is not installed.

labordep commented 5 months ago

This problem appears when updating Pyramid (from Iceberg), if a new plugin is loaded it should be installed. This is not the case. We need to find a way to load all plugins for :

Nyan11 commented 5 months ago

For "execute tests" just overwrite the setUp function in the test class to force installation of the plugins. For "Iceberg" i'm looking into it.

labordep commented 5 months ago

For "execute tests" just overwrite the setUp function in the test class to force installation of the plugins. For "Iceberg" i'm looking into it.

I have the same problem with Iceberg in Molecule. Molecule is lookup source-code modifications, including loading/unloading packages. I have developed new Iceberg/metacello events to add some events as package loading / unloading, but I need to create a pull request in Pharo. May be this is a similar problem.

See https://github.com/OpenSmock/Molecule/issues/156

Nyan11 commented 5 months ago

I need to have an event when Iceberg generate a pull from the GitLib. I don't think it is exactly the same issue.

labordep commented 5 months ago

Ok, this is the same tech feature for Molecule :) I will work on it.