moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

Try to catch more network-unavailable cases in TestSignatureInfo

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1742422 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Dominik Stadler 2016-05-05 12:22:23 +00:00
parent 26de463419
commit 56c5bd05e1
1 changed files with 37 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/ooxml/testcases/org/apache/poi/poifs/crypt/TestSignatureInfo.java
View File

@ -36,10 +36,7 @@ import java.io.FileOutputStream;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream; import java.io.OutputStream;
import java.net.ConnectException; import java.net.*;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.URL;
import java.security.Key; import java.security.Key;
import java.security.KeyPair; import java.security.KeyPair;
import java.security.KeyStore; import java.security.KeyStore;
@ -105,6 +102,7 @@ public class TestSignatureInfo {
/*** TODO : set cal to now ... only set to fixed date for debugging ... */ /*** TODO : set cal to now ... only set to fixed date for debugging ... */
cal = LocaleUtil.getLocaleCalendar(LocaleUtil.TIMEZONE_UTC); cal = LocaleUtil.getLocaleCalendar(LocaleUtil.TIMEZONE_UTC);
assertNotNull(cal);
// cal.set(2014, 7, 6, 21, 42, 12); // cal.set(2014, 7, 6, 21, 42, 12);
// cal.clear(Calendar.MILLISECOND); // cal.clear(Calendar.MILLISECOND);
@ -390,10 +388,11 @@ public class TestSignatureInfo {
if(e.getCause() == null) { if(e.getCause() == null) {
throw e; throw e;
} }
if(!(e.getCause() instanceof ConnectException)) { if(!(e.getCause() instanceof ConnectException) && !(e.getCause() instanceof SocketTimeoutException)) {
throw e; throw e;
} }
assertTrue("Only allowing ConnectException with 'timed out' as message here, but had: " + e, e.getCause().getMessage().contains("timed out")); assertTrue("Only allowing ConnectException with 'timed out' as message here, but had: " + e,
e.getCause().getMessage().contains("timed out"));
} }
// verify // verify
@ -584,17 +583,20 @@ public class TestSignatureInfo {
@Ignore @Ignore
public void testMultiSign() throws Exception { public void testMultiSign() throws Exception {
initKeyPair("KeyA", "CN=KeyA"); initKeyPair("KeyA", "CN=KeyA");
KeyPair keyPairA = keyPair; //KeyPair keyPairA = keyPair;
X509Certificate x509A = x509; //X509Certificate x509A = x509;
initKeyPair("KeyB", "CN=KeyB"); initKeyPair("KeyB", "CN=KeyB");
KeyPair keyPairB = keyPair; //KeyPair keyPairB = keyPair;
X509Certificate x509B = x509; //X509Certificate x509B = x509;
File tpl = copy(testdata.getFile("bug58630.xlsx")); File tpl = copy(testdata.getFile("bug58630.xlsx"));
OPCPackage pkg = OPCPackage.open(tpl); OPCPackage pkg = OPCPackage.open(tpl);
SignatureConfig signatureConfig = new SignatureConfig(); try {
//SignatureConfig signatureConfig = new SignatureConfig();
assertNotNull(pkg);
} finally {
pkg.close();
}
} }
private void sign(OPCPackage pkgCopy, String alias, String signerDn, int signerCount) throws Exception { private void sign(OPCPackage pkgCopy, String alias, String signerDn, int signerCount) throws Exception {