diff options
author | Fyorl <gaelreth@gmail.com> | 2012-06-30 10:41:54 +0100 |
---|---|---|
committer | Fyorl <gaelreth@gmail.com> | 2012-07-09 14:08:18 +0100 |
commit | 82ca2c8b08ca8c21b358ee8c862c639a5944977d (patch) | |
tree | 0cdf96f5b728ff74ee5424d6178a63d8deb311be /tests/functional | |
parent | 1233544c4a280e4392af448df654b5263e8c35db (diff) | |
download | forums-82ca2c8b08ca8c21b358ee8c862c639a5944977d.tar forums-82ca2c8b08ca8c21b358ee8c862c639a5944977d.tar.gz forums-82ca2c8b08ca8c21b358ee8c862c639a5944977d.tar.bz2 forums-82ca2c8b08ca8c21b358ee8c862c639a5944977d.tar.xz forums-82ca2c8b08ca8c21b358ee8c862c639a5944977d.zip |
[ticket/10941] Minor adjustments as per PR comments
Added some comments clarifying globals and lowercased fixture filenames
PHPBB3-10941
Diffstat (limited to 'tests/functional')
-rw-r--r-- | tests/functional/fileupload_test.php | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/tests/functional/fileupload_test.php b/tests/functional/fileupload_test.php index c06f7d94d7..bbcc0c37c9 100644 --- a/tests/functional/fileupload_test.php +++ b/tests/functional/fileupload_test.php @@ -51,8 +51,14 @@ class phpbb_functional_fileupload_test extends phpbb_functional_test_case public function test_remote_upload() { + // Note: we cannot check for the actual value of the error messages + // since they are passed through the translator which will result in + // blank strings within this test framework. + // Only doing this within the functional framework because we need a // URL + + // Global $config required by unique_id global $config; if (!is_array($config)) @@ -83,5 +89,7 @@ class phpbb_functional_fileupload_test extends phpbb_functional_test_case $file = $upload->remote_upload($this->root_url . 'styles/prosilver/theme/images/forum_read.gif'); $this->assertEquals(0, sizeof($file->error)); $this->assertTrue(file_exists($file->filename)); + + $config = array(); } } |