diff --git a/src/main/java/org/scijava/parsington/eval/AbstractEvaluator.java b/src/main/java/org/scijava/parsington/eval/AbstractEvaluator.java index 82502ba..229e993 100644 --- a/src/main/java/org/scijava/parsington/eval/AbstractEvaluator.java +++ b/src/main/java/org/scijava/parsington/eval/AbstractEvaluator.java @@ -30,7 +30,6 @@ package org.scijava.parsington.eval; -import java.util.Collections; import java.util.HashMap; import java.util.Map; @@ -89,9 +88,10 @@ public Object get(final String name) { return new Unresolved(name); } + @Override - public Map getAll() { - return Collections.unmodifiableMap(vars); + public Map vars() { + return vars; } @Override diff --git a/src/main/java/org/scijava/parsington/eval/Evaluator.java b/src/main/java/org/scijava/parsington/eval/Evaluator.java index 1be84d2..4409717 100644 --- a/src/main/java/org/scijava/parsington/eval/Evaluator.java +++ b/src/main/java/org/scijava/parsington/eval/Evaluator.java @@ -168,11 +168,12 @@ default Object get(final Variable v) { } /** - * Gets a map of all variable names and values. + * Gets the Evaluator variables. Not thread-safe. + * A map of all variable names and values. * - * @return A map from variable names to variable values. + * @return The map from variable names to variable values. */ - Map getAll(); + Map vars(); /** * Sets the value of a variable. diff --git a/src/test/java/org/scijava/parsington/eval/AbstractEvaluatorTest.java b/src/test/java/org/scijava/parsington/eval/AbstractEvaluatorTest.java index 62d5fc9..53b443a 100644 --- a/src/test/java/org/scijava/parsington/eval/AbstractEvaluatorTest.java +++ b/src/test/java/org/scijava/parsington/eval/AbstractEvaluatorTest.java @@ -84,16 +84,16 @@ public void testClear() { e.set("a", 1); e.set("b", 2); e.clear(); - assertEquals(new HashMap<>(), e.getAll()); + assertEquals(new HashMap<>(), e.vars()); assertThrows(IllegalArgumentException.class, () -> e.get("a")); assertThrows(IllegalArgumentException.class, () -> e.get("b")); } - /** Tests {@link Evaluator#getAll()} and {@link Evaluator#setAll(Map)}. */ + /** Tests {@link Evaluator#vars()} and {@link Evaluator#setAll(Map)}. */ @Test - public void testGetAllSetAll() { + public void testVarsSetAll() { final Evaluator e = createEvaluator(); - assertEquals(new HashMap<>(), e.getAll()); + assertEquals(new HashMap<>(), e.vars()); final Map vars = new HashMap<>(); vars.put("a", 1); @@ -101,7 +101,7 @@ public void testGetAllSetAll() { vars.put("c", 3.0); e.setAll(vars); - assertEquals(vars, e.getAll()); + assertEquals(vars, e.vars()); // Verify individual get still works after setAll. assertEquals(1, e.get("a")); @@ -110,9 +110,9 @@ public void testGetAllSetAll() { // Verify variables created at evaluation are accessible and correct. e.evaluate("d=a+c"); - assertTrue(e.getAll().containsKey("d")); + assertTrue(e.vars().containsKey("d")); final Object dVal = e.get("d"); assertEquals(4.0, dVal); - assertEquals(dVal, e.getAll().get("d")); + assertEquals(dVal, e.vars().get("d")); } }