Nice assertion values, but these messages need to be better, for example, "removing an attribute should invoke the attributeChanged callback with a null new value" or something like that. It's possible that this assertion will fire if the first or second arguments are wrong too; that's OK, because many attribute tests will probably be failing in that case. The essence of this assertion is checking for that null, really.