From 5c35cb9cf8064c25a993e42cedd4438c9846e37f Mon Sep 17 00:00:00 2001 From: Marc Alexander Date: Mon, 25 Aug 2014 22:49:13 +0200 Subject: [PATCH] [ticket/358] Remove use of T_EXT_THEME_PATH in attachments side Additionally, we'll now also test that the portal doesn't show errors while using subsilver. B3P-358 --- .../portal/modules/attachments_side.html | 2 +- tests/functional/portal_no_error_test.php | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/styles/prosilver/template/portal/modules/attachments_side.html b/styles/prosilver/template/portal/modules/attachments_side.html index 31b4f9ff..ccaa082d 100644 --- a/styles/prosilver/template/portal/modules/attachments_side.html +++ b/styles/prosilver/template/portal/modules/attachments_side.html @@ -2,7 +2,7 @@ {L_FILENAME}
-  {attach_side.FILENAME}
+   {attach_side.FILENAME}
{L_FILESIZE}{L_COLON}{attach_side.FILESIZE}
{L_DOWNLOADS}{L_COLON}{attach_side.DOWNLOAD_COUNT}

diff --git a/tests/functional/portal_no_error_test.php b/tests/functional/portal_no_error_test.php index d12b08a8..cfd58479 100644 --- a/tests/functional/portal_no_error_test.php +++ b/tests/functional/portal_no_error_test.php @@ -24,4 +24,22 @@ class phpbb_functional_portal_no_error_test extends \board3\portal\tests\testfra { self::request('GET', 'app.php/portal'); } + + public function test_enable_subsilver() + { + $crawler = self::request('GET', 'adm/index.php?i=acp_styles&mode=install&sid=' . $this->sid); + $link = $crawler->selectLink('Install style')->link()->getUri(); + $crawler = self::request('GET', substr($link, strpos($link, '/adm') + 1)); + $this->assertContains('Style "subsilver2" has been installed', $crawler->text()); + } + + /** + * @dependsOn test_enable_subsilver + */ + public function test_portal_subsilver() + { + $crawler = self::request('GET', 'app.php/portal'); + $this->assertContains('subsilver', $crawler->text()); + self::request('GET', 'app.php/portal?style=2'); + } }