Skip to content

Single fixture #24

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

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Single fixture #24

wants to merge 17 commits into from

Conversation

Hounddog
Copy link
Owner

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a80eb61 on single_fixture into 4f09fd9 on master.

@Ocramius
Copy link
Collaborator

Misses tests...

if (is_dir($fixtures)) {
$this->loader->loadFromDirectory($fixtures);
} elseif (file_exists($fixtures)) {
$classes = get_declared_classes();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this class supposed to handle this? Seems to be a case for a context aware fixture loader

@coveralls
Copy link

Coverage Status

Coverage increased (+39.31%) when pulling f8d5008 on single_fixture into 1a3571c on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e5d764f on single_fixture into 1a3571c on master.

@@ -0,0 +1,156 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should not be committed

@Ocramius Ocramius self-assigned this Nov 10, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+15.31%) to 23.0% when pulling 20c3046 on single_fixture into 1a3571c on master.

*
* @license http://www.opensource.org/licenses/lgpl-license.php LGPL
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to contact all authors of the library if you want to change the license :|

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have the license already set as MIT for the library. Unfortunately it is set differetnly in the files.

@coveralls
Copy link

Coverage Status

Coverage increased (+15.54%) to 23.23% when pulling 6a830d3 on single_fixture into 1a3571c on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants