Skip to content

Commit

Permalink
Fix: wrapper.js & adapt-contrib-spoor updated to work with Jest testi…
Browse files Browse the repository at this point in the history
…ng. (fixes adaptlearning#284) (adaptlearning#285)

* Fix: wrapper.js & adapt-contrib-spoor updated to work with Jest testing.

* Fix: Import of jquery.keycombo amnded.
  • Loading branch information
joe-allen-89 committed Jul 26, 2023
1 parent 2cde7ac commit cb3d93f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
9 changes: 4 additions & 5 deletions js/adapt-contrib-spoor.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import StatefulSession from './adapt-stateful-session';
import OfflineStorage from './adapt-offlineStorage-scorm';
import offlineStorage from 'core/js/offlineStorage';
import { shouldStart as shouldStartCookieLMS, start as startCookieLMS } from './scorm/cookieLMS';
import './../libraries/jquery.keycombo';

class Spoor extends Backbone.Controller {

Expand All @@ -27,12 +28,10 @@ class Spoor extends Backbone.Controller {
// of the course data loads
offlineStorage.get();
offlineStorage.setReadyStatus();

// setup debug window keyboard shortcut
require(['libraries/jquery.keycombo'], () => {
// listen for user holding 'd', 'e', 'v' keys together
$.onKeyCombo([68, 69, 86], () => {
Adapt.spoor.scorm.showDebugWindow();
});
$.onKeyCombo([68, 69, 86], () => {
Adapt.spoor.scorm.showDebugWindow();
});
}

Expand Down
2 changes: 2 additions & 0 deletions js/scorm/wrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -584,6 +584,8 @@ class ScormWrapper {
if (error.data.value === '') error.data.value = '\'\'';
}

if (!Adapt.course) return;

const config = Adapt.course.get('_spoor');
const messages = Object.assign({}, ScormError.defaultMessages, config && config._messages);
const message = Handlebars.compile(messages[error.name])(error.data);
Expand Down

0 comments on commit cb3d93f

Please sign in to comment.