Skip to content

Commit

Permalink
Uniformed fixed issues presentation.
Browse files Browse the repository at this point in the history
  • Loading branch information
Abramo-Bagnara committed Jan 7, 2021
1 parent 13c4980 commit dd229ec
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,8 @@ private Object createNewDetailView(final String link, final Run<?, ?> owner, fin
labelProvider.getLinkName(), url, labelProvider, sourceEncoding);
}
if ("fixed".equalsIgnoreCase(link)) {
return new FixedWarningsDetail(owner, result, fixedIssues, url, labelProvider, sourceEncoding);
return new IssuesDetail(owner, result, fixedIssues, EMPTY, EMPTY,
fixedIssues, Messages.Fixed_Warnings_Header(), url, labelProvider, sourceEncoding);
}
if ("new".equalsIgnoreCase(link)) {
return new IssuesDetail(owner, result, newIssues, newIssues, EMPTY,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,10 @@ void shouldThrowExceptionIfLinkIsNotFound() {
}

@Test
void shouldReturnFixedWarningsDetailWhenCalledWithFixedLink() {
FixedWarningsDetail details = createTrendDetails("fixed", RUN, createResult(),
void shouldReturnFixedIssuesDetailWhenCalledWithFixedLink() {
IssuesDetail details = createTrendDetails("fixed", RUN, createResult(),
ALL_ISSUES, NEW_ISSUES, OUTSTANDING_ISSUES, FIXED_ISSUES, ENCODING, createParent(),
FixedWarningsDetail.class);
IssuesDetail.class);
assertThat(details).hasIssues(FIXED_ISSUES);
assertThat(details).hasFixedIssues(FIXED_ISSUES);
assertThat(details).hasNewIssues(NO_ISSUES);
Expand Down

0 comments on commit dd229ec

Please sign in to comment.