You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2022/02/12 15:41:04 UTC

[jena] branch main updated: JENA-2280: Fix tests. Cover all cases in the report

This is an automated email from the ASF dual-hosted git repository.

andy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/jena.git


The following commit(s) were added to refs/heads/main by this push:
     new 3d23f2f  JENA-2280: Fix tests. Cover all cases in the report
     new 95e0213  Merge pull request #1197 from afs/jena2280-fix
3d23f2f is described below

commit 3d23f2f052ba78d18f7a0d8d2d738c74edcb3b1b
Author: Andy Seaborne <an...@apache.org>
AuthorDate: Sat Feb 12 15:29:27 2022 +0000

    JENA-2280: Fix tests. Cover all cases in the report
---
 .../apache/jena/sparql/exec/http/TestService.java  | 31 +++++++++++++++++++---
 1 file changed, 28 insertions(+), 3 deletions(-)

diff --git a/jena-integration-tests/src/test/java/org/apache/jena/sparql/exec/http/TestService.java b/jena-integration-tests/src/test/java/org/apache/jena/sparql/exec/http/TestService.java
index edb0ea6..8bc3487 100644
--- a/jena-integration-tests/src/test/java/org/apache/jena/sparql/exec/http/TestService.java
+++ b/jena-integration-tests/src/test/java/org/apache/jena/sparql/exec/http/TestService.java
@@ -445,14 +445,27 @@ public class TestService {
         QueryExec.dataset(localDataset()).query(queryString).ask();
     }
 
+    // JENA-2280 : No scope renaming. Tests the setup for following tests.
+    @Test public void service_scope_service_0() {
+        String queryString = StrUtils.strjoinNL
+                ("SELECT (?value as ?temp) {"
+                ,"  SELECT ?value {"
+                ,"    SERVICE <"+SERVICE+ "> { VALUES ?value { 'test' }  }"
+                ,"  }"
+                ,"}");
+        RowSet rs = QueryExec.dataset(localDataset()).query(queryString).select().materialize();
+        Binding row = rs.next();
+        assertTrue(row.contains("temp"));
+    }
+
     // JENA-2280
-    // ?value is scped as ?/value and this needs dealing with in SERVCE results.
-    @Test public void service_scope_service_1() {
+    // ?value is scoped as ?/value and this needs dealing with in SERVCE results.
+    @Test public void service_scope_service_2() {
 
         String queryString = StrUtils.strjoinNL
                 ("SELECT ?temp {"
                 ,"  SELECT (?value as ?temp) {"
-                ,"    SERVICE <http://localhost:3030/test/query> { VALUES ?value { 'test' }  }"
+                ,"    SERVICE <"+SERVICE+ "> { VALUES ?value { 'test' }  }"
                 ,"  }"
                 ,"}");
         RowSet rs = QueryExec.dataset(localDataset()).query(queryString).select().materialize();
@@ -460,6 +473,18 @@ public class TestService {
         assertTrue(row.contains("temp"));
     }
 
+    // JENA-2280
+    @Test public void service_scope_service_3() {
+        String queryString = StrUtils.strjoinNL
+                ("SELECT * {"
+                ,"  SELECT (?value as ?temp) {"
+                ,"    SERVICE <"+SERVICE+ "> { VALUES ?value { 'test' }  }"
+                ,"  }"
+                ,"}");
+        RowSet rs = QueryExec.dataset(localDataset()).query(queryString).select().materialize();
+        Binding row = rs.next();
+        assertTrue(row.contains("temp"));
+    }
 
     private static void runWithModifier(String key, HttpRequestModifier modifier, Runnable action) {
         RegistryRequestModifier.get().add(SERVICE, modifier);