Don't trim strings in readFromFixture

Author: fatfisz

src/oneLine/oneLine.test.js

@@ -4,7 +4,7 @@ import { readFromFixture } from '../testUtils';
 const val = 'amaze';
 
 test('reduces text to one line, replacing newlines with spaces', () => {
-  const expected = readFromFixture(__dirname, 'oneLine');
+  const expected = readFromFixture(__dirname, 'oneLine').trim();
   const actual = oneLine`
     wow such doge
     is very ${val}
@@ -15,7 +15,7 @@ test('reduces text to one line, replacing newlines with spaces', () => {
 });
 
 test('reduces text to one line, replacing newlines with spaces (no indentation)', () => {
-  const expected = readFromFixture(__dirname, 'oneLine');
+  const expected = readFromFixture(__dirname, 'oneLine').trim();
   const actual = oneLine`
 wow such doge
 is very ${val}
@@ -27,7 +27,7 @@ from multiline
 });
 
 test('preserves whitespace within input lines, replacing only newlines', () => {
-  const expected = readFromFixture(__dirname, 'oneLine-sentence');
+  const expected = readFromFixture(__dirname, 'oneLine-sentence').trim();
   const actual = oneLine`
     Sentences also work.  Double
     spacing is preserved.

src/oneLineCommaLists/oneLineCommaLists.test.js

@@ -5,7 +5,7 @@ const val = 'amaze';
 
 test('includes arrays as comma-separated list on one line', () => {
   const fruits = ['apple', 'banana', 'kiwi'];
-  const expected = readFromFixture(__dirname, 'oneLineCommaLists');
+  const expected = readFromFixture(__dirname, 'oneLineCommaLists').trim();
   const actual = oneLineCommaLists`
     Doge <3's these fruits: ${fruits}
     they are ${val}

src/oneLineCommaListsAnd/oneLineCommaListsAnd.test.js

@@ -5,7 +5,7 @@ const val = 'amaze';
 
 test('includes arrays as comma-separated list on one line with "and"', () => {
   const fruits = ['apple', 'banana', 'kiwi'];
-  const expected = readFromFixture(__dirname, 'oneLineCommaListsAnd');
+  const expected = readFromFixture(__dirname, 'oneLineCommaListsAnd').trim();
   const actual = oneLineCommaListsAnd`
     Doge <3's these fruits: ${fruits}
     they are ${val}
@@ -15,7 +15,10 @@ test('includes arrays as comma-separated list on one line with "and"', () => {
 
 test('only returns the first item of a single element array', () => {
   const fruits = ['apple'];
-  const expected = readFromFixture(__dirname, 'oneLineCommaListsAndSingleItem');
+  const expected = readFromFixture(
+    __dirname,
+    'oneLineCommaListsAndSingleItem',
+  ).trim();
   const actual = oneLineCommaListsAnd`
     Doge <3's these fruits: ${fruits}
     they are ${val}

src/oneLineCommaListsOr/oneLineCommaListsOr.test.js

@@ -5,7 +5,7 @@ const val = 'amaze';
 
 test('includes arrays as comma-separated list on one line with "or"', () => {
   const fruits = ['apple', 'banana', 'kiwi'];
-  const expected = readFromFixture(__dirname, 'oneLineCommaListsOr');
+  const expected = readFromFixture(__dirname, 'oneLineCommaListsOr').trim();
   const actual = oneLineCommaListsOr`
     Doge <3's these fruits: ${fruits}
     they are ${val}
@@ -15,7 +15,10 @@ test('includes arrays as comma-separated list on one line with "or"', () => {
 
 test('only returns the first item of a single element array', () => {
   const fruits = ['apple'];
-  const expected = readFromFixture(__dirname, 'oneLineCommaListsOrSingleItem');
+  const expected = readFromFixture(
+    __dirname,
+    'oneLineCommaListsOrSingleItem',
+  ).trim();
   const actual = oneLineCommaListsOr`
     Doge <3's these fruits: ${fruits}
     they are ${val}

src/oneLineInlineLists/oneLineInlineLists.test.js

@@ -5,7 +5,7 @@ const val = 'amaze';
 
 test('includes arrays as inline list on one line', () => {
   const fruits = ['apple', 'banana', 'kiwi'];
-  const expected = readFromFixture(__dirname, 'oneLineInlineLists');
+  const expected = readFromFixture(__dirname, 'oneLineInlineLists').trim();
   const actual = oneLineInlineLists`
     Doge <3's these fruits: ${fruits}
     they are ${val}

src/oneLineTrim/oneLineTrim.test.js

@@ -4,7 +4,7 @@ import { readFromFixture } from '../testUtils';
 const val = 'amaze';
 
 test('reduces to one line while trimming newlines', () => {
-  const expected = readFromFixture(__dirname, 'oneLineTrim');
+  const expected = readFromFixture(__dirname, 'oneLineTrim').trim();
   const actual = oneLineTrim`
   wow such reduction
   very absence of space
@@ -14,7 +14,7 @@ test('reduces to one line while trimming newlines', () => {
 });
 
 test('reduces to one line while trimming newlines (no indentation)', () => {
-  const expected = readFromFixture(__dirname, 'oneLineTrim');
+  const expected = readFromFixture(__dirname, 'oneLineTrim').trim();
   const actual = oneLineTrim`
 wow such reduction
 very absence of space

src/testUtils/readFromFixture/readFromFixture.js

@@ -12,5 +12,5 @@ export default function readFromFixture(dirname, name) {
     path.join(dirname, `fixtures/${name}.txt`),
     'utf8',
   );
-  return contents.replace(/\r\n/g, '\n').trim();
+  return contents.replace(/\r\n/g, '\n');
 }

src/testUtils/readFromFixture/readFromFixture.test.js

@@ -2,7 +2,7 @@ import readFromFixture from './readFromFixture';
 
 test('reads the correct fixture contents', () => {
   const actual = readFromFixture(__dirname, 'contents');
-  const expected = 'wow such doge';
+  const expected = 'wow such doge\n';
   expect(actual).toBe(expected);
 });