dweiss commented on code in PR #15468:
URL: https://github.com/apache/lucene/pull/15468#discussion_r2599712522
##########
lucene/test-framework/src/java/org/apache/lucene/tests/util/LuceneTestCase.java:
##########
@@ -350,6 +350,20 @@ public abstract class LuceneTestCase extends Assert {
String[] value();
}
+ /**
+ * Annotation for test classes that should avoid specific asserting formats
within the Asserting
+ * codec (e.g., AssertingStoredFieldsFormat) while keeping other asserting
formats active.
+ *
+ * <p>Use the simple class name of the format to suppress, e.g.,
"AssertingStoredFieldsFormat".
+ */
+ @Documented
+ @Inherited
+ @Retention(RetentionPolicy.RUNTIME)
+ @Target(ElementType.TYPE)
+ public @interface SuppressAssertingFormats {
+ String[] value();
Review Comment:
Can we make it an array of enum instead and perhaps declare this enum inside
AssertinCodec instead of Strings? It would make it a lot easier to pick and
then look up those suppression filters.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
