On Thu, Jun 20, 2013 at 02:01:54PM +0000, Daan Hoogland wrote: > Btw Sebastien, > > Newcomer as I am; how do I test the test-code tests?
The first step would be to see if you are able to build marvin $ mvn -Pdeveloper -pl :cloud-marvin clean install You can run tests suitable for basic zone within devcloud if you have it setup. See the section on "devcloud tests" in the marvin wiki. [0] Then you can try and run the checkin tests [1] (which are broken at the moment as a result of a merge) but I hope to fix that today. Automated tests will be run on the test infra @ jenkins.buildacloud.org [2] so any issues in the formatting will show up there as well. [0] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+-+Testing+with+Python [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+-+Testing+with+Python#Marvin-TestingwithPython- [2] http://jenkins.buildacloud.org/view/cloudstack-qa/job/test-smoke-matrix/ Thanks for the patches! > > -----Original Message----- > From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] > Sent: donderdag 20 juni 2013 15:43 > To: 'dev@cloudstack.apache.org' > Subject: RE: easy bug to fix for new comer > > Sure, > > One file at a time! > > -----Original Message----- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: donderdag 20 juni 2013 14:35 > To: dev@cloudstack.apache.org > Subject: easy bug to fix for new comer > > Hi, > > Here is an easy bug to fix for a newcomer to cloudstack: > > https://issues.apache.org/jira/browse/CLOUDSTACK-3096 > > install pep8 on your machine > git clone cloudstack repo > go to the marvin directory > > run pep8 like I show in the bug. > > edit the python scripts to fix the errors > > once pep8 is clean, git commit everything, create a patch and send to review > board. > > Any takers ? > > -Sebastien -- Prasanna., ------------------------ Powered by BigRock.com
