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

stelligent/cfn_nag#416 Explicit encoding when reading files in rspec tests #76

Merged
6 commits merged into from
Mar 26, 2020

Conversation

pshelby
Copy link
Contributor

@pshelby pshelby commented Mar 26, 2020

Updates stelligent/cfn_nag#416

Description

Fixes GitHub Action workflow failure.

GitHub Action runners seem to use UTF-8 as the default file encoding, which conflicted with an assumption made for two of the rspec tests. This PR explicitly sets the file encoding when reading in the file during those tests.

Testing

  1. All rspec tests now pass locally, via nektos/act, and on ubuntu-latest GitHub runners.

@pshelby pshelby added the bug label Mar 26, 2020
@pshelby pshelby requested a review from a user March 26, 2020 15:17
@pshelby pshelby self-assigned this Mar 26, 2020
@ghost ghost merged commit e46fdb5 into stelligent:master Mar 26, 2020
@pshelby pshelby deleted the feature/encoding-handling branch March 26, 2020 18:42
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant