Ignore blank target frame

2 files changed, 20 insertions, 4 deletions

diff --git a/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java b/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java
index 1ba96d1..f77eec1 100644
--- a/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java
+++ b/src/test/java/org/zwobble/mammoth/tests/docx/BodyXmlTests.java
@@ -757,7 +757,7 @@ public class BodyXmlTests {
             readSuccess(bodyReader(relationships), element),
             isHyperlink(
                 hasHref("http://example.com"),
-                hasBlankAnchor(),
+                hasNoAnchor(),
                 hasChildren(isRun(hasChildren()))
             )
         );
@@ -794,7 +794,7 @@ public class BodyXmlTests {
             readSuccess(bodyReader(), element),
             isHyperlink(
                 hasAnchor("start"),
-                hasBlankHref()
+                hasNoHref()
             )
         );
     }
@@ -820,6 +820,18 @@ public class BodyXmlTests {
     }
 
     @Test
+    public void hyperlinkEmptyTargetFrameIsIgnored() {
+        XmlElement element = element("w:hyperlink", map(
+            "w:anchor", "start",
+            "w:tgtFrame", ""
+        ));
+        assertThat(
+            readSuccess(bodyReader(), element),
+            isHyperlink(hasNoTargetFrame())
+        );
+    }
+
+    @Test
     public void goBackBookmarkIsIgnored() {
         XmlElement element = element("w:bookmarkStart", map("w:name", "_GoBack"));
         assertThat(
diff --git a/src/test/java/org/zwobble/mammoth/tests/docx/DocumentMatchers.java b/src/test/java/org/zwobble/mammoth/tests/docx/DocumentMatchers.java
index d0a1fe6..53632c1 100644
--- a/src/test/java/org/zwobble/mammoth/tests/docx/DocumentMatchers.java
+++ b/src/test/java/org/zwobble/mammoth/tests/docx/DocumentMatchers.java
@@ -91,7 +91,7 @@ public class DocumentMatchers {
         return hasProperty("href", equalTo(Optional.of(href)));
     }
 
-    static Matcher<Hyperlink> hasBlankHref() {
+    static Matcher<Hyperlink> hasNoHref() {
         return hasProperty("href", equalTo(Optional.empty()));
     }
 
@@ -99,11 +99,15 @@ public class DocumentMatchers {
         return hasProperty("anchor", equalTo(Optional.of(anchor)));
     }
 
-    static Matcher<Hyperlink> hasBlankAnchor() {
+    static Matcher<Hyperlink> hasNoAnchor() {
         return hasProperty("anchor", equalTo(Optional.empty()));
     }
 
     static Matcher<Hyperlink> hasTargetFrame(String targetFrame) {
         return hasProperty("targetFrame", equalTo(Optional.of(targetFrame)));
     }
+
+    static Matcher<Hyperlink> hasNoTargetFrame() {
+        return hasProperty("targetFrame", equalTo(Optional.empty()));
+    }
 }