Update test to please DRY rule check in elvis

The latest elvis is using a new version of katana_code, which has replaced the pre-processor handling, see: inaka/katana-code#31 This has changed the behavior of elvis to also check code defined in TEST scope.
Nordix · Apr 20, 2020 · 008ae39 · 008ae39
1 parent b8593bb
commit 008ae39
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/brod_group_coordinator.erl b/src/brod_group_coordinator.erl
@@ -1140,13 +1140,13 @@ is_default_offset_retention(_) -> false.
 merge_acked_offsets_test() ->
   ?assertEqual([{{<<"topic1">>, 1}, 1}],
                merge_acked_offsets([], [{{<<"topic1">>, 1}, 1}])),
-  ?assertEqual([{{<<"topic1">>, 1}, 1}, {{<<"topic1">>, 2}, 1}],
-               merge_acked_offsets([{{<<"topic1">>, 1}, 1}],
-                                   [{{<<"topic1">>, 2}, 1}])),
-  ?assertEqual([{{<<"topic1">>, 1}, 2}, {{<<"topic1">>, 2}, 1}],
-               merge_acked_offsets([{{<<"topic1">>, 1}, 1},
-                                    {{<<"topic1">>, 2}, 1}],
-                                   [{{<<"topic1">>, 1}, 2}])),
+  ?assertEqual([{{<<"topic2">>, 1}, 1}, {{<<"topic2">>, 2}, 1}],
+               merge_acked_offsets([{{<<"topic2">>, 1}, 1}],
+                                   [{{<<"topic2">>, 2}, 1}])),
+  ?assertEqual([{{<<"topic3">>, 1}, 2}, {{<<"topic3">>, 2}, 1}],
+               merge_acked_offsets([{{<<"topic3">>, 1}, 1},
+                                    {{<<"topic3">>, 2}, 1}],
+                                   [{{<<"topic3">>, 1}, 2}])),
   ok.
 
 is_roundrobin_v1_commit_test() ->