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.

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

Jump to bottom

[Winlogbeat] add tests for ingest node pipelines #30406

Closed
leehinman opened this issue Feb 15, 2022 · 2 comments
Closed

[Winlogbeat] add tests for ingest node pipelines #30406

leehinman opened this issue Feb 15, 2022 · 2 comments
Assignees
@efd6
Labels
8.4-candidate enhancement Winlogbeat

Comments

@leehinman
Copy link
Contributor

Describe the enhancement:

#29435 switched modules from Javascript processor to Ingest Node pipelines. The tests that were used for the Javascript processor are no longer valid. We need to:

  1. remove the old Javascript processor tests and golden files
  2. Add fixture to run sample documents through ingest node pipelines
  3. Add new tests that use the new fixture
@elasticmachine
Copy link
Collaborator

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@andrewkroh andrewkroh mentioned this issue Feb 23, 2022
Closed
@efd6 efd6 mentioned this issue Feb 23, 2022
Merged
3 tasks
@andrewkroh andrewkroh mentioned this issue Apr 6, 2022
Merged
4 tasks
@efd6 efd6 mentioned this issue Apr 29, 2022
Merged
7 tasks
@adriansr adriansr mentioned this issue May 9, 2022
Merged
4 tasks
@efd6 efd6 assigned efd6 and unassigned taylor-swanson May 26, 2022
This was referenced Jun 3, 2022
Merged
Merged
Merged
@efd6
Copy link
Contributor

efd6 commented Jun 24, 2022

Closed by #31833

@efd6 efd6 closed this as completed Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@efd6 efd6

Labels
8.4-candidate enhancement Winlogbeat
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@epixa @andrewkroh @elasticmachine @leehinman @jamiehynds @efd6 @taylor-swanson