[#12894] Per-recipient stats are calculated based on student name, not email

src/web/app/components/question-types/question-statistics/question-statistics-calculation/msq-question-statistics-calculation.ts

@@ -91,19 +91,27 @@
     const perRecipientResponse: Record<string, Record<string, number>> = {};
     const recipientToTeam: Record<string, string> = {};
     const recipientEmails: Record<string, string> = {};
+    const recipientNames: Record<string, string> = {};
     for (const response of this.responses) {
-      perRecipientResponse[response.recipient] = perRecipientResponse[response.recipient] || {};
-      recipientEmails[response.recipient] = recipientEmails[response.recipient] || response.recipientEmail || '';
+      if(!response.recipientEmail){
+        continue;
+      }
+      perRecipientResponse[response.recipientEmail] = perRecipientResponse[response.recipientEmail] || {};
+      recipientEmails[response.recipientEmail] = recipientEmails[response.recipientEmail] || response.recipientEmail || '';
+      recipientNames[response.recipientEmail] = recipientNames[response.recipientEmail] || response.recipient || '';
       for (const choice of this.question.msqChoices) {
-        perRecipientResponse[response.recipient][choice] = 0;
+        perRecipientResponse[response.recipientEmail][choice] = 0;
       }
       if (this.question.otherEnabled) {
-        perRecipientResponse[response.recipient]['Other'] = 0;
+        perRecipientResponse[response.recipientEmail]['Other'] = 0;
       }
-      recipientToTeam[response.recipient] = response.recipientTeam;
+      recipientToTeam[response.recipientEmail] = response.recipientTeam;
     }
     for (const response of this.responses) {
-      this.updateResponseCountPerOptionForResponse(response.responseDetails, perRecipientResponse[response.recipient]);
+      if(!response.recipientEmail){
+        continue;
+      }
+      this.updateResponseCountPerOptionForResponse(response.responseDetails, perRecipientResponse[response.recipientEmail]);
     }
 
     for (const recipient of Object.keys(perRecipientResponse)) {
@@ -120,7 +128,7 @@
       average = numOfResponsesForRecipient ? total / numOfResponsesForRecipient : 0;
 
       this.perRecipientResponses[recipient] = {
-        recipient,
+        recipient: recipientNames[recipient],
         recipientEmail: recipientEmails[recipient],
         total: +total.toFixed(5),
         average: +average.toFixed(2),

src/web/app/components/question-types/question-statistics/test-data/msqQuestionResponses.json

@@ -100,7 +100,7 @@
     }
   ],
   "expectedPerRecipientResponses": {
-    "Alice": {
+    "[email protected]": {
       "average": 1.5,
       "recipient": "Alice",
       "recipientEmail": "[email protected]",
@@ -112,7 +112,7 @@
       },
       "total": 3
     },
-    "Bob": {
+    "[email protected]": {
       "average": 1,
       "recipient": "Bob",
       "recipientEmail": "[email protected]",
@@ -124,7 +124,7 @@
       },
       "total": 1
     },
-    "Charles": {
+    "[email protected]": {
       "average": 0,
       "recipient": "Charles",
       "recipientEmail": "[email protected]",