Issue 6753 - Removing ticket49303_test and porting to DSLdapObject#7429
Open
aadhikar wants to merge 1 commit into389ds:mainfrom
Open
Issue 6753 - Removing ticket49303_test and porting to DSLdapObject#7429aadhikar wants to merge 1 commit into389ds:mainfrom
aadhikar wants to merge 1 commit into389ds:mainfrom
Conversation
Description: The old ticket49303_test.py had compatibility issues. This ports the functionality to tls_renegotiation_test.py using modern Encryption class from lib389. Relates: 389ds#6753 Reviewed by: ??? Assisted by: Claude
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The test mutates the nsTLSAllowClientRenegotiation setting without restoring the original value, which can leak configuration changes into subsequent tests; consider saving the initial value and restoring it in a teardown/finally block.
- The
if __name__ == '__main__':block is not needed for pytest-based tests and can be removed to keep the test module minimal.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The test mutates the nsTLSAllowClientRenegotiation setting without restoring the original value, which can leak configuration changes into subsequent tests; consider saving the initial value and restoring it in a teardown/finally block.
- The `if __name__ == '__main__':` block is not needed for pytest-based tests and can be removed to keep the test module minimal.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
Congratulations! One of the builds has completed. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
The old ticket49303_test.py had compatibility issues.
This ports the functionality to tls_renegotiation_test.py
using modern Encryption class from lib389.
Relates: #6753
Reviewed by: ???
Assisted by: Claude
Summary by Sourcery
Port TLS renegotiation coverage from the legacy ticket-based test to a new TLS suite test using the modern configuration interface.
Tests: