Skip to content

Conversation

@saw-jan
Copy link
Contributor

@saw-jan saw-jan commented Dec 16, 2025

Description

Whenever any of the test hooks fail, add them to the list of test failures so that the whole scenario is reported as failed at the end of the test run.

Related Issue

    Examples:
      | dav-path-version |
      | old              |
      | new              |
      | spaces           |
      │
      ╳  Syntax error (JsonException)
      │
      └─ @AfterScenario # FeatureContext::cleanDataAfterTests()

How Has This Been Tested?

  • test environment:

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added

Copy link
Contributor

@ScharfViktor ScharfViktor left a comment

Choose a reason for hiding this comment

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

That is correct but I'm afraid this will add instability to our CI.

@saw-jan
Copy link
Contributor Author

saw-jan commented Dec 16, 2025

That is correct but I'm afraid this will add instability to our CI.

something is not working here, a lot of tests failed. I will have a look.

@saw-jan saw-jan force-pushed the test/test-hook-failure branch from f9d7fa3 to 2d26954 Compare December 16, 2025 11:36
@saw-jan saw-jan requested a review from ScharfViktor December 16, 2025 11:58
@saw-jan saw-jan force-pushed the test/test-hook-failure branch 2 times, most recently from 8afb9bb to 4023d8f Compare December 17, 2025 07:21
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/test-hook-failure branch from 4023d8f to cb70798 Compare December 23, 2025 04:22
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.

3 participants