Commit 164459f9 by lubosdz

Fixed test break and JSON MIME type. Fixes issue #2978.

parent a97a35ba
......@@ -71,6 +71,8 @@ return [
'jpeg' => 'image/jpeg',
'jpg' => 'image/jpeg',
'js' => 'application/x-javascript',
'json' => 'application/json',
'jsonp' => 'text/javascript',
'kar' => 'audio/midi',
'latex' => 'application/x-latex',
'lha' => 'application/octet-stream',
......
......@@ -349,9 +349,13 @@ class FileHelperTest extends TestCase
file_put_contents($file, 'some text');
$this->assertEquals('text/plain', FileHelper::getMimeType($file));
// see http://stackoverflow.com/questions/477816/what-is-the-correct-json-content-type
// JSON/JSONP should not use text/plain - see http://jibbering.com/blog/?p=514
// with "fileinfo" extension enabled, returned MIME is not quite correctly "text/plain"
// without "fileinfo" it falls back to getMimeTypeByExtension() and returns application/json
$file = $this->testFilePath . DIRECTORY_SEPARATOR . 'mime_type_test.json';
file_put_contents($file, '{"a": "b"}');
$this->assertEquals('text/plain', FileHelper::getMimeType($file));
$this->assertTrue(in_array(FileHelper::getMimeType($file), array('application/json', 'text/plain')));
}
public function testNormalizePath()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment