build: adjust foundry build settings to prevent race condition when i…

…nstalling solc in CI

.github/workflows/foundry-tests.yml

@@ -24,7 +24,9 @@ jobs:
         run: npm ci
 
       - name: NPM build
-        run: npm run build
+        run: |
+          npm run build
+          npm run build:foundry
 
       - name: Run Foundry tests
         run: npm run test:foundry

foundry.toml

@@ -28,5 +28,5 @@ out = 'packages/lsp16-contracts/contracts/foundry_artifacts'
 
 [profile.lsp_smart_contracts]
 src = 'packages/lsp-smart-contracts/contracts'
-test = 'packages/lsp-smart-contracts/foundry'
+test = 'packages/lsp-smart-contracts/tests/foundry'
 out = 'packages/lsp-smart-contracts/contracts/foundry_artifacts'

package.json

@@ -28,6 +28,7 @@
   "scripts": {
     "preinstall": "npx --yes force-resolutions",
     "build": "turbo build",
+    "build:foundry": "turbo build:foundry",
     "build:js": "turbo build:js",
     "clean": "turbo clean",
     "format": "prettier --write .",

packages/lsp-smart-contracts/gasreport.ansi

@@ -0,0 +1,5 @@
+No files changed, compilation skipped
+
+No tests match the provided pattern:
+	no-match-test: `Skip`
+

packages/lsp-smart-contracts/package.json

@@ -51,6 +51,7 @@
   },
   "scripts": {
     "build": "hardhat compile --show-stack-traces",
+    "build:foundry": "forge build",
     "build:js": "unbuild",
     "build:docs": "hardhat dodoc && prettier -w ./docs",
     "clean": "hardhat clean && rm -Rf dist/",
@@ -59,7 +60,7 @@
     "lint:solidity": "solhint 'contracts/**/*.sol' && prettier --check 'contracts/**/*.sol'",
     "package": "hardhat prepare-package",
     "test": "hardhat test --no-compile tests/**/*.test.ts",
-    "test:foundry": "FOUNDRY_PROFILE=lsp_smart_contracts forge test --no-match-test Skip -vvv --gas-report > gasreport.ansi",
+    "test:foundry": "FOUNDRY_PROFILE=lsp_smart_contracts forge test --no-match-test Skip -vvv",
     "test:coverage": "hardhat coverage"
   },
   "dependencies": {

packages/lsp16-contracts/package.json

@@ -34,12 +34,13 @@
   ],
   "scripts": {
     "build": "hardhat compile --show-stack-traces",
+    "build:foundry": "forge build",
     "clean": "hardhat clean && rm -Rf dist/",
     "format": "prettier --write .",
     "lint": "eslint . --ext .ts,.js",
     "lint:solidity": "solhint 'contracts/**/*.sol' && prettier --check 'contracts/**/*.sol'",
     "test": "hardhat test --no-compile tests/*.test.ts",
-    "test:foundry": "forge test --no-match-test Skip -vvv",
+    "test:foundry": "FOUNDRY_PROFILE=lsp16 forge test --no-match-test Skip -vvv",
     "test:coverage": "hardhat coverage"
   },
   "dependencies": {

packages/lsp2-contracts/package.json

@@ -35,6 +35,7 @@
   ],
   "scripts": {
     "build": "hardhat compile --show-stack-traces",
+    "build:foundry": "forge build",
     "build:js": "unbuild",
     "clean": "hardhat clean && rm -Rf dist/",
     "format": "prettier --write .",

packages/lsp6-contracts/package.json

@@ -35,6 +35,7 @@
   ],
   "scripts": {
     "build": "hardhat compile --show-stack-traces",
+    "build:foundry": "forge build",
     "build:js": "unbuild",
     "clean": "hardhat clean && rm -Rf dist/",
     "format": "prettier --write .",

template/package.json

@@ -28,13 +28,14 @@
   ],
   "scripts": {
     "build": "hardhat compile --show-stack-traces",
+    "build:foundry": "forge build",
     "build:js": "unbuild",
     "clean": "hardhat clean && rm -Rf dist/",
     "format": "prettier --write .",
     "lint": "eslint . --ext .ts,.js",
     "lint:solidity": "solhint 'contracts/**/*.sol' && prettier --check 'contracts/**/*.sol'",
     "test": "hardhat test --no-compile tests/*.test.ts",
-    "test:foundry": "forge test --no-match-test Skip -vvv",
+    "test:foundry": "FOUNDRY_PROFILE=lspN forge test --no-match-test Skip -vvv",
     "test:coverage": "hardhat coverage"
   },
   "dependencies": {

turbo.json

@@ -7,6 +7,9 @@
       "outputs": ["**/artifacts/**", "types/**", "**/types/**"],
       "cache": true
     },
+    "build:foundry": {
+      "cache": true
+    },
     "build:js": {
       "outputs": ["**/dist/**"],
       "cache": true
@@ -29,6 +32,7 @@
       "cache": false
     },
     "test:foundry": {
+      "dependsOn": ["^build", "^build:foundry"],
       "cache": true
     }
   }