fix: only include frame-runner when running tests (#44337)

* fix: only include frame-runner when running tests

* fix: handle missing options

* test: use test runner for curriculum tests

client/src/templates/Challenges/redux/execute-challenge-saga.js

@@ -86,7 +86,8 @@ export function* executeChallengeSaga({ payload }) {
     const protect = isLoopProtected(challengeMeta);
     const buildData = yield buildChallengeData(challengeData, {
       preview: false,
-      protect
+      protect,
+      usesTestRunner: true
     });
     const document = yield getContext('document');
     const testRunner = yield call(

client/src/templates/Challenges/utils/build.js

@@ -138,12 +138,13 @@ async function getDOMTestRunner(buildData, { proxyLogger }, document) {
     runTestInTestFrame(document, testString, testTimeout);
 }
 
-export function buildDOMChallenge({
+export function buildDOMChallenge(
-  challengeFiles,
+  { challengeFiles, required = [], template = '' },
-  required = [],
+  { usesTestRunner } = { usesTestRunner: false }
-  template = ''
+) {
-}) {
+  const finalRequires = [...globalRequires, ...required];
-  const finalRequires = [...globalRequires, ...required, ...frameRunner];
+  if (usesTestRunner) finalRequires.push(...frameRunner);
+
   const loadEnzyme = challengeFiles.some(
     challengeFile => challengeFile.ext === 'jsx'
   );

curriculum/test/test-challenges.js

@@ -570,11 +570,14 @@ async function createTestRunner(
     challengeFile.editableContents = solutionFile.editableContents;
   });
 
-  const { build, sources, loadEnzyme } = await buildChallenge({
+  const { build, sources, loadEnzyme } = await buildChallenge(
-    challengeFiles,
+    {
-    required,
+      challengeFiles,
-    template
+      required,
-  });
+      template
+    },
+    { usesTestRunner: true }
+  );
 
   const code = {
     contents: sources.index,