MDL-45296 tests: ensure all submissions existv2.7.0-rc2
commit2fe085b2841f80f408050e0bb579d3cb21deae3b
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Thu, 8 May 2014 17:47:47 +0000 (8 19:47 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Thu, 8 May 2014 18:03:34 +0000 (8 20:03 +0200)
tree8ab0fa0ad05a4c3366bf5caeb2ef9e2dc2e4b2b8
parentb95db263ca2ab1498a13a5036b6b73118b682bd2
MDL-45296 tests: ensure all submissions exist

The changes performed with the change from
create_from_user() to create_from_submission()
in practice enforce a new restriction about
submissions having to exist in order to fire
their lock/unlock events.

This did not exist before the change and, also,
it seems that the assign api itself, submission->lock() ...
also accepts non existing submissions.

So I was not 100% sure about how to fix these events:

1) enforcing submission to exist.
2) firing them only if the submission exist.

I've gone with 1) for now, making tests to pass. But will
raise the question in the Tracker, just in case we have
to move to the 2) approach for any reason.
mod/assign/tests/events_test.php
mod/assign/tests/externallib_test.php