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

    https://github.com/apache/nifi/pull/2704#discussion_r189200370
  
    --- Diff: 
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/test/java/org/apache/nifi/processors/aws/s3/TestListS3.java
 ---
    @@ -311,5 +312,10 @@ public void testGetPropertyDescriptors() throws 
Exception {
             assertTrue(pd.contains(ListS3.PREFIX));
             assertTrue(pd.contains(ListS3.USE_VERSIONS));
             assertTrue(pd.contains(ListS3.MIN_AGE));
    +        
assertTrue(pd.contains(ProxyConfigurationService.PROXY_CONFIGURATION_SERVICE));
    +        assertTrue(pd.contains(ListS3.PROXY_HOST));
    --- End diff --
    
    @jvwing Good catch, thanks. I removed the duplicated PROXY_HOST and 
PROXY_HOST_PORT. The missing one was LIST_TYPE. So, 17 + 3 = 20. 3 additions 
are PROXY_USER, PROXY_PASS and LIST_TYPE.


---

Reply via email to