FogBugz (Case [Issue]10919) Kernel - Direct reference to ClassBuilder in a test case in Tests-System

A FogBugz case was opened by Martin Dias.

Case ID:      10919
Title:        Direct reference to ClassBuilder in a test case in Tests-System
Status:       Work Needed
Category:     Bug
Project:      Kernel
Area:         Misc
Priority:     3 - Must Fix
Milestone:    Later
Assigned To:  Everyone

URL:          https://pharo.fogbugz.com/default.asp?10919

I am looking forward to remove direct references to ClassBuilder. As part of such task, I found:

ClassQueryTest>>
testReferencedClasses
"
self debug: #testReferencedClasses
"
| refs |
refs := Metaclass referencedClasses.
self assert: (refs includes: ClassBuilder).

refs := self class referencedClasses.
self assert: (refs includesAllOf: {SoundService . Beeper})

---->

Since the current answer of "Metaclass referencedClasses" is:

a Set(Set TraitComposition TraitCompositionException Metaclass String ClassBuilder SystemAnnouncer)

As a fast fix we can change it to assert the inclusion of SystemAnnouncer or String.

As a better fix, it may be better to create a dummy class, compile a new method with a reference, and assert its inclusion.

Opinions?


You are subscribed to this case.  If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=10919) from this case.

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker