Fix: Missing "No newline at end of file" when comparing two texts tha…

…t do not end in newlines (kpdecker#94)
vasek · Aug 30, 2018 · 19a65d5 · 19a65d5
1 parent 04ea47c
commit 19a65d5
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/patch/create.js b/src/patch/create.js
@@ -74,7 +74,8 @@ export function structuredPatch(oldFileName, newFileName, oldStr, newStr, oldHea
             if (lines.length == 0 && !oldEOFNewline) {
               // special case: old has no eol and no trailing context; no-nl can end up before adds
               curRange.splice(hunk.oldLines, 0, '\\ No newline at end of file');
-            } else if (!oldEOFNewline || !newEOFNewline) {
+            }
+            if (!newEOFNewline) {
               curRange.push('\\ No newline at end of file');
             }
           }

diff --git a/test/patch/create.js b/test/patch/create.js
@@ -89,6 +89,22 @@ describe('patch/create', function() {
         + '+line4\n');
     });
 
+    it('should output "no newline" at end of file message on both missing nl', function() {
+      expect(createPatch('test', 'line1\nline2\nline3\nline4', 'line1\nline2\nline3\nline44', 'header1', 'header2')).to.equal(
+        'Index: test\n'
+        + '===================================================================\n'
+        + '--- test\theader1\n'
+        + '+++ test\theader2\n'
+        + '@@ -1,4 +1,4 @@\n'
+        + ' line1\n'
+        + ' line2\n'
+        + ' line3\n'
+        + '-line4\n'
+        + '\\ No newline at end of file\n'
+        + '+line44\n'
+        + '\\ No newline at end of file\n');
+    });
+
     it('should output "no newline" at end of file message on context missing nl', function() {
       expect(createPatch('test', 'line11\nline2\nline3\nline4', 'line1\nline2\nline3\nline4', 'header1', 'header2')).to.equal(
         'Index: test\n'