Skip to content
This repository has been archived by the owner on Jul 10, 2024. It is now read-only.

Commit

Permalink
EXP-660: Fix incorrect conflict resolution from merge/rebase
Browse files Browse the repository at this point in the history
Fix unit tests from 100% allocation suppression
  • Loading branch information
zacharygoodwin authored Jun 6, 2024
1 parent 571b9c9 commit 180e32c
Showing 1 changed file with 5 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ public void testToLoggingString() {
.isEmpty();
assertThat(sampleGroups.toLoggingString())
.isEqualTo(
"#A1:abtst1,#A1:bgtst0,#A1:groupwithfallbacktst2,#A1:no_definition_tst2,#A1:propertytest2");
"#A1:abtst1,#A1:bgtst0,#A1:groupwithfallbacktst2,#A1:no_definition_tst2");
}

@Test
Expand Down Expand Up @@ -317,7 +317,7 @@ public void testToLoggingStringWithExposureAndObserver() {

// getGroupsString and getAsProctorResult should not mark tests as used
final String fullLoggingString =
"#A1:abtst1,#A1:bgtst0,#A1:groupwithfallbacktst2,#A1:no_definition_tst2,#A1:propertytest2";
"#A1:abtst1,#A1:bgtst0,#A1:groupwithfallbacktst2,#A1:no_definition_tst2";
assertThat(sampleGroups.getAsProctorResult()).isNotNull();
assertThat(sampleGroups.toLoggingString()).isEqualTo(fullLoggingString);
assertThat(sampleGroups.toLongString()).isNotBlank();
Expand Down Expand Up @@ -354,8 +354,7 @@ public void testGetLoggingTestNames() {
CONTROL_SELECTED_TEST.getName(),
GROUP1_SELECTED_TEST.getName(),
GROUP_WITH_FALLBACK_TEST.getName(),
MISSING_DEFINITION_TEST.getName(),
PROPERTY_TEST.getName());
MISSING_DEFINITION_TEST.getName());
}

@Test
Expand Down Expand Up @@ -385,8 +384,7 @@ public void testAppendTestGroups() {
"#A1:bgtst0",
"#A1:abtst1",
"#A1:groupwithfallbacktst2",
"#A1:no_definition_tst2",
"#A1:propertytest2");
"#A1:no_definition_tst2");
}

@Test
Expand All @@ -395,7 +393,7 @@ public void testGetJavaScriptConfig() {
assertThat(emptyGroup.getJavaScriptConfig()).hasSize(0);

assertThat(sampleGroups.getJavaScriptConfig())
.hasSize(5)
.hasSize(6)
.containsEntry(GROUP1_SELECTED_TEST.getName(), 1)
.containsEntry(CONTROL_SELECTED_TEST.getName(), 0)
.containsEntry(GROUP_WITH_FALLBACK_TEST.getName(), 2)
Expand Down

0 comments on commit 180e32c

Please sign in to comment.