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

Already on GitHub? Sign in to your account

Make tests/cases/test/FixturesTest run on Windows. #6

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

hans-d commented Jun 13, 2012

Seems that linux based systems return NULL on file_get_contents($file) in these cases, where Windows throws a fit^h^h^hexception.
Adjusted that non-existing files cannot be read. Fixes #5

@hans-d hans-d Make tests/cases/test/FixturesTest run on Windows.
Seems that linux based systems return NULL on file_get_contents($file) in these cases, where Windows throws a fit^h^h^hexception.
Adjusted that non-existing files cannot be read
1bf7edc

@nateabele nateabele commented on the diff Jun 13, 2012

tests/cases/test/FixtureTest.php
@@ -170,42 +170,29 @@ protected function _testLoad($posts) {
* sublcass of lithium\util\Collection in the current implementation).
*/
public function testLoadResultClass() {
- $options = array(
- 'path' => dirname(dirname(__DIR__)).'/fixtures'
- );
- $ships = Fixture::load('Pirate', $options);
+ $path = dirname(dirname(__DIR__)).'/fixtures/{:file}.{:type}';
+ $options = array('path' => $path);
@nateabele

nateabele Jun 13, 2012

Collaborator

You can change this to $options = compact('path');.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment