More magic names

Author: twogee

src/main/org/apache/tools/ant/MagicNames.java

@@ -218,7 +218,7 @@ private MagicNames() {
      * Value: {@value}
      */
     public static final String PROJECT_HELPER_SERVICE =
-        "META-INF/services/org.apache.tools.ant.ProjectHelper";
+        "META-INF/services/" + PROJECT_HELPER_CLASS;
 
     /**
      * Name of ProjectHelper reference that we add to a project.

src/main/org/apache/tools/ant/taskdefs/Definer.java

@@ -465,7 +465,7 @@ public void setAntlib(String antlib) {
         }
         if (!antlib.startsWith(MagicNames.ANTLIB_PREFIX)) {
             throw new BuildException(
-                "Invalid antlib attribute - it must start with antlib:");
+                "Invalid antlib attribute - it must start with " + MagicNames.ANTLIB_PREFIX);
         }
         setURI(antlib);
         this.resource = antlib.substring(MagicNames.ANTLIB_PREFIX.length()).replace('.', '/')

src/tests/junit/org/apache/tools/ant/ExecutorTest.java

@@ -92,7 +92,7 @@ private Project getProject(String e, boolean f) {
 
     private Project getProject(String e, boolean f, boolean k) {
         Project p = buildRule.getProject();
-        p.setNewProperty("ant.executor.class", e);
+        p.setNewProperty(MagicNames.ANT_EXECUTOR_CLASSNAME, e);
         p.setKeepGoingMode(k);
         if (f) {
             p.setNewProperty("failfoo", "foo");

src/tests/junit/org/apache/tools/ant/taskdefs/optional/EchoPropertiesTest.java

@@ -37,6 +37,7 @@
 
 import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.BuildFileRule;
+import org.apache.tools.ant.MagicNames;
 import org.apache.tools.ant.util.regexp.RegexpMatcherFactory;
 import org.junit.After;
 import org.junit.Before;
@@ -182,7 +183,7 @@ public void testWithRegex() {
         buildRule.executeTarget("testWithRegex");
         // the following line has been changed from checking ant.home to ant.version
         // so the test will still work when run outside of an Ant script
-        assertThat(buildRule.getFullLog(), containsString("ant.version="));
+        assertThat(buildRule.getFullLog(), containsString(MagicNames.ANT_VERSION + "="));
     }
 
     private void testEchoPrefixVarious(String target) throws Exception {