Skip to content

Commit d20326b

Browse files
committed
e2e: exclude warmup results from final measurements
1 parent 1d5f83d commit d20326b

File tree

2 files changed

+14
-0
lines changed

2 files changed

+14
-0
lines changed

tests/e2e/server/index.ts

+11
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ type ServerInstance = {
2828
addTestResultListener: AddListener<TestResultListener>;
2929
addTestDoneListener: AddListener<TestDoneListener>;
3030
forceTestCompletion: () => void;
31+
setReadyToAcceptTestResults: (isReady: boolean) => void;
3132
start: () => Promise<void>;
3233
stop: () => Promise<Error | undefined>;
3334
};
@@ -96,6 +97,11 @@ const createServerInstance = (): ServerInstance => {
9697
const [testStartedListeners, addTestStartedListener] = createListenerState<TestStartedListener>();
9798
const [testResultListeners, addTestResultListener] = createListenerState<TestResultListener>();
9899
const [testDoneListeners, addTestDoneListener] = createListenerState<TestDoneListener>();
100+
let isReadyToAcceptTestResults = true;
101+
102+
const setReadyToAcceptTestResults = (isReady: boolean) => {
103+
isReadyToAcceptTestResults = isReady;
104+
};
99105

100106
const forceTestCompletion = () => {
101107
testDoneListeners.forEach((listener) => {
@@ -122,6 +128,10 @@ const createServerInstance = (): ServerInstance => {
122128
}
123129

124130
case Routes.testResults: {
131+
if (!isReadyToAcceptTestResults) {
132+
return res.end('ok');
133+
}
134+
125135
getPostJSONRequestData<TestResult>(req, res)?.then((data) => {
126136
if (!data) {
127137
// The getPostJSONRequestData function already handled the response
@@ -201,6 +211,7 @@ const createServerInstance = (): ServerInstance => {
201211
});
202212

203213
return {
214+
setReadyToAcceptTestResults,
204215
setTestConfig,
205216
addTestStartedListener,
206217
addTestResultListener,

tests/e2e/testRunner.ts

+3
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,7 @@ const runTests = async (): Promise<void> => {
178178
await sleep(config.BOOT_COOL_DOWN);
179179

180180
server.setTestConfig(test as TestConfig);
181+
server.setReadyToAcceptTestResults(false);
181182

182183
const warmupText = `Warmup for test '${test.name}' [${testIndex + 1}/${tests.length}]`;
183184

@@ -193,6 +194,8 @@ const runTests = async (): Promise<void> => {
193194
await runTestIteration(config.DELTA_APP_PACKAGE, `[DELTA] ${warmupText}. Iteration ${i + 1}/${iterations}`);
194195
}
195196

197+
server.setReadyToAcceptTestResults(true);
198+
196199
// For each test case we allow the test to fail three times before we stop the test run:
197200
const errorCountRef = {
198201
errorCount: 0,

0 commit comments

Comments
 (0)