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

cardano-testnet-test: fix a test stanza #6129

Merged
merged 1 commit into from
Feb 27, 2025

Conversation

smelc
Copy link
Contributor

@smelc smelc commented Feb 21, 2025

Description

FIxes the command documented to launch a test

How to trust this PR

  1. Try executing the previous stanza: cabal test cardano-testnet-test --test-options '-p "/DRepRetirement/"'
  2. Observe it doesn't launch anything
  3. Try the corrected test stanza: cabal test cardano-testnet-test --test-options '-p "/DRep Retirement/"'
  4. Observer the corresponding test is executed this time

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • CI passes. See note on CI. The following CI checks are required:
  • Self-reviewed the diff

@smelc smelc requested a review from a team as a code owner February 21, 2025 14:45
@smelc smelc changed the title cardano-testnet-test: fix a test stanza (detail) cardano-testnet-test: fix a test stanza Feb 21, 2025
@smelc smelc force-pushed the smelc/fix-test-launching-stanza branch from 680d4a2 to aac24f5 Compare February 24, 2025 13:09
@smelc smelc enabled auto-merge February 24, 2025 13:09
@smelc smelc force-pushed the smelc/fix-test-launching-stanza branch from aac24f5 to 311bfbf Compare February 26, 2025 15:40
@carbolymer carbolymer disabled auto-merge February 27, 2025 09:20
@carbolymer carbolymer merged commit 2b6bb6c into master Feb 27, 2025
15 of 16 checks passed
@carbolymer carbolymer deleted the smelc/fix-test-launching-stanza branch February 27, 2025 09:20
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.

3 participants