Skip to content

Add system tests for CEF#575

Merged
andrewkroh merged 1 commit intoelastic:masterfrom
andrewkroh:feature/cef/system-tests
Jan 29, 2021
Merged

Add system tests for CEF#575
andrewkroh merged 1 commit intoelastic:masterfrom
andrewkroh:feature/cef/system-tests

Conversation

@andrewkroh
Copy link
Copy Markdown
Member

@andrewkroh andrewkroh commented Jan 28, 2021

What does this PR do?

Add logfile and syslog (udp) system tests for the CEF package.

I removed some duplicate fields from fields.yml files.
And for checkpoint, I converted the checkpoint.event_count field to a long in the JSON.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all datasets collect metrics or logs.

Related issues

Screenshots

--- Test results for package: cef - START ---
╭─────────┬─────────────┬───────────┬───────────┬────────┬───────────────╮
│ PACKAGE │ DATA STREAM │ TEST TYPE │ TEST NAME │ RESULT │  TIME ELAPSED │
├─────────┼─────────────┼───────────┼───────────┼────────┼───────────────┤
│ cef     │ log         │ system    │ logfile   │ PASS   │ 27.426336075s │
│ cef     │ log         │ system    │ syslog    │ PASS   │ 27.180639997s │
╰─────────┴─────────────┴───────────┴───────────┴────────┴───────────────╯
--- Test results for package: cef - END   ---

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Jan 28, 2021

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

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Jan 28, 2021 •
edited
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #575 updated

    • Start Time: 2021-01-28T20:37:20.033+0000

  • Duration: 9 min 42 sec

  • Commit: 760f013

Test stats 🧪

Test Results
Failed 0
Passed 81
Skipped 0
Total 81

@andrewkroh andrewkroh mentioned this pull request Jan 28, 2021
Closed
@andrewkroh
Add system tests for CEF
760f013 
Add logfile and syslog (udp) system tests for the CEF package.

I removed some duplicate fields from fields.yml files.
And for checkpoint, I converted the `checkpoint.event_count` field to a long in the JSON.
@andrewkroh andrewkroh merged commit c24331d into elastic:master Jan 29, 2021
@adriansr adriansr mentioned this pull request Feb 3, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leehinman leehinman leehinman approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@andrewkroh @elasticmachine @leehinman