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.

Sign up for GitHub

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

Improve the Authd tests with the new force options #2184

Closed

DProvinciani opened this issue Nov 5, 2021 · 0 comments · Fixed by #2192

Labels

Contributor

DProvinciani commented Nov 5, 2021

As part of this issue, all the suggestions received as part of the pull #2171 should be implemented. These suggestions include:

Right usage of quotation.
Fix some typos.
Improve some variables creation and initialization.
Missing comments in the code to improve readability.
Modularization of some quite long functions.
Relocate some functions to a commonplace to allow the usage from other tests.

DProvinciani added the feature/authd label

DProvinciani mentioned this issue

Authd replacement configurations QA #2171

Merged

5 tasks

palaciosjeremias mentioned this issue

Implement Authd tests code improvements. #2192

Merged

5 tasks

palaciosjeremias linked a pull request

that will close this issue

Implement Authd tests code improvements. #2192

Merged

5 tasks

juliamagan added the subteam/qa-hurricane label

snaow closed this as completed in #2192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment