Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: included missing contract-abi artifacts #1014

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

quiet-node
Copy link
Member

Description:
chore: included missing contract-abi artifacts

Related issue(s):

Fixes #1013

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@quiet-node quiet-node added the internal For changes that affect the project's internal workings but not its outward-facing functionality. label Nov 13, 2024
@quiet-node quiet-node added this to the 0.11.0 milestone Nov 13, 2024
@quiet-node quiet-node self-assigned this Nov 13, 2024
@quiet-node quiet-node requested a review from a team as a code owner November 13, 2024 15:32
@quiet-node quiet-node linked an issue Nov 13, 2024 that may be closed by this pull request
Copy link

Test Results

 16 files  ± 0   83 suites  +1   11m 48s ⏱️ + 2m 13s
294 tests +25  288 ✅ +28  6 💤 +2  0 ❌  - 5 
303 runs  +15  296 ✅ +18  7 💤 +2  0 ❌  - 5 

Results for commit ca45bf9. ± Comparison against base commit 41e3d39.

This pull request removes 4 and adds 29 tests. Note that renamed tests count towards both.
"before all" hook for "should be able to increase and decrease counter on V1" ‑ Proxy Upgrade Contracts Test Suite Counter Upgradable Contract Test Suite "before all" hook for "should be able to increase and decrease counter on V1"
"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 verify it calls the correct contract and method via proxy" ‑ @OZTransparentUpgradeableProxy Test Suite "before all" hook for "should verify it calls the correct contract and method via proxy"
"before each" hook for "should be able to update fixed fee in HBARs for NFT" ‑ TokenManagmentContract Test Suite Update fees "before each" hook for "should be able to update fixed fee in HBARs for NFT"
should be able to increase and decrease counter on V1 ‑ Proxy Upgrade Contracts Test Suite Counter Upgradable Contract Test Suite should be able to increase and decrease counter on V1
should be able to increase and decrease counter on V2 ‑ Proxy Upgrade Contracts Test Suite Counter Upgradable Contract Test Suite should be able to increase and decrease counter on V2
should be able to update fixed HTS fee and royalty fee in NFT ‑ TokenManagmentContract Test Suite Update fees should be able to update fixed HTS fee and royalty fee in NFT
should be able to update fixed HTS fee for NFT ‑ TokenManagmentContract Test Suite Update fees should be able to update fixed HTS fee for NFT
should be able to update fixed fee in HBARs for NFT ‑ TokenManagmentContract Test Suite Update fees should be able to update fixed fee in HBARs for NFT
should be able to upgrade contract to V2 ‑ Proxy Upgrade Contracts Test Suite Counter Upgradable Contract Test Suite should be able to upgrade contract to V2
should deposit, buy and sell tokens from ExchangeV1 ‑ Proxy Upgrade Contracts Test Suite DEX Upgradable Contract Test Suite should deposit, buy and sell tokens from ExchangeV1
should deposit, buy and withdraw tokens from ExchangeV2 ‑ Proxy Upgrade Contracts Test Suite DEX Upgradable Contract Test Suite should deposit, buy and withdraw tokens from ExchangeV2
should fail for updateFungibleTokenCustomFees when token is not associated to fee collector ‑ TokenManagmentContract Test Suite Update fees Update fees negative cases should fail for updateFungibleTokenCustomFees when token is not associated to fee collector
should fail for updateNonFungibleTokenCustomFees when token is not associated to fee collector ‑ TokenManagmentContract Test Suite Update fees Update fees negative cases should fail for updateNonFungibleTokenCustomFees when token is not associated to fee collector
…

@quiet-node quiet-node merged commit 6c25b1d into main Nov 13, 2024
36 of 37 checks passed
@quiet-node quiet-node deleted the 1013-chore-include-missing-contract-artifacts branch November 13, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal For changes that affect the project's internal workings but not its outward-facing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: include missing contract artifacts
2 participants