scorm-again 1.5.3 → 1.7.0

package/src/Scorm12API.js

@@ -202,7 +202,7 @@ export default class Scorm12API extends BaseAPI {
    * Returns the message that corresponds to errorNumber.
    *
    * @param {*} errorNumber
-   * @param {boolean }detail
+   * @param {boolean} detail
    * @return {string}
    */
   getLmsErrorMessageDetails(errorNumber, detail) {
@@ -278,8 +278,7 @@ export default class Scorm12API extends BaseAPI {
           if (this.settings.mastery_override &&
               this.cmi.student_data.mastery_score !== '' &&
               this.cmi.core.score.raw !== '') {
-            if (parseFloat(this.cmi.core.score.raw) >=
-                parseFloat(this.cmi.student_data.mastery_score)) {
+            if (parseFloat(this.cmi.core.score.raw) >= parseFloat(this.cmi.student_data.mastery_score)) {
               this.cmi.core.lesson_status = 'passed';
             } else {
               this.cmi.core.lesson_status = 'failed';
@@ -287,8 +286,7 @@ export default class Scorm12API extends BaseAPI {
           }
         }
       } else if (this.cmi.core.lesson_mode === 'browse') {
-        if ((this.startingData?.cmi?.core?.lesson_status || '') === '' &&
-            originalStatus === 'not attempted') {
+        if ((this.startingData?.cmi?.core?.lesson_status || '') === '' && originalStatus === 'not attempted') {
           this.cmi.core.lesson_status = 'browsed';
         }
       }
@@ -297,18 +295,13 @@ export default class Scorm12API extends BaseAPI {
     const commitObject = this.renderCommitCMI(terminateCommit ||
         this.settings.alwaysSendTotalTime);
 
+    if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
+      console.debug('Commit (terminated: ' + (terminateCommit ? 'yes' : 'no') + '): ');
+      console.debug(commitObject);
+    }
     if (this.settings.lmsCommitUrl) {
-      if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
-        console.debug('Commit (terminated: ' +
-            (terminateCommit ? 'yes' : 'no') + '): ');
-        console.debug(commitObject);
-      }
-      return this.processHttpRequest(this.settings.lmsCommitUrl, commitObject,
-          terminateCommit);
+      return this.processHttpRequest(this.settings.lmsCommitUrl, commitObject, terminateCommit);
     } else {
-      console.log('Commit (terminated: ' +
-          (terminateCommit ? 'yes' : 'no') + '): ');
-      console.log(commitObject);
       return global_constants.SCORM_TRUE;
     }
   }

package/src/Scorm2004API.js

@@ -548,12 +548,12 @@ export default class Scorm2004API extends BaseAPI {
     const commitObject = this.renderCommitCMI(terminateCommit ||
         this.settings.alwaysSendTotalTime);
 
-    if (this.settings.lmsCommitUrl) {
-      if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
-        console.debug('Commit (terminated: ' +
+    if (this.apiLogLevel === global_constants.LOG_LEVEL_DEBUG) {
+      console.debug('Commit (terminated: ' +
             (terminateCommit ? 'yes' : 'no') + '): ');
-        console.debug(commitObject);
-      }
+      console.debug(commitObject);
+    }
+    if (this.settings.lmsCommitUrl) {
       const result = this.processHttpRequest(this.settings.lmsCommitUrl,
           commitObject, terminateCommit);
 
@@ -566,9 +566,6 @@ export default class Scorm2004API extends BaseAPI {
       }
       return result;
     } else {
-      console.log('Commit (terminated: ' +
-          (terminateCommit ? 'yes' : 'no') + '): ');
-      console.log(commitObject);
       return global_constants.SCORM_TRUE;
     }
   }

package/src/cmi/aicc_cmi.js

@@ -1,16 +1,40 @@
 import * as Scorm12CMI from './scorm12_cmi';
-import {BaseCMI, CMIArray, CMIScore} from './common';
+import {BaseCMI, checkValidFormat, CMIArray, CMIScore} from './common';
 import APIConstants from '../constants/api_constants';
 import Regex from '../constants/regex';
 import ErrorCodes from '../constants/error_codes';
-import {
-  check12ValidFormat,
-  throwReadOnlyError,
-} from './scorm12_cmi';
+import {AICCValidationError} from '../exceptions';
 
 const aicc_constants = APIConstants.aicc;
 const aicc_regex = Regex.aicc;
-const scorm12_error_codes = ErrorCodes.scorm12;
+const aicc_error_codes = ErrorCodes.scorm12;
+
+/**
+ * Helper method for throwing Read Only error
+ */
+function throwReadOnlyError() {
+  throw new AICCValidationError(aicc_error_codes.READ_ONLY_ELEMENT);
+}
+
+/**
+ * Helper method, no reason to have to pass the same error codes every time
+ * @param {*} value
+ * @param {string} regexPattern
+ * @param {boolean} allowEmptyString
+ * @return {boolean}
+ */
+function checkAICCValidFormat(
+    value: String,
+    regexPattern: String,
+    allowEmptyString?: boolean) {
+  return checkValidFormat(
+      value,
+      regexPattern,
+      aicc_error_codes.TYPE_MISMATCH,
+      AICCValidationError,
+      allowEmptyString
+  );
+}
 
 /**
  * CMI Class for AICC
@@ -126,8 +150,11 @@ class CMIEvaluationComments extends CMIArray {
    * Constructor for AICC Evaluation Comments object
    */
   constructor() {
-    super(aicc_constants.comments_children,
-        scorm12_error_codes.INVALID_SET_VALUE);
+    super({
+      children: aicc_constants.comments_children,
+      errorCode: aicc_error_codes.INVALID_SET_VALUE,
+      errorClass: AICCValidationError,
+    });
   }
 }
 
@@ -142,7 +169,8 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
     super(aicc_constants.student_preference_children);
 
     this.windows = new CMIArray({
-      errorCode: scorm12_error_codes.INVALID_SET_VALUE,
+      errorCode: aicc_error_codes.INVALID_SET_VALUE,
+      errorClass: AICCValidationError,
       children: '',
     });
   }
@@ -174,7 +202,7 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
    * @param {string} lesson_type
    */
   set lesson_type(lesson_type: string) {
-    if (check12ValidFormat(lesson_type, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(lesson_type, aicc_regex.CMIString256)) {
       this.#lesson_type = lesson_type;
     }
   }
@@ -192,7 +220,7 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
    * @param {string} text_color
    */
   set text_color(text_color: string) {
-    if (check12ValidFormat(text_color, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(text_color, aicc_regex.CMIString256)) {
       this.#text_color = text_color;
     }
   }
@@ -210,7 +238,7 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
    * @param {string} text_location
    */
   set text_location(text_location: string) {
-    if (check12ValidFormat(text_location, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(text_location, aicc_regex.CMIString256)) {
       this.#text_location = text_location;
     }
   }
@@ -228,7 +256,7 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
    * @param {string} text_size
    */
   set text_size(text_size: string) {
-    if (check12ValidFormat(text_size, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(text_size, aicc_regex.CMIString256)) {
       this.#text_size = text_size;
     }
   }
@@ -246,7 +274,7 @@ class AICCStudentPreferences extends Scorm12CMI.CMIStudentPreference {
    * @param {string} video
    */
   set video(video: string) {
-    if (check12ValidFormat(video, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(video, aicc_regex.CMIString256)) {
       this.#video = video;
     }
   }
@@ -374,6 +402,14 @@ export class CMIStudentDemographics extends BaseCMI {
   #telephone = '';
   #years_experience = '';
 
+  /**
+   * Getter for _children
+   * @return {string}
+   */
+  get _children() {
+    return this.#_children;
+  }
+
   /**
    * Getter for city
    * @return {string}
@@ -671,7 +707,7 @@ export class CMIPaths extends CMIArray {
    * Constructor for inline Paths Array class
    */
   constructor() {
-    super(aicc_constants.paths_children);
+    super({children: aicc_constants.paths_children});
   }
 }
 
@@ -706,7 +742,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} location_id
    */
   set location_id(location_id) {
-    if (check12ValidFormat(location_id, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(location_id, aicc_regex.CMIString256)) {
       this.#location_id = location_id;
     }
   }
@@ -724,7 +760,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} date
    */
   set date(date) {
-    if (check12ValidFormat(date, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(date, aicc_regex.CMIString256)) {
       this.#date = date;
     }
   }
@@ -742,7 +778,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} time
    */
   set time(time) {
-    if (check12ValidFormat(time, aicc_regex.CMITime)) {
+    if (checkAICCValidFormat(time, aicc_regex.CMITime)) {
       this.#time = time;
     }
   }
@@ -760,7 +796,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} status
    */
   set status(status) {
-    if (check12ValidFormat(status, aicc_regex.CMIStatus2)) {
+    if (checkAICCValidFormat(status, aicc_regex.CMIStatus2)) {
       this.#status = status;
     }
   }
@@ -778,7 +814,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} why_left
    */
   set why_left(why_left) {
-    if (check12ValidFormat(why_left, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(why_left, aicc_regex.CMIString256)) {
       this.#why_left = why_left;
     }
   }
@@ -796,7 +832,7 @@ export class CMIPathsObject extends BaseCMI {
    * @param {string} time_in_element
    */
   set time_in_element(time_in_element) {
-    if (check12ValidFormat(time_in_element, aicc_regex.CMITime)) {
+    if (checkAICCValidFormat(time_in_element, aicc_regex.CMITime)) {
       this.#time_in_element = time_in_element;
     }
   }
@@ -837,7 +873,7 @@ export class CMITries extends CMIArray {
    * Constructor for inline Tries Array class
    */
   constructor() {
-    super(aicc_constants.tries_children);
+    super({children: aicc_constants.tries_children});
   }
 }
 
@@ -855,9 +891,10 @@ export class CMITriesObject extends BaseCMI {
         {
           score_children: aicc_constants.score_children,
           score_range: aicc_regex.score_range,
-          invalidErrorCode: scorm12_error_codes.INVALID_SET_VALUE,
-          invalidTypeCode: scorm12_error_codes.TYPE_MISMATCH,
-          invalidRangeCode: scorm12_error_codes.VALUE_OUT_OF_RANGE,
+          invalidErrorCode: aicc_error_codes.INVALID_SET_VALUE,
+          invalidTypeCode: aicc_error_codes.TYPE_MISMATCH,
+          invalidRangeCode: aicc_error_codes.VALUE_OUT_OF_RANGE,
+          errorClass: AICCValidationError,
         });
   }
 
@@ -885,7 +922,7 @@ export class CMITriesObject extends BaseCMI {
    * @param {string} status
    */
   set status(status) {
-    if (check12ValidFormat(status, aicc_regex.CMIStatus2)) {
+    if (checkAICCValidFormat(status, aicc_regex.CMIStatus2)) {
       this.#status = status;
     }
   }
@@ -903,7 +940,7 @@ export class CMITriesObject extends BaseCMI {
    * @param {string} time
    */
   set time(time) {
-    if (check12ValidFormat(time, aicc_regex.CMITime)) {
+    if (checkAICCValidFormat(time, aicc_regex.CMITime)) {
       this.#time = time;
     }
   }
@@ -938,7 +975,7 @@ export class CMIAttemptRecords extends CMIArray {
    * Constructor for inline Tries Array class
    */
   constructor() {
-    super(aicc_constants.attempt_records_children);
+    super({children: aicc_constants.attempt_records_children});
   }
 }
 
@@ -956,9 +993,10 @@ export class CMIAttemptRecordsObject extends BaseCMI {
         {
           score_children: aicc_constants.score_children,
           score_range: aicc_regex.score_range,
-          invalidErrorCode: scorm12_error_codes.INVALID_SET_VALUE,
-          invalidTypeCode: scorm12_error_codes.TYPE_MISMATCH,
-          invalidRangeCode: scorm12_error_codes.VALUE_OUT_OF_RANGE,
+          invalidErrorCode: aicc_error_codes.INVALID_SET_VALUE,
+          invalidTypeCode: aicc_error_codes.TYPE_MISMATCH,
+          invalidRangeCode: aicc_error_codes.VALUE_OUT_OF_RANGE,
+          errorClass: AICCValidationError,
         });
   }
 
@@ -985,7 +1023,7 @@ export class CMIAttemptRecordsObject extends BaseCMI {
    * @param {string} lesson_status
    */
   set lesson_status(lesson_status) {
-    if (check12ValidFormat(lesson_status, aicc_regex.CMIStatus2)) {
+    if (checkAICCValidFormat(lesson_status, aicc_regex.CMIStatus2)) {
       this.#lesson_status = lesson_status;
     }
   }
@@ -1039,7 +1077,7 @@ export class CMIEvaluationCommentsObject extends BaseCMI {
    * @param {string} content
    */
   set content(content) {
-    if (check12ValidFormat(content, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(content, aicc_regex.CMIString256)) {
       this.#content = content;
     }
   }
@@ -1057,7 +1095,7 @@ export class CMIEvaluationCommentsObject extends BaseCMI {
    * @param {string} location
    */
   set location(location) {
-    if (check12ValidFormat(location, aicc_regex.CMIString256)) {
+    if (checkAICCValidFormat(location, aicc_regex.CMIString256)) {
       this.#location = location;
     }
   }
@@ -1075,7 +1113,7 @@ export class CMIEvaluationCommentsObject extends BaseCMI {
    * @param {string} time
    */
   set time(time) {
-    if (check12ValidFormat(time, aicc_regex.CMITime)) {
+    if (checkAICCValidFormat(time, aicc_regex.CMITime)) {
       this.#time = time;
     }
   }

package/src/cmi/common.js

@@ -1,7 +1,6 @@
 // @flow
 import APIConstants from '../constants/api_constants';
 import ErrorCodes from '../constants/error_codes';
-import {ValidationError} from '../exceptions';
 import Regex from '../constants/regex';
 
 const scorm12_constants = APIConstants.scorm12;
@@ -14,6 +13,7 @@ const scorm12_error_codes = ErrorCodes.scorm12;
  * @param {string} value
  * @param {string} regexPattern
  * @param {number} errorCode
+ * @param {class} errorClass
  * @param {boolean} allowEmptyString
  * @return {boolean}
  */
@@ -21,6 +21,7 @@ export function checkValidFormat(
     value: String,
     regexPattern: String,
     errorCode: number,
+    errorClass: function,
     allowEmptyString?: boolean) {
   const formatRegex = new RegExp(regexPattern);
   const matches = value.match(formatRegex);
@@ -28,7 +29,7 @@ export function checkValidFormat(
     return true;
   }
   if (value === undefined || !matches || matches[0] === '') {
-    throw new ValidationError(errorCode);
+    throw new errorClass.prototype.constructor(errorCode);
   }
   return true;
 }
@@ -39,20 +40,24 @@ export function checkValidFormat(
  * @param {*} value
  * @param {string} rangePattern
  * @param {number} errorCode
+ * @param {class} errorClass
  * @return {boolean}
  */
 export function checkValidRange(
-    value: any, rangePattern: String, errorCode: number) {
+    value: any,
+    rangePattern: String,
+    errorCode: number,
+    errorClass: function) {
   const ranges = rangePattern.split('#');
   value = value * 1.0;
   if (value >= ranges[0]) {
     if ((ranges[1] === '*') || (value <= ranges[1])) {
       return true;
     } else {
-      throw new ValidationError(errorCode);
+      throw new errorClass.prototype.constructor(errorCode);
     }
   } else {
-    throw new ValidationError(errorCode);
+    throw new errorClass.prototype.constructor(errorCode);
   }
 }
 
@@ -118,6 +123,7 @@ export class CMIScore extends BaseCMI {
    * @param {number} invalidTypeCode
    * @param {number} invalidRangeCode
    * @param {string} decimalRegex
+   * @param {class} errorClass
    */
   constructor(
       {
@@ -128,6 +134,7 @@ export class CMIScore extends BaseCMI {
         invalidTypeCode,
         invalidRangeCode,
         decimalRegex,
+        errorClass,
       }) {
     super();
 
@@ -143,6 +150,7 @@ export class CMIScore extends BaseCMI {
         scorm12_error_codes.VALUE_OUT_OF_RANGE;
     this.#_decimal_regex = decimalRegex ||
         scorm12_regex.CMIDecimal;
+    this.#_error_class = errorClass;
   }
 
   #_children;
@@ -151,6 +159,7 @@ export class CMIScore extends BaseCMI {
   #_invalid_type_code;
   #_invalid_range_code;
   #_decimal_regex;
+  #_error_class;
   #raw = '';
   #min = '';
   #max;
@@ -170,7 +179,7 @@ export class CMIScore extends BaseCMI {
    * @private
    */
   set _children(_children) {
-    throw new ValidationError(this.#_invalid_error_code);
+    throw new this.#_error_class.prototype.constructor(this.#_invalid_error_code);
   }
 
   /**
@@ -186,11 +195,9 @@ export class CMIScore extends BaseCMI {
    * @param {string} raw
    */
   set raw(raw) {
-    if (checkValidFormat(raw, this.#_decimal_regex,
-        this.#_invalid_type_code) &&
+    if (checkValidFormat(raw, this.#_decimal_regex, this.#_invalid_type_code, this.#_error_class) &&
         (!this.#_score_range ||
-            checkValidRange(raw, this.#_score_range,
-                this.#_invalid_range_code))) {
+            checkValidRange(raw, this.#_score_range, this.#_invalid_range_code, this.#_error_class))) {
       this.#raw = raw;
     }
   }
@@ -208,11 +215,9 @@ export class CMIScore extends BaseCMI {
    * @param {string} min
    */
   set min(min) {
-    if (checkValidFormat(min, this.#_decimal_regex,
-        this.#_invalid_type_code) &&
+    if (checkValidFormat(min, this.#_decimal_regex, this.#_invalid_type_code, this.#_error_class) &&
         (!this.#_score_range ||
-            checkValidRange(min, this.#_score_range,
-                this.#_invalid_range_code))) {
+            checkValidRange(min, this.#_score_range, this.#_invalid_range_code, this.#_error_class))) {
       this.#min = min;
     }
   }
@@ -230,11 +235,9 @@ export class CMIScore extends BaseCMI {
    * @param {string} max
    */
   set max(max) {
-    if (checkValidFormat(max, this.#_decimal_regex,
-        this.#_invalid_type_code) &&
+    if (checkValidFormat(max, this.#_decimal_regex, this.#_invalid_type_code, this.#_error_class) &&
         (!this.#_score_range ||
-            checkValidRange(max, this.#_score_range,
-                this.#_invalid_range_code))) {
+            checkValidRange(max, this.#_score_range, this.#_invalid_range_code, this.#_error_class))) {
       this.#max = max;
     }
   }
@@ -263,15 +266,18 @@ export class CMIArray extends BaseCMI {
    * Constructor cmi *.n arrays
    * @param {string} children
    * @param {number} errorCode
+   * @param {class} errorClass
    */
-  constructor({children, errorCode}) {
+  constructor({children, errorCode, errorClass}) {
     super();
     this.#_children = children;
     this.#errorCode = errorCode;
+    this.#errorClass = errorClass;
     this.childArray = [];
   }
 
   #errorCode;
+  #errorClass;
   #_children;
 
   /**
@@ -287,7 +293,7 @@ export class CMIArray extends BaseCMI {
    * @param {string} _children
    */
   set _children(_children) {
-    throw new ValidationError(this.#errorCode);
+    throw new this.#errorClass.prototype.constructor(this.#errorCode);
   }
 
   /**
@@ -303,7 +309,7 @@ export class CMIArray extends BaseCMI {
    * @param {number} _count
    */
   set _count(_count) {
-    throw new ValidationError(this.#errorCode);
+    throw new this.#errorClass.prototype.constructor(this.#errorCode);
   }
 
   /**

package/src/cmi/scorm12_cmi.js

@@ -9,7 +9,7 @@ import {
 import APIConstants from '../constants/api_constants';
 import ErrorCodes from '../constants/error_codes';
 import Regex from '../constants/regex';
-import {ValidationError} from '../exceptions';
+import {Scorm12ValidationError} from '../exceptions';
 import * as Utilities from '../utilities';
 import * as Util from '../utilities';
 
@@ -21,21 +21,21 @@ const scorm12_error_codes = ErrorCodes.scorm12;
  * Helper method for throwing Read Only error
  */
 export function throwReadOnlyError() {
-  throw new ValidationError(scorm12_error_codes.READ_ONLY_ELEMENT);
+  throw new Scorm12ValidationError(scorm12_error_codes.READ_ONLY_ELEMENT);
 }
 
 /**
  * Helper method for throwing Write Only error
  */
 export function throwWriteOnlyError() {
-  throw new ValidationError(scorm12_error_codes.WRITE_ONLY_ELEMENT);
+  throw new Scorm12ValidationError(scorm12_error_codes.WRITE_ONLY_ELEMENT);
 }
 
 /**
  * Helper method for throwing Invalid Set error
  */
 function throwInvalidValueError() {
-  throw new ValidationError(scorm12_error_codes.INVALID_SET_VALUE);
+  throw new Scorm12ValidationError(scorm12_error_codes.INVALID_SET_VALUE);
 }
 
 /**
@@ -49,8 +49,13 @@ export function check12ValidFormat(
     value: String,
     regexPattern: String,
     allowEmptyString?: boolean) {
-  return checkValidFormat(value, regexPattern,
-      scorm12_error_codes.TYPE_MISMATCH, allowEmptyString);
+  return checkValidFormat(
+      value,
+      regexPattern,
+      scorm12_error_codes.TYPE_MISMATCH,
+      Scorm12ValidationError,
+      allowEmptyString
+  );
 }
 
 /**
@@ -64,8 +69,13 @@ export function check12ValidRange(
     value: any,
     rangePattern: String,
     allowEmptyString?: boolean) {
-  return checkValidRange(value, rangePattern,
-      scorm12_error_codes.VALUE_OUT_OF_RANGE, allowEmptyString);
+  return checkValidRange(
+      value,
+      rangePattern,
+      scorm12_error_codes.VALUE_OUT_OF_RANGE,
+      Scorm12ValidationError,
+      allowEmptyString
+  );
 }
 
 /**
@@ -277,6 +287,7 @@ class CMICore extends BaseCMI {
           invalidErrorCode: scorm12_error_codes.INVALID_SET_VALUE,
           invalidTypeCode: scorm12_error_codes.TYPE_MISMATCH,
           invalidRangeCode: scorm12_error_codes.VALUE_OUT_OF_RANGE,
+          errorClass: Scorm12ValidationError,
         });
   }
 
@@ -583,6 +594,7 @@ class CMIObjectives extends CMIArray {
     super({
       children: scorm12_constants.objectives_children,
       errorCode: scorm12_error_codes.INVALID_SET_VALUE,
+      errorClass: Scorm12ValidationError,
     });
   }
 }
@@ -858,6 +870,7 @@ class CMIInteractions extends CMIArray {
     super({
       children: scorm12_constants.interactions_children,
       errorCode: scorm12_error_codes.INVALID_SET_VALUE,
+      errorClass: Scorm12ValidationError,
     });
   }
 }
@@ -875,10 +888,12 @@ export class CMIInteractionsObject extends BaseCMI {
 
     this.objectives = new CMIArray({
       errorCode: scorm12_error_codes.INVALID_SET_VALUE,
+      errorClass: Scorm12ValidationError,
       children: scorm12_constants.objectives_children,
     });
     this.correct_responses = new CMIArray({
       errorCode: scorm12_error_codes.INVALID_SET_VALUE,
+      errorClass: Scorm12ValidationError,
       children: scorm12_constants.correct_responses_children,
     });
   }
@@ -1082,6 +1097,7 @@ export class CMIObjectivesObject extends BaseCMI {
           invalidErrorCode: scorm12_error_codes.INVALID_SET_VALUE,
           invalidTypeCode: scorm12_error_codes.TYPE_MISMATCH,
           invalidRangeCode: scorm12_error_codes.VALUE_OUT_OF_RANGE,
+          errorClass: Scorm12ValidationError,
         });
   }