From 21828f92330d8245d464b8aa591900e5c28c476b Mon Sep 17 00:00:00 2001
From: j2blake <j2blake@440791e1-c2bf-4c1e-ad7c-caf3f8b0ebe8>
Date: Fri, 20 Apr 2012 20:04:39 +0000
Subject: [PATCH] Remove tests from
 ObjectPropertyTemplateModel_PropertyListConfigTest that had been superseded
 by tests in CustomListViewConfigFileTest

---
 ...yTemplateModel_PropertyListConfigTest.java | 163 +-----------------
 .../individual/testConfig-notValidXml.xml     |  30 ----
 .../testConfig-selectQueryNoSubNodes.xml      |  13 --
 .../testConfig-selectQueryNodeBlank.xml       |  22 ---
 .../testConfig-selectQueryNodeNotFound.xml    |  19 --
 .../testConfig-selectQuerySubNodes.xml        |  14 --
 .../testConfig-templateNodeIsEmpty.xml        |  30 ----
 .../testConfig-templateNodeNotFound.xml       |  29 ----
 8 files changed, 8 insertions(+), 312 deletions(-)
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-notValidXml.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNoSubNodes.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeBlank.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeNotFound.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQuerySubNodes.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeIsEmpty.xml
 delete mode 100644 webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeNotFound.xml

diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/ObjectPropertyTemplateModel_PropertyListConfigTest.java b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/ObjectPropertyTemplateModel_PropertyListConfigTest.java
index e5cf860d0..1ab3f53f2 100644
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/ObjectPropertyTemplateModel_PropertyListConfigTest.java
+++ b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/ObjectPropertyTemplateModel_PropertyListConfigTest.java
@@ -82,22 +82,15 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 		createConfigFile("constructQueryMissing");
 		createConfigFile("constructQueryMultiple");
 		createConfigFile("default");
-		createConfigFile("notValidXml");
 		createConfigFile("postProcessorClassNotFound");
 		createConfigFile("postProcessorClassNotSuitable");
 		createConfigFile("postProcessorConstructorThrowsException");
 		createConfigFile("postProcessorNameEmpty");
 		createConfigFile("postProcessorOK");
 		createConfigFile("postProcessorWrongConstructor");
-		createConfigFile("selectQueryNodeBlank");
-		createConfigFile("selectQueryNodeNotFound");
-		createConfigFile("selectQuerySubNodes");
-		createConfigFile("selectQueryNoSubNodes");
 		createConfigFile("selectQueryCollatedValid");
 		createConfigFile("selectQueryCollatedNoSelect");
 		createConfigFile("selectQueryCollatedNoOrder");
-		createConfigFile("templateNodeIsEmpty");
-		createConfigFile("templateNodeNotFound");
 		createConfigFile("templateDoesNotExist");
 	}
 
@@ -152,12 +145,13 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 	// TODO - baseTemplateModel shouldn't require the servlet context to be set
 	// statically!!! ServletContext shouldn't be a static field.
 
-	// ----------------------------------------------------------------------
-	// The tests
-	//
-	// TODO - remove any tests that are covered by the newer
-	// CustomListViewConfigFileTest.
-	// ----------------------------------------------------------------------
+	/*
+	 * These tests were removed because the newer CustomListViewConfigTest
+	 * covered them: configFileNotValidXml(), selectQueryNodeIsNotFound(),
+	 * selectQueryNodeIsBlank(), selectSubNodesCollatedCritical(),
+	 * selectSubNodesCollatedUncritical(), selectSubNodesUncollatedCritical(),
+	 * selectSubNodesUncollatedUncritical(), selectNoSubNodesCollatedCritical()
+	 */
 
 	//
 	// Null arguments
@@ -227,68 +221,9 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 		assertLogMessagesContains("file not found", "Can't find config file");
 	}
 
-	@Test
-	public void configFileNotValidXml() throws InvalidConfigurationException {
-		suppressSyserr();
-		captureLogsFromPropertyListConfig();
-
-		op = buildOperation("notValidXml");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-
-		assertLogMessagesContains("not valid XML", "SAXParseException");
-	}
-
-	//
-	// Problems with the <query-select> node
-	//
-
-	@Test
-	public void selectQueryNodeIsNotFound()
-			throws InvalidConfigurationException {
-		captureLogsFromPropertyListConfig();
-
-		op = buildOperation("selectQueryNodeNotFound");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-
-		assertLogMessagesContains("no select query",
-				"Missing select query specification");
-	}
-
-	@Test
-	public void selectQueryNodeIsBlank() throws InvalidConfigurationException {
-		captureLogsFromPropertyListConfig();
-
-		op = buildOperation("selectQueryNodeBlank");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-
-		assertLogMessagesContains("blank select query",
-				"Missing select query specification");
-	}
-
 	//
 	// Problems with the <template> node
 	//
