fix(util-dynamodb): revert marshall.spec.ts and minor formatting

Author: kuhe

packages/util-dynamodb/src/marshall.spec.ts

@@ -4,64 +4,62 @@ import { marshall } from "./marshall";
 jest.mock("./convertToAttr");
 
 describe("marshall", () => {
-  describe("mocked convertToAttr", () => {
-    const mockOutput = { S: "mockOutput" };
-    (convertToAttr as jest.Mock).mockReturnValue({ M: mockOutput });
+  const mockOutput = { S: "mockOutput" };
+  (convertToAttr as jest.Mock).mockReturnValue({ M: mockOutput });
 
-    afterEach(() => {
-      jest.clearAllMocks();
-    });
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
 
-    it("with object as an input", () => {
-      const input = { a: "A", b: "B" };
-      expect(marshall(input)).toEqual(mockOutput);
-      expect(convertToAttr).toHaveBeenCalledTimes(1);
-      expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
-    });
+  it("with object as an input", () => {
+    const input = { a: "A", b: "B" };
+    expect(marshall(input)).toEqual(mockOutput);
+    expect(convertToAttr).toHaveBeenCalledTimes(1);
+    expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
+  });
 
-    ["convertEmptyValues", "removeUndefinedValues"].forEach((option) => {
-      describe(`options.${option}`, () => {
-        [false, true].forEach((value) => {
-          it(`passes ${value} to convertToAttr`, () => {
-            const input = { a: "A", b: "B" };
-            expect(marshall(input, { [option]: value })).toEqual(mockOutput);
-            expect(convertToAttr).toHaveBeenCalledTimes(1);
-            expect(convertToAttr).toHaveBeenCalledWith(input, { [option]: value });
-          });
+  ["convertEmptyValues", "removeUndefinedValues"].forEach((option) => {
+    describe(`options.${option}`, () => {
+      [false, true].forEach((value) => {
+        it(`passes ${value} to convertToAttr`, () => {
+          const input = { a: "A", b: "B" };
+          expect(marshall(input, { [option]: value })).toEqual(mockOutput);
+          expect(convertToAttr).toHaveBeenCalledTimes(1);
+          expect(convertToAttr).toHaveBeenCalledWith(input, { [option]: value });
         });
       });
     });
+  });
 
-    it("with type as an input", () => {
-      type TestInputType = { a: string; b: string };
-      const input: TestInputType = { a: "A", b: "B" };
+  it("with type as an input", () => {
+    type TestInputType = { a: string; b: string };
+    const input: TestInputType = { a: "A", b: "B" };
 
-      expect(marshall(input)).toEqual(mockOutput);
-      expect(convertToAttr).toHaveBeenCalledTimes(1);
-      expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
-    });
+    expect(marshall(input)).toEqual(mockOutput);
+    expect(convertToAttr).toHaveBeenCalledTimes(1);
+    expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
+  });
 
-    it("with Interface as an input", () => {
-      interface TestInputInterface {
-        a: string;
-        b: string;
-      }
-      const input: TestInputInterface = { a: "A", b: "B" };
+  it("with Interface as an input", () => {
+    interface TestInputInterface {
+      a: string;
+      b: string;
+    }
+    const input: TestInputInterface = { a: "A", b: "B" };
 
-      expect(marshall(input)).toEqual(mockOutput);
-      expect(convertToAttr).toHaveBeenCalledTimes(1);
-      expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
-    });
+    expect(marshall(input)).toEqual(mockOutput);
+    expect(convertToAttr).toHaveBeenCalledTimes(1);
+    expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
+  });
 
-    it("with class instance as an input", () => {
-      class TestInputClass {
-        constructor(private readonly a: string, private readonly b: string) {}
-      }
-      const input = new TestInputClass("A", "B");
+  it("with class instance as an input", () => {
+    class TestInputClass {
+      constructor(private readonly a: string, private readonly b: string) {}
+    }
+    const input = new TestInputClass("A", "B");
 
-      expect(marshall(input)).toEqual(mockOutput);
-      expect(convertToAttr).toHaveBeenCalledTimes(1);
-      expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
-    });
+    expect(marshall(input)).toEqual(mockOutput);
+    expect(convertToAttr).toHaveBeenCalledTimes(1);
+    expect(convertToAttr).toHaveBeenCalledWith(input, undefined);
   });
 });

packages/util-dynamodb/src/marshallTypes.spec.ts

@@ -7,20 +7,24 @@ describe("marshall type discernment", () => {
       const value = "hello";
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals number", () => {
       const value = 1578;
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals binary", () => {
       const value = new Uint8Array([0, 1, 0, 1]);
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals boolean", () => {
       let value = false;
       expect(marshall(value)).toEqual(convertToAttr(value));
       value = true;
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals null", () => {
       const value = null;
       expect(marshall(value)).toEqual(convertToAttr(value));
@@ -29,10 +33,12 @@ describe("marshall type discernment", () => {
       const value = new Set(["a", "b"]);
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals number set", () => {
       const value = new Set([1, 2]);
       expect(marshall(value)).toEqual(convertToAttr(value));
     });
+
     it("marshals binary set", () => {
       const value = new Set([new Uint8Array([1, 0]), new Uint8Array([0, 1])]);
       expect(marshall(value)).toEqual(convertToAttr(value));
@@ -52,6 +58,7 @@ describe("marshall type discernment", () => {
         },
       });
     });
+
     it("marshals and unwraps list", () => {
       expect(marshall(["test", 2, null])).toEqual([{ S: "test" }, { N: "2" }, { NULL: true }]);
     });