Skip to content
This repository has been archived by the owner on Aug 28, 2018. It is now read-only.

Tests are now executing correctly. #42

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

Conversation

piotaixr
Copy link

There was some errors in the tests:

  • Use of Collection#count instead of count($array)
  • The Doctrine configuration mock was missing the defaultRepositoryFactory

@piotaixr
Copy link
Author

any ideas why travis fails as finding the last versions of doctrine/mongodb and is 'forced' to use a beta version of it?

@khepin
Copy link
Owner

khepin commented May 27, 2014

https://travis-ci.org/khepin/KhepinYamlFixturesBundle/jobs/25993536

Looks like you need to update some dependencies in the composer.json for the tests to pass.

@khepin
Copy link
Owner

khepin commented May 27, 2014

Seems you need to update this: https://github.com/khepin/KhepinYamlFixturesBundle/blob/master/bin/travis-mongo-installer.sh to the latest version of the mongodb extension.

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

Successfully merging this pull request may close these issues.

2 participants