Skip to content

Conversation

@codambro
Copy link

Issue being resolved: #372

Solution description

The Test Run Update API does not like specifying "case_ids" when a Test Run is configured with "include_all". The end result is the Test Run is showing 0 test cases.

Changes

Add "include_all" to the test run update API call. Only specify "case_ids" when not including all.

Potential impacts

No known impact.

Steps to test

  • Adding new test cases to an existing test run with "include_all: false"
  • Adding test results to an existing test run with "include_all: true"

PR Tasks

  • PR reference added to issue
  • README updated
  • Unit tests added/updated

@acuanico-tr-galt
Copy link
Collaborator

Hi @codambro thank you for submitting this PR. We have improved this to include additional error handling, validation and unit tests in: #375

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.

[Bug] add_results_for_cases Fails After update_run_in_plan_entry and Deletes Test Cases In Test Run

2 participants