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

feat: Quick test branch #1139

Closed
wants to merge 19 commits into from
Closed

feat: Quick test branch #1139

wants to merge 19 commits into from

Conversation

andrewb1269hg
Copy link

@andrewb1269hg andrewb1269hg commented Dec 26, 2024

Draft PR, please do not merge.

quiet-node and others added 19 commits December 23, 2024 15:28
Signed-off-by: Logan Nguyen <[email protected]>
Signed-off-by: Logan Nguyen <[email protected]>

Revert "Update erc-registry-indexer.yml"

This reverts commit e708c66.

Reapply "Update erc-registry-indexer.yml"

This reverts commit ff4d70d91f378efe423420f247ef3b1698d04885.
@andrewb1269hg andrewb1269hg added this to the 0.15.0 milestone Dec 26, 2024
@andrewb1269hg andrewb1269hg self-assigned this Dec 26, 2024
@andrewb1269hg andrewb1269hg changed the title Quick test branch feat: Quick test branch Dec 26, 2024
Copy link

Test Results

 16 files  ±0   83 suites  ±0   12m 24s ⏱️ +22s
294 tests +8  285 ✅ +6  6 💤 +2  3 ❌ ±0 
303 runs  ±0  293 ✅  - 2  7 💤 +2  3 ❌ ±0 

For more details on these failures, see this check.

Results for commit aafb257. ± Comparison against base commit f786a64.

This pull request removes 3 and adds 11 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"
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 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 not be able to change name ‑ Proxy Upgrade Contracts Test Suite Counter Upgradable Contract Test Suite should not be able to change name
should not be able to get version ‑ Proxy Upgrade Contracts Test Suite DEX Upgradable Contract Test Suite should not be able to get version
should upgrade contract to V2 ‑ Proxy Upgrade Contracts Test Suite DEX Upgradable Contract Test Suite should upgrade contract to V2
should verify it calls the correct contract and method via proxy ‑ @OZTransparentUpgradeableProxy Test Suite should verify it calls the correct contract and method via proxy
should verify it can change the underlying contract ‑ @OZTransparentUpgradeableProxy Test Suite should verify it can change the underlying contract
…

@andrewb1269hg
Copy link
Author

Testing complete, closing this PR and deleting the branch.

@andrewb1269hg andrewb1269hg deleted the quick_test_branch branch December 26, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants