merge latest - Githubissues

pharo-spec / Spec

Spec is a framework in Pharo for describing user interfaces.

MIT License

62 stars 65 forks source link

merge latest #1565

Closed estebanlm closed 4 months ago

estebanlm commented 4 months ago

hey @jecisc can you take a look at this? I can't find any undeclared or package that starts bad in my code so I guess PR is fine. What I do find is a DNU on testPackageExtensionsStartsWithProperPackageName, because packageName answer nil in some condition (in some methods that came from traits), and that may be another kind of bug...
In any case, merging this will unlock the changes on newtools too.

jecisc commented 4 months ago

I'm debugging in an image and it seems that we indeed have a bug between traits and protocols.. Protocols should not be nil

jecisc commented 4 months ago

This happens only on those 2 methods and I don't know what is special about them...

jecisc commented 4 months ago

This one is really not easy to debug :/