[WIP] Adding parallel testing ANT build configuration and related files #3703

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@Ocramius
Member
Ocramius commented Feb 7, 2013

This PR tries to introduce parallel testing in our travis-ci tests.

As of my current experiments, this was the time for builds before parallel testing:

https://travis-ci.org/zendframework/zf2/builds/4627221 (basically between 6.5 and 9 minutes)

This is the time for builds AFTER parallel testing through ANT has been introduced:

https://travis-ci.org/Ocramius/zf2/builds/4632333 (between 3 and 4 minutes)

Now it really drills down to making this prettier and safer, so please ask me anything and I shall research.

@Ocramius
Member
Ocramius commented Feb 7, 2013

(still need to fix the CS tests)

@Maks3w Maks3w commented on the diff Feb 7, 2013
build.xml
+ </parallel>
+ </target>
+
+ <target name="show-test-results" description="Display logged test results">
+ <concat>
+ <fileset dir="${basedir}/build/cs-results/"/>
+ <fileset dir="${basedir}/build/test-results/"/>
+ </concat>
+ </target>
+
+ <macrodef name="component-test">
+ <attribute name="component"/>
+ <sequential>
+ <echo output="${basedir}/build/test-results/@{component}.log" level="debug">
+
+ ZendTest/@{component}
@Maks3w
Maks3w Feb 7, 2013 Zend Framework member

use backslash here \

@Maks3w
Member
Maks3w commented Feb 7, 2013

Looks awesome @ocramius

Please add a link to a Travis build where CS fail and other more for a Test fail

@Ocramius
Member
Ocramius commented Feb 7, 2013

Will do!

Also, seems like I have to tell ANT that the target environment is unix.

@weierophinney weierophinney added a commit that referenced this pull request Feb 7, 2013
@weierophinney weierophinney Merge branch 'hotfix/3703' into develop
Forward port #3703
9b92d74
@weierophinney weierophinney added a commit that closed this pull request Feb 7, 2013
@weierophinney weierophinney Merge branch 'hotfix/3703'
Close #3703

THIS IS AMAZING! Normally takes me 3-4 minutes on my 4 core machine to run tests
and CS checks; with this, it took 52s. SECONDS! Thanks, @ocramius !!!!
de0ea57
@weierophinney
Member

Holy crap, but that was amazing - built in 52s on my machine!

@weierophinney weierophinney was assigned Feb 7, 2013
@Maks3w
Member
Maks3w commented Feb 7, 2013

@weierophinney This is a WIP. CS Fixer is not working

@weierophinney
Member

@Maks3w Testing locally, I created a CS error, and the build script notified me of it...

@Ocramius
Member
Ocramius commented Feb 7, 2013

@weierophinney builds in 30 seconds on my monster here :D Anyway, did you also fix the problem I had with the CS tester before merging? :\

@Ocramius
Member
Ocramius commented Feb 7, 2013

@Maks3w I'll take a look at it tomorrow and eventually fix the small things that were left behind

@Maks3w
Member
Maks3w commented Feb 8, 2013

Looks that there is no more "error" state in Travis

https://travis-ci.org/zendframework/zf2/jobs/4658561

@Ocramius
Member
Ocramius commented Feb 8, 2013

I think that part needs to be sync because of the downloads...

Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

On 8 February 2013 09:11, Maks notifications@github.com wrote:

@weierophinney https://github.com/weierophinney Not in Travis
https://travis-ci.org/zendframework/zf2/jobs/4653851/#L67


Reply to this email directly or view it on GitHubhttps://github.com/zendframework/zf2/pull/3703#issuecomment-13280691.

@asm89 asm89 referenced this pull request in symfony/symfony Apr 18, 2013
Closed

[WIP] Add `build.xml` to run testsuite in parallel #7708

@weierophinney weierophinney added a commit to zendframework/zend-i18n-resources that referenced this pull request May 28, 2015
@weierophinney weierophinney Merge branch 'hotfix/3703'
Close zendframework/zendframework#3703

THIS IS AMAZING! Normally takes me 3-4 minutes on my 4 core machine to run tests
and CS checks; with this, it took 52s. SECONDS! Thanks, @ocramius !!!!
86b4350
@weierophinney weierophinney added a commit to zendframework/zend-i18n-resources that referenced this pull request May 28, 2015
@weierophinney weierophinney Merge branch 'hotfix/3703' into develop e32c680
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment