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

Hani/ Testrail exercise #541

Closed
wants to merge 2 commits into from
Closed

Hani/ Testrail exercise #541

wants to merge 2 commits into from

Conversation

sv-hyacoub
Copy link
Collaborator

Relevant Links

Bugzilla: _
TestRail: _

Description of Code / Doc Changes

Leave a bullet-pointed list of changes you made.

Process Changes Required

Mark the relevant boxes:

  • Adds a dependency (rerun pipenv install)
  • Changes the BasePage
  • Changes or creates a BOM/POM (name the object model): _
  • Changes CI flow
  • Changes scheduled Beta or DevEdition
  • Changes Git hooks or Github settings
  • Changes L10n harness

Screenshots or Explanations

If you need to explain your code, do it here.

Comments or Future Work

Do we need to start another PR soon to address something you saw while working on this?

Workflow Checklist

  • Please request reviewers
  • If this is an unblocker, please post in Slack.
  • If asked to address comments, please resolve conversations.
  • If asked to change code, please re-request review from the person who wanted changes.

Thank you!

logging.info(f"Total completed automated cases found for selected suites: {len(all_completed_cases)}")

for case in all_completed_cases:
tr.update_case_field(case["id"], "custom_automation_coverage", AUTOMATION_COVERAGE_VALUE)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Comment this out, and insert a line that prints the case ids you would change if it ran live. Post that result here, and if it looks good, I'll give the go-ahead to run the script live. Thanks!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Case IDs to be updated: [127239, 127249, 127271, 2084637, 2084639, 2084641, 2084539, 2084549, 2084489, 2084490, 2084518, 2084524, 2084550, 2084552, 2084553, 2084559, 2084564, 118799, 118800, 118802, 118805, 118806, 118807, 172043, 172045, 216273, 174072, 464474, 936860, 936861, 936864, 937418]

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Total cases fetched from suite 2085: 3
Total cases fetched from suite 2525: 24
Total cases fetched from suite 5259: 5

Copy link
Collaborator

Choose a reason for hiding this comment

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

That looks right, run the script!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I ran the script successfully. Thanks.

@ben-c-at-moz
Copy link
Collaborator

Script ran successfully

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