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
[BEAM-495] Create General Verifier for File Checksum #772
[BEAM-495] Create General Verifier for File Checksum #772
Conversation
LGTM |
3a455f9
to
dca4f2a
Compare
|
||
thrown.expect(IllegalArgumentException.class); | ||
thrown.expectMessage(containsString("Expected valid checksum, but received")); | ||
new FileChecksumMatcher(null, tmpPath); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line will throw so that means that the line below will never get called
You need to either catch and verify yourself or split this into two tests, one with null and one with ""
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
dca4f2a
to
c560497
Compare
FileChecksumMatcher matcher = | ||
new FileChecksumMatcher( | ||
"90552392c28396935fe4f123bd0b5c2d0f6260c8", | ||
tmpFolder.getRoot().getPath() + "/*"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use IOChannelUtils.resolve or equivalent, your assuming all filesystems use '/' as the separator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
0a0b5e7
to
6e8eb82
Compare
PTAL @lukecwik |
LGTM |
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
[BEAM-<Jira issue #>] Description of pull request
mvn clean verify
. (Even better, enableTravis-CI on your fork and ensure the whole test matrix passes).
<Jira issue #>
in the title with the actual Jira issuenumber, if there is one.
Individual Contributor License Agreement.
JIRA(https://issues.apache.org/jira/browse/BEAM-495)
Generalize FileChecksumMatcher which is from WordCountIT. FileChecksumMatcher can be used in E2E test to verify checksum of output result.