Skip to content

Commit

Permalink
update range result comments
Browse files Browse the repository at this point in the history
  • Loading branch information
pranav-super committed Oct 21, 2024
1 parent c9b141c commit f787895
Showing 1 changed file with 5 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"A_event",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"A_event",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"A.1",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"a",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"a",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"a",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"a",
Expand Down Expand Up @@ -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<DerivedEvent> expectedResults = List.of(
new DerivedEvent(
"8",
Expand Down Expand Up @@ -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",
Expand Down

0 comments on commit f787895

Please sign in to comment.