Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix subfolder configuration in unit tests

Browse source
This commit is contained in:
ArthurHoaro 2020-09-03 14:52:34 +02:00
parent 650a5f09cb
commit b93cfeba7b
8 changed files with 55 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
application/front/controller/visitor/DailyController.php
View file

@ -132,7 +132,7 @@ class DailyController extends ShaarliVisitorController
'date' => $dayDatetime, 'date' => $dayDatetime,
'date_rss' => $dayDatetime->format(DateTime::RSS), 'date_rss' => $dayDatetime->format(DateTime::RSS),
'date_human' => format_date($dayDatetime, false, true), 'date_human' => format_date($dayDatetime, false, true),
'absolute_url' => $indexUrl . '/daily?day=' . $day, 'absolute_url' => $indexUrl . 'daily?day=' . $day,
'links' => [], 'links' => [],
]; ];

13
tests/feed/FeedBuilderTest.php
View file

@ -3,6 +3,7 @@
namespace Shaarli\Feed; namespace Shaarli\Feed;
use DateTime; use DateTime;
use PHPUnit\Framework\TestCase;
use ReferenceLinkDB; use ReferenceLinkDB;
use Shaarli\Bookmark\Bookmark; use Shaarli\Bookmark\Bookmark;
use Shaarli\Bookmark\BookmarkFileService; use Shaarli\Bookmark\BookmarkFileService;
@ -16,7 +17,7 @@ use Shaarli\History;
* *
* Unit tests for FeedBuilder. * Unit tests for FeedBuilder.
*/ */
class FeedBuilderTest extends \PHPUnit\Framework\TestCase class FeedBuilderTest extends TestCase
{ {
/** /**
* @var string locale Basque (Spain). * @var string locale Basque (Spain).
@ -44,7 +45,7 @@ class FeedBuilderTest extends \PHPUnit\Framework\TestCase
/** /**
* Called before every test method. * Called before every test method.
*/ */
public static function setUpBeforeClass() public static function setUpBeforeClass(): void
{ {
$conf = new ConfigManager('tests/utils/config/configJson'); $conf = new ConfigManager('tests/utils/config/configJson');
$conf->set('resource.datastore', self::$testDatastore); $conf->set('resource.datastore', self::$testDatastore);
@ -60,7 +61,7 @@ class FeedBuilderTest extends \PHPUnit\Framework\TestCase
'SERVER_NAME' => 'host.tld', 'SERVER_NAME' => 'host.tld',
'SERVER_PORT' => '80', 'SERVER_PORT' => '80',
'SCRIPT_NAME' => '/index.php', 'SCRIPT_NAME' => '/index.php',
'REQUEST_URI' => '/index.php?do=feed', 'REQUEST_URI' => '/feed/atom',
); );
} }
@ -81,7 +82,7 @@ class FeedBuilderTest extends \PHPUnit\Framework\TestCase
$this->assertEquals(self::$RSS_LANGUAGE, $data['language']); $this->assertEquals(self::$RSS_LANGUAGE, $data['language']);
$this->assertRegExp('/Wed, 03 Aug 2016 09:30:33 \+\d{4}/', $data['last_update']); $this->assertRegExp('/Wed, 03 Aug 2016 09:30:33 \+\d{4}/', $data['last_update']);
$this->assertEquals(true, $data['show_dates']); $this->assertEquals(true, $data['show_dates']);
$this->assertEquals('http://host.tld/index.php?do=feed', $data['self_link']); $this->assertEquals('http://host.tld/feed/atom', $data['self_link']);
$this->assertEquals('http://host.tld/', $data['index_url']); $this->assertEquals('http://host.tld/', $data['index_url']);
$this->assertFalse($data['usepermalinks']); $this->assertFalse($data['usepermalinks']);
$this->assertEquals(ReferenceLinkDB::$NB_LINKS_TOTAL, count($data['links'])); $this->assertEquals(ReferenceLinkDB::$NB_LINKS_TOTAL, count($data['links']));
@ -253,7 +254,7 @@ class FeedBuilderTest extends \PHPUnit\Framework\TestCase
'SERVER_NAME' => 'host.tld', 'SERVER_NAME' => 'host.tld',
'SERVER_PORT' => '8080', 'SERVER_PORT' => '8080',
'SCRIPT_NAME' => '/~user/shaarli/index.php', 'SCRIPT_NAME' => '/~user/shaarli/index.php',
'REQUEST_URI' => '/~user/shaarli/index.php?do=feed', 'REQUEST_URI' => '/~user/shaarli/feed/atom',
); );
$feedBuilder = new FeedBuilder( $feedBuilder = new FeedBuilder(
self::$bookmarkService, self::$bookmarkService,
@ -265,7 +266,7 @@ class FeedBuilderTest extends \PHPUnit\Framework\TestCase
$data = $feedBuilder->buildData(FeedBuilder::$FEED_ATOM, null); $data = $feedBuilder->buildData(FeedBuilder::$FEED_ATOM, null);
$this->assertEquals( $this->assertEquals(
'http://host.tld:8080/~user/shaarli/index.php?do=feed', 'http://host.tld:8080/~user/shaarli/feed/atom',
$data['self_link'] $data['self_link']
); );

2
tests/front/controller/admin/ExportControllerTest.php
View file

@ -84,7 +84,7 @@ class ExportControllerTest extends TestCase
static::assertInstanceOf(BookmarkRawFormatter::class, $formatter); static::assertInstanceOf(BookmarkRawFormatter::class, $formatter);
static::assertSame($parameters['selection'], $selection); static::assertSame($parameters['selection'], $selection);
static::assertTrue($prependNoteUrl); static::assertTrue($prependNoteUrl);
static::assertSame('http://shaarli', $indexUrl); static::assertSame('http://shaarli/subfolder/', $indexUrl);
return $bookmarks; return $bookmarks;
} }

