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 whbar tests pipeline #1267

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

natanasow
Copy link
Contributor

Description:

Currently, the WHBAR tests are not running within the CI.

Solution:

Add WHBAR tests to the CI.
Related issue(s):

Fixes #1266

Notes for reviewer:

Checklist

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

@natanasow natanasow self-assigned this Feb 27, 2025
@natanasow natanasow added the enhancement New feature or request label Feb 27, 2025
@natanasow natanasow added this to the 0.12.0 milestone Feb 27, 2025
@natanasow natanasow marked this pull request as ready for review February 27, 2025 14:39
@natanasow natanasow requested review from a team as code owners February 27, 2025 14:39
Copy link

github-actions bot commented Feb 27, 2025

Test Results

 19 files  + 1  108 suites  +2   26m 38s ⏱️ + 1m 19s
388 tests +33  370 ✅ +37  18 💤 ±0  0 ❌  - 4 
488 runs  +15  468 ✅ +19  20 💤 ±0  0 ❌  - 4 

Results for commit 35e2635. ± Comparison against base commit 49e709e.

This pull request removes 3 and adds 36 tests. Note that renamed tests count towards both.
"before all" hook for "should query allowance" ‑ TokenQueryContract Test Suite "before all" hook for "should query allowance"
"before all" hook for "should test that hollow account is created and the amount of fungible tokens is correctly transferred via system-contracts" ‑ HIP583 Test Suite - Ethereum Transfer TX via system-contracts Fungible Token Test "before all" hook for "should test that hollow account is created and the amount of fungible tokens is correctly transferred via system-contracts"
"before all" hook for "should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via system-contracts" ‑ HIP583 Test Suite - Ethereum Transfer TX via system-contracts 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 system-contracts"
should be able to approve ‑ WHBAR should be able to approve
should be able to deposit via contract`s fallback method ‑ WHBAR should be able to deposit via contract`s fallback method
should be able to deposit via contract`s receive method ‑ WHBAR should be able to deposit via contract`s receive method
should be able to transfer ‑ WHBAR should be able to transfer
should be able to transferFrom ‑ WHBAR should be able to transferFrom
should deploy the WHBAR contract ‑ WHBAR should deploy the WHBAR contract
should deposit 1 hbar and check totalSupply ‑ WHBAR should deposit 1 hbar and check totalSupply
should get decimals ‑ WHBAR should get decimals
should get name ‑ WHBAR should get name
should get symbol ‑ WHBAR should get symbol
…

♻️ This comment has been updated with latest results.

name: WHBAR Contract Test Suite
uses: ./.github/workflows/test-workflow.yml
with:
testfilter: WHBAR

Choose a reason for hiding this comment

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

I suggest to only execute it of changes on the given test or src folder for WHBAR

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.

Add WHBAR contract tests to the CI
2 participants