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

Update .travis.yml #9151

Closed
wants to merge 1 commit into from
Closed

Update .travis.yml #9151

wants to merge 1 commit into from

Conversation

ADmad
Copy link
Member

@ADmad ADmad commented Jul 21, 2016

Only run DatabaseSuite for non-mysql to speed up tests.

@ADmad ADmad added this to the 3.2.13 milestone Jul 21, 2016
@codecov-io
Copy link

codecov-io commented Jul 21, 2016

Current coverage is 89.57% (diff: 100%)

Merging #9151 into master will decrease coverage by 5.37%

@@             master      #9151   diff @@
==========================================
  Files           368        368           
  Lines         27001      28610   +1609   
  Methods        3233       3274     +41   
  Messages          0          0           
  Branches          0          0           
==========================================
- Hits          25637      25627     -10   
- Misses         1364       2983   +1619   
  Partials          0          0           

Powered by Codecov. Last update 5b0543e...3a861f1

- if [[ $DEFAULT = 1 && $TRAVIS_PHP_VERSION = 7.0 && $DB != 'mysql' ]]; then export CODECOVERAGE=1; vendor/bin/phpunit tests/TestCase/DatabaseSuite.php --coverage-clover=clover.xml; fi

- if [[ $DEFAULT = 1 && $TRAVIS_PHP_VERSION != 7.0 && $DB = 'mysql' ]]; then vendor/bin/phpunit; fi
- if [[ $DEFAULT = 1 && $TRAVIS_PHP_VERSION != 7.0 && $DB != 'mysql' ]]; then vendor/bin/phpunit tests/TestCase/DatabaseSuite.php; fi
Copy link
Contributor

Choose a reason for hiding this comment

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

Could't you use the --testsuite option here ?

@ADmad
Copy link
Member Author

ADmad commented Jul 21, 2016

Hmm codecov is not impressed. Let me see if I can get coverage to previous level else I'll close the PR.

Only run DatabaseSuite for non-mysql to speed up tests.
@thinkingmedia
Copy link
Contributor

Some of the tests must not be grouped correctly.

@ADmad
Copy link
Member Author

ADmad commented Jul 22, 2016

@thinkingmedia Maybe, but I am not inclined to review the test cases and find out :)

@ADmad ADmad deleted the travis branch May 18, 2018 12:52
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.

None yet

4 participants