Improve test coverage

Author: patrick-tolosa

__TESTS__/unit/transformation/transformation.test.ts

@@ -14,4 +14,10 @@ describe('Tests for ImageTransformation', () => {
     tx.addTransformation('w_100/w_200/w_300');
     expect(tx.toString()).toContain('w_100/w_200/w_300');
   });
+
+  it('Throws when passing a slash to an action', () => {
+    expect(() => {
+      tx.addAction('w_100/w_200/w_300');
+    }).toThrow();
+  });
 });

__TESTS__/unit/utils/base64Encode.browser.test.ts

@@ -0,0 +1,20 @@
+/**
+ * @jest-environment jsdom
+ */
+import {base64Encode} from "../../../src/internal/utils/base64Encode";
+
+
+describe('base64 test encoding', () => {
+  it('Encodes foo', () => {
+    expect(base64Encode('foo')).toBe('Zm9v');
+  });
+
+  it('Encodes ABlasjglkaskfjgkj', () => {
+    expect(base64Encode('ABlasjglkaskfjgkj')).toBe('QUJsYXNqZ2xrYXNrZmpna2o');
+  });
+
+
+  it('Encodes https://www.example.com', () => {
+    expect(base64Encode('https://www.example.com')).toBe('aHR0cHM6Ly93d3cuZXhhbXBsZS5jb20');
+  });
+});

__TESTS__/unit/utils/base64Encode.node.test.ts

@@ -0,0 +1,19 @@
+/**
+ * @jest-environment node
+ */
+import {base64Encode} from "../../../src/internal/utils/base64Encode";
+
+describe('base64 test encoding', () => {
+  it('Encodes foo', () => {
+    expect(base64Encode('foo')).toBe('Zm9v');
+  });
+
+  it('Encodes ABlasjglkaskfjgkj', () => {
+    expect(base64Encode('ABlasjglkaskfjgkj')).toBe('QUJsYXNqZ2xrYXNrZmpna2o');
+  });
+
+
+  it('Encodes https://www.example.com', () => {
+    expect(base64Encode('https://www.example.com')).toBe('aHR0cHM6Ly93d3cuZXhhbXBsZS5jb20');
+  });
+});

jest.config.json

@@ -1,6 +1,6 @@
 {
   "preset": "ts-jest",
-  "testEnvironment": "node",
+  "testEnvironment": "jsdom",
   "bail": true,
   "collectCoverageFrom": [
     "<rootDir>/src/**/*.ts",

scripts/errors/FoundNonRequiredFile.ts

@@ -1,7 +1,6 @@
 class FoundNonRequiredFile extends Error {
   constructor(filePath:string) {
-    // Required due to https://github.com/microsoft/TypeScript/issues/13029
-    super(); /* istanbul ignore next */
+    super();
     this.message = `File not required by Tree but exists in file system: ${filePath}`;
     this.name = "FoundNonRequiredFile";
   }

scripts/errors/MissingRequiredFile.ts

@@ -1,7 +1,6 @@
 class MissingRequiredFile extends Error {
   constructor(currentPath:string) {
-    // Required due to https://github.com/microsoft/TypeScript/issues/13029
-    super(); /* istanbul ignore next */
+    super();
     this.message = `File ${currentPath} does not exist, \nIf you added a new file or folder, please update the validation step`;
     this.name = "MissingRequiredFileError";
   }

src/internal/utils/base64Encode.ts

@@ -6,12 +6,16 @@
  */
 function base64Encode(input: string):string {
   // Browser
+  let encodedResult = '';
+
   if (typeof window !== 'undefined') {
-    return btoa(input);
+    encodedResult = btoa(input);
   } else {
     // NodeJS support
-    return global.Buffer.from(input).toString('base64');
+    encodedResult = global.Buffer.from(input).toString('base64');
   }
+
+  return encodedResult.replace(/=+$/, ''); // Remove ending '='
 }
 
 export {base64Encode};

src/values/source/sourceTypes/TextSource.ts

@@ -8,6 +8,7 @@ import {TextStyle} from "../../textStyle";
  */
 class TextSource extends BaseTextSource {
   constructor(fileName: string, textStyle: TextStyle) {
+    /* istanbul ignore next */
     super(fileName, textStyle);
   }
 }