exceptionfactory commented on a change in pull request #5870:
URL: https://github.com/apache/nifi/pull/5870#discussion_r828497424
##########
File path:
nifi-toolkit/nifi-toolkit-tls/src/test/java/org/apache/nifi/toolkit/tls/util/TlsHelperTest.java
##########
@@ -108,37 +95,16 @@
private KeyPairGenerator keyPairGenerator;
- private KeyStore keyStore;
-
- private String password = "changeit";
-
- @Mock
- KeyStoreSpi keyStoreSpi;
+ private final String password = "changeit";
- @Mock
- Provider keyStoreProvider;
-
- @Mock
+ @Mock(lenient = true)
OutputStreamFactory outputStreamFactory;
- @Rule
- public TemporaryFolder tempFolder = new TemporaryFolder();
-
private ByteArrayOutputStream tmpFileOutputStream;
private File file;
- private static void setUnlimitedCrypto(boolean value) {
Review comment:
Good question, this change implies that unlimited strength cryptography
support is expected. Java 8 Update 171 introduced this by default, so there
should be a separate effort to remove all checks for unlimited strength
cryptography.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
