Fix happi transfer default handling (don't past default help text)
adds a test to try and cover this
Also fixes conftest.trim_split_output because it just.. entirely didn't work. More shame
In a burst of extra effort, disables assertion checks against expected stdout-style output. The check still takes place, but simply prints (visible with pytest -s flag) if a mismatch is found. In place of this, the happi_cfg is checked to verify the requested action has actually taken place.
Tests modified are:
test_transfer_cli
test_transfer_cli_more
test_search_json (this one is comparing to json, so I still assert the stdout comparisons here)
test_add_cli
test_add_clone
Motivation and Context
closes #334 , in part (the non-list / dict comprehension parts)
How Has This Been Tested?
Adds a test
Where Has This Been Documented?
This PR
Pre-merge checklist
[x] Code works interactively
[x] Code contains descriptive docstrings, including context and API
[x] New/changed functions and methods are covered in the test suite where possible
[x] Test suite passes locally
[x] Test suite passes on GitHub Actions
[x] Ran docs/pre-release-notes.sh and created a pre-release documentation page
[x] Pre-release docs include context, functional descriptions, and contributors as appropriate
Description
Fix happi transfer default handling (don't past default help text) adds a test to try and cover this
Also fixes
conftest.trim_split_outputbecause it just.. entirely didn't work. More shameIn a burst of extra effort, disables assertion checks against expected stdout-style output. The check still takes place, but simply prints (visible with
pytest -sflag) if a mismatch is found. In place of this, the happi_cfg is checked to verify the requested action has actually taken place.Tests modified are:
test_transfer_clitest_transfer_cli_moretest_search_json(this one is comparing to json, so I still assert the stdout comparisons here)test_add_clitest_add_cloneMotivation and Context
closes #334 , in part (the non-list / dict comprehension parts)
How Has This Been Tested?
Adds a test
Where Has This Been Documented?
This PR
Pre-merge checklist
docs/pre-release-notes.shand created a pre-release documentation page