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

Work-in-progress refactor: switch gateway tests to pytest #48

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

klauer
Copy link

@klauer klauer commented Oct 27, 2022

Goal

  • Convert unittest-based tests into pytest and refactor around built-ins such as assert
    • This is cleaned/pared down from a previous effort in a separate repository for SLAC (LCLS, PCDS) to verify new gateway deployments
    • There are new tests here: a description will be added later
  • Add junit-xml output for tests (included as an artifact)
  • Add test result summary (included as an artifact which seems to render in the GH actions page as well)

Work-in-progress

Remaining to do:

  • Describe the newly-added tests that weren't just an update/conversion
  • Look into recent epics-extensions/ca-gateway changes that happened after these were converted to pytest (and forked into the other repository)
  • Open issues for expected-but-not-yet-reported test failures (mostly related to caPutLog handling)
  • Investigate differing results from what we saw previously
  • See if there are any better test summarizing tools
  • Update 3.14/3.15 to test put logging (or disable those tests entirely)

@klauer klauer closed this Sep 21, 2023
@ralphlange
Copy link
Contributor

I hope you closed it to be replaced by a rebased equivalent?!

@klauer klauer reopened this Sep 22, 2023
@klauer
Copy link
Author

klauer commented Sep 22, 2023

Ah, was cleaning up my old PRs and closed this by mistake - would you mind approving the CI to run?

@klauer
Copy link
Author

klauer commented Nov 7, 2023

@ZLLentz maybe this is something you could take over at some point? (cc @tangkong)

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.

2 participants