Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Dogfood QA Tools 3.0.0-alpha2 #117

Merged
merged 5 commits into from
Mar 3, 2017
Merged

Dogfood QA Tools 3.0.0-alpha2 #117

merged 5 commits into from
Mar 3, 2017

Conversation

rjkip
Copy link
Contributor

@rjkip rjkip commented Feb 24, 2017

This PR proposes the following changes:

  • Don't use generated Ant build file; our QA process is custom and too complex to ever want to offer using QA Tools configuration. This commit does try to stay as close as possible to the commands from the build file.
  • As QA Tools doesn't (yet?) support multiple test suites in PHPUnit, they are now organised by directory, rather than by suite. (Support multiple PHPUnit test suites #118)
  • As QA Tools doesn't (yet?) support a configured PHPUnit bootstrap file, it is passed to the binary using an option. (Support configuration of PHPUnit bootstrap file #119)

Fixes #90


  • Call Mockery::close() for tests using Mockery

@rjkip rjkip added this to the 3.0.0 milestone Feb 24, 2017
 * Don't use generated Ant build file; our QA process is custom and too complex to ever want to offer using QA Tools configuration. This commit does try to stay as close as possible to the commands from the build file.
 * As QA Tools doesn't (yet?) support multiple test suites in PHPUnit, they are now organised by directory, rather than by suite.
 * As QA Tools doesn't (yet?) support a configured PHPUnit bootstrap file, it is passed to the binary using an option
Reinier Kip added 2 commits February 24, 2017 16:41
Mockery will soon no longer support its PHPUnit TestListener, neither is its configuration supported by QA Tools 3.0.0-alpha2.
@rjkip rjkip requested a review from rjd22 February 24, 2017 15:48
Copy link

@rjd22 rjd22 left a comment

Choose a reason for hiding this comment

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

1 small thing


abstract class ContainerTestCase extends TestCase
abstract class ContainerTestCase extends \Ibuildings\QaTools\Test\MockeryTestCase
Copy link

Choose a reason for hiding this comment

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

Maybe import this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wow, you found one I missed 👏

@rjkip rjkip merged commit db028c7 into master Mar 3, 2017
@rjkip rjkip deleted the dogfood branch March 3, 2017 11:10
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.

Identify what must be done to be able to dogfood QA Tools
2 participants