change getAll() to vars()

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<String, Object> getAll() {
-		return Collections.unmodifiableMap(vars);
+	public Map<String, Object> vars() {
+		return vars;
 	}
 
 	@Override

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<String, Object> getAll();
+	Map<String, Object> vars();
 
 	/**
 	 * Sets the value of a variable.

src/test/java/org/scijava/parsington/eval/AbstractEvaluatorTest.java

@@ -84,24 +84,24 @@ 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<String, Object> vars = new HashMap<>();
 		vars.put("a", 1);
 		vars.put("b", "two");
 		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"));
 	}
 }