From 4d765cd6993d9b029a3fbb2d13d2772e918c17a0 Mon Sep 17 00:00:00 2001 From: Marina Glancy Date: Tue, 28 Feb 2023 13:42:50 +0000 Subject: [PATCH] MDL-77164 privacy: typehint test content writer --- admin/roles/tests/privacy/provider_test.php | 4 ++++ admin/tool/mobile/tests/privacy/provider_test.php | 1 + auth/mnet/tests/privacy/provider_test.php | 1 + blocks/html/tests/privacy/provider_test.php | 2 ++ calendar/tests/privacy/provider_test.php | 1 + enrol/cohort/tests/privacy/provider_test.php | 1 + enrol/meta/tests/privacy/provider_test.php | 1 + .../form/guide/tests/privacy/provider_test.php | 1 + grade/grading/tests/privacy/provider_test.php | 1 + .../report/grader/tests/privacy/provider_test.php | 2 ++ grade/report/user/tests/privacy/provider_test.php | 1 + group/tests/privacy/provider_test.php | 3 +++ lib/editor/atto/tests/privacy/provider_test.php | 3 +++ lib/editor/tests/privacy/provider_test.php | 14 ++++++++++---- lib/userkey/tests/privacy/provider_test.php | 9 +++++++++ mod/assign/tests/privacy/provider_test.php | 2 ++ .../accessrule/seb/tests/privacy/provider_test.php | 1 + mod/quiz/tests/privacy/provider_test.php | 2 ++ .../report/basic/tests/privacy/provider_test.php | 1 + .../interactions/tests/privacy/provider_test.php | 1 + .../objectives/tests/privacy/provider_test.php | 1 + notes/tests/privacy/provider_test.php | 1 + question/tests/privacy/provider_test.php | 3 +++ question/tests/privacy_helper.php | 1 + rating/tests/privacy/provider_test.php | 5 +++++ rss/tests/privacy/provider_test.php | 1 + tag/tests/privacy/provider_test.php | 2 ++ 27 files changed, 62 insertions(+), 4 deletions(-) diff --git a/admin/roles/tests/privacy/provider_test.php b/admin/roles/tests/privacy/provider_test.php index 2504be7f822..4846ac74b5e 100644 --- a/admin/roles/tests/privacy/provider_test.php +++ b/admin/roles/tests/privacy/provider_test.php @@ -53,6 +53,7 @@ class provider_test extends provider_testcase { $showadvanced = 1; set_user_preference('definerole_showadvanced', $showadvanced); provider::export_user_preferences($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context(\context_system::instance()); $prefs = $writer->get_user_preferences('core_role'); $this->assertEquals(transform::yesno($showadvanced), transform::yesno($prefs->definerole_showadvanced->value)); @@ -184,6 +185,7 @@ class provider_test extends provider_testcase { // Retrieve role capabilities and role assignments. provider::export_user_data($approvedcontextlist); foreach ($contextlist as $context) { + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); if ($context->contextlevel == CONTEXT_MODULE) { @@ -413,6 +415,7 @@ class provider_test extends provider_testcase { ]; // Test User is assigned role teacher to cohort. provider::export_user_role_to_cohort($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($contextuserassignover); $this->assertTrue($writer->has_any_data()); $exported = (array)$writer->get_related_data($subcontextteacher, 'cohortroles'); @@ -430,6 +433,7 @@ class provider_test extends provider_testcase { api::create_cohort_role_assignment($params); api::sync_all_cohort_roles(); provider::export_user_role_to_cohort($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($contextuser); $this->assertTrue($writer->has_any_data()); $exported = (array)$writer->get_related_data($subcontextteacher, 'cohortroles'); diff --git a/admin/tool/mobile/tests/privacy/provider_test.php b/admin/tool/mobile/tests/privacy/provider_test.php index 6d2fd7d324d..92add10ad65 100644 --- a/admin/tool/mobile/tests/privacy/provider_test.php +++ b/admin/tool/mobile/tests/privacy/provider_test.php @@ -111,6 +111,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $key = $DB->get_record('user_private_key', ['value' => $keyvalue]); // Validate exported data. $this->setUser($user); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); $this->export_context_data_for_user($user->id, $context, 'tool_mobile'); diff --git a/auth/mnet/tests/privacy/provider_test.php b/auth/mnet/tests/privacy/provider_test.php index 01fac8140b2..a41ac4de0f8 100644 --- a/auth/mnet/tests/privacy/provider_test.php +++ b/auth/mnet/tests/privacy/provider_test.php @@ -106,6 +106,7 @@ class provider_test extends provider_testcase { $usercontext = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($usercontext); $this->assertFalse($writer->has_any_data()); $approvedlist = new approved_contextlist($user, 'auth_mnet', [$usercontext->id]); diff --git a/blocks/html/tests/privacy/provider_test.php b/blocks/html/tests/privacy/provider_test.php index 1348d130a1a..e776d0ae662 100644 --- a/blocks/html/tests/privacy/provider_test.php +++ b/blocks/html/tests/privacy/provider_test.php @@ -185,6 +185,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export the data. $this->export_context_data_for_user($user->id, $context, 'block_html'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = \core_privacy\local\request\writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -233,6 +234,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export the data. $this->export_context_data_for_user($user->id, $context, 'block_html'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = \core_privacy\local\request\writer::with_context($context); $this->assertFalse($writer->has_any_data()); } diff --git a/calendar/tests/privacy/provider_test.php b/calendar/tests/privacy/provider_test.php index 8645f0d5966..0d4b57a908a 100644 --- a/calendar/tests/privacy/provider_test.php +++ b/calendar/tests/privacy/provider_test.php @@ -180,6 +180,7 @@ class provider_test extends provider_testcase { provider::export_user_data($approvedcontextlist); foreach ($contextlist as $context) { + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); diff --git a/enrol/cohort/tests/privacy/provider_test.php b/enrol/cohort/tests/privacy/provider_test.php index a31f2a24984..d6296134042 100644 --- a/enrol/cohort/tests/privacy/provider_test.php +++ b/enrol/cohort/tests/privacy/provider_test.php @@ -114,6 +114,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $approvedcontextlist = new approved_contextlist($user1, 'enrol_cohort', $contextlist->get_contextids()); provider::export_user_data($approvedcontextlist); foreach ($contextlist as $context) { + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $data = $writer->get_data([ get_string('pluginname', 'enrol_cohort'), diff --git a/enrol/meta/tests/privacy/provider_test.php b/enrol/meta/tests/privacy/provider_test.php index 1a278df8a38..c7c954a8a15 100644 --- a/enrol/meta/tests/privacy/provider_test.php +++ b/enrol/meta/tests/privacy/provider_test.php @@ -103,6 +103,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $approvedcontextlist = new approved_contextlist($user1, 'enrol_meta', $contextlist->get_contextids()); provider::export_user_data($approvedcontextlist); foreach ($contextlist as $context) { + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $data = $writer->get_data([ get_string('pluginname', 'enrol_meta'), diff --git a/grade/grading/form/guide/tests/privacy/provider_test.php b/grade/grading/form/guide/tests/privacy/provider_test.php index 7348ba18adb..c114b8537a5 100644 --- a/grade/grading/form/guide/tests/privacy/provider_test.php +++ b/grade/grading/form/guide/tests/privacy/provider_test.php @@ -67,6 +67,7 @@ class provider_test extends provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('gradingform_guide'); diff --git a/grade/grading/tests/privacy/provider_test.php b/grade/grading/tests/privacy/provider_test.php index 40823d75980..c24065fce62 100644 --- a/grade/grading/tests/privacy/provider_test.php +++ b/grade/grading/tests/privacy/provider_test.php @@ -110,6 +110,7 @@ class provider_test extends provider_testcase { $this->setUser($user); $context = \context_system::instance(); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); $this->export_context_data_for_user($user->id, $context, 'core_grading'); diff --git a/grade/report/grader/tests/privacy/provider_test.php b/grade/report/grader/tests/privacy/provider_test.php index ca68563f1d6..1557d1c8e79 100644 --- a/grade/report/grader/tests/privacy/provider_test.php +++ b/grade/report/grader/tests/privacy/provider_test.php @@ -77,6 +77,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data for our test user. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('gradereport_grader'); @@ -99,6 +100,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('gradereport_grader'); diff --git a/grade/report/user/tests/privacy/provider_test.php b/grade/report/user/tests/privacy/provider_test.php index e8f45ed9f92..58ac0271b85 100644 --- a/grade/report/user/tests/privacy/provider_test.php +++ b/grade/report/user/tests/privacy/provider_test.php @@ -67,6 +67,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('gradereport_user'); diff --git a/group/tests/privacy/provider_test.php b/group/tests/privacy/provider_test.php index af30c2ec835..3345b1a8f41 100644 --- a/group/tests/privacy/provider_test.php +++ b/group/tests/privacy/provider_test.php @@ -90,6 +90,7 @@ class provider_test extends provider_testcase { // Retrieve groups for user1. $this->setUser($user1); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); provider::export_groups($context, ''); @@ -134,6 +135,7 @@ class provider_test extends provider_testcase { // Retrieve groups for user1. $this->setUser($user1); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); provider::export_groups($context, 'enrol_self'); @@ -792,6 +794,7 @@ class provider_test extends provider_testcase { // Export all of the data for the context. $this->export_context_data_for_user($user1->id, $context, 'core_group'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); diff --git a/lib/editor/atto/tests/privacy/provider_test.php b/lib/editor/atto/tests/privacy/provider_test.php index 258d0385f3f..1920417e1a1 100644 --- a/lib/editor/atto/tests/privacy/provider_test.php +++ b/lib/editor/atto/tests/privacy/provider_test.php @@ -118,6 +118,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export the data for the system context. // There should be two. $this->export_context_data_for_user($user->id, $systemcontext, 'editor_atto'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = \core_privacy\local\request\writer::with_context($systemcontext); $this->assertTrue($writer->has_any_data()); @@ -535,6 +536,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Should have the data. $this->export_context_data_for_user($user->id, $usercontext, 'editor_atto'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = \core_privacy\local\request\writer::with_context($usercontext); $this->assertTrue($writer->has_any_data()); @@ -587,6 +589,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Should have the data. $this->export_context_data_for_user($user->id, $otherusercontext, 'editor_atto'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = \core_privacy\local\request\writer::with_context($otherusercontext); $this->assertTrue($writer->has_any_data()); diff --git a/lib/editor/tests/privacy/provider_test.php b/lib/editor/tests/privacy/provider_test.php index 79fb31da8b9..6bacfa61ab3 100644 --- a/lib/editor/tests/privacy/provider_test.php +++ b/lib/editor/tests/privacy/provider_test.php @@ -47,7 +47,9 @@ class provider_test extends \core_privacy\tests\provider_testcase { $this->setAdminUser(); provider::export_user_preferences($USER->id); - $this->assertFalse(writer::with_context(\context_system::instance())->has_any_data()); + /** @var \core_privacy\tests\request\content_writer $writer */ + $writer = writer::with_context(\context_system::instance()); + $this->assertFalse($writer->has_any_data()); } /** @@ -67,7 +69,9 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export test users preferences. provider::export_user_preferences($user->id); - $this->assertFalse(writer::with_context(\context_system::instance())->has_any_data()); + /** @var \core_privacy\tests\request\content_writer $writer */ + $writer = writer::with_context(\context_system::instance()); + $this->assertFalse($writer->has_any_data()); } /** @@ -87,9 +91,11 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export test users preferences. provider::export_user_preferences($user->id); - $this->assertTrue(writer::with_context(\context_system::instance())->has_any_data()); + /** @var \core_privacy\tests\request\content_writer $writer */ + $writer = writer::with_context(\context_system::instance()); + $this->assertTrue($writer->has_any_data()); - $prefs = writer::with_context(\context_system::instance())->get_user_preferences('core_editor'); + $prefs = $writer->get_user_preferences('core_editor'); $this->assertNotEmpty($prefs->htmleditor); $this->assertNotEmpty($prefs->htmleditor->value); $this->assertNotEmpty($prefs->htmleditor->description); diff --git a/lib/userkey/tests/privacy/provider_test.php b/lib/userkey/tests/privacy/provider_test.php index 8eeeaf526b0..bbca41ea486 100644 --- a/lib/userkey/tests/privacy/provider_test.php +++ b/lib/userkey/tests/privacy/provider_test.php @@ -70,6 +70,7 @@ class provider_test extends provider_testcase { provider::export_userkeys($context, $subcontext, 'core_tests'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -107,6 +108,7 @@ class provider_test extends provider_testcase { // Export all keys in core_tests. provider::export_userkeys($context, $subcontext, 'core_tests'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -143,6 +145,7 @@ class provider_test extends provider_testcase { provider::export_userkeys($context, $subcontext, 'core_tests'); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -179,11 +182,13 @@ class provider_test extends provider_testcase { // Export all keys in core_tests against instance 43 - no keys. provider::export_userkeys($context, $subcontext, 'core_tests', 43); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); // Export all keys in core_tests against instance 42. provider::export_userkeys($context, $subcontext, 'core_tests', 42); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $exported = $writer->get_related_data($subcontext, 'userkeys'); @@ -224,11 +229,13 @@ class provider_test extends provider_testcase { // Export all keys in core_tests against instance 43 - no keys. provider::export_userkeys($context, $subcontext, 'core_tests', 43); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); // Export all keys in core_tests against instance 42. provider::export_userkeys($context, $subcontext, 'core_tests', 42); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $exported = $writer->get_related_data($subcontext, 'userkeys'); @@ -278,11 +285,13 @@ class provider_test extends provider_testcase { // Export all keys in core_tests against instance 43 - no keys. provider::export_userkeys($context, $subcontext, 'core_tests', 43); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); // Export all keys in core_tests against instance 42. provider::export_userkeys($context, $subcontext, 'core_tests', 42); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $exported = $writer->get_related_data($subcontext, 'userkeys'); diff --git a/mod/assign/tests/privacy/provider_test.php b/mod/assign/tests/privacy/provider_test.php index a3efabce75c..d773bdfff12 100644 --- a/mod/assign/tests/privacy/provider_test.php +++ b/mod/assign/tests/privacy/provider_test.php @@ -299,6 +299,7 @@ class provider_test extends provider_testcase { // Give the submission a grade. $assign->save_grade($user->id, $data); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); @@ -411,6 +412,7 @@ class provider_test extends provider_testcase { $flagdata->extensionduedate = $duedate; $assign->update_user_flags($flagdata); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); diff --git a/mod/quiz/accessrule/seb/tests/privacy/provider_test.php b/mod/quiz/accessrule/seb/tests/privacy/provider_test.php index 3ece5a279a0..836326f5e3c 100644 --- a/mod/quiz/accessrule/seb/tests/privacy/provider_test.php +++ b/mod/quiz/accessrule/seb/tests/privacy/provider_test.php @@ -119,6 +119,7 @@ class provider_test extends provider_testcase { ); writer::reset(); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); provider::export_user_data($approvedcontextlist); diff --git a/mod/quiz/tests/privacy/provider_test.php b/mod/quiz/tests/privacy/provider_test.php index ffba8269df1..559ea67d732 100644 --- a/mod/quiz/tests/privacy/provider_test.php +++ b/mod/quiz/tests/privacy/provider_test.php @@ -170,6 +170,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { provider::export_user_data($approvedcontextlist); // Ensure that the quiz data was exported correctly. + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -366,6 +367,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { provider::export_user_data($approvedcontextlist); // Ensure that nothing was exported. + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data_in_any_context()); diff --git a/mod/scorm/report/basic/tests/privacy/provider_test.php b/mod/scorm/report/basic/tests/privacy/provider_test.php index da05406c1e7..8d781849570 100644 --- a/mod/scorm/report/basic/tests/privacy/provider_test.php +++ b/mod/scorm/report/basic/tests/privacy/provider_test.php @@ -68,6 +68,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('scormreport_basic'); diff --git a/mod/scorm/report/interactions/tests/privacy/provider_test.php b/mod/scorm/report/interactions/tests/privacy/provider_test.php index 6add72fe501..c87dbcb45f4 100644 --- a/mod/scorm/report/interactions/tests/privacy/provider_test.php +++ b/mod/scorm/report/interactions/tests/privacy/provider_test.php @@ -71,6 +71,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('scormreport_interactions'); diff --git a/mod/scorm/report/objectives/tests/privacy/provider_test.php b/mod/scorm/report/objectives/tests/privacy/provider_test.php index 25d4d35e48b..5f9c4ed9c0e 100644 --- a/mod/scorm/report/objectives/tests/privacy/provider_test.php +++ b/mod/scorm/report/objectives/tests/privacy/provider_test.php @@ -68,6 +68,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Validate exported data. provider::export_user_preferences($user->id); $context = \context_user::instance($user->id); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $prefs = $writer->get_user_preferences('scormreport_objectives'); diff --git a/notes/tests/privacy/provider_test.php b/notes/tests/privacy/provider_test.php index e6832d18c84..e56079bc4f3 100644 --- a/notes/tests/privacy/provider_test.php +++ b/notes/tests/privacy/provider_test.php @@ -229,6 +229,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { foreach ($contextlist as $context) { $this->assertEquals(CONTEXT_COURSE, $context->contextlevel); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); } diff --git a/question/tests/privacy/provider_test.php b/question/tests/privacy/provider_test.php index 0e6a3c883a0..78dd9360b61 100644 --- a/question/tests/privacy/provider_test.php +++ b/question/tests/privacy/provider_test.php @@ -96,6 +96,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $options = new \question_display_options(); provider::export_question_usage($testuser->id, $context, [], $quba->get_id(), $options, false); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data_in_any_context()); @@ -137,6 +138,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { // Export all questions for this attempt. $options = new \question_display_options(); provider::export_question_usage($testuser->id, $context, [], $quba->get_id(), $options, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data_in_any_context()); @@ -161,6 +163,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $options->marks = \question_display_options::HIDDEN; provider::export_question_usage($testuser->id, $context, [], $quba->get_id(), $options, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data_in_any_context()); diff --git a/question/tests/privacy_helper.php b/question/tests/privacy_helper.php index 65f3a46bfb4..521448549e4 100644 --- a/question/tests/privacy_helper.php +++ b/question/tests/privacy_helper.php @@ -93,6 +93,7 @@ trait core_question_privacy_helper { [get_string('questions', 'core_question')] ); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); foreach ($quba->get_slots() as $slotno) { diff --git a/rating/tests/privacy/provider_test.php b/rating/tests/privacy/provider_test.php index c20540ed4e0..d605c74f54d 100644 --- a/rating/tests/privacy/provider_test.php +++ b/rating/tests/privacy/provider_test.php @@ -283,6 +283,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $subcontext = []; provider::export_area_ratings($u1->id, $context, $subcontext, 'core_course', 'course', $course1->id, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $rating = $writer->get_related_data($subcontext, 'rating'); @@ -293,6 +294,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $subcontext = ['foo']; provider::export_area_ratings($u1->id, $context, $subcontext, 'core_course', 'course', $course2->id, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $result = $writer->get_related_data($subcontext, 'rating'); @@ -304,6 +306,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $subcontext = ['foo']; provider::export_area_ratings($u1->id, $context, $subcontext, 'core_course', 'course', $course3->id, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $result = $writer->get_related_data($subcontext, 'rating'); @@ -315,6 +318,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $subcontext = ['foo', 'files']; provider::export_area_ratings($u1->id, $context, $subcontext, 'core_course', 'files', $course3->id, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $result = $writer->get_related_data($subcontext, 'rating'); @@ -327,6 +331,7 @@ class provider_test extends \core_privacy\tests\provider_testcase { $subcontext = ['user']; provider::export_area_ratings($u3->id, $context, $subcontext, 'user', 'user', $u3->id, false); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); $result = $writer->get_related_data($subcontext, 'rating'); diff --git a/rss/tests/privacy/provider_test.php b/rss/tests/privacy/provider_test.php index 406db4ba938..c44f4125940 100644 --- a/rss/tests/privacy/provider_test.php +++ b/rss/tests/privacy/provider_test.php @@ -72,6 +72,7 @@ class provider_test extends provider_testcase { // Validate exported data. $this->setUser($user); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertFalse($writer->has_any_data()); $this->export_context_data_for_user($user->id, $context, 'core_rss'); diff --git a/tag/tests/privacy/provider_test.php b/tag/tests/privacy/provider_test.php index 03e1cbe9437..2f41e4e2489 100644 --- a/tag/tests/privacy/provider_test.php +++ b/tag/tests/privacy/provider_test.php @@ -77,6 +77,7 @@ class provider_test extends provider_testcase { // Retrieve tags only for this user. provider::export_item_tags($user->id, $context, $subcontext, 'core_course', 'course', $course->id, true); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $this->assertTrue($writer->has_any_data()); @@ -261,6 +262,7 @@ class provider_test extends provider_testcase { }, \core_tag_tag::get_by_name_bulk(\core_tag_collection::get_default(), ['Birdwatching', 'Computers']))); provider::export_user_data(new \core_privacy\local\request\approved_contextlist($user1, 'core_tag', [$context->id])); + /** @var \core_privacy\tests\request\content_writer $writer */ $writer = writer::with_context($context); $data = $writer->get_data(['Tags', $tagids[0]]);