added extra tests for verifying 403 on empty cmi.location. Certain content produced from other tools seems to rely on this error code in order to determine cmi.entry vs. using cmi.entry.

This added test will confirm LMS standardization if run against a platform.
Also added console errors.  When using the SCOBot Runtime API errors are generated, but on other platforms this was transparent.  So the 'getvalue' and 'setvalue' triggers within SCOBot will now trace these events to the console for these QUnit tests.

Author: cybercussion

QUnit-Tests/js/test/scobot-prod.js

@@ -38,10 +38,18 @@ var $     = SCOBotUtil,
 $.addEvent(scorm, "setvalue", function (e) {
     "use strict";
     setvalue_calls += 1;
+    // Hook for trapping SCORM errors.
+    if (e.error.code !== 0) {
+        SB.debug(e.n + " "  + e.v + "\n" + e.error.code + "\n" + e.error.message + "\n" + e.error.diagnostic, 1);
+    }
 });
 $.addEvent(scorm, "getvalue", function (e) {
     "use strict";
     getvalue_calls += 1;
+    // Hook for trapping SCORM errors.
+    if (e.error.code !== 0) {
+        SB.debug(e.n + "\n" + e.error.code + "\n" + e.error.message + "\n" + e.error.diagnostic, 1);
+    }
 });
 $.addEvent(scorm, "StoreData", function (e) {
     "use strict";
@@ -177,12 +185,20 @@ $.addEvent(SB, 'load', function (e) {
     test("Bookmarking", function () {
         if (local) {
             // There would be no bookmark unless one was manually set
+            // This means a request to get Bookmark is cached by SCOBot.  This test will work
+            // with the SCOBotBase (scorm namespace) to directly verify the error code gets set.
+            strictEqual(scorm.getvalue('cmi.location'), '', "Get empty Boomkark");
+            strictEqual(scorm.getLastError().code, 403); // should be value not initialized
+
             strictEqual(SB.setBookmark(2), 'true', 'Setting Bookmark to 2');
             strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
         } else {
             if (SB.getEntry() === "resume") {
                 strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
             } else {
+                strictEqual(scorm.getvalue('cmi.location'), '', "Get empty Boomkark");
+                strictEqual(scorm.getLastError().code, 403); // should be value not initialized
+
                 strictEqual(SB.setBookmark(2), 'true', 'Setting Bookmark to 2');
                 strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
             }

QUnit-Tests/js/test/scobot.js

@@ -38,10 +38,18 @@ var $     = SCOBotUtil,
 $.addEvent(scorm, "setvalue", function (e) {
     "use strict";
     setvalue_calls += 1;
+    // Hook for trapping SCORM errors.
+    if (e.error.code !== 0) {
+        SB.debug(e.n + " "  + e.v + "\n" + e.error.code + "\n" + e.error.message + "\n" + e.error.diagnostic, 1);
+    }
 });
 $.addEvent(scorm, "getvalue", function (e) {
     "use strict";
     getvalue_calls += 1;
+    // Hook for trapping SCORM errors.
+    if (e.error.code !== 0) {
+        SB.debug(e.n + "\n" + e.error.code + "\n" + e.error.message + "\n" + e.error.diagnostic, 1);
+    }
 });
 $.addEvent(scorm, "StoreData", function (e) {
     "use strict";
@@ -172,13 +180,21 @@ $.addEvent(SB, 'load', function (e) {
 
     test("Bookmarking", function () {
         if (local) {
+            // SCOBot will not get a cmi.location unless its entry type is resume or blank
+            // This means a request to get Bookmark is cached by SCOBot.  This test will work
+            // with the SCOBotBase (scorm namespace) to directly verify the error code gets set.
+            strictEqual(scorm.getvalue('cmi.location'), '', "Get empty Boomkark");
+            strictEqual(scorm.getLastError().code, 403); // should be value not initialized
             // There would be no bookmark unless one was manually set
             strictEqual(SB.setBookmark(2), 'true', 'Setting Bookmark to 2');
             strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
         } else {
             if (SB.getEntry() === "resume") {
                 strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
             } else {
+                strictEqual(scorm.getvalue('cmi.location'), '', "Get empty Boomkark");
+                strictEqual(scorm.getLastError().code, 403); // should be value not initialized
+
                 strictEqual(SB.setBookmark(2), 'true', 'Setting Bookmark to 2');
                 strictEqual(SB.getBookmark(), '2', 'Getting Bookmark, should be 2');
             }