[squeak-dev] The Trunk: Tests-ar.71.mcz

commits at source.squeak.org commits at source.squeak.org
Wed Apr 7 05:52:04 UTC 2010


Andreas Raab uploaded a new version of Tests to project The Trunk:
http://source.squeak.org/trunk/Tests-ar.71.mcz

==================== Summary ====================

Name: Tests-ar.71
Author: ar
Time: 6 April 2010, 10:51:32.134 pm
UUID: 117226e6-d36b-2c4d-9383-72480d201b58
Ancestors: Tests-ar.70

Fix unreliable Monticello test.

=============== Diff against Tests-ar.70 ===============

Item was changed:
  ----- Method: MCMethodDefinitionTest>>tearDown (in category 'running') -----
  tearDown
  	self restoreMocks.
  	(MCWorkingCopy forPackage: (MCPackage named: 'FooBarBaz')) unregister.
  	self class compile: 'override ^ 1' classified: 'mocks'.
+ 	self ownPackage modified: isModified.
+ 
+ 	"FIXME: Unregister Monticellomocks if it got created implicitly.
+ 	This avoids a nasty failure of MCChangeNotificationTest due to
+ 	some inconsistency about whether package names are case sensitive
+ 	or not. They're treated as case insensitive in some name lookups but not
+ 	in others; most importantly PackageOrganizer default treats package
+ 	names as being case sensitive. The package created here is Monticellomocks
+ 	(lower case mocks) and an instance of PackageInfo; the package expected
+ 	in MCChangeNotificationTest is MonticelloMocks and an instance of
+ 	MCMockPackageInfo. Since *that* lookup is case insensitive it can
+ 	find Monticellomocks instead of MonticelloMocks and fail."
+ 	PackageOrganizer default unregisterPackageNamed: 'Monticellomocks'.
+ !
- 	self ownPackage modified: isModified!




More information about the Squeak-dev mailing list