L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-66163 mod_forum: Add component to unittest sql queries

Browse Source 
files from other components may exist in the files table
This commit is contained in:
Marina Glancy
2019-07-16 20:37:15 +03:00
parent 39ef515a64
commit ccc7e00cc0
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mod/forum/tests/privacy_provider_test.php
+10 -4
View File
@@ -1470,10 +1470,13 @@ class mod_forum_privacy_provider_testcase extends \core_privacy\tests\provider_t
// 5 users * 2 forums * 1 file in each forum
// Original total: 10
// One post with file removed.
$this->assertCount(0, $DB->get_records_select('files', "itemid {$postinsql}", $postinparams));
$componentsql = "component = 'mod_forum' AND ";
$this->assertCount(0, $DB->get_records_select('files',
"{$componentsql} itemid {$postinsql}", $postinparams));
// Files for the other posts should remain.
$this->assertCount(18, $DB->get_records_select('files', "filename <> '.' AND itemid {$otherpostinsql}", $otherpostinparams));
$this->assertCount(18, $DB->get_records_select('files',
"{$componentsql} filename <> '.' AND itemid {$otherpostinsql}", $otherpostinparams));
}
/**
@@ -1678,11 +1681,14 @@ class mod_forum_privacy_provider_testcase extends \core_privacy\tests\provider_t
// 5 users * 2 forums * 1 file in each forum
// Original total: 10
// One post with file removed.
$this->assertCount(0, $DB->get_records_select('files', "itemid {$postinsql}", $postinparams));
$componentsql = "component = 'mod_forum' AND ";
$this->assertCount(0, $DB->get_records_select('files',
"{$componentsql} itemid {$postinsql}", $postinparams));
// Files for the other posts should remain.
$this->assertCount(18,
$DB->get_records_select('files', "filename <> '.' AND itemid {$otherpostinsql}", $otherpostinparams));
$DB->get_records_select('files',
"{$componentsql} filename <> '.' AND itemid {$otherpostinsql}", $otherpostinparams));
}
/**