fix: tests fs mocks

ttoss · Feb 23, 2022 · a00350a · a00350a
1 parent 9661fae
commit a00350a
Show file tree

Hide file tree

Showing 5 changed files with 3,358 additions and 5,527 deletions.
diff --git a/packages/cli/jest.config.js b/packages/cli/jest.config.js
@@ -2,7 +2,7 @@ module.exports = {
   collectCoverage: true,
   coverageThreshold: {
     global: {
-      branches: 32,
+      branches: 30,
       functions: 54,
       lines: 59,
       statements: 62,
@@ -11,4 +11,5 @@ module.exports = {
   preset: 'ts-jest',
   setupFiles: ['<rootDir>/setupTests.js'],
   silent: true,
+  timers: 'fake',
 };
diff --git a/packages/cli/src/deploy/lambda.spec.ts b/packages/cli/src/deploy/lambda.spec.ts
@@ -8,6 +8,7 @@ const existsSyncMock = jest.fn((input) => {
 });
 
 jest.mock('fs', () => ({
+  ...(jest.requireActual('fs') as any),
   existsSync: existsSyncMock,
 }));
 

diff --git a/packages/cli/src/deploy/lambda.ts b/packages/cli/src/deploy/lambda.ts
@@ -95,7 +95,7 @@ export const deployLambdaLayers = async ({
     )}] was found. Creating other layers...`,
   );
 
-  const { dependencies } = (() => {
+  const { dependencies = {} } = (() => {
     try {
       // eslint-disable-next-line global-require, import/no-dynamic-require
       return require(path.resolve(process.cwd(), 'package.json')) || {};

diff --git a/packages/cli/src/deploy/staticApp/staticApp.spec.ts b/packages/cli/src/deploy/staticApp/staticApp.spec.ts
@@ -2,13 +2,7 @@
 
 import { DescribeStackResourceCommand } from '@aws-sdk/client-cloudformation';
 import * as faker from 'faker';
-import * as fs from 'fs';
-
-// const readFileSyncMock = jest.fn();
-
-// jest.mock('fs', () => ({
-//   readFileSync: readFileSyncMock,
-// }));
+import fs from 'fs';
 
 import { AWS_DEFAULT_REGION } from '../../config';
 
@@ -54,7 +48,7 @@ jest.mock('@aws-sdk/client-cloudformation', () => ({
   })),
 }));
 
-jest.mock('fs');
+// jest.mock('fs');
 
 jest.mock('../cloudFormation.core', () => ({
   ...(jest.requireActual('../cloudFormation.core') as any),
-Original file line number
+Diff line change
@@ Expand Up / @@ -8,6 +8,7 @@ const existsSyncMock = jest.fn((input) => { @@
     });
     jest.mock('fs', () => ({
+      ...(jest.requireActual('fs') as any),
       existsSync: existsSyncMock,
     }));
@@ Expand Down @@