hashgraph / hedera-smart-contracts

Contains Hedera Smart Contract Service supporting files
Apache License 2.0
39 stars 53 forks source link

chore: cleaned up and reorganized smart contracts and tests folders (#624) #626

Closed quiet-node closed 10 months ago

quiet-node commented 10 months ago

Description: This PR mainly cleans up and reorganizes the folders. Below is a list of tasks done:

Related issue(s):

Fixes #624

Notes for reviewer:

Checklist

github-actions[bot] commented 10 months ago

Test Results

  19 files  +  6    88 suites  +19   11m 11s :stopwatch: + 3m 7s 240 tests +  5  231 :heavy_check_mark: +  2  6 :zzz: ±0  3 :x: +3  297 runs  +62  284 :heavy_check_mark: +55  8 :zzz: +2  5 :x: +5 

For more details on these failures, see this check.

Results for commit d0cda6ae. ± Comparison against base commit ee8db364.

This pull request removes 82 and adds 87 tests. Note that renamed tests count towards both. ``` Should NOT allow a non-operator to transfer tokens to another account ‑ @OZERC1155 Upgradable Vaults Tests Should NOT allow a non-operator to transfer tokens to another account Should NOT be able to change the current proxy admin if the caller is not an admin ‑ @ERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should NOT be able to change the current proxy admin if the caller is not an admin Should NOT burn insufficient amount of token ‑ @OZERC1155 Upgradable Vaults Tests Should NOT burn insufficient amount of token Should NOT transfer the ownership to another account if the caller is not owner ‑ @OZERC1155 Upgradable Vaults Tests Should NOT transfer the ownership to another account if the caller is not owner Should allow an operator to transfer a token to another account ‑ @OZERC1155 Upgradable Vaults Tests Should allow an operator to transfer a token to another account Should allow an operator to transfer tokens in batch to another account ‑ @OZERC1155 Upgradable Vaults Tests Should allow an operator to transfer tokens in batch to another account Should be able to change the current proxy admin to a new address ‑ @ERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to change the current proxy admin to a new address Should be able to get the predefined ERC1967 ADMIN_SLOT ‑ @ERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to get the predefined ERC1967 ADMIN_SLOT Should be able to get the predefined ERC1967 IMPLEMENTATION_SLOT ‑ @ERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to get the predefined ERC1967 IMPLEMENTATION_SLOT Should be able to mint a new token ‑ @OZERC1155 Upgradable Vaults Tests Should be able to mint a new token … ``` ``` Should NOT allow a non-operator to transfer tokens to another account ‑ @OZERC1155 Tests Should NOT allow a non-operator to transfer tokens to another account Should NOT be able to change the current proxy admin if the caller is not an admin ‑ @OZERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should NOT be able to change the current proxy admin if the caller is not an admin Should NOT burn insufficient amount of token ‑ @OZERC1155 Tests Should NOT burn insufficient amount of token Should NOT transfer the ownership to another account if the caller is not owner ‑ @OZERC1155 Tests Should NOT transfer the ownership to another account if the caller is not owner Should allow an operator to transfer a token to another account ‑ @OZERC1155 Tests Should allow an operator to transfer a token to another account Should allow an operator to transfer tokens in batch to another account ‑ @OZERC1155 Tests Should allow an operator to transfer tokens in batch to another account Should be able to change the current proxy admin to a new address ‑ @OZERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to change the current proxy admin to a new address Should be able to get the predefined ERC1967 ADMIN_SLOT ‑ @OZERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to get the predefined ERC1967 ADMIN_SLOT Should be able to get the predefined ERC1967 IMPLEMENTATION_SLOT ‑ @OZERC1967Upgrade Upgradable Vote Tests Proxy Contract tests Should be able to get the predefined ERC1967 IMPLEMENTATION_SLOT Should be able to mint a new token ‑ @OZERC1155 Tests Should be able to mint a new token … ```

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