Check trailing spaces before line-break

unfoldingWord · Nov 9, 2020 · b600d85 · b600d85
1 parent 5fe96ab
commit b600d85
Show file tree

Hide file tree

Showing 6 changed files with 88 additions and 58 deletions.
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "uw-content-validation",
   "description": "Functions for Checking Door43.org Scriptural Content/Resources.",
-  "version": "0.8.17_alpha14",
+  "version": "0.8.17_alpha15",
   "private": false,
   "homepage": "https://unfoldingword.github.io/uw-content-validation/",
   "repository": {

diff --git a/src/__tests__/__snapshots__/book-package-check.test.js.snap b/src/__tests__/__snapshots__/book-package-check.test.js.snap
@@ -6977,6 +6977,24 @@ Object {
       "rowID": "ab38",
       "username": "unfoldingWord",
     },
+    Object {
+      "C": "4",
+      "V": "intro",
+      "bookID": "RUT",
+      "characterIndex": 407,
+      "extra": "TN1",
+      "extract": "…in␣him.␣<br><br…",
+      "fieldName": "OccurrenceNote",
+      "filename": "en_tn_08-RUT.tsv",
+      "lineNumber": 205,
+      "location": " in en RUT book package from unfoldingWord master branch",
+      "message": "Unexpected trailing space(s) before break",
+      "priority": 94,
+      "repoCode": "TN1",
+      "repoName": "en_tn",
+      "rowID": "pz6m",
+      "username": "unfoldingWord",
+    },
     Object {
       "C": "4",
       "V": "1",

diff --git a/src/__tests__/__snapshots__/tn-table-row-check.test.js.snap b/src/__tests__/__snapshots__/tn-table-row-check.test.js.snap
@@ -13,7 +13,7 @@ Object {
       "location": " that was supplied",
       "message": "Wrong book identifier",
       "priority": 978,
-      "rowID": "d7qw",
+      "rowID": "o7qw",
     },
   ],
 }
@@ -60,7 +60,7 @@ Object {
       "location": " that was supplied",
       "message": "Link to TA should also be in OccurrenceNote",
       "priority": 787,
-      "rowID": "d7qw",
+      "rowID": "n7qw",
     },
     Object {
       "C": "1",
@@ -70,7 +70,7 @@ Object {
       "location": " that was supplied",
       "message": "Missing OccurrenceNote field",
       "priority": 274,
-      "rowID": "d7qw",
+      "rowID": "n7qw",
     },
   ],
 }
@@ -88,7 +88,7 @@ Object {
       "location": " that was supplied",
       "message": "Link to TA should also be in OccurrenceNote",
       "priority": 787,
-      "rowID": "d7qw",
+      "rowID": "m7qw",
     },
     Object {
       "C": "1",
@@ -98,7 +98,7 @@ Object {
       "location": " that was supplied",
       "message": "Field is only whitespace",
       "priority": 373,
-      "rowID": "d7qw",
+      "rowID": "m7qw",
     },
   ],
 }
@@ -145,7 +145,7 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "b7qw",
     },
   ],
 }
@@ -164,7 +164,7 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected leading space",
       "priority": 109,
-      "rowID": "d7qw",
+      "rowID": "e7qw",
     },
     Object {
       "C": "1",
@@ -177,10 +177,10 @@ Object {
       "location": " that was supplied",
       "message": "Seems original language quote might not start at the beginning of a word",
       "priority": 620,
-      "rowID": "d7qw",
+      "rowID": "e7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	e7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -197,7 +197,7 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected leading word joiner",
       "priority": 770,
-      "rowID": "d7qw",
+      "rowID": "g7qw",
     },
     Object {
       "C": "1",
@@ -209,10 +209,10 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "g7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	g7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -229,7 +229,7 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected leading zero-width joiner",
       "priority": 771,
-      "rowID": "d7qw",
+      "rowID": "k7qw",
     },
     Object {
       "C": "1",
@@ -241,10 +241,10 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "k7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	k7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -261,7 +261,7 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "i7qw",
     },
   ],
 }
@@ -280,10 +280,10 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected trailing space(s)",
       "priority": 95,
-      "rowID": "d7qw",
+      "rowID": "f7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	f7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -300,7 +300,7 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected trailing word joiner",
       "priority": 772,
-      "rowID": "d7qw",
+      "rowID": "h7qw",
     },
     Object {
       "C": "1",
@@ -312,10 +312,10 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "h7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	h7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -332,7 +332,7 @@ Object {
       "location": " that was supplied",
       "message": "Unexpected trailing zero-width joiner",
       "priority": 773,
-      "rowID": "d7qw",
+      "rowID": "l7qw",
     },
     Object {
       "C": "1",
@@ -344,10 +344,10 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "l7qw",
     },
   ],
-  "suggestion": "GEN	1	2	d7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": "GEN	1	2	l7qw	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
 }
 `;
 
@@ -364,7 +364,7 @@ Object {
       "location": " that was supplied",
       "message": "Unable to find original language quote in verse text",
       "priority": 916,
-      "rowID": "d7qw",
+      "rowID": "j7qw",
     },
   ],
 }
@@ -381,7 +381,7 @@ Object {
       "location": " that was supplied",
       "message": "Missing OrigQuote field",
       "priority": 919,
-      "rowID": "d7qw",
+      "rowID": "a7qw",
     },
   ],
 }
@@ -395,12 +395,12 @@ Object {
       "V": "2",
       "bookID": "GEN",
       "details": "(not 5)",
-      "extract": "d7q33",
+      "extract": "r7q33",
       "fieldName": "ID",
       "location": " that was supplied",
       "message": "Row ID should be exactly 4 characters",
       "priority": 778,
-      "rowID": "d7q33",
+      "rowID": "r7q33",
     },
   ],
 }
@@ -430,15 +430,15 @@ Object {
       "V": "2",
       "bookID": "GEN",
       "details": "(not 3)",
-      "extract": "d7q",
+      "extract": "q7q",
       "fieldName": "ID",
       "location": " that was supplied",
       "message": "Row ID should be exactly 4 characters",
       "priority": 778,
-      "rowID": "d7q",
+      "rowID": "q7q",
     },
   ],
-  "suggestion": "GEN	1	2	d7qj	figs-imperative	וְ⁠חֹ֖שֶׁךְ	1	Darkness	This is a command. By commanding that light should exist, God made it exist. (See: [[rc://*/ta/man/translate/figs-imperative]])",
+  "suggestion": undefined,
 }
 `;
 
@@ -1024,7 +1024,7 @@ Object {
       "location": " that was supplied",
       "message": "Resource container link should have '*' language code",
       "priority": 450,
-      "rowID": "d7qw",
+      "rowID": "u7qw",
     },
   ],
 }
@@ -1043,7 +1043,7 @@ Object {
       "location": " that was supplied",
       "message": "Wrong book identifier",
       "priority": 978,
-      "rowID": "d7qw",
+      "rowID": "t7qw",
     },
   ],
 }
@@ -1062,7 +1062,7 @@ Object {
       "location": " that was supplied",
       "message": "Wrong chapter number",
       "priority": 976,
-      "rowID": "d7qw",
+      "rowID": "s7qw",
     },
     Object {
       "C": "22",
@@ -1074,7 +1074,7 @@ Object {
       "location": " that was supplied",
       "message": "Wrong verse number",
       "priority": 975,
-      "rowID": "d7qw",
+      "rowID": "s7qw",
     },
   ],
 }