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

Daniel Stieglitz commented on NIFI-11130:
-----------------------------------------

[~exceptionfactory] I have made changes to replace the Groovy assert but there 
were also a few instances where Java unit tests were using assert so I replaced 
those also. I also noticed there were specific Spock unit tests which I am not 
familiar with and I was not sure how to refactor to Java. Those file are:
# 
nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/test/groovy/org/apache/nifi/util/SnippetUtilsSpec.groovy
# 
nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/test/groovy/org/apache/nifi/controller/PositionScalerSpec.groovy
# 
nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/groovy/org/apache/nifi/web/StandardNiFiServiceFacadeSpec.groovy
# 
nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/groovy/org/apache/nifi/web/dao/impl/StandardPolicyBasedAuthorizerDAOSpec.groovy
# 
nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/groovy/org/apache/nifi/web/dao/impl/StandardTemplateDAOSpec.groovy

Can that be left for NIFI-10047 ? 

> Refactor Groovy assert statements under nifi/nifi-nar-bundles to use JUnit 5 
> Assertions
> ---------------------------------------------------------------------------------------
>
>                 Key: NIFI-11130
>                 URL: https://issues.apache.org/jira/browse/NIFI-11130
>             Project: Apache NiFi
>          Issue Type: Sub-task
>            Reporter: Daniel Stieglitz
>            Assignee: Daniel Stieglitz
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> A regular expression search 
> {code:java}
> assert .*
> {code}
> in nifi/nifi-nar-bundles indicates there are over 100 Groovy assert 
> statements. 
> Refactor these to use JUnit 5 assertions.



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

Reply via email to