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

Update burst_pattern method to return pass/fail results if wait_until_done is True #1336

Merged
merged 8 commits into from
Mar 25, 2020

Conversation

sbethur
Copy link
Contributor

@sbethur sbethur commented Mar 19, 2020

  • This contribution adheres to CONTRIBUTING.md.

  • I've updated CHANGELOG.md if applicable.

  • I've added tests applicable for this pull request

What does this Pull Request accomplish?

Update burst_pattern method to return pass/fail results if wait_until_done is True. With this change, users can use burst_pattern is two ways:

  1. Does not wait for pattern burst to complete.
  2. Waits for pattern burst to complete and expect pass-fail results to be returned. (Majority use-case)

List issues fixed by this Pull Request below, if any.

What testing has been done?

Added and ran new system tests.

@sbethur sbethur requested a review from marcoskirsch March 19, 2020 23:38
@codecov
Copy link

codecov bot commented Mar 19, 2020

Codecov Report

Merging #1336 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1336   +/-   ##
=======================================
  Coverage   91.84%   91.84%           
=======================================
  Files          20       20           
  Lines        3616     3616           
=======================================
  Hits         3321     3321           
  Misses        295      295
Flag Coverage Δ
#codegenunittests 88.21% <ø> (ø) ⬆️
#nifakeunittests 96.3% <ø> (ø) ⬆️
#nimodinstunittests 96.19% <ø> (ø) ⬆️
#nitclkunittests 96.09% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 562ff7f...e477b19. Read the comment docs.

@marcoskirsch
Copy link
Member

please fix merge conflicts

sbethur added 4 commits March 20, 2020 15:22
# Conflicts:
#	generated/nidigital/nidigital/session.py
#	src/nidigital/system_tests/test_system_nidigital.py
# Conflicts:
#	generated/nidigital/nidigital/session.py
@sbethur
Copy link
Contributor Author

sbethur commented Mar 20, 2020

please fix merge conflicts

Done

@marcoskirsch marcoskirsch merged commit 82e05c9 into master Mar 25, 2020
@marcoskirsch marcoskirsch deleted the bug1296/burst_pattern_pass_fail branch March 25, 2020 02:07
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.

Pass-fail variant of burst_pattern is missing from nidigital API
2 participants