From 6de738fd04f773ba6b05b3768c5214847b55cf50 Mon Sep 17 00:00:00 2001 From: Mark Payne Date: Tue, 7 Jun 2016 15:33:13 -0400 Subject: [PATCH] NIFI-1660: Fix formatting; update to NOTICE files of nars that bring in EL. Closed Reader that was opened in TestQuery. --- .../attribute/expression/language/Query.java | 5 ++- .../expression/language/TestQuery.java | 34 +++++++++---------- .../src/main/resources/META-INF/NOTICE | 8 +++++ .../src/main/resources/META-INF/NOTICE | 8 +++++ 4 files changed, 34 insertions(+), 21 deletions(-) diff --git a/nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/Query.java b/nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/Query.java index 9bd653a3a9..6985bfcd7e 100644 --- a/nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/Query.java +++ b/nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/Query.java @@ -1359,9 +1359,8 @@ public class Query { return addToken(new JsonPathEvaluator(toStringEvaluator(subjectEvaluator), toStringEvaluator(argEvaluators.get(0), "first argument to jsonPath")), "jsonPath"); } - default: - throw new AttributeExpressionLanguageParsingException( - "Expected a Function-type expression but got " + tree.toString()); + default: + throw new AttributeExpressionLanguageParsingException("Expected a Function-type expression but got " + tree.toString()); } } diff --git a/nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java b/nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java index 39622cb649..05bd0179a2 100644 --- a/nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java +++ b/nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java @@ -241,7 +241,7 @@ public class TestQuery { } @Test - public void testJsonPath() { + public void testJsonPath() throws IOException { final Map attributes = new HashMap<>(); attributes.put("json", getResourceAsString("/json/address-book.json")); verifyEquals("${json:jsonPath('$.firstName')}", attributes, "John"); @@ -1315,24 +1315,22 @@ public class TestQuery { assertEquals(expectedResult, result.getValue()); } - private String getResourceAsString(String resourceName) { - Reader reader = new InputStreamReader(new BufferedInputStream(getClass().getResourceAsStream(resourceName))); - int n = 0; - char[] buf = new char[1024]; - StringBuilder sb = new StringBuilder(); - while (n != -1) { - try { - n = reader.read(buf, 0, buf.length); - } catch (IOException e) { - throw new RuntimeException("failed to read resource", e); - } - if (n > 0) { - sb.append(buf, 0, n); + private String getResourceAsString(String resourceName) throws IOException { + try (final Reader reader = new InputStreamReader(new BufferedInputStream(getClass().getResourceAsStream(resourceName)))) { + int n = 0; + char[] buf = new char[1024]; + StringBuilder sb = new StringBuilder(); + while (n != -1) { + try { + n = reader.read(buf, 0, buf.length); + } catch (IOException e) { + throw new RuntimeException("failed to read resource", e); + } + if (n > 0) { + sb.append(buf, 0, n); + } } + return sb.toString(); } - return sb.toString(); - - } - } diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework-nar/src/main/resources/META-INF/NOTICE b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework-nar/src/main/resources/META-INF/NOTICE index ac7f0b7ae6..8de7a18f8a 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework-nar/src/main/resources/META-INF/NOTICE +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework-nar/src/main/resources/META-INF/NOTICE @@ -100,6 +100,14 @@ The following binary components are provided under the Apache Software License v Swagger Core 1.5.3-M1 Copyright 2015 Reverb Technologies, Inc. + (ASLv2) JSON-SMART + The following NOTICE information applies: + Copyright 2011 JSON-SMART authors + + (ASLv2) JsonPath + The following NOTICE information applies: + Copyright 2011 JsonPath authors + ************************ Common Development and Distribution License 1.1 ************************ diff --git a/nifi-nar-bundles/nifi-update-attribute-bundle/nifi-update-attribute-nar/src/main/resources/META-INF/NOTICE b/nifi-nar-bundles/nifi-update-attribute-bundle/nifi-update-attribute-nar/src/main/resources/META-INF/NOTICE index 419dc19c78..1e90f9f70b 100644 --- a/nifi-nar-bundles/nifi-update-attribute-bundle/nifi-update-attribute-nar/src/main/resources/META-INF/NOTICE +++ b/nifi-nar-bundles/nifi-update-attribute-bundle/nifi-update-attribute-nar/src/main/resources/META-INF/NOTICE @@ -17,3 +17,11 @@ The following binary components are provided under the Apache Software License v This product includes software from the Spring Framework, under the Apache License 2.0 (see: StringUtils.containsWhitespace()) + + (ASLv2) JSON-SMART + The following NOTICE information applies: + Copyright 2011 JSON-SMART authors + + (ASLv2) JsonPath + The following NOTICE information applies: + Copyright 2011 JsonPath authors