6
tests/front/controller/admin/ToolsControllerTest.php
View file

@ -8,7 +8,7 @@ use PHPUnit\Framework\TestCase;
use Slim\Http\Request; use Slim\Http\Request;
use Slim\Http\Response; use Slim\Http\Response;
class ToolsControllerTestControllerTest extends TestCase class ToolsControllerTest extends TestCase
{ {
use FrontAdminControllerMockHelper; use FrontAdminControllerMockHelper;
@ -41,7 +41,7 @@ class ToolsControllerTestControllerTest extends TestCase
static::assertSame(200, $result->getStatusCode()); static::assertSame(200, $result->getStatusCode());
static::assertSame('tools', (string) $result->getBody()); static::assertSame('tools', (string) $result->getBody());
static::assertSame('https://shaarli', $assignedVariables['pageabsaddr']); static::assertSame('https://shaarli/', $assignedVariables['pageabsaddr']);
static::assertTrue($assignedVariables['sslenabled']); static::assertTrue($assignedVariables['sslenabled']);
} }
@ -63,7 +63,7 @@ class ToolsControllerTestControllerTest extends TestCase
static::assertSame(200, $result->getStatusCode()); static::assertSame(200, $result->getStatusCode());
static::assertSame('tools', (string) $result->getBody()); static::assertSame('tools', (string) $result->getBody());
static::assertSame('http://shaarli', $assignedVariables['pageabsaddr']); static::assertSame('http://shaarli/', $assignedVariables['pageabsaddr']);
static::assertFalse($assignedVariables['sslenabled']); static::assertFalse($assignedVariables['sslenabled']);
} }
} }

12
tests/front/controller/visitor/DailyControllerTest.php
View file

@ -392,8 +392,8 @@ class DailyControllerTest extends TestCase
static::assertStringContainsString('application/rss', $result->getHeader('Content-Type')[0]); static::assertStringContainsString('application/rss', $result->getHeader('Content-Type')[0]);
static::assertSame('dailyrss', (string) $result->getBody()); static::assertSame('dailyrss', (string) $result->getBody());
static::assertSame('Shaarli', $assignedVariables['title']); static::assertSame('Shaarli', $assignedVariables['title']);
static::assertSame('http://shaarli', $assignedVariables['index_url']); static::assertSame('http://shaarli/subfolder/', $assignedVariables['index_url']);
static::assertSame('http://shaarli/daily-rss', $assignedVariables['page_url']); static::assertSame('http://shaarli/subfolder/daily-rss', $assignedVariables['page_url']);
static::assertFalse($assignedVariables['hide_timestamps']); static::assertFalse($assignedVariables['hide_timestamps']);
static::assertCount(2, $assignedVariables['days']); static::assertCount(2, $assignedVariables['days']);
@ -402,7 +402,7 @@ class DailyControllerTest extends TestCase
static::assertEquals($dates[0], $day['date']); static::assertEquals($dates[0], $day['date']);
static::assertSame($dates[0]->format(\DateTime::RSS), $day['date_rss']); static::assertSame($dates[0]->format(\DateTime::RSS), $day['date_rss']);
static::assertSame(format_date($dates[0], false), $day['date_human']); static::assertSame(format_date($dates[0], false), $day['date_human']);
static::assertSame('http://shaarli/daily?day='. $dates[0]->format('Ymd'), $day['absolute_url']); static::assertSame('http://shaarli/subfolder/daily?day='. $dates[0]->format('Ymd'), $day['absolute_url']);
static::assertCount(1, $day['links']); static::assertCount(1, $day['links']);
static::assertSame(1, $day['links'][0]['id']); static::assertSame(1, $day['links'][0]['id']);
static::assertSame('http://domain.tld/1', $day['links'][0]['url']); static::assertSame('http://domain.tld/1', $day['links'][0]['url']);
@ -413,7 +413,7 @@ class DailyControllerTest extends TestCase
static::assertEquals($dates[1], $day['date']); static::assertEquals($dates[1], $day['date']);
static::assertSame($dates[1]->format(\DateTime::RSS), $day['date_rss']); static::assertSame($dates[1]->format(\DateTime::RSS), $day['date_rss']);
static::assertSame(format_date($dates[1], false), $day['date_human']); static::assertSame(format_date($dates[1], false), $day['date_human']);
static::assertSame('http://shaarli/daily?day='. $dates[1]->format('Ymd'), $day['absolute_url']); static::assertSame('http://shaarli/subfolder/daily?day='. $dates[1]->format('Ymd'), $day['absolute_url']);
static::assertCount(2, $day['links']); static::assertCount(2, $day['links']);
static::assertSame(2, $day['links'][0]['id']); static::assertSame(2, $day['links'][0]['id']);
@ -468,8 +468,8 @@ class DailyControllerTest extends TestCase
static::assertStringContainsString('application/rss', $result->getHeader('Content-Type')[0]); static::assertStringContainsString('application/rss', $result->getHeader('Content-Type')[0]);
static::assertSame('dailyrss', (string) $result->getBody()); static::assertSame('dailyrss', (string) $result->getBody());
static::assertSame('Shaarli', $assignedVariables['title']); static::assertSame('Shaarli', $assignedVariables['title']);
static::assertSame('http://shaarli', $assignedVariables['index_url']); static::assertSame('http://shaarli/subfolder/', $assignedVariables['index_url']);
static::assertSame('http://shaarli/daily-rss', $assignedVariables['page_url']); static::assertSame('http://shaarli/subfolder/daily-rss', $assignedVariables['page_url']);
static::assertFalse($assignedVariables['hide_timestamps']); static::assertFalse($assignedVariables['hide_timestamps']);
static::assertCount(0, $assignedVariables['days']); static::assertCount(0, $assignedVariables['days']);
} }

