(jena) branch main updated: Remove unstable assert

[andy]

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 20a9ced373 Remove unstable assert
20a9ced373 is described below

commit 20a9ced3731a8e5914036e40dc315a01e5fb04eb
Author: Andy Seaborne <a...@apache.org>
AuthorDate: Sun Nov 19 13:24:03 2023 +0000

    Remove unstable assert
---
 .../jena/riot/system/AbstractTestPrefixMap.java    | 64 ++++++++++------------
 1 file changed, 28 insertions(+), 36 deletions(-)

diff --git 
a/jena-arq/src/test/java/org/apache/jena/riot/system/AbstractTestPrefixMap.java 
b/jena-arq/src/test/java/org/apache/jena/riot/system/AbstractTestPrefixMap.java
index e5155b5ef6..87badeebc8 100644
--- 
a/jena-arq/src/test/java/org/apache/jena/riot/system/AbstractTestPrefixMap.java
+++ 
b/jena-arq/src/test/java/org/apache/jena/riot/system/AbstractTestPrefixMap.java
@@ -37,56 +37,50 @@ public abstract class AbstractTestPrefixMap {
     protected abstract PrefixMap getPrefixMap();
 
     @Test
-    public void prefixMap_basic_01()
-    {
+    public void prefixMap_basic_01() {
         PrefixMap pmap = getPrefixMap();
-        assertTrue(pmap.isEmpty()) ;
-        assertEquals(0, pmap.size()) ;
-        assertTrue(pmap.getMapping().isEmpty()) ;
-        assertTrue(pmap.getMappingCopy().isEmpty()) ;
+        assertTrue(pmap.isEmpty());
+        assertEquals(0, pmap.size());
+        assertTrue(pmap.getMapping().isEmpty());
+        assertTrue(pmap.getMappingCopy().isEmpty());
     }
 
     @Test
-    public void prefixMap_basic_02()
-    {
+    public void prefixMap_basic_02() {
         PrefixMap pmap = getPrefixMap();
-        pmap.add("", "http://example/";) ;
-        assertFalse(pmap.isEmpty()) ;
-        assertEquals(1, pmap.size()) ;
-        assertTrue(pmap.containsPrefix("")) ;
-        assertTrue(pmap.getMapping().containsKey("")) ;
-        assertTrue(pmap.getMappingCopy().containsKey("")) ;
+        pmap.add("", "http://example/";);
+        assertFalse(pmap.isEmpty());
+        assertEquals(1, pmap.size());
+        assertTrue(pmap.containsPrefix(""));
+        assertTrue(pmap.getMapping().containsKey(""));
+        assertTrue(pmap.getMappingCopy().containsKey(""));
     }
 
     @Test
-    public void prefixMap_basic_03()
-    {
+    public void prefixMap_basic_03() {
         PrefixMap pmap = getPrefixMap();
-        pmap.add("", "http://example/";) ;
-        pmap.add("org", "http://example.org/";) ;
-        assertTrue(pmap.containsPrefix("")) ;
-        assertTrue(pmap.containsPrefix("org")) ;
-        assertFalse(pmap.isEmpty()) ;
-        assertEquals(2, pmap.size()) ;
+        pmap.add("", "http://example/";);
+        pmap.add("org", "http://example.org/";);
+        assertTrue(pmap.containsPrefix(""));
+        assertTrue(pmap.containsPrefix("org"));
+        assertFalse(pmap.isEmpty());
+        assertEquals(2, pmap.size());
     }
 
     @Test
-    public void prefixMap_basic_04()
-    {
+    public void prefixMap_basic_04() {
         PrefixMap pmap1 = getPrefixMap();
-        pmap1.add("", "http://example/";) ;
-        pmap1.add("org", "http://example.org/";) ;
+        pmap1.add("", "http://example/";);
+        pmap1.add("org", "http://example.org/";);
         PrefixMap pmap2 = getPrefixMap();
-        pmap2.putAll(pmap1) ;
+        pmap2.putAll(pmap1);
 
-        assertTrue(pmap2.containsPrefix("")) ;
-        assertTrue(pmap2.containsPrefix("org")) ;
-        assertFalse(pmap2.isEmpty()) ;
-        assertEquals(2, pmap2.size()) ;
+        assertTrue(pmap2.containsPrefix(""));
+        assertTrue(pmap2.containsPrefix("org"));
+        assertFalse(pmap2.isEmpty());
+        assertEquals(2, pmap2.size());
     }
 
-
-
     /**
      * Simple expand test
      */
@@ -322,11 +316,9 @@ public abstract class AbstractTestPrefixMap {
         String iriStr2 = nsIRI+p.getRight();
         assertEquals(iriStr, iriStr2);
 
-        // Agree with abbreviate? (due to multiple choices, this isn't 
guaranteed).
+        // Does "abbrev" agree with "abbreviate"? There are multiple choices 
in the way the URI is abbreviated
         String x1 = pm.abbreviate(iriStr);
         String abbrevString = ns+":"+ln;
-        assertEquals(x1, abbrevString);
-
         for ( String possible : expected ) {
             if ( possible.equals(abbrevString) ) {
                 assertTrue(iriStr.startsWith(nsIRI));