Ryan's feedback rnd3

deephaven · Jan 10, 2024 · f11bed1 · f11bed1
1 parent e820fdd
commit f11bed1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/engine/table/src/main/java/io/deephaven/engine/table/impl/InstrumentedTableUpdateSource.java b/engine/table/src/main/java/io/deephaven/engine/table/impl/InstrumentedTableUpdateSource.java
@@ -22,6 +22,8 @@ public InstrumentedTableUpdateSource(
             final BaseTable<?> table,
             final String description) {
         super(updateSourceRegistrar, description);
+        // verify that the updateSourceRegistrar's update graph is the same as the table's update graph (if applicable)
+        updateSourceRegistrar.getUpdateGraph(table);
         tableReference = new WeakReference<>(table);
     }
 

diff --git a/...st-utils/src/main/java/io/deephaven/engine/testutil/testcase/RefreshingTableTestCase.java b/...st-utils/src/main/java/io/deephaven/engine/testutil/testcase/RefreshingTableTestCase.java
@@ -136,7 +136,7 @@ public <T> T allowingError(Supplier<T> function, Predicate<List<Throwable>> erro
         } finally {
             setExpectError(original);
         }
-        if (errors == null && !errorsAcceptable.test(errors)) {
+        if (errors != null && !errorsAcceptable.test(errors)) {
             TestCase.fail("Unacceptable errors: " + errors);
         }
         return retval;