PrivateCoffee/alltube
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix cookie settings in tests

Browse source
This commit is contained in:
Pierre Rudloff 2017-12-05 14:37:23 +01:00
parent 5cd6ea0c44
commit 6b73026194
3 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
classes/LocaleManager.php
View file

@ -42,7 +42,6 @@ class LocaleManager
{
$session_factory = new \Aura\Session\SessionFactory();
$session = $session_factory->newInstance($cookies);
$session->setCookieParams(['httponly' => true]);
$this->sessionSegment = $session->getSegment('Alltube\LocaleManager');
$cookieLocale = $this->sessionSegment->get('locale');
if (isset($cookieLocale)) {

2
tests/LocaleManagerTest.php
View file

@ -27,6 +27,7 @@ class LocaleManagerTest extends TestCase
protected function setUp()
{
$this->localeManager = new LocaleManager();
$_SESSION['Alltube\LocaleManager']['locale'] = 'foo_BAR';
}
/**
@ -36,7 +37,6 @@ class LocaleManagerTest extends TestCase
*/
public function testConstructorWithCookies()
{
$_SESSION['Alltube\LocaleManager']['locale'] = 'foo_BAR';
$localeManager = new LocaleManager([]);
$this->assertEquals('foo_BAR', (string) $localeManager->getLocale());
}

2
tests/bootstrap.php
View file

@ -9,6 +9,8 @@ use Alltube\PlaylistArchiveStream;
*/
require_once __DIR__.'/../vendor/autoload.php';
ini_set('session.use_cookies', 0);
session_cache_limiter('');
session_start();
stream_wrapper_register('playlist', PlaylistArchiveStream::class);