From b106d0694fb1594b80331121a746d09cabc7acf9 Mon Sep 17 00:00:00 2001 From: PKulkoRaccoonGang Date: Thu, 25 Sep 2025 14:29:30 +0300 Subject: [PATCH] fix: fixed some problems with tests --- package-lock.json | 1 + package.json | 5 +++-- .../bulk-email-tool/test/BulkEmailTool.test.jsx | 4 ---- .../page-container/test/PageContainer.test.jsx | 4 ---- src/index.test.jsx | 2 +- tsconfig.json | 13 +++++++++++++ 6 files changed, 18 insertions(+), 11 deletions(-) create mode 100644 tsconfig.json diff --git a/package-lock.json b/package-lock.json index db11b00..ae49a82 100644 --- a/package-lock.json +++ b/package-lock.json @@ -42,6 +42,7 @@ "devDependencies": { "@edx/browserslist-config": "^1.2.0", "@edx/reactifex": "^2.1.1", + "@edx/typescript-config": "^1.1.0", "@openedx/frontend-build": "^14.6.2", "@testing-library/jest-dom": "^6.6.3", "@testing-library/react": "^16.2.0", diff --git a/package.json b/package.json index 1fb763e..e0fdc47 100644 --- a/package.json +++ b/package.json @@ -12,8 +12,8 @@ "scripts": { "build": "fedx-scripts webpack", "i18n_extract": "fedx-scripts formatjs extract", - "lint": "fedx-scripts eslint --ext .js --ext .jsx .", - "lint:fix": "fedx-scripts eslint --fix --ext .js --ext .jsx .", + "lint": "fedx-scripts eslint --ext .js --ext .jsx src/", + "lint:fix": "fedx-scripts eslint --fix --ext .js --ext .jsx src/", "snapshot": "fedx-scripts jest --updateSnapshot", "start": "fedx-scripts webpack-dev-server --progress", "dev": "PUBLIC_PATH=/communications/ MFE_CONFIG_API_URL='http://localhost:8000/api/mfe_config/v1' fedx-scripts webpack-dev-server --progress --host apps.local.openedx.io", @@ -62,6 +62,7 @@ "devDependencies": { "@edx/browserslist-config": "^1.2.0", "@edx/reactifex": "^2.1.1", + "@edx/typescript-config": "^1.1.0", "@openedx/frontend-build": "^14.6.2", "@testing-library/jest-dom": "^6.6.3", "@testing-library/react": "^16.2.0", diff --git a/src/components/bulk-email-tool/test/BulkEmailTool.test.jsx b/src/components/bulk-email-tool/test/BulkEmailTool.test.jsx index 1fc0f43..9ea6c9b 100644 --- a/src/components/bulk-email-tool/test/BulkEmailTool.test.jsx +++ b/src/components/bulk-email-tool/test/BulkEmailTool.test.jsx @@ -23,10 +23,6 @@ jest.mock('react-router-dom', () => ({ courseId: 'test-course-id', })), })); -jest.mock('@edx/frontend-component-header', () => ({ - __esModule: true, - default: () => null, -})); describe('BulkEmailTool', () => { beforeEach(() => jest.resetModules()); diff --git a/src/components/page-container/test/PageContainer.test.jsx b/src/components/page-container/test/PageContainer.test.jsx index 5727874..0d6ac3b 100644 --- a/src/components/page-container/test/PageContainer.test.jsx +++ b/src/components/page-container/test/PageContainer.test.jsx @@ -23,10 +23,6 @@ jest.mock('react-router-dom', () => ({ courseId: 'test-course-id', })), })); -jest.mock('@edx/frontend-component-header', () => ({ - __esModule: true, - default: () => null, -})); describe('PageContainer', () => { beforeEach(() => jest.resetModules()); diff --git a/src/index.test.jsx b/src/index.test.jsx index 734a1ee..ee64eb0 100644 --- a/src/index.test.jsx +++ b/src/index.test.jsx @@ -3,7 +3,7 @@ import { } from '@edx/frontend-platform'; // Jest needs this for module resolution -import * as app from '.'; // eslint-disable-line no-unused-vars +import * as app from '.'; // eslint-disable-line @typescript-eslint/no-unused-vars // These need to be var not let so they get hoisted // and can be used by jest.mock (which is also hoisted) diff --git a/tsconfig.json b/tsconfig.json new file mode 100644 index 0000000..4150a9e --- /dev/null +++ b/tsconfig.json @@ -0,0 +1,13 @@ +{ + "extends": "@edx/typescript-config", + "compilerOptions": { + "baseUrl": "./src", + "paths": { + "*": ["*"] + }, + "rootDir": ".", + "outDir": "dist" + }, + "include": ["*.js", ".eslintrc.js", "src/**/*", "plugins/**/*", "jest.config.ts"], + "exclude": ["*.js", ".eslintrc.js", "dist", "node_modules"] +}