You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by "Daniel Stieglitz (Jira)" <ji...@apache.org> on 2022/12/14 15:59:00 UTC

[jira] [Comment Edited] (NIFI-10941) Remove TestNG

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

Daniel Stieglitz edited comment on NIFI-10941 at 12/14/22 3:58 PM:
-------------------------------------------------------------------

[~exceptionfactory] I am not sure you saw my comment on the PR yesterday but I noticed yesterday that ImportFlowIt on line 59 still has
{code:java}
import org.junit.Test;{code}
instead of
{code:java}
import org.junit.jupiter.api.Test;{code}


was (Author: JIRAUSER294662):
[~exceptionfactory] I am not sure you saw my comment on the PR yesterday but I noticed yesterday that ImportFlowIt on line 59 still has

{{}}
{code:java}
import org.junit.Test;{code}
{{}}

instead of

{{}}
{code:java}
import org.junit.jupiter.api.Test;{code}
{{}}

> Remove TestNG
> -------------
>
>                 Key: NIFI-10941
>                 URL: https://issues.apache.org/jira/browse/NIFI-10941
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework, Tools and Build
>            Reporter: David Handermann
>            Assignee: David Handermann
>            Priority: Minor
>             Fix For: 1.20.0, 1.19.1
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Several tests in {{nifi-framework-bundle}} reference TestNG assertions instead of JUnit assertions, and several {{nifi-toolkit}} modules include TestNG transitive dependencies through {{groovy-all}}. All test references should be converted to JUnit 5, and TestNG should be excluded from all modules to avoid inadvertent references.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)