[MERGE #6272 @rhuanjl] ModuleEvaluation should return undefined

Merge pull request #6272 from rhuanjl:moduleReturnUndefined

Picking up on discussion in #6266 this reverts an unintended side-effect of #6171

The successful completion value of a module is `undefined` - this was not tested and the change in #6171 had resulted in it being  `{done : true}` instead.

Author: Kevin Smith

lib/Runtime/Language/SourceTextModuleRecord.cpp

@@ -1103,6 +1103,7 @@ namespace Js
             {
                 ResumeYieldData yieldData(scriptContext->GetLibrary()->GetUndefined(), nullptr);
                 ret = gen->CallGenerator(&yieldData, _u("Module Global"));
+                ret = JavascriptOperators::GetProperty(VarTo<RecyclableObject>(ret), PropertyIds::value, scriptContext);
             }
             END_SAFE_REENTRANT_CALL
         }