Remove redundant Verify() calls in scalar static tests + add a Multip…

…ly test to use r2
Autarkysoft · Jan 26, 2024 · 6b1adc7 · 6b1adc7
1 parent 95bfdd8
commit 6b1adc7
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/Src/Tests/Bitcoin/Cryptography/EllipticCurve/Scalar8x32Tests.cs b/Src/Tests/Bitcoin/Cryptography/EllipticCurve/Scalar8x32Tests.cs
@@ -1480,24 +1480,22 @@ public void Libsecp256k1_StaticTest(byte[] xba, byte[] yba, byte[] r1ba, byte[]
             Assert.False(overflow);
 
             Scalar8x32 z = x.Multiply(y);
-            Assert.True(z.Verify());
             Assert.Equal(r1, z);
 
             if (!y.IsZero)
             {
                 Scalar8x32 zz = y.Inverse();
-                Assert.True(zz.Verify());
                 Scalar8x32 zzv = y.InverseVar();
                 Assert.Equal(zzv, zz);
                 // x*y*y' == x*1 == x  (y' represents inverse of y)
                 z = z.Multiply(zz);
-                Assert.True(z.Verify());
                 Assert.Equal(x, z);
                 // y'*y == 1
                 zz = zz.Multiply(y);
-                Assert.True(zz.Verify());
                 Assert.Equal(Scalar8x32.One, zz);
             }
+            z = x.Multiply(x);
+            Assert.Equal(r2, z);
         }