SpreadsheetMetadataPropertyNameViewportTest.testParseUrlFragmentSaveValue2

Author: mP1

src/main/java/walkingkooka/spreadsheet/meta/SpreadsheetMetadataPropertyNameViewport.java

@@ -19,6 +19,7 @@
 
 
 import walkingkooka.locale.LocaleContext;
+import walkingkooka.net.UrlFragment;
 import walkingkooka.spreadsheet.reference.SpreadsheetSelection;
 import walkingkooka.spreadsheet.viewport.SpreadsheetViewport;
 
@@ -77,6 +78,8 @@ void accept(final SpreadsheetViewport value,
 
     @Override
     SpreadsheetViewport parseUrlFragmentSaveValueNonNull(final String value) {
-        return this.failParseUrlFragmentSaveValueUnsupported();
+        return SpreadsheetViewport.fromUrlFragment(
+            UrlFragment.parse(value)
+        );
     }
 }

src/test/java/walkingkooka/spreadsheet/meta/SpreadsheetMetadataPropertyNameTestCase.java

@@ -182,32 +182,41 @@ final void extractLocaleValueAwareAndCheck(final LocaleContext context,
     public final void testParseUrlFragmentSaveValue() {
         final SpreadsheetMetadataPropertyName<V> propertyName = this.createName();
 
-        final String text;
-
-        final V value = this.propertyValue();
-        if (value instanceof HasUrlFragment) {
-            final HasUrlFragment has = (HasUrlFragment) value;
-            text = has.urlFragment().value();
-        } else {
-            if (value instanceof HasText) {
-                final HasText has = (HasText) value;
-                text = has.text();
+        if (false == propertyName instanceof SpreadsheetMetadataPropertyNameAuditInfo &&
+            false == propertyName instanceof SpreadsheetMetadataPropertyNameSpreadsheetId &&
+            false == propertyName instanceof SpreadsheetMetadataPropertyNameStyle) {
+            final String text;
+
+            final V value = this.propertyValue();
+            if (value instanceof HasUrlFragment) {
+                final HasUrlFragment has = (HasUrlFragment) value;
+                text = has.urlFragment().value();
             } else {
-                text = String.valueOf(value);
+                if (value instanceof HasText) {
+                    final HasText has = (HasText) value;
+                    text = has.text();
+                } else {
+                    text = String.valueOf(value);
+                }
             }
-        }
 
-        try {
-            this.checkEquals(
-                value,
-                propertyName.parseUrlFragmentSaveValue(text),
-                () -> "parseUrlFragmentSaveValue " + CharSequences.quoteAndEscape(text)
+            this.parseUrlFragmentSaveValueAndCheck(
+                propertyName,
+                text,
+                value
             );
-        } catch (final UnsupportedOperationException ignore) {
-
         }
     }
 
+    final void parseUrlFragmentSaveValueAndCheck(final SpreadsheetMetadataPropertyName<V> propertyName,
+                                                 final String urlFragment,
+                                                 final V value) {
+        this.checkEquals(
+            value,
+            propertyName.parseUrlFragmentSaveValue(urlFragment)
+        );
+    }
+
     // isConverterSelector...................................................................................
 
     @Test

src/test/java/walkingkooka/spreadsheet/meta/SpreadsheetMetadataPropertyNameViewportTest.java

@@ -23,6 +23,8 @@
 import walkingkooka.spreadsheet.reference.SpreadsheetSelection;
 import walkingkooka.spreadsheet.viewport.SpreadsheetViewport;
 import walkingkooka.spreadsheet.viewport.SpreadsheetViewportAnchor;
+import walkingkooka.spreadsheet.viewport.SpreadsheetViewportNavigationList;
+import walkingkooka.spreadsheet.viewport.SpreadsheetViewportRectangle;
 
 import java.util.Locale;
 import java.util.Optional;
@@ -104,6 +106,31 @@ public void testExtractLocaleAwareValue() {
         );
     }
 
+    @Test
+    public void testParseUrlFragmentSaveValue2() {
+        final SpreadsheetViewport viewport = SpreadsheetViewportRectangle.with(
+                SpreadsheetSelection.A1,
+                200,
+                300
+            ).viewport()
+            .setIncludeFrozenColumnsRows(true)
+            .setAnchoredSelection(
+                Optional.of(
+                    SpreadsheetSelection.parseCellRange("B2:C3")
+                        .setAnchor(SpreadsheetViewportAnchor.TOP_LEFT)
+                )
+            ).setNavigations(
+                SpreadsheetViewportNavigationList.parse("right 400px")
+            );
+
+        this.parseUrlFragmentSaveValueAndCheck(
+            SpreadsheetMetadataPropertyNameViewport.instance(),
+            viewport.urlFragment()
+                .value(),
+            viewport
+        );
+    }
+
     @Test
     public void testToString() {
         this.toStringAndCheck(