-	@Test
-	public void templateNodeNotFound() throws InvalidConfigurationException {
-		captureLogsFromPropertyListConfig();
-
-		op = buildOperation("templateNodeNotFound");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-
-		assertLogMessagesContains("no template node",
-				"Config file must contain a template element");
-	}
-
-	@Test
-	public void templateNodeIsEmpty() throws InvalidConfigurationException {
-		captureLogsFromPropertyListConfig();
-
-		op = buildOperation("templateNodeIsEmpty");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-
-		assertLogMessagesContains("empty template node",
-				"In a config file, the <template> element must not be empty.");
-	}
 
 	@Test
 	public void templateDoesNotExist() throws InvalidConfigurationException {
@@ -302,52 +237,9 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 	}
 
 	//
-	// Optional tags in the select query.
+	// Check for valid query.
 	//
 
-	@Test
-	public void selectSubNodesCollatedCritical()
-			throws InvalidConfigurationException {
-		op = buildOperation("selectQuerySubNodes");
-		optm = new SimpleCollatingOPTM(op, subject, vreq, false);
-		assertSelectQuery("collated, critical",
-				"Plain collated plain critical plain collated plain.");
-	}
-
-	@Test
-	public void selectSubNodesCollatedUncritical()
-			throws InvalidConfigurationException {
-		op = buildOperation("selectQuerySubNodes");
-		optm = new SimpleCollatingOPTM(op, subject, vreq, true);
-		assertSelectQuery("collated, UNcritical",
-				"Plain collated plain plain collated plain.");
-	}
-
-	@Test
-	public void selectSubNodesUncollatedCritical()
-			throws InvalidConfigurationException {
-		op = buildOperation("selectQuerySubNodes");
-		optm = new NonCollatingOPTM(op, subject, vreq, false);
-		assertSelectQuery("UNcollated, critical",
-				"Plain plain critical plain plain.");
-	}
-
-	@Test
-	public void selectSubNodesUncollatedUncritical()
-			throws InvalidConfigurationException {
-		op = buildOperation("selectQuerySubNodes");
-		optm = new NonCollatingOPTM(op, subject, vreq, true);
-		assertSelectQuery("UNcollated, UNcritical", "Plain plain plain plain.");
-	}
-
-	@Test
-	public void selectNoSubNodesCollatedCritical()
-			throws InvalidConfigurationException {
-		op = buildOperation("selectQueryNoSubNodes");
-		optm = new SimpleCollatingOPTM(op, subject, vreq, false);
-		assertSelectQuery("simple collated, critical", "Plain.");
-	}
-
 	@Test
 	public void collatedNoSubclassSelector()
 			throws InvalidConfigurationException {
@@ -509,20 +401,6 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 				+ expected);
 	}
 
-	private void assertSelectQuery(String message, String expected) {
-		String actual = "BOGUS";
-		try {
-			Method m = ObjectPropertyTemplateModel.class.getDeclaredMethod(
-					"getSelectQuery", new Class<?>[0]);
-			m.setAccessible(true);
-			actual = (String) m.invoke(optm, new Object[0]);
-		} catch (Exception e) {
-			fail(message + " - " + e);
-		}
-
-		assertEquals(message, expected, actual);
-	}
-
 	@SuppressWarnings("unchecked")
 	private void assertConstructQueries(String message, String... expectedArray) {
 		Set<String> expected = new HashSet<String>(Arrays.asList(expectedArray));
@@ -587,31 +465,6 @@ public class ObjectPropertyTemplateModel_PropertyListConfigTest extends
 
 	}
 
