From f787895dc797f56e0719238e16bcac96c6301057 Mon Sep 17 00:00:00 2001 From: pranav-super Date: Mon, 21 Oct 2024 12:55:59 -0700 Subject: [PATCH] update range result comments --- .../jpl/aerie/database/ExternalEventTests.java | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/db-tests/src/test/java/gov/nasa/jpl/aerie/database/ExternalEventTests.java b/db-tests/src/test/java/gov/nasa/jpl/aerie/database/ExternalEventTests.java index bc785a5e92..5e893b109b 100644 --- a/db-tests/src/test/java/gov/nasa/jpl/aerie/database/ExternalEventTests.java +++ b/db-tests/src/test/java/gov/nasa/jpl/aerie/database/ExternalEventTests.java @@ -664,7 +664,6 @@ void testBackground() throws SQLException { var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present final List expectedResults = List.of( new DerivedEvent( "A_event", @@ -827,7 +826,6 @@ void testAmalgamation() throws SQLException { var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present final List expectedResults = List.of( new DerivedEvent( "A_event", @@ -954,7 +952,7 @@ void rule1_solitary() throws SQLException { // ensure the result has the right size and keys final var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present + // the range should only have one element final List expectedResults = List.of( new DerivedEvent( "A.1", @@ -1028,7 +1026,7 @@ void rule1_bookended() throws SQLException { // verify the expected keys are included final var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present + // the ranges should only have a single element and be fully present final List expectedResults = List.of( new DerivedEvent( "a", @@ -1113,7 +1111,7 @@ void rule2() throws SQLException { // verify the expected keys final var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present + // all ranges should only have a single element and be fully present final List expectedResults = List.of( new DerivedEvent( "a", @@ -1371,7 +1369,7 @@ void rule4() throws SQLException { // verify expected keys final var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present + // this range should only have a single element final List expectedResults = List.of( new DerivedEvent( "a", @@ -1478,7 +1476,7 @@ void nEventsAtSameTime() throws SQLException { // all 3 keys should be present! var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present + // all ranges should only have a single element and be fully present final List expectedResults = List.of( new DerivedEvent( "a", @@ -1949,7 +1947,6 @@ void superDerivedEvents() throws SQLException { // verify everything is present var results = getDerivedEvents(); - // both ranges should only have a single element and be fully present List expectedResults = List.of( new DerivedEvent( "8", @@ -2098,7 +2095,6 @@ void superDerivedEvents() throws SQLException { // verify for a given DERIVATION_GROUP expected keys are correct, no overlap inside DERIVATION_GROUP results = getDerivedEvents("WHERE derivation_group_name = '%s' ORDER BY start_time;".formatted(dg2)); - // both ranges should only have a single element and be fully present expectedResults = List.of( new DerivedEvent( "1",