test(unit): add mock data, rename tests (#610)

Separate from #562
thymikee · Nov 5, 2020 · 29074dd · 29074dd
1 parent 74b3e0f
commit 29074dd
Show file tree

Hide file tree

Showing 16 changed files with 68 additions and 42 deletions.
diff --git a/jest.config.js b/jest.config.js
@@ -7,5 +7,5 @@ module.exports = {
       tsconfig: 'tsconfig.spec.json',
     },
   },
-  testPathIgnorePatterns: ['/e2e/'],
+  testPathIgnorePatterns: ['/e2e/', '/src/__tests__/__mocks__/'],
 };
diff --git a/src/__tests__/__mocks__/app.component.ts b/src/__tests__/__mocks__/app.component.ts
@@ -0,0 +1,10 @@
+import { Component } from '@angular/core';
+
+@Component({
+  selector: 'app-root',
+  templateUrl: './app.component.html',
+  styleUrls: ['./app.component.scss'],
+})
+export class AppComponent {
+  title = 'test-app-v10';
+}
diff --git a/src/__tests__/__mocks__/foo.component.ts b/src/__tests__/__mocks__/foo.component.ts
@@ -0,0 +1,9 @@
+import { Component } from '@angular/core';
+
+@Component({
+  selector: 'foo-root',
+  template: `<h1>Hello World !!</h1>`,
+})
+export class FooComponent {
+  title: number = 'test-app-v10';
+}
diff --git a/src/__tests__/__mocks__/foo.service.ts b/src/__tests__/__mocks__/foo.service.ts
@@ -0,0 +1,9 @@
+import { Injectable } from '@angular/core';
+
+export class ClassInject {}
+
+@Injectable()
+export class MyService {
+  // eslint-disable-next-line
+  constructor(_v: ClassInject) {}
+}
diff --git a/src/__tests__/__mocks__/foo.spec.ts b/src/__tests__/__mocks__/foo.spec.ts
@@ -0,0 +1,5 @@
+import { getFoo } from './foo';
+
+jest.mock('./foo');
+
+console.log(getFoo());
diff --git a/src/__tests__/__mocks__/foo.ts b/src/__tests__/__mocks__/foo.ts
@@ -0,0 +1,3 @@
+export function getFoo(): string {
+  return 'foo';
+}
diff --git a/..._/__snapshots__/inline-files.test.ts.snap → ..._/__snapshots__/inline-files.spec.ts.snap b/..._/__snapshots__/inline-files.test.ts.snap → ..._/__snapshots__/inline-files.spec.ts.snap
diff --git a/src/__tests__/__snapshots__/ng-jest-config.spec.ts.snap b/src/__tests__/__snapshots__/ng-jest-config.spec.ts.snap
@@ -2,11 +2,28 @@
 
 exports[`NgJestConfig readTsConfig should return config including Angular compiler config with tsconfig as a string from ts-jest option 1`] = `
 Object {
-  "enableIvy": false,
-  "fullTemplateTypeCheck": true,
+  "enableIvy": true,
+  "esModuleInterop": true,
+  "experimentalDecorators": true,
+  "forceConsistentCasingInFileNames": true,
+  "importHelpers": true,
+  "importsNotUsedAsValues": 2,
+  "lib": Array [
+    "lib.esnext.d.ts",
+    "lib.dom.d.ts",
+  ],
   "module": 1,
-  "strictInjectionParameters": true,
+  "noFallthroughCasesInSwitch": true,
+  "noImplicitReturns": true,
+  "noUnusedLocals": true,
+  "noUnusedParameters": true,
+  "skipLibCheck": true,
+  "strict": true,
   "target": 1,
+  "types": Array [
+    "node",
+    "jest",
+  ],
 }
 `;
 

diff --git a/..._/__snapshots__/strip-styles.test.ts.snap → ..._/__snapshots__/strip-styles.spec.ts.snap b/..._/__snapshots__/strip-styles.test.ts.snap → ..._/__snapshots__/strip-styles.spec.ts.snap
diff --git a/src/__tests__/inline-files.test.ts → src/__tests__/inline-files.spec.ts b/src/__tests__/inline-files.test.ts → src/__tests__/inline-files.spec.ts
diff --git a/src/__tests__/json.spec.ts b/src/__tests__/json.spec.ts
diff --git a/src/__tests__/ng-jest-config.spec.ts b/src/__tests__/ng-jest-config.spec.ts
@@ -4,13 +4,15 @@ import { join } from 'path';
 import { NgJestConfig } from '../config/ng-jest-config';
 
 describe('NgJestConfig', () => {
+  const specifiedTsCfgPath = join(__dirname, '..', '..', 'tsconfig.spec.json');
+  const defaultTsCfgPath = join(__dirname, '..', '..', 'tsconfig.json');
+
   describe('readTsConfig', () => {
     test('should return config including Angular compiler config with tsconfig as a string from ts-jest option', () => {
-      const configFilePath = join(__dirname, 'tsconfig-ve.json');
       const ngJestConfig = new NgJestConfig({
         globals: {
           'ts-jest': {
-            tsconfig: configFilePath,
+            tsconfig: specifiedTsCfgPath,
           },
         },
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -19,7 +21,7 @@ describe('NgJestConfig', () => {
 
       delete config.options.basePath;
       delete config.options.baseUrl;
-      expect(config.options.configFilePath as string).toEqual(normalizeSeparators(configFilePath));
+      expect(config.options.configFilePath).toEqual(normalizeSeparators(specifiedTsCfgPath));
       delete config.options.configFilePath;
       delete config.options.genDir;
       expect(config.options).toMatchSnapshot();
@@ -41,9 +43,7 @@ describe('NgJestConfig', () => {
 
       delete config.options.basePath;
       delete config.options.baseUrl;
-      expect(config.options.configFilePath as string).toEqual(
-        normalizeSeparators(join(__dirname, '..', '..', 'tsconfig.json')),
-      );
+      expect(config.options.configFilePath).toEqual(normalizeSeparators(defaultTsCfgPath));
       delete config.options.configFilePath;
       delete config.options.genDir;
       expect(config.options).toMatchSnapshot();
@@ -55,9 +55,7 @@ describe('NgJestConfig', () => {
 
       delete config.options.basePath;
       delete config.options.baseUrl;
-      expect(config.options.configFilePath as string).toEqual(
-        normalizeSeparators(join(__dirname, '..', '..', 'tsconfig.json')),
-      );
+      expect(config.options.configFilePath).toEqual(normalizeSeparators(defaultTsCfgPath));
       delete config.options.configFilePath;
       delete config.options.genDir;
       expect(config.options).toMatchSnapshot();

diff --git a/src/__tests__/reflect-metadata.test.ts → src/__tests__/reflect-metadata.spec.ts b/src/__tests__/reflect-metadata.test.ts → src/__tests__/reflect-metadata.spec.ts
diff --git a/src/__tests__/strip-styles.test.ts → src/__tests__/strip-styles.spec.ts b/src/__tests__/strip-styles.test.ts → src/__tests__/strip-styles.spec.ts
diff --git a/src/__tests__/tsconfig-ve.json b/src/__tests__/tsconfig-ve.json
diff --git a/tsconfig.spec.json b/tsconfig.spec.json
@@ -3,5 +3,8 @@
   "compilerOptions": {
     "target": "ES5"
   },
-  "include": [] // boost test's speed trick
+  "include": ["**/__tests__"],
+  "exclude": [
+    "*.spec.ts"
+  ]
 }