Skip to content

Commit 751c2d5

Browse files
authored
Fix reversed assertEquals parameter order in Long modification tests (#237)
Fixed incorrect parameter order in assertEquals calls where the actual value was placed before the expected value. The JUnit convention expects the expected value as the first parameter. Added 'L' suffix to numeric literals to ensure proper Long type matching. Related to tls-attacker/TLS-Attacker-Development#1321
1 parent f4072ac commit 751c2d5

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/test/java/de/rub/nds/modifiablevariable/mlong/LongAddModificationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ void testToString() {
8787
void testConstructors() {
8888
LongAddModification constructor = new LongAddModification(5L);
8989
assertNotNull(constructor);
90-
assertEquals(constructor.getSummand(), 5);
90+
assertEquals(5L, constructor.getSummand());
9191

9292
// Test copy constructor
9393
LongAddModification copy = new LongAddModification(modification);

src/test/java/de/rub/nds/modifiablevariable/mlong/LongExplicitValueModificationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ void testConstructors() {
9191
// Test default constructor
9292
LongExplicitValueModification constructor = new LongExplicitValueModification(5L);
9393
assertNotNull(constructor);
94-
assertEquals(constructor.getExplicitValue(), 5);
94+
assertEquals(5L, constructor.getExplicitValue());
9595

9696
// Test copy constructor
9797
LongExplicitValueModification copy = new LongExplicitValueModification(modification);

0 commit comments

Comments
 (0)