ADL TEST: CM-01

Note - CM -01 is still failing as next activity isn't launched after first activity is complete.

this could do with a peer review too - I'll try to look at this a bit closer unless another dev beats me to it!

Hi Dan,

Could you add testing instruction for this issue?

Thanks

While reviewing this issue, I couldn't get it to passed the first page. I'm using the 'LMSTestPackage_CM-01.zip'

Am I missing any setting?

Hi Rosie - I probably need to write a decent set of testing info for SCORM 2004 packages - it only works in certain browsers, you are supposed to have the adl test engine running etc etc. If you wouldn't mind doing an initial look at the code around coding guidelines/structure etc that would be really useful at this stage.

I'll try to throw a decent test doc explaining how to test a single package later this week

thanks!

Hi Dan,

I tried to setup adl test engine on my machine but it kept killing my machine.

So I follow your suggestion to review the code itself.

On mod/scorm/locallib.php, add break line between if statement.

Then, on line 1703, is there any reason for re-set $scoes as $sco. If it is necessary, it should probably assigned as clone($scoes).

On mod/scorm/module.js line 297 to 308, the conditional for "var scorm_skipprev = function(node) {}" could be simplified to:

if (node.previousSibling && typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') {
    ...
} else if (node.depth > 0 && typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') {
    ....
}    

or

if (typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') {
    var prevscoid = scoes_nav[mod_scorm_launch_sco].prevscoid;
    
    if (node.previousSibling) {
        node.previousSibling.title = scoes_nav[prevscoid].url;
        return node.previousSibling;
    }else if (node.depth > 0) {
        node.parent.title = scoes_nav[prevscoid].url;
        return node.parent;
    }                 
}

On mod/scorm/module.js line 325, the conditional for "var scorm_skipnext = function(node) {}", should the elseif statement check for "typeof scoes_nav[mod_scorm_launch_sco].nextscoid != 'undefined'"?

Let me know if you have any question.

Thanks! - that's helpful feedback - Mayank can you please take a look at Rossiani's comments and make changes as required?

Mayanks patch contains fixes for more than just this test - have reabased and moved his patch into MDL-35418 - will link all the tests that his patch fixes after testing it. Thanks Mayank.

Moodle 2.4 now passes the CM-01 test due to the work in MDL-35418