From 0eac64e82b4de1e9f6a26d197a7c6032a6ea21f5 Mon Sep 17 00:00:00 2001
From: Aries Lai <aries@gis.tw>
Date: Fri, 13 Jan 2023 14:16:06 +0800
Subject: [PATCH 1/3] Fixes #9 Remove SERVICE_URL parameter

---
 .../org/opengis/cite/wps20/CommonFixture.java   |  4 ++--
 .../cite/wps20/SuiteFixtureListener.java        | 17 +++++++++--------
 .../java/org/opengis/cite/wps20/TestRunArg.java |  7 ++++---
 src/main/scripts/ctl/wps20-suite.ctl            | 11 +----------
 .../cite/wps20/VerifySuiteFixtureListener.java  |  7 ++++---
 .../cite/wps20/VerifyTestNGController.java      |  6 +++---
 6 files changed, 23 insertions(+), 29 deletions(-)

diff --git a/src/main/java/org/opengis/cite/wps20/CommonFixture.java b/src/main/java/org/opengis/cite/wps20/CommonFixture.java
index c141a1d..9afe3bf 100644
--- a/src/main/java/org/opengis/cite/wps20/CommonFixture.java
+++ b/src/main/java/org/opengis/cite/wps20/CommonFixture.java
@@ -72,10 +72,10 @@ public void initCommonFixture(ITestContext testContext) {
         if (null != obj) {
             this.client = Client.class.cast(obj);
         }
-        obj = testContext.getSuite().getAttribute(SuiteAttribute.TEST_SUBJECT.getName());
+        /*obj = testContext.getSuite().getAttribute(SuiteAttribute.TEST_SUBJECT.getName());
         if (null == obj) {
             throw new SkipException("Test subject not found in ITestContext.");
-        }
+        }*/
          
         /* Define SERVICE_URL parameter */
         Object ServiceUrlObj = testContext.getSuite().getAttribute(SuiteAttribute.SERVICE_URL.getName());
diff --git a/src/main/java/org/opengis/cite/wps20/SuiteFixtureListener.java b/src/main/java/org/opengis/cite/wps20/SuiteFixtureListener.java
index ce4c54a..0dcc427 100644
--- a/src/main/java/org/opengis/cite/wps20/SuiteFixtureListener.java
+++ b/src/main/java/org/opengis/cite/wps20/SuiteFixtureListener.java
@@ -57,7 +57,7 @@ public void onFinish(ISuite suite) {
     void processSuiteParameters(ISuite suite) {
         Map<String, String> params = suite.getXmlSuite().getParameters();
         TestSuiteLogger.log(Level.CONFIG, "Suite parameters\n" + params.toString());
-        String iutParam = params.get(TestRunArg.IUT.toString());
+        /*String iutParam = params.get(TestRunArg.IUT.toString());
         if ((null == iutParam) || iutParam.isEmpty()) {
             throw new IllegalArgumentException("Required test run parameter not found: " + TestRunArg.IUT.toString());
         }
@@ -77,16 +77,16 @@ void processSuiteParameters(ISuite suite) {
         } catch (Exception x) {
             throw new RuntimeException("Failed to parse resource retrieved from " + iutRef, x);
         }
-        suite.setAttribute(SuiteAttribute.TEST_SUBJECT.getName(), iutDoc);
+        suite.setAttribute(SuiteAttribute.TEST_SUBJECT.getName(), iutDoc);*/
         
         /* Define SERVICE_URL parameter */
-        String ServiceUrlParam = params.get(TestRunArg.SERVICE_URL.toString());
+        String ServiceUrlParam = params.get(TestRunArg.IUT.toString());
 		if ((null == ServiceUrlParam) || ServiceUrlParam.isEmpty()) {
 			throw new IllegalArgumentException(
-					"Required test run parameter not found: " + TestRunArg.SERVICE_URL.toString());
+					"Required test run parameter not found: " + TestRunArg.IUT.toString());
 		}
-		String ServiceUrlRefString = params.get(TestRunArg.SERVICE_URL.toString());
-		suite.setAttribute(SuiteAttribute.SERVICE_URL.getName(), URI.create(ServiceUrlRefString));
+		//mapping TestRunArg.IUT to SuiteAttribute.SERVICE_URL
+		suite.setAttribute(SuiteAttribute.SERVICE_URL.getName(), URI.create(ServiceUrlParam));
 		
 		/* Define SERVICE_URL parameter */
         String EchoProcessIdParam = params.get(TestRunArg.ECHO_PROCESS_ID.toString());
@@ -136,8 +136,9 @@ void processSuiteParameters(ISuite suite) {
         
         if (TestSuiteLogger.isLoggable(Level.FINE)) {
             StringBuilder logMsg = new StringBuilder("Parsed resource retrieved from ");
-            logMsg.append(iutRef).append("\n");
-            logMsg.append(XMLUtils.writeNodeToString(iutDoc));
+            //logMsg.append(iutRef).append("\n");
+            //logMsg.append(XMLUtils.writeNodeToString(iutDoc));
+            logMsg.append(ServiceUrlParam).append("\n");
             TestSuiteLogger.log(Level.FINE, logMsg.toString());
         }
     }
diff --git a/src/main/java/org/opengis/cite/wps20/TestRunArg.java b/src/main/java/org/opengis/cite/wps20/TestRunArg.java
index 3ea694c..0567dac 100644
--- a/src/main/java/org/opengis/cite/wps20/TestRunArg.java
+++ b/src/main/java/org/opengis/cite/wps20/TestRunArg.java
@@ -10,13 +10,14 @@ public enum TestRunArg {
      * metadata about it.
      */
     IUT,
-	SERVICE_URL,
-	ECHO_PROCESS_ID,
+	ECHO_PROCESS_ID
+	/*SERVICE_URL,
 	GC_XML_URI,
 	DP_XML_URI,
 	EX_SNC_XML_URI,
 	EX_ANC_XML_URI,
-	EX_ATO_XML_URI;
+	EX_ATO_XML_URI*/
+	;
 
     @Override
     public String toString() {
diff --git a/src/main/scripts/ctl/wps20-suite.ctl b/src/main/scripts/ctl/wps20-suite.ctl
index fe09881..5d96e3a 100644
--- a/src/main/scripts/ctl/wps20-suite.ctl
+++ b/src/main/scripts/ctl/wps20-suite.ctl
@@ -47,15 +47,9 @@
                        border-width: medium; padding:4px">Implementation under test</legend>
                <p>
                  <label for="uri">
-                   <h4 style="margin-bottom: 0.5em">Location of IUT (absolute http: or file: URI)</h4>
+                   <h4 style="margin-bottom: 0.5em">Location of IUT (absolute http: URI)</h4>
                  </label>
                  <input id="uri" name="uri" size="128" type="text" value="https://geoprocessing.demo.52north.org/javaps/service" />
-               </p>
-			   <p>
-                 <label for="service_url">
-                   <h4 style="margin-bottom: 0.5em">Endpoint for testing WPS Service</h4>
-                 </label>
-                 <input id="service_url" name="service_url" size="128" type="text" value="https://geoprocessing.demo.52north.org/javaps/service" />
                </p>
 			   <p>
                  <label for="echo_process_id">
@@ -105,9 +99,6 @@
             <xsl:value-of select="normalize-space($form-data/values/value[@key='uri'])"/>
           </entry>
           <entry key="ICS"><xsl:value-of select="$form-data/values/value[@key='level']"/></entry>
-		  <entry key="SERVICE_URL">
-            <xsl:value-of select="normalize-space($form-data/values/value[@key='service_url'])"/>
-          </entry>
 		  <entry key="ECHO_PROCESS_ID">
             <xsl:value-of select="normalize-space($form-data/values/value[@key='echo_process_id'])"/>
           </entry>
diff --git a/src/test/java/org/opengis/cite/wps20/VerifySuiteFixtureListener.java b/src/test/java/org/opengis/cite/wps20/VerifySuiteFixtureListener.java
index addb180..a1b8f25 100644
--- a/src/test/java/org/opengis/cite/wps20/VerifySuiteFixtureListener.java
+++ b/src/test/java/org/opengis/cite/wps20/VerifySuiteFixtureListener.java
@@ -5,6 +5,7 @@
 import static org.mockito.Mockito.when;
 
 import java.io.File;
+import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.URL;
 import java.util.HashMap;
@@ -59,14 +60,14 @@ public void processIUTParameter() throws URISyntaxException {
         URL url = this.getClass().getResource("/atom-feed.xml");
         Map<String, String> params = new HashMap<String, String>();
         params.put(TestRunArg.IUT.toString(), url.toURI().toString());
-        params.put(TestRunArg.SERVICE_URL.toString(), url.toURI().toString());
+        //params.put(TestRunArg.SERVICE_URL.toString(), url.toURI().toString());
         params.put(TestRunArg.ECHO_PROCESS_ID.toString(), url.toURI().toString());
         when(xmlSuite.getParameters()).thenReturn(params);
         SuiteFixtureListener iut = new SuiteFixtureListener();
         iut.onStart(suite);
         verify(suite).setAttribute(
-                Matchers.eq(SuiteAttribute.TEST_SUBJ_FILE.getName()), 
-                Matchers.isA(File.class));
+                Matchers.eq(SuiteAttribute.SERVICE_URL.getName()), 
+                Matchers.isA(URI.class));
     }
 
 }
diff --git a/src/test/java/org/opengis/cite/wps20/VerifyTestNGController.java b/src/test/java/org/opengis/cite/wps20/VerifyTestNGController.java
index 921361a..93af3f9 100644
--- a/src/test/java/org/opengis/cite/wps20/VerifyTestNGController.java
+++ b/src/test/java/org/opengis/cite/wps20/VerifyTestNGController.java
@@ -53,9 +53,9 @@ public void loadDefaultTestRunProperties()
 
     @Test
     public void doTestRun() throws Exception {
-        URL testSubject = getClass().getResource("/atom-feed-2.xml");
-        this.testRunProps.setProperty(TestRunArg.IUT.toString(), testSubject
-                .toURI().toString());
+        //URL testSubject = getClass().getResource("/atom-feed-2.xml");
+        //this.testRunProps.setProperty(TestRunArg.IUT.toString(), testSubject
+        //        .toURI().toString());
         ByteArrayOutputStream outStream = new ByteArrayOutputStream(1024);
         this.testRunProps.storeToXML(outStream, "Integration test");
         Document testRunArgs = docBuilder.parse(new ByteArrayInputStream(

From 4295efd64ac1c5954034a0e005f69d35031f80a3 Mon Sep 17 00:00:00 2001
From: Aries Lai <aries@gis.tw>
Date: Fri, 13 Jan 2023 14:16:26 +0800
Subject: [PATCH 2/3] Update changelog.adoc

---
 src/site/asciidoc/changelog.adoc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/site/asciidoc/changelog.adoc b/src/site/asciidoc/changelog.adoc
index 83553fe..a0c9c29 100644
--- a/src/site/asciidoc/changelog.adoc
+++ b/src/site/asciidoc/changelog.adoc
@@ -1,4 +1,9 @@
 = Release Notes
+== 0.3 (2023-01-16)
+* [#26](https://github.com/opengeospatial/ets-wps20/issues/26) Enhance documentation by description of basic conformance classes
+* [#21](https://github.com/opengeospatial/ets-wps20/issues/21) Organise the packages into separate Conformance Classes
+* [#9](https://github.com/opengeospatial/ets-wps20/issues/9) Remove SERVICE_URL parameter
+* [#8](https://github.com/opengeospatial/ets-wps20/issues/8) Separate the tests into different java packages
 
 == 0.2 (2022-11-23)
 * [#23](https://github.com/opengeospatial/ets-wps20/issues/23) Update documentation for v0.2

From b3ec635d0dcfbe2595c0e66b0819c480d7805414 Mon Sep 17 00:00:00 2001
From: Gobe Hobona <ghobona@opengeospatial.org>
Date: Fri, 13 Jan 2023 08:04:32 +0000
Subject: [PATCH 3/3] updated documentation regarding certification

---
 src/main/scripts/ctl/wps20-suite.ctl | 5 +++++
 src/site/asciidoc/index.adoc         | 5 +++++
 2 files changed, 10 insertions(+)

diff --git a/src/main/scripts/ctl/wps20-suite.ctl b/src/main/scripts/ctl/wps20-suite.ctl
index 5d96e3a..40f50e6 100644
--- a/src/main/scripts/ctl/wps20-suite.ctl
+++ b/src/main/scripts/ctl/wps20-suite.ctl
@@ -40,6 +40,11 @@
                  <li>Synchronous WPS</li>
                  <li>Asynchronous WPS</li>
                </ul>
+               <p>To quality for Compliance certification, a product has to:</p>
+               <ul>
+                 <li>Pass both the Basic and Synchronous WPS tests, or</li>
+                 <li>Pass both the Basic and Asynchronous WPS tests</li>
+               </ul>               
              </div>
              <fieldset style="background:#ccffff">
                <legend style="font-family: sans-serif; color: #000099; 
diff --git a/src/site/asciidoc/index.adoc b/src/site/asciidoc/index.adoc
index 8307f7f..000b267 100644
--- a/src/site/asciidoc/index.adoc
+++ b/src/site/asciidoc/index.adoc
@@ -18,6 +18,11 @@ Support for the following conformance classes is tested:
 * Synchronous WPS 
 * Asynchronous WPS
 
+To quality for Compliance certification, a product has to:
+
+* Pass both the Basic and Synchronous WPS tests, or
+* Pass both the Basic and Asynchronous WPS tests
+
 == Test requirements
 
 The documents listed below stipulate requirements that must be satisfied by a