The current implementation of this plug-in results in the ignoring of test failures if the property maven.test.failure.ignore is set TO ANY VALUE. In other words, even if maven.test.failure.ignore=false test failures will be ignored. This is not logical given the highly descriptive property name. I propose that the plug-in be refactored so that its behavior is predicated on the value of this property.