PromucFlow_constructor/app/client/src/utils/WorkerUtil.test.ts

225 lines
7.3 KiB
TypeScript
Raw Normal View History

import { GracefulWorkerService } from "./WorkerUtil";
import { runSaga } from "redux-saga";
const MessageType = "message";
interface extraWorkerProperties {
callback: CallableFunction;
noop: CallableFunction;
delayMilliSeconds: number;
running: boolean;
}
type WorkerClass = Worker & extraWorkerProperties;
class MockWorkerClass implements WorkerClass {
2020-12-30 13:26:44 +00:00
// Implement interface
onmessage: any;
onmessageerror: any;
dispatchEvent: any;
onerror: any;
callback: CallableFunction;
noop: CallableFunction;
messages: Array<any>;
delayMilliSeconds: number;
instance: WorkerClass | undefined;
responses: Set<number>;
running: boolean;
resetInstance() {
this.instance = undefined;
}
constructor() {
/* eslint-disable @typescript-eslint/no-empty-function */
this.noop = () => {};
this.callback = this.noop;
this.messages = [];
this.delayMilliSeconds = 0;
this.responses = new Set<number>();
this.instance = this;
this.running = true;
}
2020-12-30 13:26:44 +00:00
addEventListener(msgType: string, callback: any) {
expect(msgType).toEqual(MessageType);
this.callback = callback;
}
2020-12-30 13:26:44 +00:00
removeEventListener(msgType: string, callback: any) {
expect(msgType).toEqual(MessageType);
expect(callback).toEqual(this.callback);
this.callback = this.noop;
}
postMessage(message: any) {
expect(this.running).toEqual(true);
expect(this.callback).not.toEqual(this.noop);
this.messages.push(message);
const counter = setTimeout(() => {
const response = {
messageId: message.messageId,
messageType: "RESPONSE",
body: { data: message.body.data },
};
this.sendEvent({ data: response });
chore: update Styled components to latest version and related cleanup (#19284) ## Description We need to upgrade `styled-components`, so that it will become easy to upgrade to version 6.0 when it is out. This is because, v6.0 has an important functionality which isn't available in today's version. ### Tasks completed - Update Styled components to latest version. - Prepare codebase by cleaning up the styled components functions that will be deprecated in version 6 - We are still using the `withTheme` HOC, we should instead use the `useTheme` hook (best practices) - Remove the `AnyStyledComponent` type it is un-necessary and will be deprecated Fixes #19463 ## Type of change - Non breaking change. The application should work as before and should not effect any visual elements or UI. ## How Has This Been Tested? - Manual @appsmithorg/qa please refer to the test plan for areas of interest. - Cypress: All existing test cases must pass. ### Test Plan - We need to do a sanity check on the Product Updates Modal, Release section. - We also need to do a sanity check on the Login, Signup, ResetPassword pages. - I think we can merge this Pull Request and continue with our weekly regression, because there are no style changes in this Pull Request, everything should work as expected. ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-13 11:05:59 +00:00
// @ts-expect-error: setTimeout return type mismatch
this.responses.delete(counter);
}, this.delayMilliSeconds);
chore: update Styled components to latest version and related cleanup (#19284) ## Description We need to upgrade `styled-components`, so that it will become easy to upgrade to version 6.0 when it is out. This is because, v6.0 has an important functionality which isn't available in today's version. ### Tasks completed - Update Styled components to latest version. - Prepare codebase by cleaning up the styled components functions that will be deprecated in version 6 - We are still using the `withTheme` HOC, we should instead use the `useTheme` hook (best practices) - Remove the `AnyStyledComponent` type it is un-necessary and will be deprecated Fixes #19463 ## Type of change - Non breaking change. The application should work as before and should not effect any visual elements or UI. ## How Has This Been Tested? - Manual @appsmithorg/qa please refer to the test plan for areas of interest. - Cypress: All existing test cases must pass. ### Test Plan - We need to do a sanity check on the Product Updates Modal, Release section. - We also need to do a sanity check on the Login, Signup, ResetPassword pages. - I think we can merge this Pull Request and continue with our weekly regression, because there are no style changes in this Pull Request, everything should work as expected. ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-01-13 11:05:59 +00:00
// @ts-expect-error: setTimeout return type mismatch
this.responses.add(counter);
}
sendEvent(ev: any) {
expect(this.running).toEqual(true);
expect(this.callback).not.toEqual(this.noop);
this.callback(ev);
}
terminate() {
this.running = false;
expect(this.callback).toEqual(this.noop);
this.responses.forEach((counter) => {
clearTimeout(counter);
});
this.responses = new Set<number>();
}
}
describe("GracefulWorkerService", () => {
test("Worker should start", async () => {
const MockWorker = new MockWorkerClass();
const w = new GracefulWorkerService(MockWorker);
// wait for worker to start
await runSaga({}, w.start);
if (MockWorker.instance === undefined) {
expect(MockWorker.instance).toBeDefined();
return;
}
expect(MockWorker.instance.callback).not.toEqual(MockWorker.instance.noop);
});
test("Independent requests should respond independently irrespective of order", async () => {
const MockWorker = new MockWorkerClass();
const w = new GracefulWorkerService(MockWorker);
await runSaga({}, w.start);
const message1 = { tree: "hello" };
const message2 = { tree: "world" };
// Send requests in order
const result1 = await runSaga({}, w.request, "test", message1);
const result2 = await runSaga({}, w.request, "test", message2);
// wait for responses out of order
const resp2 = await result2.toPromise();
const resp1 = await result1.toPromise();
expect(resp1).toEqual(message1);
expect(resp2).toEqual(message2);
});
test("Request should wait for ready", async () => {
const MockWorker = new MockWorkerClass();
const w = new GracefulWorkerService(MockWorker);
const message = { hello: "world" };
// Send a request before starting
const result = await runSaga({}, w.request, "test", message);
// trigger start after the worker is already waiting
runSaga({}, w.start);
const resp = await result.toPromise();
expect(resp).toEqual(message);
});
test("Worker should wait to drain in-flight requests before shutdown", async () => {
const MockWorker = new MockWorkerClass();
const w = new GracefulWorkerService(MockWorker);
const message = { hello: "world" };
await runSaga({}, w.start);
const start = performance.now();
// Need this to work with eslint
if (MockWorker.instance === undefined) {
expect(MockWorker.instance).toBeDefined();
return;
}
// Typical run takes less than 10ms
// we add a delay of 100ms to check if shutdown waited for pending requests.
MockWorker.instance.delayMilliSeconds = 100;
const result = await runSaga({}, w.request, "test", message);
// wait for shutdown
await (await runSaga({}, w.shutdown)).toPromise();
// Shutdown shouldn't happen till we get a response
expect(performance.now() - start).toBeGreaterThanOrEqual(
MockWorker.instance.delayMilliSeconds,
);
const resp = await result.toPromise();
expect(resp).toEqual(message);
});
test("Worker restart should work", async () => {
const MockWorker = new MockWorkerClass();
let w = new GracefulWorkerService(MockWorker);
const message1 = { tree: "hello" };
await runSaga({}, w.start);
// Need this to work with eslint
if (MockWorker.instance === undefined) {
expect(MockWorker.instance).toBeDefined();
return;
}
// Keep a reference to the old instance to check later
const oldInstance = MockWorker.instance;
const result1 = await runSaga({}, w.request, "test", message1);
expect(await result1.toPromise()).toEqual(message1);
// stop the worker
await (await runSaga({}, w.shutdown)).toPromise();
// Should have called terminate on worker
expect(oldInstance.running).toEqual(false);
// Send a message to the new worker before starting it
const newMockWorker = new MockWorkerClass();
w = new GracefulWorkerService(newMockWorker);
const message2 = { tree: "world" };
const result2 = await runSaga({}, w.request, "test", message2);
await runSaga({}, w.start);
// We should have a new instance of the worker
expect(newMockWorker.instance).not.toEqual(oldInstance);
// The new worker should get the correct message
expect(await result2.toPromise()).toEqual(message2);
});
2020-12-30 13:26:44 +00:00
test("Cancelling saga before starting up should not crash", async () => {
const MockWorker = new MockWorkerClass();
2020-12-30 13:26:44 +00:00
const w = new GracefulWorkerService(MockWorker);
const message = { tree: "hello" };
const task = await runSaga({}, w.request, "cancel_test", message);
// Start shutting down
const shutdown = await runSaga({}, w.shutdown);
task.cancel();
// wait for shutdown
await shutdown.toPromise();
expect(await task.toPromise()).not.toEqual(message);
});
test("Cancelled saga should clean up", async () => {
const MockWorker = new MockWorkerClass();
2020-12-30 13:26:44 +00:00
const w = new GracefulWorkerService(MockWorker);
const message = { tree: "hello" };
await runSaga({}, w.start);
// Need this to work with eslint
if (MockWorker.instance === undefined) {
expect(MockWorker.instance).toBeDefined();
return;
}
// Make sure we get a chance to cancel before the worker can respond
MockWorker.instance.delayMilliSeconds = 100;
const task = await runSaga({}, w.request, "cancel_test", message);
// Start shutting down
const shutdown = await runSaga({}, w.shutdown);
task.cancel();
// wait for shutdown
await shutdown.toPromise();
expect(await task.toPromise()).not.toEqual(message);
});
});