3
tests/front/controller/visitor/FrontControllerMockHelper.php
View file

@ -79,8 +79,9 @@ trait FrontControllerMockHelper
$this->container->environment = [ $this->container->environment = [
'SERVER_NAME' => 'shaarli', 'SERVER_NAME' => 'shaarli',
'SERVER_PORT' => '80', 'SERVER_PORT' => '80',
'REQUEST_URI' => '/daily-rss', 'REQUEST_URI' => '/subfolder/daily-rss',
'REMOTE_ADDR' => '1.2.3.4', 'REMOTE_ADDR' => '1.2.3.4',
'SCRIPT_NAME' => '/subfolder/index.php',
]; ];
$this->container->basePath = '/subfolder'; $this->container->basePath = '/subfolder';

35
tests/front/controller/visitor/InstallControllerTest.php
View file

@ -257,6 +257,39 @@ class InstallControllerTest extends TestCase
static::assertSame('/subfolder/login', $result->getHeader('location')[0]); static::assertSame('/subfolder/login', $result->getHeader('location')[0]);
static::assertSame('UTC', $confSettings['general.timezone']); static::assertSame('UTC', $confSettings['general.timezone']);
static::assertSame('Shared bookmarks on http://shaarli', $confSettings['general.title']); static::assertSame('Shared bookmarks on http://shaarli/subfolder/', $confSettings['general.title']);
}
/**
* Same test as testSaveInstallDefaultValues() but for an instance install in root directory.
*/
public function testSaveInstallDefaultValuesWithoutSubfolder(): void
{
$confSettings = [];
$this->container->environment = [
'SERVER_NAME' => 'shaarli',
'SERVER_PORT' => '80',
'REQUEST_URI' => '/install',
'REMOTE_ADDR' => '1.2.3.4',
'SCRIPT_NAME' => '/index.php',
];
$this->container->basePath = '';
$request = $this->createMock(Request::class);
$response = new Response();
$this->container->conf->method('set')->willReturnCallback(function (string $key, $value) use (&$confSettings) {
$confSettings[$key] = $value;
});
$result = $this->controller->save($request, $response);
static::assertSame(302, $result->getStatusCode());
static::assertSame('/login', $result->getHeader('location')[0]);
static::assertSame('UTC', $confSettings['general.timezone']);
static::assertSame('Shared bookmarks on http://shaarli/', $confSettings['general.title']);
} }
} }

2
tests/front/controller/visitor/OpenSearchControllerTest.php
View file

@ -39,6 +39,6 @@ class OpenSearchControllerTest extends TestCase
$result->getHeader('Content-Type')[0] $result->getHeader('Content-Type')[0]
); );
static::assertSame('opensearch', (string) $result->getBody()); static::assertSame('opensearch', (string) $result->getBody());
static::assertSame('http://shaarli', $assignedVariables['serverurl']); static::assertSame('http://shaarli/subfolder/', $assignedVariables['serverurl']);
} }
} }