search

cedarbdd / cedar

BDD-style testing using Objective-C
http://groups.google.com/group/cedar-discuss
1.19k stars 140 forks source link

Updated XCTest swizzling for test bundle setup #380

Closed idoru closed 8 years ago

idoru commented 8 years ago
idoru commented 8 years ago

This alternative to #379 does not even attempt to use the XCTestObservationCenter in any way, so it avoids both the original issue with test output suppression on Xcode 7.3 as well as the problem of owning the NSPrincipalClass setting on the bundle, making it unavailable for other frameworks.

With Xcode 7.3, +[XCTestSuite allTests] doesn't appear to be called anymore. In fact I suspect it may not have been called for a while before Xcode 7.3?

Anyway, from my experimentation, it appears that +[XCTestSuite testSuiteForTestConfiguration:] is called instead. I opted to remove all the swizzling and just add this method for now, but I'm not sure about how much backwards compatibility support we need here?

idoru commented 8 years ago

Tried a few things on Travis by throwing up commits on this PR without much luck :(

Going to get an older Xcode set up locally here so I can do some deeper digging.

idoru commented 8 years ago

Going with #382 to address this for now. Closing.