hashgraph / hedera-smart-contracts

Contains Hedera Smart Contract Service supporting files
Apache License 2.0
37 stars 51 forks source link

test: ci migration testing #733

Closed natanasow closed 1 month ago

natanasow commented 2 months ago

Description:

Add a CI pipeline that uses hardcoded service's versions for pre/post migration images, and execute the following flow:

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

github-actions[bot] commented 2 months ago

Test Results

  14 files  ±  0    55 suites   - 17   4m 41s :stopwatch: - 3m 44s 146 tests  - 96  130 :heavy_check_mark:  - 106  4 :zzz:  - 2  12 :x: +12  202 runs   - 40  186 :heavy_check_mark:  -   50  4 :zzz:  - 2  12 :x: +12 

For more details on these failures, see this check.

Results for commit 1566088c. ± Comparison against base commit eb00d80e.

This pull request removes 107 and adds 11 tests. Note that renamed tests count towards both. ``` should NOT be able to use transferFrom on NFT tokens without approval ‑ TokenTransferContract Test Suite should NOT be able to use transferFrom on NFT tokens without approval should NOT be able to use transferFrom on fungible tokens without approval ‑ TokenTransferContract Test Suite should NOT be able to use transferFrom on fungible tokens without approval should be able to approve another account ‑ IERC20 Test Suite should be able to approve another account should be able to burn token ‑ TokenManagmentContract Test Suite should be able to burn token should be able to change ADMIN key to ECDSA_secp256k and perform admin action with same contract ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change ADMIN key to ECDSA_secp256k and perform admin action with same contract should be able to change ADMIN key to contractId and perform admin action with same contract ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to ECDSA_secp256k Positive should be able to change ADMIN key to contractId and perform admin action with same contract should be able to change FREEZE key to ECDSA_secp256k and freeze the token with the same account ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change FREEZE key to ECDSA_secp256k and freeze the token with the same account should be able to change FREEZE key to contractId and freeze the token with same contract ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to ECDSA_secp256k Positive should be able to change FREEZE key to contractId and freeze the token with same contract should be able to change PAUSE key to ECDSA_secp256k and pause the token with the same account ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId Positive should be able to change PAUSE key to ECDSA_secp256k and pause the token with the same account should be able to change PAUSE key to contractId and pause the token with same contract ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to ECDSA_secp256k Positive should be able to change PAUSE key to contractId and pause the token with same contract … ``` ``` "before all" hook for "should NOT be able to use transferFrom on fungible tokens without approval" ‑ TokenTransferContract Test Suite "before all" hook for "should NOT be able to use transferFrom on fungible tokens without approval" "before all" hook for "should be able to delete token" ‑ TokenManagmentContract Test Suite "before all" hook for "should be able to delete token" "before all" hook for "should be able to execute burnToken" ‑ TokenCreateContract Test Suite "before all" hook for "should be able to execute burnToken" "before all" hook for "should be able to execute name()" ‑ @OZERC20 Test Suite "before all" hook for "should be able to execute name()" "before all" hook for "should be able to get token name" ‑ IERC20 Test Suite "before all" hook for "should be able to get token name" "before all" hook for "should check if create/mint the erc20 tokens happened when contract created" ‑ @OZERC20Votes Test Suite "before all" hook for "should check if create/mint the erc20 tokens happened when contract created" "before all" hook for "should deposit, buy and sell tokens from ExchangeV1" ‑ Proxy Upgrade Contracts Test Suite DEX Upgradable Contract Test Suite "before all" hook for "should deposit, buy and sell tokens from ExchangeV1" "before all" hook for "should test that hollow account is created and the amount of fungible tokens is correctly transferred via precompile" ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Fungible Token Test "before all" hook for "should test that hollow account is created and the amount of fungible tokens is correctly transferred via precompile" "before all" hook for "should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile" ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Non-Fungible Token Test "before all" hook for "should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile" "before all" hook in "@OZERC20Extensions Test Suite" ‑ @OZERC20Extensions Test Suite "before all" hook in "@OZERC20Extensions Test Suite" … ```

:recycle: This comment has been updated with latest results.