Include leading zeros in each byte of hex fingerprint output

rkluszczynski · Nov 8, 2017 · 5222725 · 5222725
1 parent f793b6d
commit 5222725
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/io/github/rkluszczynski/avro/cli/command/fingerprint/SchemaFingerprint.java b/src/main/java/io/github/rkluszczynski/avro/cli/command/fingerprint/SchemaFingerprint.java
@@ -33,7 +33,7 @@ public String execute(CliMainParameters mainParameters) {
 
             String fingerprintString = "";
             for (byte b : bytes) {
-                fingerprintString += String.format("%x", b);
+                fingerprintString += String.format("%02x", b);
             }
             return fingerprintString;
         } catch (NoSuchAlgorithmException e) {

diff --git a/src/test/groovy/io/github/rkluszczynski/avro/cli/command/FingerprintCommandTest.groovy b/src/test/groovy/io/github/rkluszczynski/avro/cli/command/FingerprintCommandTest.groovy
@@ -23,7 +23,7 @@ class FingerprintCommandTest extends BaseTestSpecification {
         algorithmName | fingerprintValue
         'CRC-64-AVRO' | '8db714e9ffa989a2'
         'MD5'         | 'f53ea32559669f2659949fe064c091c3'
-        'SHA-256'     | 'd8df4d61a04ac76369f63ef7861b98f9a9dddc28ef0d3d65326de5493c9'
+        'SHA-256'     | 'd8df4d61a004ac76369f63ef7861b908f90a9dddc28ef00d3d65326de50493c9'
     }
 
     def 'should calculate Rabin fingerprint'() {
@@ -34,7 +34,7 @@ class FingerprintCommandTest extends BaseTestSpecification {
         trimmedOutput() == '0918bd124db26deb'
     }
 
-    def 'should fail for no existing algorihtm'() {
+    def 'should fail for no existing algorithm'() {
         when:
         commandService.executeCommand('fingerprint', '-a', 'NOT-EXISTING-ALGORITHM',
                 '--schema', prepareSchemaPath('schema-no-fields.avsc')