Skip to content

Commit

Permalink
test: add necessary tests
Browse files Browse the repository at this point in the history
  • Loading branch information
YamenMerhi committed Sep 4, 2023
1 parent 628f24d commit d6d4e5a
Show file tree
Hide file tree
Showing 8 changed files with 738 additions and 3 deletions.
11 changes: 11 additions & 0 deletions contracts/Mocks/FallbackExtensions/Buy.sol
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
// SPDX-License-Identifier: Apache-2.0
pragma solidity ^0.8.4;

/**
* @dev This contract is used only for testing purposes
*/
contract BuyExtension {
function buy() public payable returns (uint256) {
return address(this).balance;
}
}
2 changes: 1 addition & 1 deletion contracts/Mocks/FallbackExtensions/CheckerExtension.sol
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ contract CheckerExtension {
function checkMsgVariable(
address originalMsgSender,
uint256 originalMsgValue
) public pure returns (bool) {
) public payable returns (bool) {
if (msg.data.length != 120) revert();
if (
originalMsgSender !=
Expand Down
4 changes: 2 additions & 2 deletions tests/LSP17ContractExtension/LSP17Extendable.behaviour.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@ import { ERC725YDataKeys } from '../../constants';

export type LSP17TestContext = {
accounts: SignerWithAddress[];
contract: LSP0ERC725Account | LSP9Vault;
deployParams: { owner: SignerWithAddress };
contract: LSP0ERC725Account | LSP9Vault | any;
deployParams: any;
};

export const shouldBehaveLikeLSP17 = (buildContext: () => Promise<LSP17TestContext>) => {
Expand Down
Loading

0 comments on commit d6d4e5a

Please sign in to comment.