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

    https://github.com/apache/nifi/pull/2748#discussion_r193230207
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestReplaceText.java
 ---
    @@ -81,15 +81,16 @@ public void testSimple() throws IOException {
         @Test
         public void testWithEscaped$InReplacement() throws IOException {
             final TestRunner runner = getRunner();
    -        runner.setProperty(ReplaceText.SEARCH_VALUE, "(?s:^.*$)");
    +        //runner.setProperty(ReplaceText.SEARCH_VALUE, "(?s:^.*$)");
    +        runner.setProperty(ReplaceText.SEARCH_VALUE, "(?s)(^.*$)");
             runner.setProperty(ReplaceText.REPLACEMENT_VALUE, "a\\$b");
     
             runner.enqueue("a$a,b,c,d");
             runner.run();
     
             runner.assertAllFlowFilesTransferred(ReplaceText.REL_SUCCESS, 1);
             final MockFlowFile out = 
runner.getFlowFilesForRelationship(ReplaceText.REL_SUCCESS).get(0);
    -        out.assertContentEquals("a\\$b".getBytes("UTF-8"));
    +        out.assertContentEquals("a$b".getBytes("UTF-8"));
    --- End diff --
    
    Was this is a change in the way that ReplaceText behaves or was this unit 
test incorrect before?  Is there a way to obtain t he output "a\$b" from 
ReplaceText now?


---

Reply via email to