Skip to content

Fix asset test runner#224

Merged
mtojek merged 2 commits intoelastic:masterfrom
mtojek:fix-asset-tests-log-package
Jan 18, 2021
Merged

Fix asset test runner#224
mtojek merged 2 commits intoelastic:masterfrom
mtojek:fix-asset-tests-log-package

Conversation

@mtojek
Copy link
Copy Markdown
Contributor

@mtojek mtojek commented Jan 18, 2021

This PR adjusts the asset test runner with a couple of fixes.

Changes:

  • ingest pipeline may be undefined to logs-type packages
  • use asset ID from inside of a Kibana saved object file
  • format loaded assets in failure message

@mtojek mtojek self-assigned this Jan 18, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 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 #224 opened

    • Start Time: 2021-01-18T13:23:44.409+0000
  • Duration: 12 min 59 sec

  • Commit: 8487e86

Test stats 🧪

Test Results
Failed 0
Passed 68
Skipped 0
Total 68

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Jan 18, 2021

self, approved. I need to fix elastic/integrations#536 .

@mtojek mtojek merged commit ec5c00a into elastic:master Jan 18, 2021
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