X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/58fa7679bccafd00f9a50bcd4a87e96876331b03..refs/pull/3593/head:/tests/Unit/ConfigTest.php diff --git a/tests/Unit/ConfigTest.php b/tests/Unit/ConfigTest.php index 0833ffbd8..3d15005af 100644 --- a/tests/Unit/ConfigTest.php +++ b/tests/Unit/ConfigTest.php @@ -1,4 +1,6 @@ -runWithEnv('STORAGE_TYPE', 'local_secure', function() { + $this->runWithEnv('STORAGE_TYPE', 'local_secure', function () { $this->checkEnvConfigResult('STORAGE_IMAGE_TYPE', 's3', 'filesystems.images', 's3'); $this->checkEnvConfigResult('STORAGE_IMAGE_TYPE', null, 'filesystems.images', 'local_secure'); }); @@ -23,7 +22,7 @@ class ConfigTest extends TestCase public function test_filesystem_attachments_falls_back_to_storage_type_var() { - $this->runWithEnv('STORAGE_TYPE', 'local_secure', function() { + $this->runWithEnv('STORAGE_TYPE', 'local_secure', function () { $this->checkEnvConfigResult('STORAGE_ATTACHMENT_TYPE', 's3', 'filesystems.attachments', 's3'); $this->checkEnvConfigResult('STORAGE_ATTACHMENT_TYPE', null, 'filesystems.attachments', 'local_secure'); }); @@ -46,11 +45,11 @@ class ConfigTest extends TestCase ]); $temp = tempnam(sys_get_temp_dir(), 'bs-test'); - $original = ini_set( 'error_log', $temp); + $original = ini_set('error_log', $temp); Log::channel('errorlog_plain_webserver')->info('Aww, look, a cute puppy'); - ini_set( 'error_log', $original); + ini_set('error_log', $original); $output = file_get_contents($temp); $this->assertStringContainsString('Aww, look, a cute puppy', $output); @@ -77,17 +76,37 @@ class ConfigTest extends TestCase ); } + public function test_dompdf_remote_fetching_controlled_by_allow_untrusted_server_fetching_false() + { + $this->checkEnvConfigResult('ALLOW_UNTRUSTED_SERVER_FETCHING', 'false', 'dompdf.options.enable_remote', false); + $this->checkEnvConfigResult('ALLOW_UNTRUSTED_SERVER_FETCHING', 'true', 'dompdf.options.enable_remote', true); + } + + public function test_dompdf_paper_size_options_are_limited() + { + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'cat', 'dompdf.options.default_paper_size', 'a4'); + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'letter', 'dompdf.options.default_paper_size', 'letter'); + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'a4', 'dompdf.options.default_paper_size', 'a4'); + } + + public function test_snappy_paper_size_options_are_limited() + { + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'cat', 'snappy.pdf.options.page-size', 'A4'); + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'letter', 'snappy.pdf.options.page-size', 'Letter'); + $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'a4', 'snappy.pdf.options.page-size', 'A4'); + } + /** * Set an environment variable of the given name and value * then check the given config key to see if it matches the given result. * Providing a null $envVal clears the variable. + * * @param mixed $expectedResult */ protected function checkEnvConfigResult(string $envName, ?string $envVal, string $configKey, $expectedResult) { - $this->runWithEnv($envName, $envVal, function() use ($configKey, $expectedResult) { + $this->runWithEnv($envName, $envVal, function () use ($configKey, $expectedResult) { $this->assertEquals($expectedResult, config($configKey)); }); } - -} \ No newline at end of file +}