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.

Sign up for GitHub

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

ECS-6433 heuristic loading #231 #348

Merged
merged 26 commits into from
Oct 21, 2024
+118 −10
Merged

ECS-6433 heuristic loading #231 #348

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
d8ba42c
Updated cli.py so 'happi load' can handle glob searches, like 'happi …
janeliu-slac Oct 9, 2024
6450a1e
Added pre-commit changes.
janeliu-slac Oct 9, 2024
779086d
Updated cli.py with logic to handle user input with glob and non-glob…
janeliu-slac Oct 10, 2024
06274c5
Updated cli.py with documentation for the new feature and pre-commit …
janeliu-slac Oct 10, 2024
9cfbf0c
Added release notes.
janeliu-slac Oct 10, 2024
85a13c0
Removed the line "if '*' not in (' '.join(search_criteria))".
janeliu-slac Oct 10, 2024
137bb18
Recoded the feature to gather the results of each search term individ…
janeliu-slac Oct 11, 2024
37d9c06
Removed some print statements used for testing.
janeliu-slac Oct 11, 2024
7b25139
Updated cli.py so 'happi load' can handle glob searches, like 'happi …
janeliu-slac Oct 9, 2024
ef16b22
Added pre-commit changes.
janeliu-slac Oct 9, 2024
87013ad
Updated cli.py with logic to handle user input with glob and non-glob…
janeliu-slac Oct 10, 2024
8386902
Updated cli.py with documentation for the new feature and pre-commit …
janeliu-slac Oct 10, 2024
4f4982a
Added release notes.
janeliu-slac Oct 10, 2024
5e86159
Removed the line "if '*' not in (' '.join(search_criteria))".
janeliu-slac Oct 10, 2024
987daed
Recoded the feature to gather the results of each search term individ…
janeliu-slac Oct 11, 2024
c7ca35f
Removed some print statements used for testing.
janeliu-slac Oct 11, 2024
aa3fa70
Merge remote-tracking branch 'refs/remotes/origin/ECS-6433_heuristic_…
janeliu-slac Oct 15, 2024
997f39c
In cli.py load() function removed 'term' and replaced with '[item]' f…
janeliu-slac Oct 16, 2024
226a012
Tried to pass name values from SearchResult objects obtained from cli…
janeliu-slac Oct 16, 2024
d2c5ff3
Updated test_load_glob_args() function with names of mock devices in …
janeliu-slac Oct 18, 2024
39777ff
Added a second glob parameter to test_load_glob_args() in test_cli.py.
janeliu-slac Oct 18, 2024
733be20
Created a second unit test for the happi load() function to handle mu…
janeliu-slac Oct 18, 2024
0fa9f5e
Added release notes.
janeliu-slac Oct 19, 2024
a790d08
Delete docs/source/upcoming_release_notes/348-heuristic_loading.rst
janeliu-slac Oct 19, 2024
3d2cc7a
Updated release notes.
janeliu-slac Oct 19, 2024
fd38612
Updated release notes.
janeliu-slac Oct 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion happi/tests/test_cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -649,9 +649,10 @@ def test_load_glob_args(

with mock.patch.object(IPython, 'start_ipython') as m:
_ = runner.invoke(
happi_cli, ['--path', happi_cfg, 'load', 'tst_*']
happi_cli, ['--path', happi_cfg, 'load', 'tst_*', 'device_class=types*']
janeliu-slac marked this conversation as resolved.
Show resolved Hide resolved
)
m.assert_called_once_with(argv=['--quick'], user_ns=devices)

with caplog.at_level(logging.INFO):
assert "Creating shell with devices" in caplog.text

Expand Down