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

2384: Fix edge case bug for @setup in dry run mode #2519

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

codehackerr
Copy link
Contributor

@codehackerr codehackerr commented Feb 26, 2024

There are steps with no parent scenarios. Ex: background steps.

  • Added null check to handle this scenario.
  • Added a background step to dry run feature test

Before Fix


Before Fix

After Fix

After Fix

Description

Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

  • Relevant Issues : (compulsory)
  • Relevant PRs : (optional)
  • Type of change :
    • New feature
    • Bug fix for existing feature
    • Code quality improvement
    • Addition or Improvement of tests
    • Addition or Improvement of documentation

There are steps with no parent scenarios. Ex: background
Added null check.
@ptrthomas ptrthomas merged commit 4012901 into karatelabs:develop Apr 3, 2024
1 check passed
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