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.

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

Port to pharo13 bugfix issue 16702: Trait class side methods sometimes not installed on user classes. #16758

Open
wants to merge 2 commits into
base: Pharo13
Choose a base branch
from

Conversation

bouraqadi
Copy link
Contributor

Bug fix initially implemented for P12

@MarcusDenker
Copy link
Member

Lots of tests are failing:

 Unix64.Collections.Sequenceable.Tests.ArrayTest.testShuffleBy	1.2 sec	1
 Unix64.Collections.Support.Tests.CollectionTest.testAtRandomWeighting	10 sec	1
 Unix64.Collections.Support.Tests.CollectionTest.testAtRandomWeightingMultiple	10 sec	1
 Unix64.Fuel.Core.Tests.FLCreateClassSerializationTest.testCreateWithTraitOnClassSide	10 sec	1
 Unix64.Fuel.Core.Tests.FLCreateTraitSerializationTest.testCreateWithTraitOnClassSide	10 sec	1
 Unix64.Kernel.Extended.Tests.UnicodeTest.testRNG	8.7 sec	1
 Unix64.Network.Tests.UUIDPrimitivesTest.testCreationNodeBased	9 sec	1
 Unix64.PharoDocComment.Tests.CommentTestCase.nil	5 sec	1
 Unix64.PharoDocComment.Tests.CommentTestCase.nil	6.7 sec	1
 Unix64.PharoDocComment.Tests.CommentTestCase.nil	7.3 sec	1
 Unix64.RTree.Tests.RTreeTest.testRemoveLeaf1	10 sec	1
 Unix64.Random.Tests.RandomTest.testDistribution	8.8 sec	1
 Unix64.Random.Tests.RandomTest.testPrimitiveRandomGeneration1	8.7 sec	1
 Unix64.Random.Tests.RandomTest.testPrimitiveRandomGeneration2	9 sec	1
 Unix64.Random.Tests.RandomTest.testPrimitiveRandomGeneration3	8.6 sec	1
 Unix64.Roassal.Global.Tests.RSForceBasedLayoutTest.testAddNodesAndEdges	0.49 sec	1
 Unix64.Roassal.Global.Tests.RSRTreeTest.testRemoveLeaf1	0.67 sec	1
 Unix64.Slot.Tests.SlotTraitsTest.testClassWithClassTrait	1.6 sec	1
 Unix64.Slot.Tests.SlotTraitsTest.testModifyClassTraitComposition	1.5 sec	1
 Unix64.Slot.Tests.SlotTraitsTest.testRemoveClassTrait	1.8 sec	1
 Unix64.Slot.Tests.SlotTraitsTest.testTraitUsersAfterClassReshape	1.6 sec	1
 Unix64.Slot.Tests.SlotTraitsTest.testTraitUsersAfterMetaclassReshape	1.5 sec	1
 Unix64.Traits.Tests.TraitFileOutTest.testFileOutCategory	9.1 sec	1
 Unix64.Traits.Tests.TraitFileOutTest.testFileOutTrait	9 sec	1
 Unix64.Traits.Tests.TraitFileOutTest.testRecompiling	8.8 sec	1
 Unix64.Traits.Tests.TraitFileOutTest.testRemovingMethods

@MarcusDenker MarcusDenker added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Jun 7, 2024
@Ducasse
Copy link
Member

Ducasse commented Jun 14, 2024

This is strange. Are you sure that we got a fix in P12 that is not in P13

@MarcusDenker
Copy link
Member

No, the same tests fail on Pharo12, too.

There are now three PRs for this fix, maybe we should close all but Pharo13 and backport when we now that it works

@MarcusDenker
Copy link
Member

The random tests are not related,added an issue: #16763

But there are tests failing like TraitFileOutTest and more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Need more work The issue is nearly ready. Waiting some last bits.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants