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

Fix tests fails caused by stats socket changes #3271

Merged
merged 4 commits into from
Oct 26, 2022

Conversation

jmv74211
Copy link
Contributor

@jmv74211 jmv74211 commented Sep 8, 2022

Related issue
#3113

Description

This PR aims to fix the tests that started failing after the changes done in the PR associated to #3113.
The Wazuh/Wazuh PR #13713 changed the analysisd, remoted and wazuhdb sockets, which caused tests to fail, requiring changes in commands passed in and the expected outputs.

Added

  • One new test case for test_wazuhdb_getconfig.py

Updated

  • Updated remoted socket path
  • All Test cases inputs and outputs in test_wazuhdb_getconfig.py
  • Updated test_remoted modules that required updates in input or output checks
  • Update regex to matchexpected responses for:test_get_configuration_sock.pyandtest_rules_verbose.pyfrom thetest_logtest` suite.
  • Update EPS limit tests according to the new changes in analysisd.state (now the statistics are reset only when the wazuh-manager is restarted)

@jmv74211 jmv74211 force-pushed the revert-3270-revert-3146-fix-wdb-getconfig branch from 172a68f to 67ec2a1 Compare October 20, 2022 13:52
@jmv74211 jmv74211 changed the title IT: Fix tests fails caused by stats socket changes Fix tests fails caused by stats socket changes Oct 25, 2022
Copy link
Member

@Rebits Rebits left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Rebits Rebits merged commit cfad3be into 4.4 Oct 26, 2022
@Rebits Rebits deleted the revert-3270-revert-3146-fix-wdb-getconfig branch October 26, 2022 15:17
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