X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/f84bf8e883e8e5bd9a24b908e2f90a2742d36d19..refs/heads/ldap_host_failover:/tests/Unit/ConfigTest.php diff --git a/tests/Unit/ConfigTest.php b/tests/Unit/ConfigTest.php index 69b737d7d..3d15005af 100644 --- a/tests/Unit/ConfigTest.php +++ b/tests/Unit/ConfigTest.php @@ -1,20 +1,20 @@ -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'); }); @@ -22,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'); }); @@ -36,20 +36,77 @@ class ConfigTest extends TestCase $this->checkEnvConfigResult('APP_URL', $oldDefault, 'app.url', ''); } + public function test_errorlog_plain_webserver_channel() + { + // We can't full test this due to it being targeted for the SAPI logging handler + // so we just overwrite that component so we can capture the error log output. + config()->set([ + 'logging.channels.errorlog_plain_webserver.handler_with' => [0], + ]); + + $temp = tempnam(sys_get_temp_dir(), 'bs-test'); + $original = ini_set('error_log', $temp); + + Log::channel('errorlog_plain_webserver')->info('Aww, look, a cute puppy'); + + ini_set('error_log', $original); + + $output = file_get_contents($temp); + $this->assertStringContainsString('Aww, look, a cute puppy', $output); + $this->assertStringNotContainsString('INFO', $output); + $this->assertStringNotContainsString('info', $output); + $this->assertStringNotContainsString('testing', $output); + } + + public function test_session_cookie_uses_sub_path_from_app_url() + { + $this->checkEnvConfigResult('APP_URL', 'https://example.com', 'session.path', '/'); + $this->checkEnvConfigResult('APP_URL', 'https://a.com/b', 'session.path', '/b'); + $this->checkEnvConfigResult('APP_URL', 'https://a.com/b/d/e', 'session.path', '/b/d/e'); + $this->checkEnvConfigResult('APP_URL', '', 'session.path', '/'); + } + + public function test_saml2_idp_authn_context_string_parsed_as_space_separated_array() + { + $this->checkEnvConfigResult( + 'SAML2_IDP_AUTHNCONTEXT', + 'urn:federation:authentication:windows urn:federation:authentication:linux', + 'saml2.onelogin.security.requestedAuthnContext', + ['urn:federation:authentication:windows', 'urn:federation:authentication:linux'] + ); + } + + 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 string $envName - * @param string|null $envVal - * @param string $configKey - * @param string $expectedResult + * + * @param mixed $expectedResult */ - protected function checkEnvConfigResult(string $envName, $envVal, string $configKey, string $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 +}