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

IT Authd/Wazuh DB: Refactor for failures caused by new development of Wazuh DB. #2584

Closed
Tracked by #2504
damarisg opened this issue Feb 18, 2022 · 2 comments · Fixed by #2777
Closed
Tracked by #2504

IT Authd/Wazuh DB: Refactor for failures caused by new development of Wazuh DB. #2584

damarisg opened this issue Feb 18, 2022 · 2 comments · Fixed by #2777

Comments

@damarisg
Copy link
Member

damarisg commented Feb 18, 2022

Description

Working on Migrate agent-group files to wazuh-db, the core team detected that there are affected tests by the new changes, and at the same time that there are deprecated tests.

The objective of this issue is to perform the refactor of each affected test.

Authd - Manager

Test Path Total Test Requires Change
test_authd\test_authd_worker.py 1 Update Maximum allows is 128 not 256
test_authd\test_authd.py 1 Update Maximum allows is 128 not 256
test_authd\test_authd_key_hash.py 1 Update Maximum allows is 128 not 256
test_authd\test_authd_agents_ctx.py 2 Update Now not exist '/var/ossec/queue/agent-groups', checks with the DB

Wazuh DB - Manager

Test Path Total Requires Change
test_wazuh_db\test_wazuh_db.py: test_wazuh_db_range_checksum 1 Update fixture: prepare_range_checksum_data
test_wazuh_db\test_wazuh_db.py: test_wazuh_db_messages_global 1 Update log messages

Notes:

test_wazuh_db\test_wazuh_db.py: test_wazuh_db_range_checksum

Try to insert an agent with id 1 because it already exists 'err Unable to execute global database query; UNIQUE constraint failed: agent.id'

test_wazuh_db\test_wazuh_db.py: test_wazuh_db_messages_global

  • global get-agent-info after insert: don't expect "group_sync_status" column
  • global update-agent-group success: command "update-agent-group" does not exist.
  • global select-agent-group success: agent group is not updated and select fails
  • global get-all-agents success: expect 3 agents but there are only 2 in the DB.
  • global sync-agent-info-get success: an agent is expected in "connection_status":"active" state but it is in "connection_status":"never_connected"
  • global insert-agent-belong success: The "insert-agent-belong" command no longer exists.
  • global checking sync_status before reset-agents-connection: agent 2 not found.
  • global get-agents-by-connection-status pending success: agent 2 not found.
  • global disconnect-agents success: agent 2 not found.
  • global delete-group-belong: The "delete-group-belong" command does not exist.
  • global manager get-agent-info success pre update: the "get-agent-info" command returns more columns than the test expects.
@damarisg damarisg changed the title IT Wazuh DB - Refactor for failures caused by new development. IT Remoted - Authd - Wazuh DB: Refactor for failures caused by new development. Feb 18, 2022
@damarisg damarisg changed the title IT Remoted - Authd - Wazuh DB: Refactor for failures caused by new development. IT Remoted/Authd/Wazuh DB: Refactor for failures caused by new development of Wazuh DB. Feb 18, 2022
@damarisg damarisg changed the title IT Remoted/Authd/Wazuh DB: Refactor for failures caused by new development of Wazuh DB. IT Authd/Wazuh DB: Refactor for failures caused by new development of Wazuh DB. Feb 18, 2022
@damarisg damarisg self-assigned this Feb 23, 2022
@Deblintrake09
Copy link
Contributor

Updates

  • 2022/05/04- 2022/05/06 - Finished Fixing authd tests
  • 2022/05/07 - Fix WDB global_messages tests
  • Launch Jenkins instances

@damarisg
Copy link
Member Author

It was merged on qa-10771-agent-groups-files-to-wazuh-db branch and closed.

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

Successfully merging a pull request may close this issue.

3 participants