andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Rename testmethods to make these more clear (#10231)

Browse Source 
Motivation:

The currently used method names don't make a lot of sense.

Modifications:

Rename to cleanup

Result:

Cleanup
This commit is contained in:
Norman Maurer 2020-04-29 17:52:43 +02:00
parent f22993a530
commit f23c33822c
5 changed files with 15 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
handler/src/test/java/io/netty/handler/ssl/JdkSslClientContextTest.java
View File

@ -20,7 +20,7 @@ import java.io.File;
public class JdkSslClientContextTest extends SslContextTest {
@Override
protected SslContext newServerContext(File crtFile, File keyFile, String pass) throws SSLException {
protected SslContext newSslContext(File crtFile, File keyFile, String pass) throws SSLException {
return SslContextBuilder.forClient()
.sslProvider(SslProvider.JDK)
.keyManager(crtFile, keyFile, pass)

2
handler/src/test/java/io/netty/handler/ssl/JdkSslServerContextTest.java
View File

@ -21,7 +21,7 @@ import java.io.File;
public class JdkSslServerContextTest extends SslContextTest {
@Override
protected SslContext newServerContext(File crtFile, File keyFile, String pass) throws SSLException {
protected SslContext newSslContext(File crtFile, File keyFile, String pass) throws SSLException {
return SslContextBuilder.forServer(crtFile, keyFile, pass).sslProvider(SslProvider.JDK).build();
}
}

2
handler/src/test/java/io/netty/handler/ssl/OpenSslClientContextTest.java
View File

@ -31,7 +31,7 @@ public class OpenSslClientContextTest extends SslContextTest {
}
@Override
protected SslContext newServerContext(File crtFile, File keyFile, String pass) throws SSLException {
protected SslContext newSslContext(File crtFile, File keyFile, String pass) throws SSLException {
return SslContextBuilder.forClient()
.sslProvider(SslProvider.OPENSSL)
.trustManager(InsecureTrustManagerFactory.INSTANCE)

2
handler/src/test/java/io/netty/handler/ssl/OpenSslServerContextTest.java
View File

@ -32,7 +32,7 @@ public class OpenSslServerContextTest extends SslContextTest {
}
@Override
protected SslContext newServerContext(File crtFile, File keyFile, String pass) throws SSLException {
protected SslContext newSslContext(File crtFile, File keyFile, String pass) throws SSLException {
Assume.assumeTrue(OpenSsl.isAvailable());
return SslContextBuilder.forServer(crtFile, keyFile, pass).sslProvider(SslProvider.OPENSSL).build();
}

22
handler/src/test/java/io/netty/handler/ssl/SslContextTest.java
View File

@ -64,35 +64,35 @@ public abstract class SslContextTest {
}
@Test
public void testSslServerWithEncryptedPrivateKey() throws SSLException {
public void testSslContextWithEncryptedPrivateKey() throws SSLException {
File keyFile = ResourcesUtil.getFile(getClass(), "test_encrypted.pem");
File crtFile = ResourcesUtil.getFile(getClass(), "test.crt");
newServerContext(crtFile, keyFile, "12345");
newSslContext(crtFile, keyFile, "12345");
}
@Test
public void testSslServerWithEncryptedPrivateKey2() throws SSLException {
public void testSslContextWithEncryptedPrivateKey2() throws SSLException {
File keyFile = ResourcesUtil.getFile(getClass(), "test2_encrypted.pem");
File crtFile = ResourcesUtil.getFile(getClass(), "test2.crt");
newServerContext(crtFile, keyFile, "12345");
newSslContext(crtFile, keyFile, "12345");
}
@Test
public void testSslServerWithUnencryptedPrivateKey() throws SSLException {
public void testSslContextWithUnencryptedPrivateKey() throws SSLException {
File keyFile = ResourcesUtil.getFile(getClass(), "test_unencrypted.pem");
File crtFile = ResourcesUtil.getFile(getClass(), "test.crt");
newServerContext(crtFile, keyFile, null);
newSslContext(crtFile, keyFile, null);
}
@Test(expected = IllegalArgumentException.class)
public void testSslServerWithUnencryptedPrivateKeyEmptyPass() throws SSLException {
public void testSslContextWithUnencryptedPrivateKeyEmptyPass() throws SSLException {
File keyFile = ResourcesUtil.getFile(getClass(), "test_unencrypted.pem");
File crtFile = ResourcesUtil.getFile(getClass(), "test.crt");
newServerContext(crtFile, keyFile, "");
newSslContext(crtFile, keyFile, "");
}
@Test
@ -112,14 +112,14 @@ public abstract class SslContextTest {
File keyFile = ResourcesUtil.getFile(getClass(), "test_unencrypted.pem");
File crtFile = ResourcesUtil.getFile(getClass(), "test.crt");
SslContext sslContext = newServerContext(crtFile, keyFile, null);
SslContext sslContext = newSslContext(crtFile, keyFile, null);
assertFalse(sslContext.cipherSuites().contains(unsupportedCipher));
}
@Test(expected = CertificateException.class)
public void test() throws CertificateException {
public void testUnsupportedParams() throws CertificateException {
SslContext.toX509Certificates(new File(getClass().getResource("ec_params_unsupported.pem").getFile()));
}
protected abstract SslContext newServerContext(File crtFile, File keyFile, String pass) throws SSLException;
protected abstract SslContext newSslContext(File crtFile, File keyFile, String pass) throws SSLException;
}