Skip to content

Fixed expected error message part. #36

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 12, 2016
Merged

Fixed expected error message part. #36

merged 1 commit into from
Apr 12, 2016

Conversation

@sormuras
Copy link
Contributor

@sormuras sormuras commented Mar 31, 2016

See error message composition in Main.java line 255 ff:

          errWriter
              .append(fileName)
              .append(": could not read file: ")
              .append(e.getMessage())
...

The e.getMessage() produces an absolute file path on my machine, which yields a failing test.

@cushon cushon merged commit de2bad1 into google:master Apr 12, 2016
@sormuras sormuras deleted the test_fix branch April 13, 2016 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sormuras @cushon @googlebot