-	/*
-	 * No populated properties and we don't do syntax checking on the select
-	 * query.
-	 */
-	private class SimpleCollatingOPTM extends
-			CollatedObjectPropertyTemplateModel {
-		SimpleCollatingOPTM(ObjectProperty op, Individual subject,
-				VitroRequest vreq, boolean editing)
-				throws InvalidConfigurationException {
-			super(op, subject, vreq, editing, Collections
-					.<ObjectProperty> emptyList());
-		}
-
-		@Override
-		public ConfigError checkQuery(String queryString) {
-			return null;
-		}
-
-		@Override
-		protected TemplateLoader getFreemarkerTemplateLoader() {
-			return ObjectPropertyTemplateModel_PropertyListConfigTest.this.tl;
-		}
-
-	}
-
 	/** No populated properties but we do check the syntax of the select query. */
 	private class CheckingCollatingOPTM extends
 			CollatedObjectPropertyTemplateModel {
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-notValidXml.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-notValidXml.xml
deleted file mode 100644
index 548af0c27..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-notValidXml.xml
+++ /dev/null
@@ -1,30 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	This would be valid except for the XML error.  
--->
-
-<list-view-config>
-    <query-select>
-        SELECT <collated> ?subclass </collated>
-               ?object
-        WHERE {
-            ?subject ?property ?object
-            <collated>
-            ?object a ?subclass.
-            </collated>
-        }  
-    </query-select>
-
-    <query-construct/>
-        CONSTRUCT {
-            ?subject ?property ?object .
-        } WHERE {
-            ?subject ?property ?object 
-        }
-    </query-construct>
-    
-    <template>propStatement-default.ftl</template>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNoSubNodes.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNoSubNodes.xml
deleted file mode 100644
index 67d048eb6..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNoSubNodes.xml
+++ /dev/null
@@ -1,13 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	Test this with collated and uncollated, critical and uncritical, to insure that we don't need the nested tags.  
--->
-
-<list-view-config>
-    <query-select>Plain.</query-select>
-    <query-construct> Construct </query-construct>
-    <template>propStatement-default.ftl</template>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeBlank.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeBlank.xml
deleted file mode 100644
index b5089efba..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeBlank.xml
+++ /dev/null
@@ -1,22 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	Invalid - <query-select> is blank.  
--->
-
-<list-view-config>
-    <query-select>
-    </query-select>
-
-    <query-construct>
-        CONSTRUCT {
-            ?subject ?property ?object .
-        } WHERE {
-            ?subject ?property ?object 
-        }
-    </query-construct>
-    
-    <template>propStatement-default.ftl</template>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeNotFound.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeNotFound.xml
deleted file mode 100644
index 0c39e7564..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQueryNodeNotFound.xml
+++ /dev/null
@@ -1,19 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	Invalid - no <query-select> node.  
--->
-
-<list-view-config>
-    <query-construct>
-        CONSTRUCT {
-            ?subject ?property ?object .
-        } WHERE {
-            ?subject ?property ?object 
-        }
-    </query-construct>
-    
-    <template>propStatement-default.ftl</template>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQuerySubNodes.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQuerySubNodes.xml
deleted file mode 100644
index d1eb22d33..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-selectQuerySubNodes.xml
+++ /dev/null
@@ -1,14 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	Test this with collated and uncollated, critical and non-critical, 
-	to be sure that we include or omit the appropriate text.  
--->
-
-<list-view-config>
-    <query-select>Plain <collated>collated </collated>plain <critical-data-required>critical </critical-data-required>plain <collated>collated </collated>plain.</query-select>
-    <query-construct> Construct </query-construct>
-    <template>propStatement-default.ftl</template>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeIsEmpty.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeIsEmpty.xml
deleted file mode 100644
index 5b2915975..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeIsEmpty.xml
+++ /dev/null
@@ -1,30 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	Template node is empty - should throw an exception.  
--->
-
-<list-view-config>
-    <query-select>
-        SELECT <collated> ?subclass </collated>
-               ?object
-        WHERE {
-            ?subject ?property ?object
-            <collated>
-            ?object a ?subclass.
-            </collated>
-        }  
-    </query-select>
-
-    <query-construct>
-        CONSTRUCT {
-            ?subject ?property ?object .
-        } WHERE {
-            ?subject ?property ?object 
-        }
-    </query-construct>
-    
-    <template/>
-</list-view-config>
diff --git a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeNotFound.xml b/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeNotFound.xml
deleted file mode 100644
index 6d1a37738..000000000
--- a/webapp/test/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/testConfig-templateNodeNotFound.xml
+++ /dev/null
@@ -1,29 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1"?>
-
-<!-- $This file is distributed under the terms of the license in /doc/license.txt$ -->
-
-<!--
-	No template node. Should throw an exception.  
--->
-
-<list-view-config>
-    <query-select>
-        SELECT <collated> ?subclass </collated>
-               ?object
-        WHERE {
-            ?subject ?property ?object
-            <collated>
-            ?object a ?subclass.
-            </collated>
-        }  
-    </query-select>
-
-    <query-construct>
-        CONSTRUCT {
-            ?subject ?property ?object .
-        } WHERE {
-            ?subject ?property ?object 
-        }
-    </query-construct>
-    
-</list-view-config>