Merge pull request #217 from beakerandjake/feature/216-fix-input-file-trimmed

Feature/216 fix input file trimmed

Author: beakerandjake

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- `downloadInput()` no longer trims the start of the text ([#216](https://github.com/beakerandjake/advent-of-code-runner/issues/216))
+
 ## [1.3.3] - 2023-02-09
 ### Fixed
 - Removed WIP warning message on README

src/api/api.js

@@ -53,7 +53,7 @@ export const downloadInput = async (year, day, authenticationToken) => {
   }
 
   // expect text of response is the input.
-  const text = (await response.text())?.trim() || '';
+  const text = (await response.text())?.trimEnd() || '';
   logger.debug('downloaded: %s', sizeOfStringInKb(text));
 
   if (!text) {

tests/api/api.test.js

@@ -98,9 +98,35 @@ describe('api', () => {
       const result = await downloadInput(2022, 1, 'ASDF');
       expect(result).toBe(expected);
     });
+
+    test('does not trim start of input', async () => {
+      const expected = '\t1234\n5678\n9101112';
+      mockFetch.mockImplementation(() =>
+        Promise.resolve({
+          ok: true,
+          status: 200,
+          text: () => Promise.resolve(expected),
+        })
+      );
+      const result = await downloadInput(2022, 1, 'ASDF');
+      expect(result).toBe(expected);
+    });
+
+    test('trims end of input', async () => {
+      const expected = '1234\n5678\n9101112';
+      mockFetch.mockImplementation(() =>
+        Promise.resolve({
+          ok: true,
+          status: 200,
+          text: () => Promise.resolve(`${expected}\t\n`),
+        })
+      );
+      const result = await downloadInput(2022, 1, 'ASDF');
+      expect(result).toBe(expected);
+    });
   });
 
-  describe('submitSolution', () => {
+  describe('submitSolution()', () => {
     test.each([undefined, null, ''])(
       'throws if authentication token is: "%s"',
       async (value) => {