Merge pull request #229 from oasisprotocol/CedarMist/hashtests-fix

contracts: hashing tests weren't running, which meant sha384 error wasn't caught
oasisprotocol · Nov 26, 2023 · 5f83294 · 5f83294
2 parents 9fdd2bb + 669d49a
commit 5f83294
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/contracts/contracts/Sapphire.sol b/contracts/contracts/Sapphire.sol
@@ -34,7 +34,7 @@ library Sapphire {
     address internal constant SHA512 =
         0x0100000000000000000000000000000000000102;
     address internal constant SHA384 =
-        0x0100000000000000000000000000000000000103;
+        0x0100000000000000000000000000000000000104;
 
     type Curve25519PublicKey is bytes32;
     type Curve25519SecretKey is bytes32;

diff --git a/contracts/test/hashes.ts b/contracts/test/hashes.ts
@@ -32,14 +32,14 @@ describe('Hashes', () => {
   }
 
   it('SHA512-256', async () => {
-    testHashes('SHA512-256', contract.testSHA512_256.bind(contract));
+    await testHashes('SHA512-256', contract.testSHA512_256.bind(contract));
   });
 
   it('SHA512', async () => {
-    testHashes('SHA512', contract.testSHA512.bind(contract));
+    await testHashes('SHA512', contract.testSHA512.bind(contract));
   });
 
   it('SHA384', async () => {
-    testHashes('SHA384', contract.testSHA384.bind(contract));
+    await testHashes('SHA384', contract.testSHA384.bind(contract));
   });
 });