test(cli): exclude

wevm · Jul 8, 2024 · 12d7319 · 12d7319
1 parent 2010b9f
commit 12d7319
Show file tree

Hide file tree

Showing 5 changed files with 35 additions and 8 deletions.
diff --git a/.changeset/five-mice-kiss.md b/.changeset/five-mice-kiss.md
@@ -2,4 +2,4 @@
 "@wagmi/cli": patch
 ---
 
-Fixed an issue that made the exclude options of the hardhat & Foundry plugins ignored
+Fixed an issue where the Foundry and Hardhat plugins' `exclude` option was ignored.
diff --git a/packages/cli/src/plugins/__fixtures__/foundry/src/Foo.sol b/packages/cli/src/plugins/__fixtures__/foundry/src/Foo.sol
@@ -0,0 +1,11 @@
+// SPDX-License-Identifier: UNLICENSED
+pragma solidity ^0.8.13;
+
+contract Foo {
+  string public bar;
+
+  function setFoo(string memory baz) public {
+    bar = baz;
+  }
+}
+
diff --git a/packages/cli/src/plugins/__fixtures__/hardhat/contracts/Foo.sol b/packages/cli/src/plugins/__fixtures__/hardhat/contracts/Foo.sol
@@ -0,0 +1,10 @@
+// SPDX-License-Identifier: UNLICENSED
+pragma solidity ^0.8.13;
+
+contract Foo {
+  string public bar;
+
+  function setFoo(string memory baz) public {
+    bar = baz;
+  }
+}
diff --git a/packages/cli/src/plugins/foundry.test.ts b/packages/cli/src/plugins/foundry.test.ts
@@ -12,7 +12,7 @@ afterEach(() => {
 
 test('forge not installed', async () => {
   const dir = f.temp()
-  await expect(
+  expect(
     foundry({
       project: dir,
       forge: {
@@ -44,13 +44,14 @@ test('validates without project', async () => {
   const spy = vi.spyOn(process, 'cwd')
   spy.mockImplementation(() => dir)
 
-  await expect(foundry().validate()).resolves.toBeUndefined()
+  expect(foundry().validate()).resolves.toBeUndefined()
 })
 
-test('contracts', async () => {
-  await expect(
+test('contracts', () => {
+  expect(
     foundry({
       project: resolve(__dirname, '__fixtures__/foundry/'),
+      exclude: ['Foo.sol/**'],
     }).contracts(),
   ).resolves.toMatchInlineSnapshot(`
       [
@@ -102,7 +103,11 @@ test('contracts without project', async () => {
   const spy = vi.spyOn(process, 'cwd')
   spy.mockImplementation(() => dir)
 
-  await expect(foundry().contracts()).resolves.toMatchInlineSnapshot(`
+  expect(
+    foundry({
+      exclude: ['Foo.sol/**'],
+    }).contracts(),
+  ).resolves.toMatchInlineSnapshot(`
       [
         {
           "abi": [

diff --git a/packages/cli/src/plugins/hardhat.test.ts b/packages/cli/src/plugins/hardhat.test.ts
@@ -12,7 +12,7 @@ afterEach(() => {
 
 test('validate', async () => {
   const temp = f.temp()
-  await expect(
+  expect(
     hardhat({ project: temp }).validate(),
   ).rejects.toThrowErrorMatchingInlineSnapshot(
     '[Error: hardhat must be installed to use Hardhat plugin.]',
@@ -34,9 +34,10 @@ test('project does not exist', async () => {
 })
 
 test('contracts', async () => {
-  await expect(
+  expect(
     hardhat({
       project: resolve(__dirname, '__fixtures__/hardhat/'),
+      exclude: ['Foo.sol/**'],
     }).contracts(),
   ).resolves.toMatchInlineSnapshot(`
       [