Skip to content

Commit

Permalink
Manifest checker: test for different_case earlier (see #119)
Browse files Browse the repository at this point in the history
This ensures that the test won’t flag the file it just added
for any case where the filename isn’t already lowercased.
  • Loading branch information
acdha committed Jul 9, 2018
1 parent 2b7002e commit 6040c62
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -123,9 +123,9 @@ private static void checkManifestPayload(final Path manifestFile, final Charset
String path = parsePath(line);

path = checkForManifestCreatedWithMD5SumTools(path, warnings, warningsToIgnore);
paths.add(path.toLowerCase());

checkForDifferentCase(path, paths, manifestFile, warnings, warningsToIgnore);
paths.add(path.toLowerCase());

if(encoding.name().startsWith("UTF")){
checkNormalization(path, manifestFile.getParent(), warnings, warningsToIgnore);
}
Expand Down

0 comments on commit 6040c62

Please sign in to comment.