trying to fix ci

dl-solarity · Dec 30, 2024 · d866a18 · d866a18
1 parent d4c262d
commit d866a18
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 13 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -16,11 +16,24 @@ jobs:
     steps:
       - name: Checkout the repository
         uses: actions/checkout@v3
+
       - name: Setup
         uses: ./.github/actions/setup
-      - name: Compile
+
+      - name: Compile only
+        env:
+          NODE_OPTIONS: "--max_old_space_size=16384"
+        run: npx hardhat zkit compile
+
+      - name: Update hardhat.config
+        run: |
+        sed -i "/compilationSettings: {/a \ skipFiles:\n[\n\"mock\/ec\/scalarMult.circom\",\n\"mock\/ec\/scalarMultBrainpoolP256r1.circom\",\n\"mock\/signatures\/ecdsaNum.circom\",\n]," hardhat.config.ts
+        cat hardhat.config.ts
+
+      - name: Compile and setup
         env:
           NODE_OPTIONS: "--max_old_space_size=16384"
         run: npm run compile
+
       - name: Run tests
         run: npm run test
diff --git a/hardhat.config.ts b/hardhat.config.ts
@@ -41,14 +41,7 @@ const config: HardhatUserConfig = {
   },
   zkit: {
     compilationSettings: {
-      //onlyFiles: [],
-      skipFiles: [
-        "mock/ec/scalarMult.circom",
-        "mock/ec/scalarMultBrainpoolP256r1.circom",
-        "mock/signatures/ecdsaBits.circom",
-        "mock/signatures/ecdsaNum.circom",
-      ],
-    },
+    }
   },
 };
 

diff --git a/test/ec/EcMult.test.ts b/test/ec/EcMult.test.ts
@@ -258,7 +258,7 @@ describe("BrainpoolP256r1 generator multiplication test", function () {
   });
 });
 
-describe.skip("Scalar point multiplication test", function () {
+describe("Scalar point multiplication test", function () {
   //This circuit requires more than 16GB of RAM to generate the ZKey file,
   //so this test is limited to witness testing only
   this.timeout(10000000);
@@ -293,7 +293,7 @@ describe.skip("Scalar point multiplication test", function () {
   });
 });
 
-describe.skip("BrainpoolP256r1 scalar point multiplication test", function () {
+describe("BrainpoolP256r1 scalar point multiplication test", function () {
   //This circuit requires more than 16GB of RAM to generate the ZKey file,
   //so this test is limited to witness testing only
   this.timeout(10000000);

diff --git a/test/signatures/Ecdsa.test.ts b/test/signatures/Ecdsa.test.ts
@@ -142,7 +142,7 @@ async function testVerBits(
   return proofStruct;
 }
 
-describe.skip("Ecdsa num test", function () {
+describe("Ecdsa num test", function () {
   //This circuit requires more than 16GB of RAM to generate the ZKey file,
   //so this test is limited to witness testing only
   this.timeout(10000000);
@@ -181,7 +181,7 @@ describe.skip("Ecdsa num test", function () {
   });
 });
 
-describe.skip("Ecdsa bits test", function () {
+describe("Ecdsa bits test", function () {
   this.timeout(10000000);
   const reverter = new Reverter();