## Description We are shifting the linter engine for jshint to eslint. This PR adds the package needed for the same and also adds the required configs. The eslint feature is behing a rollout feature flag, it's currently at 0% right now. > [!NOTE] > This PR is part of a series of [stacked diffs](https://newsletter.pragmaticengineer.com/p/stacked-diffs) and might have changes from it's parent PRs. Untill the blocking parent PRs are merged, this diff would show more changes than are relevant for this PR. Blocking PRs: - #36548 Fixes #37254 ## Automation /test js sanity ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/11965021215> > Commit: 579e4772f918936da292ba2815621ca80499c212 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=11965021215&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.JS, @tag.Sanity` > Spec: > <hr>Fri, 22 Nov 2024 02:49:47 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes - **New Features** - Integrated ESLint support alongside existing JSHint functionality for enhanced linting capabilities. - Added a new dependency for ESLint linting. - **Improvements** - Enhanced error handling and reporting for linting errors, ensuring consistency across different linting tools. - Updated testing framework to accommodate multiple linter types without duplicating logic. - Introduced new functions for sanitizing and converting ESLint errors to the application's format. - Added support for structured cloning in the testing setup. - Improved logic for determining main actions and error handling in conditional expressions. - **Bug Fixes** - Improved logic for handling lint errors, particularly in differentiating between ESLint and JSHint errors. - **Documentation** - Updated comments for clarity on linting processes and configurations. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
114 lines
2.7 KiB
TypeScript
114 lines
2.7 KiB
TypeScript
import { setupServer } from "msw/node";
|
|
import { handlers } from "./__mocks__/apiHandlers";
|
|
import "../src/polyfills/requestIdleCallback";
|
|
import { Crypto } from "@peculiar/webcrypto";
|
|
|
|
// since global crypto is immutable, we need to first delete it and then use the
|
|
// peculiar crypto lisrc/sagas/helper.test.tsb
|
|
delete global['crypto'];
|
|
global.crypto = new Crypto();
|
|
|
|
export const server = setupServer(...handlers);
|
|
|
|
jest.mock("api/Api", () => ({
|
|
__esModule: true,
|
|
default: class Api { },
|
|
}));
|
|
|
|
window.scrollTo = jest.fn();
|
|
Element.prototype.scrollIntoView = jest.fn();
|
|
Element.prototype.scrollBy = jest.fn();
|
|
|
|
jest.mock("../src/api/Api.ts", () => ({
|
|
__esModule: true,
|
|
default: class Api { },
|
|
}));
|
|
|
|
// Polyfill for `structuredClone` if not available
|
|
// This is needed for eslint jest tests
|
|
if (typeof global.structuredClone === "undefined") {
|
|
global.structuredClone = (obj) => {
|
|
return JSON.parse(JSON.stringify(obj));
|
|
};
|
|
}
|
|
|
|
beforeAll(() => {
|
|
window.IntersectionObserver = jest
|
|
.fn()
|
|
.mockImplementation((fn: (entry: any) => any) => {
|
|
return {
|
|
observe: () => {
|
|
fn([
|
|
{
|
|
isIntersecting: true,
|
|
boundingClientRect: {
|
|
top: 64,
|
|
left: 293,
|
|
},
|
|
intersectionRect: {
|
|
width: 1296,
|
|
height: 424,
|
|
top: 64,
|
|
left: 293,
|
|
},
|
|
},
|
|
]);
|
|
},
|
|
unobserve: jest.fn(),
|
|
disconnect: jest.fn(),
|
|
};
|
|
});
|
|
|
|
window.ResizeObserver = jest.fn().mockImplementation(() => {
|
|
return {
|
|
observe: jest.fn(),
|
|
unobserve: jest.fn(),
|
|
disconnect: jest.fn(),
|
|
};
|
|
});
|
|
});
|
|
|
|
// establish API mocking before all tests
|
|
beforeAll(() => server.listen());
|
|
// reset any request handlers that are declared as a part of our tests
|
|
// (i.e. for testing one-time error scenarios)
|
|
afterEach(() => server.resetHandlers());
|
|
// clean up once the tests are done
|
|
afterAll(() => server.close());
|
|
|
|
// popper.js fix for jest tests
|
|
document.createRange = () => {
|
|
const range = new Range();
|
|
|
|
range.getBoundingClientRect = jest.fn();
|
|
|
|
range.getClientRects = () => {
|
|
return {
|
|
item: () => null,
|
|
length: 0,
|
|
[Symbol.iterator]: jest.fn(),
|
|
};
|
|
};
|
|
|
|
return range;
|
|
};
|
|
|
|
// jest events doesnt seem to be handling scrollTo
|
|
Element.prototype.scrollTo = () => { };
|
|
|
|
class WorkerStub {
|
|
url: string;
|
|
onmessage: CallableFunction;
|
|
constructor(stringUrl: string) {
|
|
this.url = stringUrl;
|
|
// eslint-disable-next-line @typescript-eslint/no-empty-function
|
|
this.onmessage = () => { };
|
|
}
|
|
|
|
postMessage(msg) {
|
|
this.onmessage(msg);
|
|
}
|
|
}
|
|
|
|
window.Worker = WorkerStub as any;
|