Skip to content

Class Satooshi\...\Configuration not found in CoverageCollector.php #13

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

Closed
sjparkinson opened this issue Jul 20, 2014 · 4 comments
Closed

Comments

@sjparkinson
Copy link

Heya, I'm trying to setup code coverage on my PHP project and I'm hitting the following error when running ./vendor/bin/test-reporter locally.

Something I'm missing/doing wrong?

PHP Fatal error: 
    Class 'Satooshi\Bundle\CoverallsV1Bundle\Config\Configuration' not found
    in /Users/Samuel/Repositories/Personal/StaticReview/vendor/codeclimate/php-test-reporter/src/CodeClimate/Bundle/TestReporterBundle/CoverageCollector.php
    on line 22

The commands I've run:

composer update --dev --prefer-source --optimize-autoloader

vendor/bin/phpunit --bootstrap vendor/autoload.php --coverage-clover build/logs/clover.xml tests

CODECLIMATE_REPO_TOKEN=[...] ./vendor/bin/test-reporter
@sjparkinson
Copy link
Author

And here's the Travis CI output: https://travis-ci.org/sjparkinson/static-review/jobs/30413307#L127

martinssipenko pushed a commit to martinssipenko/php-test-reporter that referenced this issue Jul 21, 2014
martinssipenko pushed a commit to martinssipenko/php-test-reporter that referenced this issue Jul 21, 2014
martinssipenko pushed a commit to martinssipenko/php-test-reporter that referenced this issue Jul 21, 2014
@martinssipenko
Copy link
Contributor

Turns out this package was built on top development version of satooshi/php-coveralls where they have changed namespaces, so setting >=0.6.1 in composer.json resulted in to loading a version where old namespaces were used.

I've submitted a pull request #14 that should fix the issue. Creating a tag on this repo is also required 0.1.2.

This was missed in Travis CI tests because this package had "minimum-stability": "dev" in it's composer file, which I have now removed.

@sjparkinson
Copy link
Author

Awesome! 👍

Closed as should be addressed in pull request #14.

@martinssipenko
Copy link
Contributor

In case there is a plan to implement #6 in near future we are safe.

Otherwise there should be a ticket created to migrate to satooshi/php-coveralls version 0.7.0. Development branch could be used to do so, I'm happy to contribute if it's required.

marshally added a commit that referenced this issue Jul 22, 2014
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

No branches or pull requests

2 participants