Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Don't test permissions of headless browser files #100

Merged
merged 1 commit into from
Sep 17, 2018
Merged

Don't test permissions of headless browser files #100

merged 1 commit into from
Sep 17, 2018

Conversation

jarpy
Copy link
Contributor

@jarpy jarpy commented Sep 17, 2018

Kibana installs headless browsers at runtime. We can't control the
permissions of these files at build time, so we should not make
assertions about those permissions.

This patch excludes new files that were added by elastic/kibana#21788.

Fixes #99

@jarpy jarpy requested a review from mgreau September 17, 2018 03:53
@jarpy jarpy added the 6.x label Sep 17, 2018
Kibana installs headless browsers at run time. We can't control the
permissions of these files at build time, so we should not make
assertions about those permissions.

This patch excludes new files that were added by elastic/kibana#21788.

Fixes #99
Copy link
Member

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

LGTM

@jarpy jarpy merged commit c1de1a6 into master Sep 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants