intg: remove test_session_recording.py#8243
Conversation
There was a problem hiding this comment.
Code Review
This pull request removes test_session_recording.py, which appears to be the complete integration test suite for the SSSD session recording feature. Based on other files in the repository such as src/man/sssd-session-recording.5.xml and src/util/session_recording.c, this feature is still active. Deleting its tests introduces a critical risk of regressions going undetected. It is essential to maintain test coverage for existing features. If these tests are being replaced, the new tests should ideally be part of the same pull request.
f3bbebb to
b55b9d4
Compare
Adding the necessary tag, but AFAIK, we are leaving 2-7 and 2-8 alone. |
|
This has been discussed, the priority to remove these tests are greater than having these intg tests. These tests will be written for the new test framework at a later time. |
Removing intg session recording test file and removing from Makefile. Reviewed-by: Dan Lavu <dlavu@redhat.com>
b55b9d4 to
fbdd22a
Compare
No description provided.