makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-07 20:08:53 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/10851] Add unit and functional tests for checking attachments

Browse source 
These tests will make sure that fileuploads now work as expected regarding
the check for disallowed content.

PHPBB3-10851
This commit is contained in:
Marc Alexander 2014-04-03 18:26:39 +02:00
parent e4e3df0b37
commit 91c3ad07ec
3 changed files with 35 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
tests/functional/fileupload_form_test.php
View file

@ -76,6 +76,39 @@ class phpbb_functional_fileupload_form_test extends phpbb_functional_test_case
$this->assertEquals($this->lang('DISALLOWED_EXTENSION', 'bif'), $crawler->filter('p.error')->text());
}
public function test_disallowed_content()
{
$this->login();
$crawler = $this->upload_file('disallowed.jpg', 'image/jpeg');
$this->assertEquals($this->lang('DISALLOWED_CONTENT'), $crawler->filter('p.error')->text());
}
public function test_disallowed_content_no_check()
{
$this->login();
$this->admin_login();
$this->add_lang('ucp');
$crawler = self::request('GET', 'adm/index.php?sid=' . $this->sid . '&i=acp_attachments&mode=attach');
$form = $crawler->selectButton('Submit')->form();
$values = $form->getValues();
$values["config[check_attachment_content]"] = 0;
$form->setValues($values);
$crawler = self::submit($form);
// Logout and back in for correct URL
$this->logout();
$this->login();
$crawler = $this->upload_file('disallowed.jpg', 'image/jpeg');
// Hitting the UNABLE_GET_IMAGE_SIZE error means we passed the
// DISALLOWED_CONTENT check
$this->assertEquals($this->lang('UNABLE_GET_IMAGE_SIZE'), $crawler->filter('p.error')->text());
}
public function test_too_large()
{
$this->create_user('fileupload');

BIN
tests/functional/fixtures/files/disallowed.jpg Normal file
View file

Binary file not shown.

2
tests/upload/filespec_test.php
View file

@ -143,6 +143,8 @@ class phpbb_filespec_test extends phpbb_test_case
$disallowed_content = explode('|', $this->config['mime_triggers']);
$filespec = $this->get_filespec(array('tmp_name' => $this->path . $filename));
$this->assertEquals($expected, $filespec->check_content($disallowed_content));
// All files should pass if $disallowed_content is empty
$this->assertEquals(true, $filespec->check_content(array()));
}
public function clean_filename_variables()