Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Skip to content

Draft: BECWidgets e2e test extension

appel_c requested to merge tests/add_e2e_test_gui_rpc into pre_release_v2

Description

MR to add additional e2e tests using the rpc interface to BECWidgets. This will test all widgets available through the rpc interface (anything in gui.available_widgets). In addition, various tests were added to test certain use cases simulating the work at the beamline.

Note:

  • This MR also changed the current behavior of e2e tests. It now runs without flushing REDIS, which IMO is closer to the actual use case at the beamline.

Related Issues

closes #481 (closed) closes #462 (closed) (!462 (3f15132a)) closes #497 (!462 (46fa1901))

Type of Change

  • Session scoped fixture for test_rpc_widgets_e2e. This significantly improves the speed for the e2e tests. Happy for input on name, this could be more specific, i.e. in bec it is distributed in test_scans_lib_e2e and test_scans_e2e. In my understanding, test_scans_e2e simulates usages as it would be during a beamtime. test_rpc_widgets_e2e would be similar. We might want to name things similar in BECWidgets, so happy for input

Potential side effects

None

Additional Comments

In future it might be nice to have a template for this test. Maybe we can clean it a bit more and move some more things to conftest.

Definition of Done

  • Documentation is up-to-date.
Edited by appel_c

Merge request reports