Skip to content

Commit

Permalink
Add tests to CI + small test fix
Browse files Browse the repository at this point in the history
  • Loading branch information
nmlinaric committed Oct 17, 2023
1 parent 6b7386d commit 6b48590
Show file tree
Hide file tree
Showing 2 changed files with 45 additions and 1 deletion.
43 changes: 43 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
name: Test

on:
pull_request:
types: [opened, synchronize, reopened]

jobs:
test:
name: Test
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x]
steps:
- name: Checkout code
uses: actions/[email protected]
- uses: actions/[email protected]
with:
path: ~/.npm
key: ${{ runner.os }}-node-${{ hashFiles('**/yarn.lock') }}
restore-keys: |
${{ runner.os }}-node-
- name: Install Node.js ${{ matrix.node-version }}
uses: actions/[email protected]
with:
node-version: ${{ matrix.node-version }}
- name: Install dependencies
run: yarn install
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
- name: Install OZ contracts
run: forge install OpenZeppelin/openzeppelin-contracts --no-commit
- name: Compile contracts
run: forge compile --hardhat
working-directory: ./sygma-x-solidity/
- name: Store contract artifacts
uses: actions/upload-artifact@v3
with:
name: contracts-artifacts
path: artifacts
- name: Hardhat Tests
run:
yarn run test
3 changes: 2 additions & 1 deletion test/e2e/erc20/decimals/roundingLoss.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,8 @@ describe("E2E ERC20 - Two EVM Chains both with decimal places != 18 with roundin
originDomainID,
destinationResourceID,
expectedDepositNonce,
destinationDepositData,
await recipientAccount.getAddress(),
destinationDepositData.toLowerCase(),
toHex(destinationRelayerConvertedAmount.toString(), 32),
);

Expand Down

0 comments on commit 6b48590

Please sign in to comment.