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

Test Fixed, change decoder/rule tittle for correct execution #4496

Merged
merged 3 commits into from
Sep 12, 2022

Conversation

Mayons95
Copy link
Contributor

@Mayons95 Mayons95 commented Sep 12, 2022

Related issue
closes #4494

Description:

This PR implements a fix to Create new decoder/rule test for Wazuh Dashboard.
The main problem is related to the decoder name, the new decoder needs to have a unique name

Solution:
The proposed solution is to use a random name for the new decoders based on the timestamp. I add this code in a new function in the controller so we can implement it in other test cases as well.

E2E tests

image

@Mayons95 Mayons95 marked this pull request as draft September 12, 2022 13:55
@Mayons95 Mayons95 self-assigned this Sep 12, 2022
@Mayons95 Mayons95 linked an issue Sep 12, 2022 that may be closed by this pull request
@Mayons95
Copy link
Contributor Author

PR Tested on
Wazuh Dashoard 1.2.0
Kibana 7.16.3 - Xpack Security
image

@Mayons95 Mayons95 marked this pull request as ready for review September 12, 2022 14:20
@Mayons95 Mayons95 changed the title Test Fixed, change decoder tittle for correct execution Test Fixed, change decoder/rule tittle for correct execution Sep 12, 2022
Copy link
Member

@Machi3mfl Machi3mfl left a comment

Choose a reason for hiding this comment

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

LGTM!

@Mayons95 Mayons95 merged commit ec3e4f6 into 4.4-7.10 Sep 12, 2022
@Mayons95 Mayons95 deleted the fix/show-restart-warning-after-saves-decoder branch September 12, 2022 15:15
github-actions bot pushed a commit that referenced this pull request Sep 12, 2022
* Test Fixed, change decoder tittle for correct execution

* Adding random name based on timestamp

* Add new writing behaviour, same as decoder

(cherry picked from commit ec3e4f6)
github-actions bot pushed a commit that referenced this pull request Sep 12, 2022
* Test Fixed, change decoder tittle for correct execution

* Adding random name based on timestamp

* Add new writing behaviour, same as decoder

(cherry picked from commit ec3e4f6)
Mayons95 added a commit that referenced this pull request Sep 12, 2022
…t execution (#4497)

Test Fixed, change decoder/rule tittle for correct execution (#4496)

* Test Fixed, change decoder tittle for correct execution

* Adding random name based on timestamp

* Add new writing behaviour, same as decoder

(cherry picked from commit ec3e4f6)

Co-authored-by: Mayons95 <[email protected]>
Mayons95 added a commit that referenced this pull request Sep 12, 2022
…rect execution (#4498)

Test Fixed, change decoder/rule tittle for correct execution (#4496)

* Test Fixed, change decoder tittle for correct execution

* Adding random name based on timestamp

* Add new writing behaviour, same as decoder

(cherry picked from commit ec3e4f6)

Co-authored-by: Mayons95 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation automation type/test Test issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automation - Fix - Decoder/Rules - Restart warning after saves
3 participants