package to.etc.webapp.testsupport;

import java.util.*;

import javax.annotation.*;

/**
 * Accessor for test-related properties, which "knows" the ways that properties can
 * be gotten: either from a .test.properties file or System.getProperty parameters.
 *
 *
 * @author <a href="mailto:jal@etc.to">Frits Jalvingh</a>
 * Created on Sep 11, 2014
 */
final public class TestProperties {
	@Nonnull
	final private Properties m_properties;

	final private boolean m_present;

	public TestProperties(@Nonnull Properties properties, boolean present) {
		m_properties = properties;
		m_present = present;
	}

	@Nullable
	public String getProperty(@Nonnull String key) {
		String value = System.getProperty(key);
		if(null != value)
			return value;
		return m_properties.getProperty(key);
	}

	@Nullable
	public String getProperty(@Nonnull String key, @Nullable String deflt) {
		String res = getProperty(key);
		return res == null ? deflt : res;
	}

	public boolean isPresent() {
		return m_present;
	}
}

@@ -76,16 +76,54 @@
 ation.*;
+%0Aimport javax.annotation.concurrent.*;
 %0A%0A/**%0A *
@@ -383,16 +383,27 @@
 014%0A */%0A
+@Immutable%0A
 final pu

Fix review comment