From 026de4f8c61bdf4d2101020e0e2bf5ad1701f720 Mon Sep 17 00:00:00 2001
From: Mathieu Delrocq <delrocq.mathieu@gmail.com>
Date: Wed, 22 Mar 2023 17:40:35 +0100
Subject: [PATCH] fix javadoc + format test

---
 .../signature/RpmFileSignatureProcessor.java  |  2 +-
 .../RpmFileSignatureProcessorTest.java        | 21 ++++++++++---------
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/rpm/src/main/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessor.java b/rpm/src/main/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessor.java
index b53b577..130f05b 100644
--- a/rpm/src/main/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessor.java
+++ b/rpm/src/main/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessor.java
@@ -43,7 +43,7 @@
 /**
  * 
  * Sign existing RPM file by calling
- * {@link #perform(InputStream, InputStream, String)}
+ * {@link #perform(File, InputStream, String, OutputStream)}
  * 
  *
  */
diff --git a/rpm/src/test/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessorTest.java b/rpm/src/test/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessorTest.java
index a20eee8..f894dc7 100644
--- a/rpm/src/test/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessorTest.java
+++ b/rpm/src/test/java/org/eclipse/packager/rpm/signature/RpmFileSignatureProcessorTest.java
@@ -29,7 +29,6 @@
 import org.bouncycastle.openpgp.bc.BcPGPPublicKeyRing;
 import org.eclipse.packager.rpm.RpmSignatureTag;
 import org.eclipse.packager.rpm.Rpms;
-import org.eclipse.packager.rpm.app.Dumper;
 import org.eclipse.packager.rpm.parse.InputHeader;
 import org.eclipse.packager.rpm.parse.RpmInputStream;
 import org.junit.jupiter.api.AfterAll;
@@ -42,6 +41,9 @@
 @TestMethodOrder(OrderAnnotation.class)
 public class RpmFileSignatureProcessorTest {
 
+    private static final String SOURCE_FILE_PATH = "src/test/resources/data/org.eclipse.scada-0.2.1-1.noarch.rpm";
+    private static final String PRIVATE_KEY_PATH = "src/test/resources/key/private_key.txt";
+    private static final String PUBLIC_KEY_PATH = "src/test/resources/key/public_key.txt";
     private static final String RESULT_FILE_PATH = "src/test/resources/result/org.eclipse.scada-0.2.1-1.noarch.rpm";
     private static final String RESULT_DIR = "src/test/resources/result";
 
@@ -49,9 +51,9 @@ public class RpmFileSignatureProcessorTest {
     @Order(1)
     public void testSigningExistingRpm() throws IOException, PGPException {
         // Read files
-        String passPhrase = "testkey";
-        File rpm = new File("src/test/resources/data/org.eclipse.scada-0.2.1-1.noarch.rpm");
-        File private_key = new File("src/test/resources/key/private_key.txt");
+        final String passPhrase = "testkey"; // Do not change
+        File rpm = new File(SOURCE_FILE_PATH);
+        File private_key = new File(PRIVATE_KEY_PATH);
         if (!rpm.exists() || !private_key.exists()) {
             fail("Input files rpm or private_key does not exist");
         }
@@ -69,14 +71,12 @@ public void testSigningExistingRpm() throws IOException, PGPException {
             // Read the initial (non signed) rpm file
             RpmInputStream initialRpm = new RpmInputStream(new FileInputStream(rpm));
             initialRpm.available();
-            System.out.println("#########################################################################");
-            Dumper.dumpAll(initialRpm);
             initialRpm.close();
-            System.out.println("#########################################################################");
             InputHeader<RpmSignatureTag> initialHeader = initialRpm.getSignatureHeader();
+
+            // Read the signed rpm file
             RpmInputStream rpmSigned = new RpmInputStream(new FileInputStream(signedRpm));
             rpmSigned.available();
-            Dumper.dumpAll(rpmSigned);
             rpmSigned.close();
             InputHeader<RpmSignatureTag> signedHeader = rpmSigned.getSignatureHeader();
 
@@ -86,7 +86,7 @@ public void testSigningExistingRpm() throws IOException, PGPException {
             String initialSha1 = initialHeader.getEntry(RpmSignatureTag.SHA1HEADER).get().getValue().toString();
             String initialMd5 = Rpms.dumpValue(initialHeader.getEntry(RpmSignatureTag.MD5).get().getValue());
 
-            // Read information of the signed rpm file
+            // Get informations of the signed rpm file
             int signedSize = (int) signedHeader.getEntry(RpmSignatureTag.SIZE).get().getValue();
             int signedPayloadSize = (int) signedHeader.getEntry(RpmSignatureTag.PAYLOAD_SIZE).get().getValue();
             String signedSha1 = signedHeader.getEntry(RpmSignatureTag.SHA1HEADER).get().getValue().toString();
@@ -98,6 +98,7 @@ public void testSigningExistingRpm() throws IOException, PGPException {
             assertEquals(initialPayloadSize, signedPayloadSize);
             assertEquals(initialSha1, signedSha1);
             assertEquals(initialMd5, signedMd5);
+
             // Verify if signature is present
             assertNotNull(pgpSignature);
         }
@@ -107,7 +108,7 @@ public void testSigningExistingRpm() throws IOException, PGPException {
     @Order(2)
     @Disabled
     public void verifyRpmSignature() throws IOException, PGPException {
-        File public_key = new File("src/test/resources/key/public_key.txt");
+        File public_key = new File(PUBLIC_KEY_PATH);
         File signedRpm = new File(RESULT_FILE_PATH);
         if (!public_key.exists() || !signedRpm.exists()) {
             fail("Input files signedRpm or public_key does not exist");