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

Technical Debt: add unit test WildcardPatternFileProviderTest #1354

Merged
merged 2 commits into from
Dec 17, 2017

Conversation

Bertk
Copy link
Contributor

@Bertk Bertk commented Dec 16, 2017

This change is Reviewable

@Bertk Bertk requested a review from guwirth December 16, 2017 13:14
@Bertk Bertk self-assigned this Dec 16, 2017
@Bertk Bertk added this to the 0.9.9 milestone Dec 16, 2017
Copy link
Collaborator

@guwirth guwirth left a comment

Choose a reason for hiding this comment

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

@Bertk looks good except absolute_paths_with_current_and_parent_folder_access


@Test
public void absolute_paths_with_current_and_parent_folder_access() {
assertThat(listFiles(new File(tmp.getRoot(), path("a", "..", ".", "foo.txt")).getAbsolutePath()))
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Bertk testing three times the same thing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This finding is correct. I removed the redundant statements.

@guwirth guwirth merged commit eb620d0 into SonarOpenCommunity:master Dec 17, 2017
@guwirth
Copy link
Collaborator

guwirth commented Dec 17, 2017

@Bertk thx

@guwirth guwirth changed the title add unit test WildcardPatternFileProviderTest Technical Debt: add unit test WildcardPatternFileProviderTest Jan 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants