Skip to content

Commit 3727805

Browse files
committed
Allow CRLF line endings in some PGP tests
1 parent 1545fc7 commit 3727805

File tree

3 files changed

+27
-22
lines changed

3 files changed

+27
-22
lines changed

pg/src/test/java/org/bouncycastle/openpgp/api/test/OpenPGPDetachedSignatureProcessorTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import org.bouncycastle.openpgp.api.SignatureParameters;
2323
import org.bouncycastle.openpgp.api.SignatureSubpacketsFunction;
2424
import org.bouncycastle.openpgp.operator.PGPKeyPairGenerator;
25+
import org.bouncycastle.util.Strings;
2526

2627
public class OpenPGPDetachedSignatureProcessorTest
2728
extends APITest
@@ -63,7 +64,7 @@ private void createVerifyV4Signature(OpenPGPApi api)
6364
OpenPGPSignature.OpenPGPDocumentSignature signature = signatures.get(0);
6465
isEquals(4, signature.getSignature().getVersion());
6566
String armored = signature.toAsciiArmoredString();
66-
isTrue(armored.startsWith("-----BEGIN PGP SIGNATURE-----\n"));
67+
isTrue(armored.startsWith("-----BEGIN PGP SIGNATURE-----" + Strings.lineSeparator()));
6768

6869
// Verify detached signatures
6970
OpenPGPDetachedSignatureProcessor processor = api.verifyDetachedSignature();
@@ -90,7 +91,7 @@ private void createVerifyV6Signature(OpenPGPApi api)
9091
OpenPGPSignature.OpenPGPDocumentSignature signature = signatures.get(0);
9192
isEquals(6, signature.getSignature().getVersion());
9293
String armored = signature.toAsciiArmoredString();
93-
isTrue(armored.startsWith("-----BEGIN PGP SIGNATURE-----\n"));
94+
isTrue(armored.startsWith("-----BEGIN PGP SIGNATURE-----" + Strings.lineSeparator()));
9495

9596
// Verify detached signatures
9697
OpenPGPDetachedSignatureProcessor processor = api.verifyDetachedSignature();

pg/src/test/java/org/bouncycastle/openpgp/api/test/OpenPGPMessageGeneratorTest.java

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
import org.bouncycastle.openpgp.api.OpenPGPKey;
1414
import org.bouncycastle.openpgp.api.OpenPGPMessageGenerator;
1515
import org.bouncycastle.openpgp.api.OpenPGPMessageOutputStream;
16+
import org.bouncycastle.util.Strings;
1617
import org.bouncycastle.util.encoders.Hex;
1718

1819
public class OpenPGPMessageGeneratorTest
@@ -52,12 +53,13 @@ private void armoredLiteralDataPacket(OpenPGPApi api)
5253

5354
msgOut.close();
5455

55-
isEquals(
56-
"-----BEGIN PGP MESSAGE-----\n" +
57-
"\n" +
58-
"yxNiAAAAAABIZWxsbywgV29ybGQh\n" +
59-
"-----END PGP MESSAGE-----\n",
60-
bOut.toString());
56+
String nl = Strings.lineSeparator();
57+
String expected =
58+
"-----BEGIN PGP MESSAGE-----" + nl +
59+
nl +
60+
"yxNiAAAAAABIZWxsbywgV29ybGQh" + nl +
61+
"-----END PGP MESSAGE-----" + nl;
62+
isEquals(expected, bOut.toString());
6163
}
6264

6365
private void unarmoredLiteralDataPacket(OpenPGPApi api)
@@ -93,11 +95,13 @@ private void armoredCompressedLiteralDataPacket(OpenPGPApi api)
9395

9496
msgOut.close();
9597

96-
isEquals("-----BEGIN PGP MESSAGE-----\n" +
97-
"\n" +
98-
"yBUBOy2cxAACHqk5Ofk6CuH5RTkpigA=\n" +
99-
"-----END PGP MESSAGE-----\n",
100-
bOut.toString());
98+
String nl = Strings.lineSeparator();
99+
String expected =
100+
"-----BEGIN PGP MESSAGE-----" + nl +
101+
nl +
102+
"yBUBOy2cxAACHqk5Ofk6CuH5RTkpigA=" + nl +
103+
"-----END PGP MESSAGE-----" + nl;
104+
isEquals(expected, bOut.toString());
101105
}
102106

103107
private void unarmoredCompressedLiteralDataPacket(OpenPGPApi api)

pg/src/test/java/org/bouncycastle/openpgp/test/ArmorCRCTest.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,17 +19,17 @@
1919
public class ArmorCRCTest
2020
extends SimpleTest
2121
{
22-
22+
private static final String NL = Strings.lineSeparator();
2323
private static final String WITHOUT_CRC = "" +
24-
"-----BEGIN PGP MESSAGE-----\n" +
25-
"\n" +
26-
"yxR0AAAAAABIZWxsbywgV29ybGQhCg==\n" +
27-
"-----END PGP MESSAGE-----\n";
24+
"-----BEGIN PGP MESSAGE-----" + NL +
25+
NL +
26+
"yxR0AAAAAABIZWxsbywgV29ybGQhCg==" + NL +
27+
"-----END PGP MESSAGE-----" + NL;
2828
private static final String FAULTY_CRC = "" +
29-
"-----BEGIN PGP MESSAGE-----\n" +
30-
"\n" +
31-
"yxR0AAAAAABIZWxsbywgV29ybGQhCg==\n" +
32-
"=TRA9\n" +
29+
"-----BEGIN PGP MESSAGE-----" + NL +
30+
NL +
31+
"yxR0AAAAAABIZWxsbywgV29ybGQhCg==" + NL +
32+
"=TRA9" + NL +
3333
"-----END PGP MESSAGE-----";
3434

3535
@Override

0 commit comments

Comments
 (0)