Add tests for constructor checker

scripts/test-setup.sh

@@ -6,7 +6,7 @@ trap 'for killable in $(jobs -p); do kill -9 $killable; done' EXIT
 
 # setup example repo
 rm -rf starknet-hardhat-example
-EXAMPLE_REPO_BRANCH="plugin"
+EXAMPLE_REPO_BRANCH="constructor-checker"
 if [[ "$CIRCLE_BRANCH" == "master" ]] && [[ "$EXAMPLE_REPO_BRANCH" != "plugin" ]]; then
     echo "Invalid example repo branch: $EXAMPLE_REPO_BRANCH"
     exit 1

test/general-tests/constructor-test/check.ts

@@ -1,4 +1,33 @@
-import { hardhatStarknetCompileDeprecated, hardhatStarknetTest } from "../../utils/cli-functions";
+import path from "path";
+import {
+    hardhatStarknetCompile,
+    hardhatStarknetCompileDeprecated,
+    hardhatStarknetTest
+} from "../../utils/cli-functions";
+import { copyFileSync } from "fs";
+import { assertContains, rmrfSync } from "../../utils/utils";
+
+const prefix = path.join(__dirname);
+const contract1 = "duplicate_constructor.cairo";
+const contract1Path = path.join("cairo1-contracts", contract1);
+
+const contract2 = "no_constructor.cairo";
+const contract2Path = path.join("cairo1-contracts", contract2);
+
+const contract3 = "mute_constructor.cairo";
+const contract3Path = path.join("cairo1-contracts", contract3);
+
+copyFileSync(path.join(prefix, contract1), contract1Path);
+copyFileSync(path.join(prefix, contract2), contract2Path);
+copyFileSync(path.join(prefix, contract3), contract3Path);
+
+const exptectedErrorMsg = "Error: Expected at most one constructor.";
+const execution = hardhatStarknetCompile([contract1Path], true);
+assertContains(execution.stderr, exptectedErrorMsg);
+rmrfSync(contract1Path);
+
+// Compile cairo1 contracts
+hardhatStarknetCompile(["cairo1-contracts/", "--add-pythonic-hints"]);
 
 hardhatStarknetCompileDeprecated(
     "contracts/contract.cairo contracts/simple_storage.cairo contracts/empty_constructor.cairo".split(

test/general-tests/constructor-test/duplicate_constructor.cairo

@@ -0,0 +1,26 @@
+#[contract]
+mod Contract {
+    struct Storage {
+        balance: felt252,
+    }
+
+    #[constructor]
+    fn constructor(initial_balance: felt252) {
+        balance::write(initial_balance);
+    }
+
+    #[constructor]
+    fn constructor2(initial_balance: felt252) {
+        balance::write(initial_balance);
+    }
+
+    #[external]
+    fn increase_balance(amount1: felt252, amount2: felt252) {
+        balance::write(balance::read() + amount1 + amount2);
+    }
+
+    #[view]
+    fn get_balance() -> felt252 {
+        balance::read()
+    }
+}

test/general-tests/constructor-test/mute_constructor.cairo

@@ -0,0 +1,21 @@
+#[contract]
+mod Contract {
+    struct Storage {
+        balance: felt252,
+    }
+
+    // #[constructor]
+    fn constructor(initial_balance: felt252) {
+        balance::write(initial_balance);
+    }
+
+    #[external]
+    fn increase_balance(amount1: felt252, amount2: felt252) {
+        balance::write(balance::read() + amount1 + amount2);
+    }
+
+    #[view]
+    fn get_balance() -> felt252 {
+        balance::read()
+    }
+}

test/general-tests/constructor-test/no_constructor.cairo

@@ -0,0 +1,16 @@
+#[contract]
+mod Contract {
+    struct Storage {
+        balance: felt252,
+    }
+
+    #[external]
+    fn increase_balance(amount1: felt252, amount2: felt252) {
+        balance::write(balance::read() + amount1 + amount2);
+    }
+
+    #[view]
+    fn get_balance() -> felt252 {
+        balance::read()
+    }
+}