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

Add unit test to capture-stop some streams #883

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Commits on Nov 8, 2024

  1. Add capture-stop unit test for some streams

    Also tweaked the _send_data helper and how it returned X- and B-engine
    results for incorporation of this test.
    
    Contributes to: NGC-1155.
    amishatishpatel committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    e0cd770 View commit details
    Browse the repository at this point in the history
  2. Complete test_capture_stop_some_streams

    And refine the test itself. Also fix verify_beam_sensors after the
    previous _send_data update.
    
    Contributes to: NGC-1155.
    amishatishpatel committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    deec5db View commit details
    Browse the repository at this point in the history
  3. Remove unnecessary xbengine fixture from unit test

    Turns out it doesn't need to be around at all for the newest xbgpu unit
    test.
    
    Contributes to: NGC-1155.
    amishatishpatel committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    2ac82e0 View commit details
    Browse the repository at this point in the history
  4. Use chain to combine lists in unit test

    I imported it to use in the end-to-end test, figured I'd use it in the
    new test as well.
    
    Contributes to: NGC-1155.
    amishatishpatel committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    45b1d91 View commit details
    Browse the repository at this point in the history