[ 
https://issues.apache.org/jira/browse/NIFI-2519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15417198#comment-15417198
 ] 

ASF GitHub Bot commented on NIFI-2519:
--------------------------------------

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

    https://github.com/apache/nifi/pull/827#discussion_r74418332
  
    --- Diff: 
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
 ---
    @@ -106,26 +69,17 @@
                 .addValidator(StandardValidators.PORT_VALIDATOR)
                 .build();
     
    -    protected static final PropertyDescriptor SMTP_HOSTNAME = new 
PropertyDescriptor.Builder()
    -            .name("SMTP_HOSTNAME")
    -            .displayName("SMTP hostname")
    -            .description("The hostname to be embedded into the banner 
displayed when an " +
    -                    "SMTP client connects to the processor TCP port .")
    -            .required(true)
    -            .expressionLanguageSupported(false)
    -            .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
    -            .build();
    -
    -    protected static final PropertyDescriptor SMTP_MAXIMUM_CONNECTIONS = 
new PropertyDescriptor.Builder()
    +    static final PropertyDescriptor SMTP_MAXIMUM_CONNECTIONS = new 
PropertyDescriptor.Builder()
                 .name("SMTP_MAXIMUM_CONNECTIONS")
                 .displayName("Maximum number of SMTP connection")
                 .description("The maximum number of simultaneous SMTP 
connections.")
                 .required(true)
    +            .defaultValue("1")
                 .expressionLanguageSupported(false)
                 .addValidator(StandardValidators.INTEGER_VALIDATOR)
                 .build();
     
    -    protected static final PropertyDescriptor SMTP_TIMEOUT = new 
PropertyDescriptor.Builder()
    +    static final PropertyDescriptor SMTP_TIMEOUT = new 
PropertyDescriptor.Builder()
    --- End diff --
    
    Is SMTP_TIMEOUT being enforced at the DATA stage?
    
    The server seems to hang around the data stage with the connection open no 
matter how long the client idles for.
    
    Typing . and quit do not work
    
    ```
    $ time telnet 0 2525
    Trying 0.0.0.0...
    Connected to 0.
    Escape character is '^]'.
    220 localhost ESMTP Apache NiFi
    helo .
    250 localhost
    mail from: x
    250 Ok
    rcpt to: a
    250 Ok
    data
    354 End data with <CR><LF>.<CR><LF>
    Hmmmm....
    
    .
    ^]
    telnet> quit
    Connection closed.
    
    real    1m24.348s
    user    0m0.000s
    sys     0m0.002s
    ```


> TestListenSMTP ValidEmail fails during parallel build
> -----------------------------------------------------
>
>                 Key: NIFI-2519
>                 URL: https://issues.apache.org/jira/browse/NIFI-2519
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Extensions
>            Reporter: Joseph Witt
>            Assignee: Oleg Zhurakousky
>             Fix For: 1.0.0
>
>
> While running a full NiFi parallel build received the following. So there is 
> some test issue at least that is impacting build stability.
> [INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
> nifi-email-processors ---
> [INFO] Changes detected - recompiling the module!
> [INFO] Compiling 4 source files to 
> /home/travis/build/apache/nifi/nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/target/test-classes
> [WARNING] 
> /home/travis/build/apache/nifi/nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/java/org/apache/nifi/processors/email/TestListenSMTP.java:[122,24]
>  [deprecation] stop() in Thread has been deprecated
> [WARNING] 
> /home/travis/build/apache/nifi/nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/java/org/apache/nifi/processors/email/TestListenSMTP.java:[186,24]
>  [deprecation] stop() in Thread has been deprecated
> [WARNING] 
> /home/travis/build/apache/nifi/nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/java/org/apache/nifi/processors/email/TestListenSMTP.java:[307,24]
>  [deprecation] stop() in Thread has been deprecated
> [INFO] 
> [INFO] --- maven-compiler-plugin:3.2:testCompile (groovy-tests) @ 
> nifi-email-processors ---
> [INFO] Changes detected - recompiling the module!
> [INFO] Nothing to compile - all classes are up to date
> [INFO] 
> [INFO] --- maven-surefire-plugin:2.18:test (default-test) @ 
> nifi-email-processors ---
> [INFO] Surefire report directory: 
> /home/travis/build/apache/nifi/nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/target/surefire-reports
> [INFO] Using configured provider 
> org.apache.maven.surefire.junit4.JUnit4Provider
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> Running org.apache.nifi.processors.email.TestListenSMTP
> Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.473 sec <<< 
> FAILURE! - in org.apache.nifi.processors.email.TestListenSMTP
> ValidEmail(org.apache.nifi.processors.email.TestListenSMTP)  Time elapsed: 
> 0.038 sec  <<< FAILURE!
> java.lang.AssertionError: Sending email failed
>       at org.junit.Assert.fail(Assert.java:88)
>       at org.junit.Assert.assertTrue(Assert.java:41)
>       at org.junit.Assert.assertFalse(Assert.java:64)
>       at 
> org.apache.nifi.processors.email.TestListenSMTP.ValidEmail(TestListenSMTP.java:188)
> Running org.apache.nifi.processors.email.TestExtractEmailAttachments
> Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.1 sec - in 
> org.apache.nifi.processors.email.TestExtractEmailAttachments
> Running org.apache.nifi.processors.email.TestExtractEmailHeaders
> Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.029 sec - 
> in org.apache.nifi.processors.email.TestExtractEmailHeaders
> Results :
> Failed tests: 
>   TestListenSMTP.ValidEmail:188 Sending email failed



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to