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 robustness to test_authd_valid_name_ip IT on wazuh 4.4 #4164

Merged
merged 2 commits into from
May 23, 2023

Conversation

QU3B1M
Copy link
Member

@QU3B1M QU3B1M commented May 12, 2023

Related issue
#3455

Description

This PR fixes the errors reported on the issue #3455 were the test_authd/test_authd_valid_name_ip.py IT were failing sporadically (locally it happens more often in wazuh 4.3.X running on Ubuntu).
The error occurred because all the cases were executed one after the other without cleaning the environment in between, so sometimes (after the invalid cases) it tries to validate before the environment is clean and ready.

To make sure the env is clean after each execution the fixtures restart_authd_function, wait_for_authd_startup_function and clean_client_keys_file_function were applied.

Updated

  • deps/wazuh_testing/wazuh_testing/authd.py
  • tests/integration/test_authd/data/test_authd_valid_name_ip.yaml
  • tests/integration/test_authd/test_authd_valid_name_ip.py

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@QU3B1M (Developer) test_authd/test_authd_valid_name_ip.py ⚫⚫⚫ 🟢 🟢 🟢 Ubuntu 20.04 - manager 6eef7ed Nothing to highlight
@damarisg (Reviewer) test_authd/ 🟢 🚫 🚫 🚫 centOS - Manager 6eef7ed

@QU3B1M QU3B1M changed the base branch from master to 4.4 May 12, 2023 18:38
@QU3B1M QU3B1M self-assigned this May 12, 2023
@QU3B1M QU3B1M marked this pull request as ready for review May 12, 2023 18:41
@QU3B1M QU3B1M changed the title 3455 apply fix to new cases Add robustness to test_authd_valid_name_ip IT on wazuh 4.4 May 12, 2023
@damarisg damarisg linked an issue May 23, 2023 that may be closed by this pull request
@damarisg damarisg merged commit bf7982a into 4.4 May 23, 2023
@damarisg damarisg deleted the 3455-apply-fix-to-new-cases branch May 23, 2023 13:53
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.

Authd force options integration tests failures
2 participants