mirror of https://github.com/apache/nifi.git
1d560e2b02
Fixed test failures in nifi-couchbase-processors, BinaryDocument matcher replaced with ByteArrayDocument Fixed test failures in nifi-riemann-processors, anyInt() matcher replaced with anyLong() matcher, calling method passes a long, not int Removed unnecessary method mocks from nifi-toolkit-tls tests, TlsCertificateAuthorityServiceHandlerTest and TlsCertificateSigningRequestPerformerTest, since those were flagged by Mockito as unnecessary (they're unused) Removed explicit mockito dependency version in nifi-gcp-processors pom to inherit version from nifi's pom.xml Updated ArgumentMatchers in Kafka 0.10, 0.11, 1.0, and 2.0 processor tests, since in Mockito 2.x, the "any" matchers no longer allow nulls Updated ArgumentMatchers in nifi-jolt-transform-json-ui, since in Mockito 2.x, the "any" matchers no longer allow nulls Removed unnecessary method mocks from MetricsReportingTaskTest Updated TestStandardRemoteGroupPort to return Long instead of Integer for test flowfile.size() invocations Updated AbstractCassandraProcessor to include keyspaceProperty.getValue() in null check Updated SimpleProcessLogger and TestSimpleProcessLogger, vararg matching does not work the same in Java 8 and 11 Updated TestStandardProcessScheduler to allow null values during mock invocations, Mockito 2.x no longer allows nulls in those matchers Updated TestPutHiveStreaming to allow null values during mock invocations, Mockito 2.x no longer allows nulls in those matchers Updated FetchParquetTest to allow null values during mock invocations, Mockito 2.x no longer allows nulls in those matchers Updated ControllerSearchServiceTest to allow null values during mock invocations, Mockito 2.x no longer allows nulls in those matchers Removed usage of Whitebox from GetAzureEventHubTest due to Mockito 2.x, replaced with FieldUtils Removed usage of Whitebox from StandardOidcIdentityProviderTest due to Mockito 2.x, replaced with FieldUtils Updated apache-rat-plugin configuration in root POM to make use of useIdeaDefaultExcludes which makes the rat plugin exclude IntelliJ artifacts Updated several modules to use mockito-core instead of mockito-all (discontinued in Mockito 2.x) Updated nifi-site-to-site-reporting-task tests to be compatible with Mockito 2.x Ignored TestPutJMS tests; the tests need to be refactored to work with Mockito 2.x, but the processor is deprecated. Refactor may be done in a separate PR. Adjusted several mock interaction iterations to 0 for TestPublishKafkaRecord_* tests. Mockito 2.x flagged several interactions as unused and were adjusted to 0 interactions. Updated PowerMock and Mockito dependencies to exclude transitive dependency on bytebuddy, added explicit dependency on bytebuddy 1.9.10 so that PowerMock and Mockito use the same version. Bytebuddy 1.9.3 (used by PowerMock 2.0.2) did not allow for the mocking of final/private classes, bytebuddy 1.9.10 (used by Mockito 2.28.2) does. Updated TestSiteToSiteProvenanceReportingTask use of InvocationOnMock.getArgument to use objects for the resulting object rather than primitives Removed unnecessary stubs from evtx tests, Mockito 2.x defaults to strict mocks Fixed classloader issue with tests in nifi-windows-event-log-processors module that use JNAJUnitRunner when Mockito mocked JNA classes (Kernel32) Addressed Mockito-related deprecation warnings Import cleanup This closes #3533 Signed-off-by: Mike Thomsen <mikerthomsen@gmail.com> |
||
|---|---|---|
| .. | ||
| nifi-toolkit-admin | ||
| nifi-toolkit-api | ||
| nifi-toolkit-assembly | ||
| nifi-toolkit-cli | ||
| nifi-toolkit-encrypt-config | ||
| nifi-toolkit-flowanalyzer | ||
| nifi-toolkit-flowfile-repo | ||
| nifi-toolkit-s2s | ||
| nifi-toolkit-tls | ||
| nifi-toolkit-zookeeper-migrator | ||
| pom.xml | ||