Create default registerInfosFrom() method

gselzer · ctrueden · commit 0dca4e455260 · 2022-06-30T17:02:03.000-05:00
TODO: Make sure we don't get two instances of OpInfos using this along
with the natural discovery of Ops.
diff --git a/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/OpEnvironment.java b/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/OpEnvironment.java
@@ -225,6 +225,15 @@ default OpBuilder op(final String opName) {
 	 */
 	Collection<OpInfo> infosFrom(Object o);
 
+	/**
+	 * Creates some {@link OpInfo}s from {@code o}
+	 * @param o the {@link Object} to create {@link OpInfo}s from
+	 * @return a {@link Collection} of {@link OpInfo}s
+	 */
+	default void registerInfosFrom(Object o) {
+		register(infosFrom(o));
+	}
+
 	/**
 	 * Sets the {@link Hints} for the {@link OpEnvironment}. Every Call to
 	 * {@link #op} that <b>does not</b> pass a {@link Hints} will <b>copy</b> the
diff --git a/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/impl/TherapiBasedOpTest.java b/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/impl/TherapiBasedOpTest.java
@@ -35,8 +35,8 @@ public static void setUp() {
 			ServiceLoader::load));
 		parser = ServiceLoader.load(Parser.class).findFirst().get();
 		ops = barebonesEnvironment();
-		ops.register(ops.infosFrom(new TherapiBasedOpTest()));
-		ops.register(ops.infosFrom(new TherapiOpClass()));
+		ops.registerInfosFrom(new TherapiBasedOpTest());
+		ops.registerInfosFrom(new TherapiOpClass());
 	}
 
 	@AfterClass
