Github user ottobackwards commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2748#discussion_r193422077
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestReplaceText.java
 ---
    @@ -1074,12 +1090,11 @@ public void testRegexWithBadCaptureGroup() throws 
IOException {
             runner.setProperty(ReplaceText.REPLACEMENT_STRATEGY, 
ReplaceText.REGEX_REPLACE);
             runner.setProperty(ReplaceText.EVALUATION_MODE, 
ReplaceText.ENTIRE_TEXT);
     
    +        exception.expect(AssertionError.class);
    +        exception.expectMessage("java.lang.IndexOutOfBoundsException: No 
group 1");
    --- End diff --
    
    I'm looking into it, but the processor already has handing for this at 
runtime not at validation time, where it escapes back references that are over 
the actual capture count


---

Reply via email to