jprdonnelly/freeCodeCamp
1
0
Fork 0
mirror of https://github.com/freeCodeCamp/freeCodeCamp.git synced 2026-05-26 22:01:30 -04:00
Code Issues Projects Releases Wiki Activity

refactor: simplify test-evaluator (#59037)

Browse Source
This commit is contained in:
Oliver Eyton-Williams
2025-03-10 06:54:37 +01:00
committed by GitHub
parent c142d46b84
commit 44c43e3c0a
1 changed files with 18 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
tools/client-plugins/browser-scripts/test-evaluator.ts
View File

@@ -1,6 +1,6 @@
import chai from 'chai';
import { toString as __toString } from 'lodash-es';
import * as helpers from '@freecodecamp/curriculum-helpers';
import * as curriculumHelpers from '@freecodecamp/curriculum-helpers';
import { format as __format } from './utils/format';
const ctx: Worker & typeof globalThis = self as unknown as Worker &
@@ -80,10 +80,23 @@ const __utils = (() => {
};
})();
// We freeze these two to prevent learners from getting the tester into a weird
// state.
// Fake Deep Equal dependency
// eslint-disable-next-line @typescript-eslint/no-unused-vars
const DeepEqual = (a: unknown, b: unknown) =>
JSON.stringify(a) === JSON.stringify(b);
// We can't simply import these because of how webpack names them when building
// the bundle. Since both assert and __helpers have to exist in the global
// scope, we have to declare them.
const assert = chai.assert;
const __helpers = curriculumHelpers;
// We freeze to prevent learners from getting the tester into a weird
// state by modifying these objects.
Object.freeze(self);
Object.freeze(__utils);
Object.freeze(assert);
Object.freeze(__helpers);
interface TestEvaluatorEvent extends MessageEvent {
data: {
@@ -104,18 +117,8 @@ interface TestEvaluatorEvent extends MessageEvent {
/* Run the test if there is one. If not just evaluate the user code */
ctx.onmessage = async (e: TestEvaluatorEvent) => {
/* eslint-disable @typescript-eslint/no-unused-vars */
const code = (e.data?.code?.contents || '').slice();
const editableContents = (e.data?.code?.editableContents || '').slice();
const assert = chai.assert;
const __helpers = helpers;
// Similarly to self and __utils, if the learner tries to modify these, weird
// behavior may result. Freezing them means they get an error instead.
Object.freeze(assert);
Object.freeze(__helpers);
// Fake Deep Equal dependency
const DeepEqual = (a: unknown, b: unknown) =>
JSON.stringify(a) === JSON.stringify(b);
const code = e.data?.code?.contents || '';
const editableContents = e.data?.code?.editableContents || '';
// Build errors should be reported, but only once:
__utils.toggleProxyLogger(e.data.firstTest);