Search moodle.org's
Developer Documentation

See Release Notes

  • Bug fixes for general core bugs in 4.3.x will end 7 October 2024 (12 months).
  • Bug fixes for security issues in 4.3.x will end 21 April 2025 (18 months).
  • PHP version: minimum PHP 8.0.0 Note: minimum PHP version has increased since Moodle 4.1. PHP 8.2.x is supported too.

Differences Between: [Versions 310 and 403] [Versions 39 and 403]

   1  <?php
   2  // This file is part of Moodle - http://moodle.org/
   3  //
   4  // Moodle is free software: you can redistribute it and/or modify
   5  // it under the terms of the GNU General Public License as published by
   6  // the Free Software Foundation, either version 3 of the License, or
   7  // (at your option) any later version.
   8  //
   9  // Moodle is distributed in the hope that it will be useful,
  10  // but WITHOUT ANY WARRANTY; without even the implied warranty of
  11  // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  12  // GNU General Public License for more details.
  13  //
  14  // You should have received a copy of the GNU General Public License
  15  // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
  16  
  17  namespace core_calendar;
  18  
  19  use action_event_test_factory;
  20  use core_calendar\local\event\data_access\event_vault;
  21  use core_calendar\local\event\strategies\raw_event_retrieval_strategy;
  22  
  23  defined('MOODLE_INTERNAL') || die();
  24  
  25  global $CFG;
  26  require_once($CFG->dirroot . '/calendar/tests/helpers.php');
  27  
  28  /**
  29   * This file contains the class that handles testing of the calendar event vault.
  30   *
  31   * @package core_calendar
  32   * @copyright 2017 Ryan Wyllie <ryan@moodle.com>
  33   * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  34   */
  35  class event_vault_test extends \advanced_testcase {
  36  
  37      /**
  38       * Test that get_action_events_by_timesort returns events after the
  39       * provided timesort value.
  40       */
  41      public function test_get_action_events_by_timesort_after_time() {
  42          $this->resetAfterTest(true);
  43  
  44          $user = $this->getDataGenerator()->create_user();
  45          $factory = new action_event_test_factory();
  46          $strategy = new raw_event_retrieval_strategy();
  47          $vault = new event_vault($factory, $strategy);
  48  
  49          $this->setUser($user);
  50  
  51          for ($i = 1; $i < 6; $i++) {
  52              create_event([
  53                  'name' => sprintf('Event %d', $i),
  54                  'eventtype' => 'user',
  55                  'userid' => $user->id,
  56                  'timesort' => $i,
  57                  'type' => CALENDAR_EVENT_TYPE_ACTION
  58              ]);
  59          }
  60  
  61          $events = $vault->get_action_events_by_timesort($user, 3);
  62  
  63          $this->assertCount(3, $events);
  64          $this->assertEquals('Event 3', $events[0]->get_name());
  65          $this->assertEquals('Event 4', $events[1]->get_name());
  66          $this->assertEquals('Event 5', $events[2]->get_name());
  67  
  68          $events = $vault->get_action_events_by_timesort($user, 3, null, null, 1);
  69  
  70          $this->assertCount(1, $events);
  71          $this->assertEquals('Event 3', $events[0]->get_name());
  72  
  73          $events = $vault->get_action_events_by_timesort($user, 6);
  74  
  75          $this->assertCount(0, $events);
  76      }
  77  
  78      /**
  79       * Test that get_action_events_by_timesort returns events before the
  80       * provided timesort value.
  81       */
  82      public function test_get_action_events_by_timesort_before_time() {
  83          $this->resetAfterTest(true);
  84          $this->setAdminuser();
  85  
  86          $user = $this->getDataGenerator()->create_user();
  87          $factory = new action_event_test_factory();
  88          $strategy = new raw_event_retrieval_strategy();
  89          $vault = new event_vault($factory, $strategy);
  90  
  91          for ($i = 1; $i < 6; $i++) {
  92              create_event([
  93                  'name' => sprintf('Event %d', $i),
  94                  'eventtype' => 'user',
  95                  'userid' => $user->id,
  96                  'timesort' => $i,
  97                  'type' => CALENDAR_EVENT_TYPE_ACTION,
  98                  'courseid' => 1
  99              ]);
 100          }
 101  
 102          $events = $vault->get_action_events_by_timesort($user, null, 3);
 103  
 104          $this->assertCount(3, $events);
 105          $this->assertEquals('Event 1', $events[0]->get_name());
 106          $this->assertEquals('Event 2', $events[1]->get_name());
 107          $this->assertEquals('Event 3', $events[2]->get_name());
 108  
 109          $events = $vault->get_action_events_by_timesort($user, null, 3, null, 1);
 110  
 111          $this->assertCount(1, $events);
 112          $this->assertEquals('Event 1', $events[0]->get_name());
 113  
 114          $events = $vault->get_action_events_by_timesort($user, 6);
 115  
 116          $this->assertCount(0, $events);
 117      }
 118  
 119      /**
 120       * Test that get_action_events_by_timesort returns events between the
 121       * provided timesort values.
 122       */
 123      public function test_get_action_events_by_timesort_between_time() {
 124          $this->resetAfterTest(true);
 125          $this->setAdminuser();
 126  
 127          $user = $this->getDataGenerator()->create_user();
 128          $factory = new action_event_test_factory();
 129          $strategy = new raw_event_retrieval_strategy();
 130          $vault = new event_vault($factory, $strategy);
 131  
 132          for ($i = 1; $i < 6; $i++) {
 133              create_event([
 134                  'name' => sprintf('Event %d', $i),
 135                  'eventtype' => 'user',
 136                  'userid' => $user->id,
 137                  'timesort' => $i,
 138                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 139                  'courseid' => 1
 140              ]);
 141          }
 142  
 143          $events = $vault->get_action_events_by_timesort($user, 2, 4);
 144  
 145          $this->assertCount(3, $events);
 146          $this->assertEquals('Event 2', $events[0]->get_name());
 147          $this->assertEquals('Event 3', $events[1]->get_name());
 148          $this->assertEquals('Event 4', $events[2]->get_name());
 149  
 150          $events = $vault->get_action_events_by_timesort($user, 2, 4, null, 1);
 151  
 152          $this->assertCount(1, $events);
 153          $this->assertEquals('Event 2', $events[0]->get_name());
 154      }
 155  
 156      /**
 157       * Test that get_action_events_by_timesort returns events between the
 158       * provided timesort values and after the last seen event when one is
 159       * provided.
 160       */
 161      public function test_get_action_events_by_timesort_between_time_after_event() {
 162          $this->resetAfterTest(true);
 163          $this->setAdminuser();
 164  
 165          $user = $this->getDataGenerator()->create_user();
 166          $factory = new action_event_test_factory();
 167          $strategy = new raw_event_retrieval_strategy();
 168          $vault = new event_vault($factory, $strategy);
 169  
 170          $records = [];
 171          for ($i = 1; $i < 21; $i++) {
 172              $records[] = create_event([
 173                  'name' => sprintf('Event %d', $i),
 174                  'eventtype' => 'user',
 175                  'userid' => $user->id,
 176                  'timesort' => $i,
 177                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 178                  'courseid' => 1
 179              ]);
 180          }
 181  
 182          $aftereventid = $records[6]->id;
 183          $afterevent = $vault->get_event_by_id($aftereventid);
 184          $events = $vault->get_action_events_by_timesort($user, 3, 15, $afterevent);
 185  
 186          $this->assertCount(8, $events);
 187          $this->assertEquals('Event 8', $events[0]->get_name());
 188  
 189          $events = $vault->get_action_events_by_timesort($user, 3, 15, $afterevent, 3);
 190  
 191          $this->assertCount(3, $events);
 192      }
 193  
 194      /**
 195       * Test that get_action_events_by_timesort returns events between the
 196       * provided timesort values and the last seen event can be provided to
 197       * get paginated results.
 198       */
 199      public function test_get_action_events_by_timesort_between_time_skip_even_records() {
 200          $this->resetAfterTest(true);
 201          $this->setAdminuser();
 202  
 203          $user = $this->getDataGenerator()->create_user();
 204          // The factory will return every event that is divisible by 2.
 205          $factory = new action_event_test_factory(function($actionevent) {
 206              static $count = 0;
 207              $count++;
 208              return ($count % 2) ? true : false;
 209          });
 210          $strategy = new raw_event_retrieval_strategy();
 211          $vault = new event_vault($factory, $strategy);
 212  
 213          for ($i = 1; $i < 41; $i++) {
 214              create_event([
 215                  'name' => sprintf('Event %d', $i),
 216                  'eventtype' => 'user',
 217                  'userid' => $user->id,
 218                  'timesort' => $i,
 219                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 220                  'courseid' => 1
 221              ]);
 222          }
 223  
 224          $events = $vault->get_action_events_by_timesort($user, 3, 35, null, 5);
 225  
 226          $this->assertCount(5, $events);
 227          $this->assertEquals('Event 3', $events[0]->get_name());
 228          $this->assertEquals('Event 5', $events[1]->get_name());
 229          $this->assertEquals('Event 7', $events[2]->get_name());
 230          $this->assertEquals('Event 9', $events[3]->get_name());
 231          $this->assertEquals('Event 11', $events[4]->get_name());
 232  
 233          $afterevent = $events[4];
 234          $events = $vault->get_action_events_by_timesort($user, 3, 35, $afterevent, 5);
 235  
 236          $this->assertCount(5, $events);
 237          $this->assertEquals('Event 13', $events[0]->get_name());
 238          $this->assertEquals('Event 15', $events[1]->get_name());
 239          $this->assertEquals('Event 17', $events[2]->get_name());
 240          $this->assertEquals('Event 19', $events[3]->get_name());
 241          $this->assertEquals('Event 21', $events[4]->get_name());
 242      }
 243  
 244      /**
 245       * Test that get_action_events_by_timesort returns events between the
 246       * provided timesort values. The database will continue to be read until the
 247       * number of events requested has been satisfied. In this case the first
 248       * five events are rejected so it should require two database requests.
 249       */
 250      public function test_get_action_events_by_timesort_between_time_skip_first_records() {
 251          $this->resetAfterTest(true);
 252          $this->setAdminuser();
 253  
 254          $user = $this->getDataGenerator()->create_user();
 255          $limit = 5;
 256          $seen = 0;
 257          // The factory will skip the first $limit events.
 258          $factory = new action_event_test_factory(function($actionevent) use (&$seen, $limit) {
 259              if ($seen < $limit) {
 260                  $seen++;
 261                  return false;
 262              } else {
 263                  return true;
 264              }
 265          });
 266          $strategy = new raw_event_retrieval_strategy();
 267          $vault = new event_vault($factory, $strategy);
 268  
 269          for ($i = 1; $i < 21; $i++) {
 270              create_event([
 271                  'name' => sprintf('Event %d', $i),
 272                  'eventtype' => 'user',
 273                  'userid' => $user->id,
 274                  'timesort' => $i,
 275                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 276                  'courseid' => 1
 277              ]);
 278          }
 279  
 280          $events = $vault->get_action_events_by_timesort($user, 1, 20, null, $limit);
 281  
 282          $this->assertCount($limit, $events);
 283          $this->assertEquals(sprintf('Event %d', $limit + 1), $events[0]->get_name());
 284          $this->assertEquals(sprintf('Event %d', $limit + 2), $events[1]->get_name());
 285          $this->assertEquals(sprintf('Event %d', $limit + 3), $events[2]->get_name());
 286          $this->assertEquals(sprintf('Event %d', $limit + 4), $events[3]->get_name());
 287          $this->assertEquals(sprintf('Event %d', $limit + 5), $events[4]->get_name());
 288      }
 289  
 290      /**
 291       * Test that get_action_events_by_timesort returns events between the
 292       * provided timesort values and after the last seen event when one is
 293       * provided. This should work even when the event ids aren't ordered the
 294       * same as the timesort order.
 295       */
 296      public function test_get_action_events_by_timesort_non_consecutive_ids() {
 297          $this->resetAfterTest(true);
 298          $this->setAdminuser();
 299  
 300          $user = $this->getDataGenerator()->create_user();
 301          $factory = new action_event_test_factory();
 302          $strategy = new raw_event_retrieval_strategy();
 303          $vault = new event_vault($factory, $strategy);
 304  
 305          /*
 306           * The events should be ordered by timesort as follows:
 307           *
 308           * 1 event 1
 309           * 2 event 1
 310           * 1 event 2
 311           * 2 event 2
 312           * 1 event 3
 313           * 2 event 3
 314           * 1 event 4
 315           * 2 event 4
 316           * 1 event 5
 317           * 2 event 5
 318           * 1 event 6
 319           * 2 event 6
 320           * 1 event 7
 321           * 2 event 7
 322           * 1 event 8
 323           * 2 event 8
 324           * 1 event 9
 325           * 2 event 9
 326           * 1 event 10
 327           * 2 event 10
 328           */
 329          $records = [];
 330          for ($i = 1; $i < 11; $i++) {
 331              $records[] = create_event([
 332                  'name' => sprintf('1 event %d', $i),
 333                  'eventtype' => 'user',
 334                  'userid' => $user->id,
 335                  'timesort' => $i,
 336                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 337                  'courseid' => 1
 338              ]);
 339          }
 340  
 341          for ($i = 1; $i < 11; $i++) {
 342              $records[] = create_event([
 343                  'name' => sprintf('2 event %d', $i),
 344                  'eventtype' => 'user',
 345                  'userid' => $user->id,
 346                  'timesort' => $i,
 347                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 348                  'courseid' => 1
 349              ]);
 350          }
 351  
 352          /*
 353           * Expected result set:
 354           *
 355           * 2 event 4
 356           * 1 event 5
 357           * 2 event 5
 358           * 1 event 6
 359           * 2 event 6
 360           * 1 event 7
 361           * 2 event 7
 362           * 1 event 8
 363           * 2 event 8
 364           */
 365          $aftereventid = $records[3]->id;
 366          $afterevent = $vault->get_event_by_id($aftereventid);
 367          // Offset results by event with name "1 event 4" which has the same timesort
 368          // value as the lower boundary of this query (3). Confirm that the given
 369          // $afterevent is used to ignore events with the same timesortfrom values.
 370          $events = $vault->get_action_events_by_timesort($user, 3, 8, $afterevent);
 371  
 372          $this->assertCount(9, $events);
 373          $this->assertEquals('2 event 4', $events[0]->get_name());
 374          $this->assertEquals('2 event 8', $events[8]->get_name());
 375  
 376          /*
 377           * Expected result set:
 378           *
 379           * 2 event 4
 380           * 1 event 5
 381           */
 382          $events = $vault->get_action_events_by_timesort($user, 3, 8, $afterevent, 2);
 383  
 384          $this->assertCount(2, $events);
 385          $this->assertEquals('2 event 4', $events[0]->get_name());
 386          $this->assertEquals('1 event 5', $events[1]->get_name());
 387  
 388          /*
 389           * Expected result set:
 390           *
 391           * 2 event 8
 392           */
 393          $aftereventid = $records[7]->id;
 394          $afterevent = $vault->get_event_by_id($aftereventid);
 395          // Offset results by event with name "1 event 8" which has the same timesort
 396          // value as the upper boundary of this query (8). Confirm that the given
 397          // $afterevent is used to ignore events with the same timesortto values.
 398          $events = $vault->get_action_events_by_timesort($user, 3, 8, $afterevent);
 399  
 400          $this->assertCount(1, $events);
 401          $this->assertEquals('2 event 8', $events[0]->get_name());
 402  
 403          /*
 404           * Expected empty result set.
 405           */
 406          $aftereventid = $records[18]->id;
 407          $afterevent = $vault->get_event_by_id($aftereventid);
 408          // Offset results by event with name "2 event 9" which has a timesort
 409          // value larger than the upper boundary of this query (9 > 8). Confirm
 410          // that the given $afterevent is used for filtering events.
 411          $events = $vault->get_action_events_by_timesort($user, 3, 8, $afterevent);
 412          $this->assertEmpty($events);
 413      }
 414  
 415      /**
 416       * There are subtle cases where the priority of an event override may be identical to another.
 417       * For example, if you duplicate a group override, but make it apply to a different group. Now
 418       * there are two overrides with exactly the same overridden dates. In this case the priority of
 419       * both is 1.
 420       *
 421       * In this situation:
 422       * - A user in group A should see only the A override
 423       * - A user in group B should see only the B override
 424       * - A user in both A and B should see both
 425       */
 426      public function test_get_action_events_by_timesort_with_identical_group_override_priorities() {
 427          $this->resetAfterTest();
 428          $this->setAdminuser();
 429  
 430          $course = $this->getDataGenerator()->create_course();
 431  
 432          // Create an assign instance.
 433          $assigngenerator = $this->getDataGenerator()->get_plugin_generator('mod_assign');
 434          $assigninstance = $assigngenerator->create_instance(['course' => $course->id]);
 435  
 436          // Create users.
 437          $users = [
 438              'Only in group A'  => $this->getDataGenerator()->create_user(),
 439              'Only in group B'  => $this->getDataGenerator()->create_user(),
 440              'In group A and B' => $this->getDataGenerator()->create_user(),
 441              'In no groups'     => $this->getDataGenerator()->create_user()
 442          ];
 443  
 444          // Enrol users.
 445          foreach ($users as $user) {
 446              $this->getDataGenerator()->enrol_user($user->id, $course->id);
 447          }
 448  
 449          // Create groups.
 450          $groupa = $this->getDataGenerator()->create_group(['courseid' => $course->id]);
 451          $groupb = $this->getDataGenerator()->create_group(['courseid' => $course->id]);
 452  
 453          // Add members to groups.
 454          // Group A.
 455          $this->getDataGenerator()->create_group_member(['groupid' => $groupa->id, 'userid' => $users['Only in group A']->id]);
 456          $this->getDataGenerator()->create_group_member(['groupid' => $groupa->id, 'userid' => $users['In group A and B']->id]);
 457  
 458          // Group B.
 459          $this->getDataGenerator()->create_group_member(['groupid' => $groupb->id, 'userid' => $users['Only in group B']->id]);
 460          $this->getDataGenerator()->create_group_member(['groupid' => $groupb->id, 'userid' => $users['In group A and B']->id]);
 461  
 462          // Events with the same module name, instance and event type.
 463          $events = [
 464              [
 465                  'name' => 'Assignment 1 due date - Group A override',
 466                  'description' => '',
 467                  'format' => 1,
 468                  'courseid' => $course->id,
 469                  'groupid' => $groupa->id,
 470                  'userid' => 2,
 471                  'modulename' => 'assign',
 472                  'instance' => $assigninstance->id,
 473                  'eventtype' => 'due',
 474                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 475                  'timestart' => 1,
 476                  'timeduration' => 0,
 477                  'visible' => 1,
 478                  'priority' => 1
 479              ],
 480              [
 481                  'name' => 'Assignment 1 due date - Group B override',
 482                  'description' => '',
 483                  'format' => 1,
 484                  'courseid' => $course->id,
 485                  'groupid' => $groupb->id,
 486                  'userid' => 2,
 487                  'modulename' => 'assign',
 488                  'instance' => $assigninstance->id,
 489                  'eventtype' => 'due',
 490                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 491                  'timestart' => 1,
 492                  'timeduration' => 0,
 493                  'visible' => 1,
 494                  'priority' => 1
 495              ],
 496              [
 497                  'name' => 'Assignment 1 due date',
 498                  'description' => '',
 499                  'format' => 1,
 500                  'courseid' => $course->id,
 501                  'groupid' => 0,
 502                  'userid' => 2,
 503                  'modulename' => 'assign',
 504                  'instance' => $assigninstance->id,
 505                  'eventtype' => 'due',
 506                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 507                  'timestart' => 1,
 508                  'timeduration' => 0,
 509                  'visible' => 1,
 510                  'priority' => null,
 511              ]
 512          ];
 513  
 514          foreach ($events as $event) {
 515              \calendar_event::create($event, false);
 516          }
 517  
 518          $factory = new action_event_test_factory();
 519          $strategy = new raw_event_retrieval_strategy();
 520          $vault = new event_vault($factory, $strategy);
 521  
 522          $usersevents = array_reduce(array_keys($users), function($carry, $description) use ($users, $vault) {
 523              // NB: This is currently needed to make get_action_events_by_timesort return the right thing.
 524              // It needs to be fixed, see MDL-58736.
 525              $this->setUser($users[$description]);
 526              return $carry + ['For user ' . lcfirst($description) => $vault->get_action_events_by_timesort($users[$description])];
 527          }, []);
 528  
 529          foreach ($usersevents as $description => $userevents) {
 530              if ($description == 'For user in group A and B') {
 531                  // User is in both A and B, so they should see the override for both
 532                  // given that the priority is the same.
 533                  $this->assertCount(2, $userevents);
 534                  continue;
 535              }
 536  
 537              // Otherwise there should be only one assign event for each user.
 538              $this->assertCount(1, $userevents);
 539          }
 540  
 541          // User in only group A should see the group A override.
 542          $this->assertEquals('Assignment 1 due date - Group A override', $usersevents['For user only in group A'][0]->get_name());
 543  
 544          // User in only group B should see the group B override.
 545          $this->assertEquals('Assignment 1 due date - Group B override', $usersevents['For user only in group B'][0]->get_name());
 546  
 547          // User in group A and B should see see both overrides since the priorities are the same.
 548          $this->assertEquals('Assignment 1 due date - Group A override', $usersevents['For user in group A and B'][0]->get_name());
 549          $this->assertEquals('Assignment 1 due date - Group B override', $usersevents['For user in group A and B'][1]->get_name());
 550  
 551          // User in no groups should see the plain assignment event.
 552          $this->assertEquals('Assignment 1 due date', $usersevents['For user in no groups'][0]->get_name());
 553      }
 554  
 555      /**
 556       * Test that if a user is suspended that events related to that course are not shown.
 557       * User 1 is suspended. User 2 is active.
 558       */
 559      public function test_get_action_events_by_timesort_with_suspended_user() {
 560          $this->resetAfterTest();
 561          $user1 = $this->getDataGenerator()->create_user();
 562          $user2 = $this->getDataGenerator()->create_user();
 563          $course = $this->getDataGenerator()->create_course();
 564          $this->setAdminuser();
 565          $lesson = $this->getDataGenerator()->create_module('lesson', [
 566                  'name' => 'Lesson 1',
 567                  'course' => $course->id,
 568                  'available' => time(),
 569                  'deadline' => (time() + (60 * 60 * 24 * 5))
 570              ]
 571          );
 572          $this->getDataGenerator()->enrol_user($user1->id, $course->id, null, 'manual', 0, 0, ENROL_USER_SUSPENDED);
 573          $this->getDataGenerator()->enrol_user($user2->id, $course->id);
 574  
 575          $factory = new action_event_test_factory();
 576          $strategy = new raw_event_retrieval_strategy();
 577          $vault = new event_vault($factory, $strategy);
 578  
 579          $user1events = $vault->get_action_events_by_timesort($user1, null, null, null, 20, true);
 580          $this->assertEmpty($user1events);
 581          $user2events = $vault->get_action_events_by_timesort($user2, null, null, null, 20, true);
 582          $this->assertCount(1, $user2events);
 583          $this->assertEquals('Lesson 1 closes', $user2events[0]->get_name());
 584      }
 585  
 586      /**
 587       * Test that get_action_events_by_course returns events after the
 588       * provided timesort value.
 589       */
 590      public function test_get_action_events_by_course_after_time() {
 591          $user = $this->getDataGenerator()->create_user();
 592          $course1 = $this->getDataGenerator()->create_course();
 593          $course2 = $this->getDataGenerator()->create_course();
 594          $factory = new action_event_test_factory();
 595          $strategy = new raw_event_retrieval_strategy();
 596          $vault = new event_vault($factory, $strategy);
 597  
 598          $this->resetAfterTest(true);
 599          $this->setAdminuser();
 600          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 601          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 602  
 603          for ($i = 1; $i < 6; $i++) {
 604              create_event([
 605                  'name' => sprintf('Event %d', $i),
 606                  'eventtype' => 'user',
 607                  'userid' => $user->id,
 608                  'timesort' => $i,
 609                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 610                  'courseid' => $course1->id,
 611              ]);
 612          }
 613  
 614          for ($i = 6; $i < 12; $i++) {
 615              create_event([
 616                  'name' => sprintf('Event %d', $i),
 617                  'eventtype' => 'user',
 618                  'userid' => $user->id,
 619                  'timesort' => $i,
 620                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 621                  'courseid' => $course2->id,
 622              ]);
 623          }
 624  
 625          $events = $vault->get_action_events_by_course($user, $course1, 3);
 626          $this->assertCount(3, $events);
 627          $this->assertEquals('Event 3', $events[0]->get_name());
 628          $this->assertEquals('Event 4', $events[1]->get_name());
 629          $this->assertEquals('Event 5', $events[2]->get_name());
 630  
 631          $events = $vault->get_action_events_by_course($user, $course1, 3, null, null, 1);
 632  
 633          $this->assertCount(1, $events);
 634          $this->assertEquals('Event 3', $events[0]->get_name());
 635  
 636          $events = $vault->get_action_events_by_course($user, $course1, 6);
 637  
 638          $this->assertCount(0, $events);
 639      }
 640  
 641      /**
 642       * Test that get_action_events_by_course returns events before the
 643       * provided timesort value.
 644       */
 645      public function test_get_action_events_by_course_before_time() {
 646          $user = $this->getDataGenerator()->create_user();
 647          $course1 = $this->getDataGenerator()->create_course();
 648          $course2 = $this->getDataGenerator()->create_course();
 649          $factory = new action_event_test_factory();
 650          $strategy = new raw_event_retrieval_strategy();
 651          $vault = new event_vault($factory, $strategy);
 652  
 653          $this->resetAfterTest(true);
 654          $this->setAdminuser();
 655          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 656          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 657  
 658          for ($i = 1; $i < 6; $i++) {
 659              create_event([
 660                  'name' => sprintf('Event %d', $i),
 661                  'eventtype' => 'user',
 662                  'userid' => $user->id,
 663                  'timesort' => $i,
 664                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 665                  'courseid' => $course1->id,
 666              ]);
 667          }
 668  
 669          for ($i = 6; $i < 12; $i++) {
 670              create_event([
 671                  'name' => sprintf('Event %d', $i),
 672                  'eventtype' => 'user',
 673                  'userid' => $user->id,
 674                  'timesort' => $i,
 675                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 676                  'courseid' => $course2->id,
 677              ]);
 678          }
 679  
 680          $events = $vault->get_action_events_by_course($user, $course1, null, 3);
 681  
 682          $this->assertCount(3, $events);
 683          $this->assertEquals('Event 1', $events[0]->get_name());
 684          $this->assertEquals('Event 2', $events[1]->get_name());
 685          $this->assertEquals('Event 3', $events[2]->get_name());
 686  
 687          $events = $vault->get_action_events_by_course($user, $course1, null, 3, null, 1);
 688  
 689          $this->assertCount(1, $events);
 690          $this->assertEquals('Event 1', $events[0]->get_name());
 691  
 692          $events = $vault->get_action_events_by_course($user, $course1, 6);
 693  
 694          $this->assertCount(0, $events);
 695      }
 696  
 697      /**
 698       * Test that get_action_events_by_course returns events between the
 699       * provided timesort values.
 700       */
 701      public function test_get_action_events_by_course_between_time() {
 702          $user = $this->getDataGenerator()->create_user();
 703          $course1 = $this->getDataGenerator()->create_course();
 704          $course2 = $this->getDataGenerator()->create_course();
 705          $factory = new action_event_test_factory();
 706          $strategy = new raw_event_retrieval_strategy();
 707          $vault = new event_vault($factory, $strategy);
 708  
 709          $this->resetAfterTest(true);
 710          $this->setAdminuser();
 711          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 712          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 713  
 714          for ($i = 1; $i < 6; $i++) {
 715              create_event([
 716                  'name' => sprintf('Event %d', $i),
 717                  'eventtype' => 'user',
 718                  'userid' => $user->id,
 719                  'timesort' => $i,
 720                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 721                  'courseid' => $course1->id,
 722              ]);
 723          }
 724  
 725          for ($i = 6; $i < 12; $i++) {
 726              create_event([
 727                  'name' => sprintf('Event %d', $i),
 728                  'eventtype' => 'user',
 729                  'userid' => $user->id,
 730                  'timesort' => $i,
 731                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 732                  'courseid' => $course2->id,
 733              ]);
 734          }
 735  
 736          $events = $vault->get_action_events_by_course($user, $course1, 2, 4);
 737  
 738          $this->assertCount(3, $events);
 739          $this->assertEquals('Event 2', $events[0]->get_name());
 740          $this->assertEquals('Event 3', $events[1]->get_name());
 741          $this->assertEquals('Event 4', $events[2]->get_name());
 742  
 743          $events = $vault->get_action_events_by_course($user, $course1, 2, 4, null, 1);
 744  
 745          $this->assertCount(1, $events);
 746          $this->assertEquals('Event 2', $events[0]->get_name());
 747      }
 748  
 749      /**
 750       * Test that get_action_events_by_course returns events between the
 751       * provided timesort values and after the last seen event when one is
 752       * provided.
 753       */
 754      public function test_get_action_events_by_course_between_time_after_event() {
 755          $user = $this->getDataGenerator()->create_user();
 756          $course1 = $this->getDataGenerator()->create_course();
 757          $course2 = $this->getDataGenerator()->create_course();
 758          $factory = new action_event_test_factory();
 759          $strategy = new raw_event_retrieval_strategy();
 760          $vault = new event_vault($factory, $strategy);
 761          $records = [];
 762  
 763          $this->resetAfterTest(true);
 764          $this->setAdminuser();
 765          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 766          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 767  
 768          for ($i = 1; $i < 21; $i++) {
 769              $records[] = create_event([
 770                  'name' => sprintf('Event %d', $i),
 771                  'eventtype' => 'user',
 772                  'userid' => $user->id,
 773                  'timesort' => $i,
 774                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 775                  'courseid' => $course1->id,
 776              ]);
 777          }
 778  
 779          for ($i = 21; $i < 41; $i++) {
 780              $records[] = create_event([
 781                  'name' => sprintf('Event %d', $i),
 782                  'eventtype' => 'user',
 783                  'userid' => $user->id,
 784                  'timesort' => $i,
 785                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 786                  'courseid' => $course2->id,
 787              ]);
 788          }
 789  
 790          $aftereventid = $records[6]->id;
 791          $afterevent = $vault->get_event_by_id($aftereventid);
 792          $events = $vault->get_action_events_by_course($user, $course1, 3, 15, $afterevent);
 793  
 794          $this->assertCount(8, $events);
 795          $this->assertEquals('Event 8', $events[0]->get_name());
 796  
 797          $events = $vault->get_action_events_by_course($user, $course1, 3, 15, $afterevent, 3);
 798  
 799          $this->assertCount(3, $events);
 800      }
 801  
 802      /**
 803       * Test that get_action_events_by_course returns events between the
 804       * provided timesort values and the last seen event can be provided to
 805       * get paginated results.
 806       */
 807      public function test_get_action_events_by_course_between_time_skip_even_records() {
 808          $user = $this->getDataGenerator()->create_user();
 809          $course1 = $this->getDataGenerator()->create_course();
 810          $course2 = $this->getDataGenerator()->create_course();
 811          // The factory will return every event that is divisible by 2.
 812          $factory = new action_event_test_factory(function($actionevent) {
 813              static $count = 0;
 814              $count++;
 815              return ($count % 2) ? true : false;
 816          });
 817          $strategy = new raw_event_retrieval_strategy();
 818          $vault = new event_vault($factory, $strategy);
 819  
 820          $this->resetAfterTest(true);
 821          $this->setAdminuser();
 822          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 823          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 824  
 825          for ($i = 1; $i < 41; $i++) {
 826              create_event([
 827                  'name' => sprintf('Event %d', $i),
 828                  'eventtype' => 'user',
 829                  'userid' => $user->id,
 830                  'timesort' => $i,
 831                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 832                  'courseid' => $course1->id,
 833              ]);
 834          }
 835  
 836          for ($i = 41; $i < 81; $i++) {
 837              create_event([
 838                  'name' => sprintf('Event %d', $i),
 839                  'eventtype' => 'user',
 840                  'userid' => $user->id,
 841                  'timesort' => $i,
 842                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 843                  'courseid' => $course2->id,
 844              ]);
 845          }
 846  
 847          $events = $vault->get_action_events_by_course($user, $course1, 3, 35, null, 5);
 848  
 849          $this->assertCount(5, $events);
 850          $this->assertEquals('Event 3', $events[0]->get_name());
 851          $this->assertEquals('Event 5', $events[1]->get_name());
 852          $this->assertEquals('Event 7', $events[2]->get_name());
 853          $this->assertEquals('Event 9', $events[3]->get_name());
 854          $this->assertEquals('Event 11', $events[4]->get_name());
 855  
 856          $afterevent = $events[4];
 857          $events = $vault->get_action_events_by_course($user, $course1, 3, 35, $afterevent, 5);
 858  
 859          $this->assertCount(5, $events);
 860          $this->assertEquals('Event 13', $events[0]->get_name());
 861          $this->assertEquals('Event 15', $events[1]->get_name());
 862          $this->assertEquals('Event 17', $events[2]->get_name());
 863          $this->assertEquals('Event 19', $events[3]->get_name());
 864          $this->assertEquals('Event 21', $events[4]->get_name());
 865      }
 866  
 867      /**
 868       * Test that get_action_events_by_course returns events between the
 869       * provided timesort values. The database will continue to be read until the
 870       * number of events requested has been satisfied. In this case the first
 871       * five events are rejected so it should require two database requests.
 872       */
 873      public function test_get_action_events_by_course_between_time_skip_first_records() {
 874          $user = $this->getDataGenerator()->create_user();
 875          $course1 = $this->getDataGenerator()->create_course();
 876          $course2 = $this->getDataGenerator()->create_course();
 877          $limit = 5;
 878          $seen = 0;
 879          // The factory will skip the first $limit events.
 880          $factory = new action_event_test_factory(function($actionevent) use (&$seen, $limit) {
 881              if ($seen < $limit) {
 882                  $seen++;
 883                  return false;
 884              } else {
 885                  return true;
 886              }
 887          });
 888          $strategy = new raw_event_retrieval_strategy();
 889          $vault = new event_vault($factory, $strategy);
 890  
 891          $this->resetAfterTest(true);
 892          $this->setAdminuser();
 893          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 894          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 895  
 896          for ($i = 1; $i < 21; $i++) {
 897              create_event([
 898                  'name' => sprintf('Event %d', $i),
 899                  'eventtype' => 'user',
 900                  'userid' => $user->id,
 901                  'timesort' => $i,
 902                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 903                  'courseid' => $course1->id,
 904              ]);
 905          }
 906  
 907          for ($i = 21; $i < 41; $i++) {
 908              create_event([
 909                  'name' => sprintf('Event %d', $i),
 910                  'eventtype' => 'user',
 911                  'userid' => $user->id,
 912                  'timesort' => $i,
 913                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 914                  'courseid' => $course2->id,
 915              ]);
 916          }
 917  
 918          $events = $vault->get_action_events_by_course($user, $course1, 1, 20, null, $limit);
 919  
 920          $this->assertCount($limit, $events);
 921          $this->assertEquals(sprintf('Event %d', $limit + 1), $events[0]->get_name());
 922          $this->assertEquals(sprintf('Event %d', $limit + 2), $events[1]->get_name());
 923          $this->assertEquals(sprintf('Event %d', $limit + 3), $events[2]->get_name());
 924          $this->assertEquals(sprintf('Event %d', $limit + 4), $events[3]->get_name());
 925          $this->assertEquals(sprintf('Event %d', $limit + 5), $events[4]->get_name());
 926      }
 927  
 928      /**
 929       * Test that get_action_events_by_course returns events between the
 930       * provided timesort values and after the last seen event when one is
 931       * provided. This should work even when the event ids aren't ordered the
 932       * same as the timesort order.
 933       */
 934      public function test_get_action_events_by_course_non_consecutive_ids() {
 935          $this->resetAfterTest(true);
 936          $this->setAdminuser();
 937  
 938          $user = $this->getDataGenerator()->create_user();
 939          $course1 = $this->getDataGenerator()->create_course();
 940          $course2 = $this->getDataGenerator()->create_course();
 941          $factory = new action_event_test_factory();
 942          $strategy = new raw_event_retrieval_strategy();
 943          $vault = new event_vault($factory, $strategy);
 944  
 945          $this->setAdminuser();
 946          $this->getDataGenerator()->enrol_user($user->id, $course1->id);
 947          $this->getDataGenerator()->enrol_user($user->id, $course2->id);
 948  
 949          /*
 950           * The events should be ordered by timesort as follows:
 951           *
 952           * 1 event 1
 953           * 2 event 1
 954           * 1 event 2
 955           * 2 event 2
 956           * 1 event 3
 957           * 2 event 3
 958           * 1 event 4
 959           * 2 event 4
 960           * 1 event 5
 961           * 2 event 5
 962           * 1 event 6
 963           * 2 event 6
 964           * 1 event 7
 965           * 2 event 7
 966           * 1 event 8
 967           * 2 event 8
 968           * 1 event 9
 969           * 2 event 9
 970           * 1 event 10
 971           * 2 event 10
 972           */
 973          $records = [];
 974          for ($i = 1; $i < 11; $i++) {
 975              $records[] = create_event([
 976                  'name' => sprintf('1 event %d', $i),
 977                  'eventtype' => 'user',
 978                  'userid' => $user->id,
 979                  'timesort' => $i,
 980                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 981                  'courseid' => $course1->id,
 982              ]);
 983          }
 984  
 985          for ($i = 1; $i < 11; $i++) {
 986              $records[] = create_event([
 987                  'name' => sprintf('2 event %d', $i),
 988                  'eventtype' => 'user',
 989                  'userid' => $user->id,
 990                  'timesort' => $i,
 991                  'type' => CALENDAR_EVENT_TYPE_ACTION,
 992                  'courseid' => $course1->id,
 993              ]);
 994          }
 995  
 996          // Create events for the other course.
 997          for ($i = 1; $i < 11; $i++) {
 998              $records[] = create_event([
 999                  'name' => sprintf('3 event %d', $i),
1000                  'eventtype' => 'user',
1001                  'userid' => $user->id,
1002                  'timesort' => $i,
1003                  'type' => CALENDAR_EVENT_TYPE_ACTION,
1004                  'courseid' => $course2->id,
1005              ]);
1006          }
1007  
1008          /*
1009           * Expected result set:
1010           *
1011           * 2 event 4
1012           * 1 event 5
1013           * 2 event 5
1014           * 1 event 6
1015           * 2 event 6
1016           * 1 event 7
1017           * 2 event 7
1018           * 1 event 8
1019           * 2 event 8
1020           */
1021          $aftereventid = $records[3]->id;
1022          $afterevent = $vault->get_event_by_id($aftereventid);
1023          // Offset results by event with name "1 event 4" which has the same timesort
1024          // value as the lower boundary of this query (3). Confirm that the given
1025          // $afterevent is used to ignore events with the same timesortfrom values.
1026          $events = $vault->get_action_events_by_course($user, $course1, 3, 8, $afterevent);
1027  
1028          $this->assertCount(9, $events);
1029          $this->assertEquals('2 event 4', $events[0]->get_name());
1030          $this->assertEquals('2 event 8', $events[8]->get_name());
1031  
1032          /*
1033           * Expected result set:
1034           *
1035           * 2 event 4
1036           * 1 event 5
1037           */
1038          $events = $vault->get_action_events_by_course($user, $course1, 3, 8, $afterevent, 2);
1039  
1040          $this->assertCount(2, $events);
1041          $this->assertEquals('2 event 4', $events[0]->get_name());
1042          $this->assertEquals('1 event 5', $events[1]->get_name());
1043  
1044          /*
1045           * Expected result set:
1046           *
1047           * 2 event 8
1048           */
1049          $aftereventid = $records[7]->id;
1050          $afterevent = $vault->get_event_by_id($aftereventid);
1051          // Offset results by event with name "1 event 8" which has the same timesort
1052          // value as the upper boundary of this query (8). Confirm that the given
1053          // $afterevent is used to ignore events with the same timesortto values.
1054          $events = $vault->get_action_events_by_course($user, $course1, 3, 8, $afterevent);
1055  
1056          $this->assertCount(1, $events);
1057          $this->assertEquals('2 event 8', $events[0]->get_name());
1058  
1059          /*
1060           * Expected empty result set.
1061           */
1062          $aftereventid = $records[18]->id;
1063          $afterevent = $vault->get_event_by_id($aftereventid);
1064          // Offset results by event with name "2 event 9" which has a timesort
1065          // value larger than the upper boundary of this query (9 > 8). Confirm
1066          // that the given $afterevent is used for filtering events.
1067          $events = $vault->get_action_events_by_course($user, $course1, 3, 8, $afterevent);
1068  
1069          $this->assertEmpty($events);
1070      }
1071  
1072      /**
1073       * There are subtle cases where the priority of an event override may be identical to another.
1074       * For example, if you duplicate a group override, but make it apply to a different group. Now
1075       * there are two overrides with exactly the same overridden dates. In this case the priority of
1076       * both is 1.
1077       *
1078       * In this situation:
1079       * - A user in group A should see only the A override
1080       * - A user in group B should see only the B override
1081       * - A user in both A and B should see both
1082       */
1083      public function test_get_action_events_by_course_with_identical_group_override_priorities() {
1084          $this->resetAfterTest();
1085          $this->setAdminuser();
1086  
1087          $course = $this->getDataGenerator()->create_course();
1088  
1089          // Create an assign instance.
1090          $assigngenerator = $this->getDataGenerator()->get_plugin_generator('mod_assign');
1091          $assigninstance = $assigngenerator->create_instance(['course' => $course->id]);
1092  
1093          // Create users.
1094          $users = [
1095              'Only in group A'  => $this->getDataGenerator()->create_user(),
1096              'Only in group B'  => $this->getDataGenerator()->create_user(),
1097              'In group A and B' => $this->getDataGenerator()->create_user(),
1098              'In no groups'     => $this->getDataGenerator()->create_user()
1099          ];
1100  
1101          // Enrol users.
1102          foreach ($users as $user) {
1103              $this->getDataGenerator()->enrol_user($user->id, $course->id);
1104          }
1105  
1106          // Create groups.
1107          $groupa = $this->getDataGenerator()->create_group(['courseid' => $course->id]);
1108          $groupb = $this->getDataGenerator()->create_group(['courseid' => $course->id]);
1109  
1110          // Add members to groups.
1111          // Group A.
1112          $this->getDataGenerator()->create_group_member(['groupid' => $groupa->id, 'userid' => $users['Only in group A']->id]);
1113          $this->getDataGenerator()->create_group_member(['groupid' => $groupa->id, 'userid' => $users['In group A and B']->id]);
1114  
1115          // Group B.
1116          $this->getDataGenerator()->create_group_member(['groupid' => $groupb->id, 'userid' => $users['Only in group B']->id]);
1117          $this->getDataGenerator()->create_group_member(['groupid' => $groupb->id, 'userid' => $users['In group A and B']->id]);
1118  
1119          // Events with the same module name, instance and event type.
1120          $events = [
1121              [
1122                  'name' => 'Assignment 1 due date - Group A override',
1123                  'description' => '',
1124                  'format' => 1,
1125                  'courseid' => $course->id,
1126                  'groupid' => $groupa->id,
1127                  'userid' => 2,
1128                  'modulename' => 'assign',
1129                  'instance' => $assigninstance->id,
1130                  'eventtype' => 'due',
1131                  'type' => CALENDAR_EVENT_TYPE_ACTION,
1132                  'timestart' => 1,
1133                  'timeduration' => 0,
1134                  'visible' => 1,
1135                  'priority' => 1
1136              ],
1137              [
1138                  'name' => 'Assignment 1 due date - Group B override',
1139                  'description' => '',
1140                  'format' => 1,
1141                  'courseid' => $course->id,
1142                  'groupid' => $groupb->id,
1143                  'userid' => 2,
1144                  'modulename' => 'assign',
1145                  'instance' => $assigninstance->id,
1146                  'eventtype' => 'due',
1147                  'type' => CALENDAR_EVENT_TYPE_ACTION,
1148                  'timestart' => 1,
1149                  'timeduration' => 0,
1150                  'visible' => 1,
1151                  'priority' => 1
1152              ],
1153              [
1154                  'name' => 'Assignment 1 due date',
1155                  'description' => '',
1156                  'format' => 1,
1157                  'courseid' => $course->id,
1158                  'groupid' => 0,
1159                  'userid' => 2,
1160                  'modulename' => 'assign',
1161                  'instance' => $assigninstance->id,
1162                  'eventtype' => 'due',
1163                  'type' => CALENDAR_EVENT_TYPE_ACTION,
1164                  'timestart' => 1,
1165                  'timeduration' => 0,
1166                  'visible' => 1,
1167                  'priority' => null,
1168              ]
1169          ];
1170  
1171          foreach ($events as $event) {
1172              \calendar_event::create($event, false);
1173          }
1174  
1175          $factory = new action_event_test_factory();
1176          $strategy = new raw_event_retrieval_strategy();
1177          $vault = new event_vault($factory, $strategy);
1178  
1179          $usersevents = array_reduce(array_keys($users), function($carry, $description) use ($users, $course, $vault) {
1180              // NB: This is currently needed to make get_action_events_by_timesort return the right thing.
1181              // It needs to be fixed, see MDL-58736.
1182              $this->setUser($users[$description]);
1183              return $carry + [
1184                  'For user ' . lcfirst($description) => $vault->get_action_events_by_course($users[$description], $course)
1185              ];
1186          }, []);
1187  
1188          foreach ($usersevents as $description => $userevents) {
1189              if ($description == 'For user in group A and B') {
1190                  // User is in both A and B, so they should see the override for both
1191                  // given that the priority is the same.
1192                  $this->assertCount(2, $userevents);
1193                  continue;
1194              }
1195  
1196              // Otherwise there should be only one assign event for each user.
1197              $this->assertCount(1, $userevents);
1198          }
1199  
1200          // User in only group A should see the group A override.
1201          $this->assertEquals('Assignment 1 due date - Group A override', $usersevents['For user only in group A'][0]->get_name());
1202  
1203          // User in only group B should see the group B override.
1204          $this->assertEquals('Assignment 1 due date - Group B override', $usersevents['For user only in group B'][0]->get_name());
1205  
1206          // User in group A and B should see see both overrides since the priorities are the same.
1207          $this->assertEquals('Assignment 1 due date - Group A override', $usersevents['For user in group A and B'][0]->get_name());
1208          $this->assertEquals('Assignment 1 due date - Group B override', $usersevents['For user in group A and B'][1]->get_name());
1209  
1210          // User in no groups should see the plain assignment event.
1211          $this->assertEquals('Assignment 1 due date', $usersevents['For user in no groups'][0]->get_name());
1212      }
1213  }