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

Fixing tests #31

Closed
schmittjoh opened this issue Mar 5, 2012 · 6 comments · Fixed by #32
Closed

Fixing tests #31

schmittjoh opened this issue Mar 5, 2012 · 6 comments · Fixed by #32

Comments

@schmittjoh
Copy link
Member

@jmikola, we have a failing test case related to the refactoring of the RegisterEventListenersSubscribersPass, could you take a look?

@jmikola
Copy link
Member

jmikola commented Mar 5, 2012

Will do.

@jmikola
Copy link
Member

jmikola commented Mar 5, 2012

Are you sure?

[avocado: DoctrineBundle] master $ ./composer.phar install --install-suggests
Installing dependencies
  - Package twig/twig (1.6.1)
    Downloading                                                                 
    Unpacking archive
    Cleaning up

  - Package doctrine/dbal (2.2.1)
    Downloading                                                                 
    Unpacking archive
    Cleaning up

  - Package doctrine/orm (2.2.1)
    Downloading                                                                 
    Unpacking archive
    Cleaning up

  - Package doctrine/common (2.2.1)
    Downloading                                                                 
    Unpacking archive
    Cleaning up

  - Package symfony/symfony (dev-master)
    Cloning 294b57e1b15f28541444f667be5c98d90ab8c668

Writing lock file
Generating autoload files
[avocado: DoctrineBundle] master $ phpunit 
PHPUnit 3.6.5 by Sebastian Bergmann.

Configuration read from /home/jmikola/workspace/DoctrineBundle/phpunit.xml.dist

................................................................. 65 / 86 ( 75%)
.....................

Time: 1 second, Memory: 69.00Mb

OK (86 tests, 382 assertions)

One problem I noticed is that ``--install-suggestsdoesn't installdoctrine/orm` despite it being listed as a suggested dep in `composer.json`. I had to move the package into the "require" block to actually get the dependency. Without ORM installed, the test suite fails hard:

[avocado: DoctrineBundle] master $ phpunit 
PHPUnit 3.6.5 by Sebastian Bergmann.

Configuration read from /home/jmikola/workspace/DoctrineBundle/phpunit.xml.dist

.EE.............................................................. 65 / 86 ( 75%)
........PHP Fatal error:  Class 'Doctrine\ORM\Mapping\ClassMetadataInfo' not found in /home/jmikola/workspace/DoctrineBundle/Tests/Mapping/MetadataFactoryTest.php on line 23

Fatal error: Class 'Doctrine\ORM\Mapping\ClassMetadataInfo' not found in /home/jmikola/workspace/DoctrineBundle/Tests/Mapping/MetadataFactoryTest.php on line 23

@schmittjoh
Copy link
Member Author

Maybe it's just travisci that fails?

http://travis-ci.org/#!/doctrine/DoctrineBundle/jobs/799379

@jmikola
Copy link
Member

jmikola commented Mar 5, 2012

Woah, it looks like I was testing my own outdated fork. Fixing.

@schmittjoh
Copy link
Member Author

Thanks!

@jmikola
Copy link
Member

jmikola commented Mar 5, 2012

Thanks for catching this. I wasn't aware the bundle was set up for Travis CI - I just enabled it on my fork so this shouldn't happen again :)

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 a pull request may close this issue.

2 participants