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

Hydra-plutus tests are actually red and golden tests got side-stepped #1458

Closed
ch1bo opened this issue May 29, 2024 · 2 comments
Closed

Hydra-plutus tests are actually red and golden tests got side-stepped #1458

ch1bo opened this issue May 29, 2024 · 2 comments
Labels
bug 🐛 Something isn't working

Comments

@ch1bo
Copy link
Collaborator

ch1bo commented May 29, 2024

Context & versions

On current master cb5a8c8, we have a green CI https://github.com/input-output-hk/hydra/actions/runs/9280096242.

BUT, the hydra-plutus tests are actually RED.

Steps to reproduce

  1. Checkout cb5a8c8e9ed0af7a2f8ba422ab78f042d4ee4920
  2. cd hydra-plutus
  3. cabal run hydra-plutus:exe:tests

This is related to #1449 as to verify this, you need to run the tests in the right directory. And a false sense of security is created by seeing the tests green (although they generated new golden files).

Actual behavior

Our CI allowed a change to hydra-plutus to be merged to master without

Expected behavior

The CI to actually run the hydra-plutus tests, detect breaking changes by seeing golden tests red (and their intended consequence of documenting detected breaking changes)

Hypothesis

@ch1bo ch1bo added the bug 🐛 Something isn't working label May 29, 2024
@locallycompact
Copy link
Contributor

#1459

@ch1bo
Copy link
Collaborator Author

ch1bo commented May 29, 2024

Fixed by #1459

@ch1bo ch1bo closed this as completed May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants