Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Session validation improvement #889

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update ConfigTest.php
  • Loading branch information
kristuff committed Jun 6, 2020
commit bb6ed3390b212983d218d8e274a578473ed602e7
8 changes: 8 additions & 0 deletions tests/core/ConfigTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ public function tearDown()

/**
* Checks if the correct config file for an EXISTING environment / config is called.
*
* @runInSeparateProcess
*/
public function testGetDefaultEnvironment()
{
Expand All @@ -36,8 +38,14 @@ public function testGetDefaultEnvironment()
$this->assertEquals('index', Config::get('DEFAULT_ACTION'));
}

/**
* @runInSeparateProcess
*/
public function testGetFailingEnvironment()
{
// for testing
header_remove();

// manually set environment to "foobar" (and non-existing environment)
putenv('APPLICATION_ENV=foobar');

Expand Down