The Trunk: Tests-ar.71.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

The Trunk: Tests-ar.71.mcz

commits-2
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!