Improve Unit Test

karma.conf.js

@@ -6,12 +6,49 @@ const runChrome = typeof argv.chrome !== 'undefined';
 
 const rootPath = __dirname;
 
+const failedTestsReporter = function (baseReporterDecorator) {
+    baseReporterDecorator(this);
+
+    const failedTests = [];
+
+    const RED = '\x1b[31m';
+    const RESET = '\x1b[0m';
+    const BOLD = '\x1b[1m';
+    const GREEN = '\x1b[32m';
+
+    this.onSpecComplete = function (browser, result) {
+        if (!result.success) {
+            failedTests.push({
+                description: result.description,
+                suite: result.suite.join(' > '),
+                log: result.log,
+            });
+        }
+    };
+
+    this.onRunComplete = function () {
+        if (failedTests.length > 0) {
+            console.log(`\n\n${RED}${BOLD}=== ❌ Failed Tests Summary ===${RESET}\n`);
+            failedTests.forEach((test, index) => {
+                console.log(`${index + 1}. ${test.suite} > ${test.description}`);
+                test.log.forEach(log => console.log(`   ${log}`));
+                console.log('');
+            });
+        } else {
+            console.log(`${GREEN}✅ All tests passed successfully${RESET}`);
+        }
+    };
+};
+
+failedTestsReporter.$inject = ['baseReporterDecorator'];
+
 module.exports = function (config) {
     const plugins = [
         'karma-webpack',
         'karma-phantomjs-launcher',
         'karma-jasmine',
         'karma-sourcemap-loader',
+        { 'reporter:failed-summary': ['type', failedTestsReporter] },
     ];
     const launcher = [];
 
@@ -113,10 +150,11 @@ module.exports = function (config) {
         // Concurrency level
         // how many browser should be started simultaneous
         concurrency: Infinity,
+        reporters: ['failed-summary'],
     };
 
     if (runCoverage) {
-        settings.reporters = ['coverage-istanbul'];
+        settings.reporters = ['coverage-istanbul', 'failed-summary'];
         settings.coverageIstanbulReporter = {
             reports: ['html', 'lcovonly', 'text-summary'],
             dir: './dist/deploy/coverage',

packages/roosterjs-content-model-core/test/command/createModelFromHtml/createDomToModelContextForSanitizingTest.ts

@@ -2,6 +2,7 @@ import * as createDomToModelContext from 'roosterjs-content-model-dom/lib/domToM
 import * as createPasteEntityProcessor from '../../../lib/override/pasteEntityProcessor';
 import * as createPasteGeneralProcessor from '../../../lib/override/pasteGeneralProcessor';
 import { containerSizeFormatParser } from '../../../lib/override/containerSizeFormatParser';
+import { createDefaultDomToModelContext } from '../../testUtils';
 import { createDomToModelContextForSanitizing } from '../../../lib/command/createModelFromHtml/createDomToModelContextForSanitizing';
 import { DomToModelOptionForSanitizing } from 'roosterjs-content-model-types';
 import { pasteBlockEntityParser } from '../../../lib/override/pasteCopyBlockEntityParser';
@@ -13,15 +14,7 @@ describe('createDomToModelContextForSanitizing', () => {
     const mockedPasteGeneralProcessor = 'GENERALPROCESSOR' as any;
     const mockedPasteEntityProcessor = 'ENTITYPROCESSOR' as any;
     const mockedResult = 'CONTEXT' as any;
-    const defaultOptions: DomToModelOptionForSanitizing = {
-        processorOverride: {},
-        formatParserOverride: {},
-        additionalFormatParsers: {},
-        additionalAllowedTags: [],
-        additionalDisallowedTags: [],
-        styleSanitizers: {},
-        attributeSanitizers: {},
-    };
+    const defaultOptions: DomToModelOptionForSanitizing = createDefaultDomToModelContext();
     let createDomToModelContextSpy: jasmine.Spy;
 
     let createPasteGeneralProcessorSpy: jasmine.Spy;

packages/roosterjs-content-model-core/test/command/paste/generatePasteOptionFromPluginsTest.ts

@@ -1,3 +1,4 @@
+import { createDefaultDomToModelContext } from '../../testUtils';
 import { generatePasteOptionFromPlugins } from '../../../lib/command/paste/generatePasteOptionFromPlugins';
 import { IEditor } from 'roosterjs-content-model-types';
 
@@ -66,15 +67,7 @@ describe('generatePasteOptionFromPlugins', () => {
             htmlAfter: htmlAfter,
             htmlAttributes: mockedMetadata,
             pasteType: 'normal',
-            domToModelOption: {
-                additionalAllowedTags: [],
-                additionalDisallowedTags: [],
-                additionalFormatParsers: {},
-                formatParserOverride: {},
-                processorOverride: {},
-                styleSanitizers: {},
-                attributeSanitizers: {},
-            },
+            domToModelOption: createDefaultDomToModelContext(),
             containsBlockElements: false,
         });
         expect(triggerPluginEventSpy).toHaveBeenCalledWith(
@@ -205,15 +198,7 @@ describe('generatePasteOptionFromPlugins', () => {
             htmlAfter: '',
             htmlAttributes: mockedMetadata,
             pasteType: 'mergeFormat',
-            domToModelOption: {
-                additionalAllowedTags: [],
-                additionalDisallowedTags: [],
-                additionalFormatParsers: {},
-                formatParserOverride: {},
-                processorOverride: {},
-                styleSanitizers: {},
-                attributeSanitizers: {},
-            },
+            domToModelOption: createDefaultDomToModelContext(),
             containsBlockElements: false,
         });
     });

packages/roosterjs-content-model-core/test/testUtils.ts

@@ -0,0 +1,13 @@
+import { DomToModelOptionForSanitizing } from 'roosterjs-content-model-types';
+
+export const createDefaultDomToModelContext = (): DomToModelOptionForSanitizing => {
+    return {
+        additionalAllowedTags: [],
+        additionalDisallowedTags: [],
+        additionalFormatParsers: {},
+        formatParserOverride: {},
+        processorOverride: {},
+        styleSanitizers: {},
+        attributeSanitizers: {},
+    };
+};

packages/roosterjs-content-model-dom/test/testUtils.ts

@@ -1,3 +1,5 @@
+import { DomToModelOptionForSanitizing } from 'roosterjs-content-model-types';
+
 export function expectHtml(actualHtml: string, expectedHtml: string | string[]) {
     expectedHtml = Array.isArray(expectedHtml) ? expectedHtml : [expectedHtml];
     expect(expectedHtml.indexOf(actualHtml)).toBeGreaterThanOrEqual(0, actualHtml);
@@ -29,3 +31,15 @@ export function itChromeOnly(
     const func = __karma__.config.browser == 'Chrome' ? it : xit;
     return func(expectation, assertion, timeout);
 }
+
+export const createDefaultDomToModelContext = (): DomToModelOptionForSanitizing => {
+    return {
+        additionalAllowedTags: [],
+        additionalDisallowedTags: [],
+        additionalFormatParsers: {},
+        formatParserOverride: {},
+        processorOverride: {},
+        styleSanitizers: {},
+        attributeSanitizers: {},
+    };
+};

packages/roosterjs-content-model-plugins/test/TestHelper.ts

@@ -1,10 +1,11 @@
+import { Editor } from 'roosterjs-content-model-core';
 import {
     ContentModelDocument,
     CoreApiMap,
+    DomToModelOptionForSanitizing,
     EditorPlugin,
     IEditor,
 } from 'roosterjs-content-model-types';
-import { Editor } from 'roosterjs-content-model-core';
 
 export function initEditor(
     id: string,
@@ -39,3 +40,15 @@ export function removeElement(id: string) {
         node.parentNode.removeChild(node);
     }
 }
+
+export const createDefaultDomToModelContext = (): DomToModelOptionForSanitizing => {
+    return {
+        additionalAllowedTags: [],
+        additionalDisallowedTags: [],
+        additionalFormatParsers: {},
+        formatParserOverride: {},
+        processorOverride: {},
+        styleSanitizers: {},
+        attributeSanitizers: {},
+    };
+};

packages/roosterjs-content-model-plugins/test/paste/createBeforePasteEventMock.ts

@@ -1,3 +1,4 @@
+import { createDefaultDomToModelContext } from '../TestHelper';
 import type { BeforePasteEvent, ClipboardData } from 'roosterjs-content-model-types';
 
 export function createBeforePasteEventMock(
@@ -12,14 +13,6 @@ export function createBeforePasteEventMock(
         htmlAfter: '',
         htmlAttributes: {},
         pasteType: 'normal',
-        domToModelOption: {
-            additionalAllowedTags: [],
-            additionalDisallowedTags: [],
-            additionalFormatParsers: {},
-            attributeSanitizers: {},
-            formatParserOverride: {},
-            processorOverride: {},
-            styleSanitizers: {},
-        },
+        domToModelOption: createDefaultDomToModelContext(),
     };
 }

packages/roosterjs-content-model-plugins/test/paste/plugin/ContentModelPastePluginTest.ts

@@ -6,6 +6,7 @@ import * as PowerPointFile from '../../../lib/paste/PowerPoint/processPastedCont
 import * as setProcessor from '../../../lib/paste/utils/setProcessor';
 import * as WacFile from '../../../lib/paste/WacComponents/processPastedContentWacComponents';
 import { BeforePasteEvent, DOMCreator, IEditor } from 'roosterjs-content-model-types';
+import { createDefaultDomToModelContext } from '../../TestHelper';
 import { PastePlugin } from '../../../lib/paste/PastePlugin';
 import { PastePropertyNames } from '../../../lib/paste/pasteSourceValidations/constants';
 
@@ -47,15 +48,7 @@ describe('Content Model Paste Plugin Test', () => {
                 htmlAfter: '',
                 htmlAttributes: {},
                 pasteType: 'normal',
-                domToModelOption: {
-                    additionalAllowedTags: [],
-                    additionalDisallowedTags: [],
-                    additionalFormatParsers: {},
-                    formatParserOverride: {},
-                    processorOverride: {},
-                    styleSanitizers: {},
-                    attributeSanitizers: {},
-                } as any,
+                domToModelOption: createDefaultDomToModelContext(),
             };
         });
 

packages/roosterjs-content-model-plugins/test/paste/powerPoint/processPastedContentFromPowerPointTest.ts

@@ -1,4 +1,5 @@
 import * as moveChildNodes from 'roosterjs-content-model-dom/lib/domUtils/moveChildNodes';
+import { createDefaultDomToModelContext } from '../../TestHelper';
 import { processPastedContentFromPowerPoint } from '../../../lib/paste/PowerPoint/processPastedContentFromPowerPoint';
 import type { BeforePasteEvent, ClipboardData, DOMCreator } from 'roosterjs-content-model-types';
 
@@ -11,15 +12,7 @@ const getPasteEvent = (): BeforePasteEvent => {
         htmlAfter: '',
         htmlAttributes: {},
         pasteType: 'normal',
-        domToModelOption: {
-            additionalAllowedTags: [],
-            additionalDisallowedTags: [],
-            additionalFormatParsers: {},
-            formatParserOverride: {},
-            processorOverride: {},
-            styleSanitizers: {},
-            attributeSanitizers: {},
-        },
+        domToModelOption: createDefaultDomToModelContext(),
     };
 };
 

packages/roosterjs-editor-adapter/test/editor/utils/eventConverterTest.ts

@@ -1,4 +1,5 @@
 import * as selectionConvert from '../../../lib/editor/utils/selectionConverter';
+import { createDefaultDomToModelContext } from '../../testUtils';
 import { newEventToOldEvent, oldEventToNewEvent } from '../../../lib/editor/utils/eventConverter';
 import {
     EntityOperation,
@@ -105,15 +106,7 @@ describe('oldEventToNewEvent', () => {
                 eventType: 'beforePaste',
                 clipboardData: mockedClipboardData,
                 customizedMerge: undefined,
-                domToModelOption: {
-                    additionalAllowedTags: [],
-                    additionalDisallowedTags: [],
-                    additionalFormatParsers: {},
-                    formatParserOverride: {},
-                    processorOverride: {},
-                    styleSanitizers: {},
-                    attributeSanitizers: {},
-                },
+                domToModelOption: createDefaultDomToModelContext(),
                 eventDataCache: mockedDataCache,
                 fragment: mockedFragment,
                 htmlAfter: mockedHtmlAfter,

packages/roosterjs-editor-adapter/test/testUtils.ts

@@ -0,0 +1,13 @@
+import { DomToModelOptionForSanitizing } from 'roosterjs-content-model-types';
+
+export const createDefaultDomToModelContext = (): DomToModelOptionForSanitizing => {
+    return {
+        additionalAllowedTags: [],
+        additionalDisallowedTags: [],
+        additionalFormatParsers: {},
+        formatParserOverride: {},
+        processorOverride: {},
+        styleSanitizers: {},
+        attributeSanitizers: {},
+    };
+};