Skip to content

Commit

Permalink
added comment to overriding compareResult
Browse files Browse the repository at this point in the history
  • Loading branch information
garyluoex committed Sep 8, 2016
1 parent d26b632 commit ac4c349
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ class LookupDimensionGroupingDataServletSpec extends BaseDataServletComponentSpe

@Override
boolean compareResult(String result, String expectedResult, JsonSortStrategy sortStrategy) {
// Using default SORT_MAPS provided by Util
// Override "test druid query" feature test's compareResult which calls with SORT_BOTH to be default SORT_MAP
GroovyTestUtils.compareJson(result, expectedResult)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class LookupDimensionGroupingFilteringDataServletSpec extends BaseDataServletCom

@Override
boolean compareResult(String result, String expectedResult, JsonSortStrategy sortStrategy) {
// Using default SORT_MAPS provided by Util
// Override "test druid query" feature test's compareResult which calls with SORT_BOTH to be default SORT_MAP
GroovyTestUtils.compareJson(result, expectedResult)
}

Expand Down

0 comments on commit ac4c349

Please sign in to comment.