AnchoredSpreadsheetSelection.urlFragment leading slash added

- inserted sample comment into AnchoredSpreadsheetSelection#urlFragment

Author: mP1

src/main/java/walkingkooka/spreadsheet/viewport/AnchoredSpreadsheetSelection.java

@@ -106,16 +106,21 @@ public void printTree(final IndentingPrinter printer) {
 
     // HasUrlFragment...................................................................................................
 
+    // /A1
+    // /A2:B3/top-left
+
     @Override
     public UrlFragment urlFragment() {
         final UrlFragment selection = this.selection()
             .urlFragment();
         final SpreadsheetViewportAnchor anchor = this.anchor();
 
-        return SpreadsheetViewportAnchor.NONE != anchor ?
-            selection.append(UrlFragment.SLASH)
-                .append(anchor.urlFragment()) :
-            selection;
+        return UrlFragment.SLASH.append(
+            SpreadsheetViewportAnchor.NONE != anchor ?
+                selection.append(UrlFragment.SLASH)
+                    .append(anchor.urlFragment()) :
+                selection
+        );
     }
 
     // Object..........................................................................................................

src/test/java/walkingkooka/spreadsheet/viewport/AnchoredSpreadsheetSelectionTest.java

@@ -20,7 +20,7 @@
 import org.junit.jupiter.api.Test;
 import walkingkooka.HashCodeEqualsDefinedTesting2;
 import walkingkooka.ToStringTesting;
-import walkingkooka.net.UrlFragment;
+import walkingkooka.net.HasUrlFragmentTesting;
 import walkingkooka.reflect.ClassTesting;
 import walkingkooka.reflect.JavaVisibility;
 import walkingkooka.spreadsheet.reference.SpreadsheetSelection;
@@ -36,7 +36,8 @@ public final class AnchoredSpreadsheetSelectionTest implements ClassTesting<Anch
     HashCodeEqualsDefinedTesting2<AnchoredSpreadsheetSelection>,
     ToStringTesting<AnchoredSpreadsheetSelection>,
     JsonNodeMarshallingTesting<AnchoredSpreadsheetSelection>,
-    TreePrintableTesting {
+    TreePrintableTesting,
+    HasUrlFragmentTesting {
 
     private final static SpreadsheetSelection SELECTION = SpreadsheetSelection.parseCellRange("A1:B2");
     private final static SpreadsheetViewportAnchor ANCHOR = SpreadsheetViewportAnchor.BOTTOM_RIGHT;
@@ -202,21 +203,30 @@ public void testTreePrint() {
     }
 
     // HasUrlFragment...................................................................................................
+
     @Test
-    public void testHasUrlFragmentCell() {
-        this.checkEquals(
-            UrlFragment.parse("A1"),
-            SpreadsheetSelection.A1
-                .urlFragment()
+    public void testUrlFragmentCellWithCell() {
+        this.urlFragmentAndCheck(
+            SpreadsheetSelection.A1.setDefaultAnchor(),
+            "/A1"
         );
     }
 
     @Test
-    public void testHasUrlFragmentCellRange() {
-        this.checkEquals(
-            UrlFragment.parse("A1:B2/bottom-right"),
-            this.createObject()
-                .urlFragment()
+    public void testUrlFragmentCellWithCellRange() {
+        this.urlFragmentAndCheck(
+            SpreadsheetSelection.parseCellRange("B2:C3")
+                .setAnchor(SpreadsheetViewportAnchor.TOP_LEFT),
+            "/B2:C3/top-left"
+        );
+    }
+
+    @Test
+    public void testUrlFragmentCellWithLabel() {
+        this.urlFragmentAndCheck(
+            SpreadsheetSelection.labelName("Label123")
+                .setDefaultAnchor(),
+            "/Label123"
         );
     }
 

src/test/java/walkingkooka/spreadsheet/viewport/SpreadsheetViewportTest.java

@@ -427,7 +427,7 @@ public void testTreePrintRectangleAnchoredSelectionRowRangeNavigations() {
     public void testUrlFragmentCell() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.A1.setDefaultAnchor(),
-            "A1"
+            "/A1"
         );
     }
 
@@ -436,7 +436,7 @@ public void testUrlFragmentCellRangeTopLeft() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseCellRange("B2:C3")
                 .setAnchor(SpreadsheetViewportAnchor.TOP_LEFT),
-            "B2:C3/top-left"
+            "/B2:C3/top-left"
         );
     }
 
@@ -445,7 +445,7 @@ public void testUrlFragmentCellRangeTopRight() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseCellRange("B2:C3")
                 .setAnchor(SpreadsheetViewportAnchor.TOP_RIGHT),
-            "B2:C3/top-right"
+            "/B2:C3/top-right"
         );
     }
 
@@ -454,7 +454,7 @@ public void testUrlFragmentColumnNone() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseColumn("Z")
                 .setDefaultAnchor(),
-            "Z"
+            "/Z"
         );
     }
 
@@ -463,7 +463,7 @@ public void testUrlFragmentColumnRangeLeft() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseColumnRange("X:Y")
                 .setAnchor(SpreadsheetViewportAnchor.LEFT),
-            "X:Y/left"
+            "/X:Y/left"
         );
     }
 
@@ -472,7 +472,7 @@ public void testUrlFragmentColumnRangeRight() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseColumnRange("X:Y")
                 .setAnchor(SpreadsheetViewportAnchor.RIGHT),
-            "X:Y/right"
+            "/X:Y/right"
         );
     }
 
@@ -481,7 +481,7 @@ public void testUrlFragmentLabelNone() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseCellOrLabel("Label123")
                 .setDefaultAnchor(),
-            "Label123"
+            "/Label123"
         );
     }
 
@@ -490,7 +490,7 @@ public void testUrlFragmentLabelBottomRight() {
         this.urlFragmentAndCheck(
             SpreadsheetSelection.parseCellOrLabel("Label123")
                 .setAnchor(SpreadsheetViewportAnchor.BOTTOM_RIGHT),
-            "Label123/bottom-right"
+            "/Label123/bottom-right"
         );
     }