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: add HTS Token Proxy reference contract #886

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Conversation

acuarica
Copy link
Contributor

Description:

This PR includes the HTS Token Proxy contract for reference.

Related issue(s):

Fixes #885.

Notes for reviewer:

Checklist

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

@acuarica acuarica linked an issue Jul 31, 2024 that may be closed by this pull request
@acuarica acuarica changed the title Add HTS Token Proxy reference contract chore: add HTS Token Proxy reference contract Jul 31, 2024
Copy link

github-actions bot commented Jul 31, 2024

Test Results

  16 files  +  3    77 suites  +17   7m 17s ⏱️ + 1m 57s
254 tests +77  247 ✔️ +79  7 💤 ±0  0  - 2 
263 runs  +75  255 ✔️ +76  8 💤 +1  0  - 2 

Results for commit c7c9c76. ± Comparison against base commit 8cc3ef8.

This pull request removes 2 and adds 79 tests. Note that renamed tests count towards both.
"before all" hook in "Admin key set to contractId" ‑ TokenManagmentContract Test Suite Extended update token info and keys test suite Admin key set to contractId "before all" hook in "Admin key set to contractId"
"before each" hook for "Should set a new token URI" ‑ @OZERC1155Token Test Suite "before each" hook for "Should set a new token URI"
Should NOT allow a non-operator to transfer tokens to another account ‑ @OZERC1155Token Test Suite Should NOT allow a non-operator to transfer tokens to another account
Should NOT allow a spender to spend hbar on behalf of owner without an allowance grant ‑ @CryptoAllowance Test Suite Should NOT allow a spender to spend hbar on behalf of owner without an allowance grant
Should NOT allow an approval on behalf of hbar owner WITHOUT its signature ‑ @CryptoAllowance Test Suite Should NOT allow an approval on behalf of hbar owner WITHOUT its signature
Should NOT burn insufficient amount of token ‑ @OZERC1155Token Test Suite Should NOT burn insufficient amount of token
Should allow a crypto owner contract account to grant an allowance to a spender contract account to transfer allowance to a receiver on behalf of owner contract account ‑ @CryptoAllowance Test Suite Should allow a crypto owner contract account to grant an allowance to a spender contract account to transfer allowance to a receiver on behalf of owner contract account
Should allow an approval on behalf of hbar owner WITH its signature ‑ @CryptoAllowance Test Suite Should allow an approval on behalf of hbar owner WITH its signature
Should allow an operator to transfer a token to another account ‑ @OZERC1155Token Test Suite Should allow an operator to transfer a token to another account
Should allow an operator to transfer tokens in batch to another account ‑ @OZERC1155Token Test Suite Should allow an operator to transfer tokens in batch to another account
Should burn token in batch ‑ @OZERC1155Token Test Suite Should burn token in batch
Should burn token ‑ @OZERC1155Token Test Suite Should burn token
…

♻️ This comment has been updated with latest results.

Copy link
Contributor

@Ivo-Yankov Ivo-Yankov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@acuarica acuarica merged commit dd32ca5 into main Aug 2, 2024
25 of 31 checks passed
@acuarica acuarica deleted the 885-hts-token-proxy branch August 2, 2024 12:29
@quiet-node quiet-node added this to the 0.10.0 milestone Aug 12, 2024
@quiet-node quiet-node added the enhancement New feature or request label Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include HTS Token Proxy contract for reference
3 participants