test: update for the new behaivior

lukso-network · Sep 27, 2023 · 9ef792e · 9ef792e
1 parent 8d247a8
commit 9ef792e
Show file tree

Hide file tree

Showing 3 changed files with 52 additions and 24 deletions.
diff --git a/tests/LSP17ContractExtension/LSP17ExtendableTokens.behaviour.ts b/tests/LSP17ContractExtension/LSP17ExtendableTokens.behaviour.ts
@@ -145,30 +145,6 @@ export const shouldBehaveLikeLSP17 = (buildContext: () => Promise<LSP17TestConte
       '00000000000000000000000000000000'; // zero padded
   });
 
-  describe('when calling the contract with empty calldata', () => {
-    describe('when making a call without any value', () => {
-      it('should revert', async () => {
-        await expect(
-          context.accounts[0].sendTransaction({
-            to: context.contract.address,
-          }),
-        ).to.be.revertedWithCustomError(context.contract, 'InvalidFunctionSelector');
-      });
-    });
-
-    describe('when making a call with sending value', () => {
-      it('should revert', async () => {
-        const amountSent = 200;
-        await expect(
-          context.accounts[0].sendTransaction({
-            to: context.contract.address,
-            value: amountSent,
-          }),
-        ).to.be.revertedWithCustomError(context.contract, 'InvalidFunctionSelector');
-      });
-    });
-  });
-
   describe('when calling the contract with calldata', () => {
     describe("when calling method that doesn't exist", () => {
       describe('when there is no extension for the function called', () => {

diff --git a/tests/LSP7DigitalAsset/LSP7DigitalAsset.behaviour.ts b/tests/LSP7DigitalAsset/LSP7DigitalAsset.behaviour.ts
@@ -2144,6 +2144,32 @@ export const shouldBehaveLikeLSP7 = (buildContext: () => Promise<LSP7TestContext
       });
     });
   });
+
+  describe('when calling the contract with empty calldata', () => {
+    describe('when making a call without any value', () => {
+      it('should revert', async () => {
+        await expect(
+          context.accounts.anyone.sendTransaction({
+            to: context.lsp7.address,
+          }),
+        )
+          .to.be.revertedWithCustomError(context.lsp7, 'InvalidFunctionSelector')
+          .withArgs('0x00000000');
+      });
+    });
+
+    describe('when making a call with sending value', () => {
+      it('should revert', async () => {
+        const amountSent = 200;
+        await expect(
+          context.accounts.anyone.sendTransaction({
+            to: context.lsp7.address,
+            value: amountSent,
+          }),
+        ).to.be.revertedWithCustomError(context.lsp7, 'LSP7TokenContractCannotHoldValue');
+      });
+    });
+  });
 };
 
 export type LSP7InitializeTestContext = {

diff --git a/tests/LSP8IdentifiableDigitalAsset/LSP8IdentifiableDigitalAsset.behaviour.ts b/tests/LSP8IdentifiableDigitalAsset/LSP8IdentifiableDigitalAsset.behaviour.ts
@@ -1765,6 +1765,32 @@ export const shouldBehaveLikeLSP8 = (buildContext: () => Promise<LSP8TestContext
       });
     });
   });
+
+  describe('when calling the contract with empty calldata', () => {
+    describe('when making a call without any value', () => {
+      it('should revert', async () => {
+        await expect(
+          context.accounts.anyone.sendTransaction({
+            to: context.lsp8.address,
+          }),
+        )
+          .to.be.revertedWithCustomError(context.lsp8, 'InvalidFunctionSelector')
+          .withArgs('0x00000000');
+      });
+    });
+
+    describe('when making a call with sending value', () => {
+      it('should revert', async () => {
+        const amountSent = 200;
+        await expect(
+          context.accounts.anyone.sendTransaction({
+            to: context.lsp8.address,
+            value: amountSent,
+          }),
+        ).to.be.revertedWithCustomError(context.lsp8, 'LSP8TokenContractCannotHoldValue');
+      });
+    });
+  });
 };
 
 export type LSP8InitializeTestContext = {