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(template-sync-test): updated tests to use new template syncing method #1247

Open
wants to merge 13 commits into
base: development
Choose a base branch
from

Conversation

ksrichard
Copy link
Collaborator

Description

Updated tests to use the new template publishing method and possibly fixed the random cucumber test hanging without even exiting.

Important: #1242 must be merged to development before this PR could be

Motivation and Context

How Has This Been Tested?

What process can a PR reviewer use to test or verify this change?

Breaking Changes

  • None
  • Requires data directory to be deleted
  • Other - Please specify

@ksrichard ksrichard changed the title Feature/template sync tests feat(template-sync-test): updated tests to use new template syncing method Jan 17, 2025
Copy link

github-actions bot commented Jan 17, 2025

Test Results (CI)

607 tests  ±0   607 ✅ ±0   3h 48m 37s ⏱️ + 7m 29s
 66 suites ±0     0 💤 ±0 
  2 files   ±0     0 ❌ ±0 

Results for commit ffe2920. ± Comparison against base commit 9500de9.

This pull request removes 10 and adds 10 tests. Note that renamed tests count towards both.
Scenario: Claim and transfer confidential assets via wallet daemon: tests/features/wallet_daemon.feature:88:3
Scenario: Confidential transfer to unexisting account: tests/features/transfer.feature:160:3
Scenario: Counter template registration and invocation multiple times: tests/features/counter.feature:48:3
Scenario: Create and mint account NFT: tests/features/wallet_daemon.feature:130:3
Scenario: Create resource and mint in one transaction: tests/features/nft.feature:81:3
Scenario: Indexer GraphQL filtering and pagination of events: tests/features/indexer.feature:133:3
Scenario: Indexer GraphQL requests events over network substate indexing: tests/features/indexer.feature:101:3
Scenario: Indexer GraphQL requests work: tests/features/indexer.feature:91:3
Scenario: Template registration and invocation in a 2-VN committee: tests/features/committee.feature:9:3
Scenario: Transfer tokens to existing account: tests/features/transfer.feature:85:3
Scenario: Claim and transfer confidential assets via wallet daemon: tests/features/wallet_daemon.feature:83:3
Scenario: Confidential transfer to unexisting account: tests/features/transfer.feature:149:3
Scenario: Counter template registration and invocation multiple times: tests/features/counter.feature:46:3
Scenario: Create and mint account NFT: tests/features/wallet_daemon.feature:125:3
Scenario: Create resource and mint in one transaction: tests/features/nft.feature:80:3
Scenario: Indexer GraphQL filtering and pagination of events: tests/features/indexer.feature:129:3
Scenario: Indexer GraphQL requests events over network substate indexing: tests/features/indexer.feature:97:3
Scenario: Indexer GraphQL requests work: tests/features/indexer.feature:87:3
Scenario: Template registration and invocation in a 2-VN committee: tests/features/committee.feature:10:3
Scenario: Transfer tokens to existing account: tests/features/transfer.feature:80:3

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants