Dynamite Tom. Regardless once one or two of us jump on improving both sonar stats and test issues the codebase will be better for it.
On Friday, October 16, 2015, Tom Barber <[email protected]> wrote: > https://builds.apache.org/job/oodt-coverage/13/clover-report/dashboard.html > > The rum has worn off, I'm back at work. I did a bit more fiddling and saw > you can publish the reports via jenkins, so there you go. Per build clover > coverage that you don't need to login to Jenkins for. > > Tom > > On Fri, Oct 16, 2015 at 12:14 AM, Lewis John Mcgibbney < > [email protected] <javascript:;>> wrote: > > > Hi Tom, > > Done. The page is available here > > http://oodt.apache.org/development.html#testing > > Ta, > > Have a good weekend mate. > > > > On Thu, Oct 15, 2015 at 4:08 PM, Tom Barber <[email protected] > <javascript:;>> > > wrote: > > > > > Can you do me a favour, I can't find that development page in CMS and I > > > need to drink run, > > > > > > change the mvn command to: > > > > > > mvn clean com.atlassian.maven.plugins:maven-clover2-plugin:instrument > > > com.atlassian.maven.plugins:maven-clover2-plugin:setup > > > com.atlassian.maven.plugins:maven-clover2-plugin:aggregate > > > com.atlassian.maven.plugins:maven-clover2-plugin:clover test install > > > -Dmaven.clover.skipGroverJar=true > > > > > > and then put if they want to find out more about Clover visit: > > > > > > > > > https://confluence.atlassian.com/display/CLOVER/Clover-for-Maven+2+and+3+User's+Guide > > > > > > Thanks! > > > > > > On Fri, Oct 16, 2015 at 12:06 AM, Lewis John Mcgibbney < > > > [email protected] <javascript:;>> wrote: > > > > > > > bewm > > > > > > > > On Thu, Oct 15, 2015 at 4:02 PM, Tom Barber <[email protected] > <javascript:;>> > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > https://builds.apache.org/job/oodt-coverage/ws/target/site/clover/dashboard.html > > > > > > > > > > There you go, don't say I don't do anything for ya, I think it > might > > > be a > > > > > bit up and down over the next few days whilst I clean it up, but it > > > seems > > > > > to be working fine currently. > > > > > > > > > > Tom > > > > > > > > > > On Thu, Oct 15, 2015 at 6:11 PM, Lewis John Mcgibbney < > > > > > [email protected] <javascript:;>> wrote: > > > > > > > > > > > Hi Tom. > > > > > > Thanks for getting this together > > > > > > http://oodt.apache.org/development.html#testing > > > > > > Thanks > > > > > > Lewis > > > > > > > > > > > > On Thu, Oct 15, 2015 at 9:47 AM, Tom Barber < > > [email protected] <javascript:;> > > > > > > > > > > wrote: > > > > > > > > > > > > > Okay, the build works but it says we have 0% test coverage > cause > > > its > > > > > not > > > > > > > finding the tests I'll fiddle around with it later, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://builds.apache.org/job/oodt-coverage/ws/target/site/clover/dashboard.html > > > > > > > > > > > > > > Tom > > > > > > > > > > > > > > On Thu, Oct 15, 2015 at 4:20 PM, Lewis John Mcgibbney < > > > > > > > [email protected] <javascript:;>> wrote: > > > > > > > > > > > > > > > If the output it retained within the Jenkins job workspace > then > > > we > > > > > can > > > > > > at > > > > > > > > least look into it. > > > > > > > > Would be a good point of reference for developers. > > > > > > > > > > > > > > > > > > > > > > > > On Thursday, October 15, 2015, Tom Barber < > > > [email protected] <javascript:;> > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Hi Lewis, > > > > > > > > > > > > > > > > > > No problem, I thought about adding it to Jenkins but I > don't > > > > think > > > > > we > > > > > > > > > publish the site docs automatically currently, so there is > > > little > > > > > > > point. > > > > > > > > If > > > > > > > > > someone knows how we can publish the site docs after each > > > build, > > > > > then > > > > > > > yes > > > > > > > > > it would be great to add it to the jenkins build. > > > > > > > > > > > > > > > > > > Tom > > > > > > > > > > > > > > > > > > On Thu, Oct 15, 2015 at 4:13 PM, Lewis John Mcgibbney < > > > > > > > > > [email protected] <javascript:;> <javascript:;>> > wrote: > > > > > > > > > > > > > > > > > > > Nice Tom. > > > > > > > > > > Will add the directions to the website documentation as I > > > think > > > > > > there > > > > > > > > is > > > > > > > > > an > > > > > > > > > > area for eat coverage. > > > > > > > > > > We adding this to the Jenkins build yes? > > > > > > > > > > > > > > > > > > > > On Thursday, October 15, 2015, Tom Barber < > > > > > [email protected] <javascript:;> > > > > > > > > > <javascript:;>> wrote: > > > > > > > > > > > > > > > > > > > > > Okay they sent me the new license, its in our root > > pom.xml > > > > but > > > > > > > > doesn't > > > > > > > > > > have > > > > > > > > > > > any effect on the build, you need to run it manually: > > > > > > > > > > > > > > > > > > > > > > mvn clover2:setup test clover2:aggregate clover2:clover > > > > > > > > > > > -Dmaven.test.failure.ignore=true > > > > > > > > > > > > > > > > > > > > > > for example and it will report to the site folder. > > > > > > > > > > > > > > > > > > > > > > The plugin is in the committers donated licenses folder > > in > > > > SVN, > > > > > > it > > > > > > > > is a > > > > > > > > > > > license for org.apache.* so if you want to use it on > > > another > > > > > > > project > > > > > > > > to > > > > > > > > > > use > > > > > > > > > > > the same license. > > > > > > > > > > > > > > > > > > > > > > Tom > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 15, 2015 at 6:57 AM, Lewis John Mcgibbney < > > > > > > > > > > > [email protected] <javascript:;> > <javascript:;> <javascript:;>> > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > Sound > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Oct 13, 2015 at 12:37 PM, Tom Barber < > > > > > > > > > [email protected] <javascript:;> <javascript:;> > > > > > > > > > > > <javascript:;>> > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > Okay, had a bit of a search of the mailing list > > archive > > > > and > > > > > > > > found a > > > > > > > > > > > > Clover > > > > > > > > > > > > > license, so I added it to the pom, sadly its out of > > > date! > > > > > > > > > > > > > > > > > > > > > > > > > > I'll check with Atlassian and see if we can get a > > > > renewal, > > > > > > the > > > > > > > > old > > > > > > > > > > one > > > > > > > > > > > > was > > > > > > > > > > > > > licensed for all org.apache.* packages so all > > projects > > > > > could > > > > > > > use > > > > > > > > > it, > > > > > > > > > > > > > hopefully we can get another one the same. > > > > > > > > > > > > > > > > > > > > > > > > > > Tom > > > > > > > > > > > > > > > > > > > > > > > > > > On Sun, Oct 11, 2015 at 9:51 PM, Tom Barber < > > > > > > > > > [email protected] <javascript:;> <javascript:;> > > > > > > > > > > > <javascript:;>> > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > Yeah I use the clover report all the time in real > > > life > > > > > and > > > > > > > its > > > > > > > > > > great > > > > > > > > > > > > for > > > > > > > > > > > > > > stuff like OODT but its an Atlassian product, so > I > > > just > > > > > ran > > > > > > > it > > > > > > > > on > > > > > > > > > > my > > > > > > > > > > > > > laptop > > > > > > > > > > > > > > using my own open source license. Technically we > > > would > > > > > > > qualify > > > > > > > > > for > > > > > > > > > > an > > > > > > > > > > > > > open > > > > > > > > > > > > > > source license, but I don't know what the ASF > > policy > > > > is, > > > > > > or, > > > > > > > > they > > > > > > > > > > > might > > > > > > > > > > > > > > already have a license in which case, absolutely, > > its > > > > > just > > > > > > > > > > published > > > > > > > > > > > > with > > > > > > > > > > > > > > the standard Maven Site. > > > > > > > > > > > > > > > > > > > > > > > > > > > > If no one else knows, I can check with Infra, > Sonar > > > > also > > > > > is > > > > > > > > > > supposed > > > > > > > > > > > to > > > > > > > > > > > > > > have code coverage but its not visible so I don't > > > know > > > > if > > > > > > > they > > > > > > > > > > > disable > > > > > > > > > > > > > it, > > > > > > > > > > > > > > also Clover reports can be used by Sonar, so > there > > is > > > > > some > > > > > > > > > > crossover > > > > > > > > > > > > > there. > > > > > > > > > > > > > > > > > > > > > > > > > > > > Tom > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Sun, Oct 11, 2015 at 9:48 PM, Lewis John > > > Mcgibbney < > > > > > > > > > > > > > > [email protected] <javascript:;> > <javascript:;> > > > > <javascript:;>> > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > >> This is also extremely useful. How did you > > generate > > > > the > > > > > > > > report? > > > > > > > > > > Can > > > > > > > > > > > we > > > > > > > > > > > > > >> automate this as part of builds against master? > > > > > > > > > > > > > >> > > > > > > > > > > > > > >> On Saturday, October 10, 2015, Tom Barber < > > > > > > > > > > [email protected] <javascript:;> <javascript:;> > > > > > > > > > > > <javascript:;>> > > > > > > > > > > > > > >> wrote: > > > > > > > > > > > > > >> > > > > > > > > > > > > > >> > On top of the Sonar stuff, I ran Atlassian > > Clover > > > > over > > > > > > the > > > > > > > > > whole > > > > > > > > > > > > > >> project to > > > > > > > > > > > > > >> > get some coverage stats. > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> > The results of which you can find here: > > > > > > > > > > > > > >> > http://www.meteorite.bi/downloads/clover/ > > > > > > > > > > > > > >> > As you can see we currently stand as 26% > > coverage. > > > > > > Handily > > > > > > > > the > > > > > > > > > > > > clover > > > > > > > > > > > > > >> > report tells us some stuff about what's > missing. > > > So > > > > on > > > > > > top > > > > > > > > of > > > > > > > > > > > > cleaning > > > > > > > > > > > > > >> up > > > > > > > > > > > > > >> > the code, I can finally look at adding to the > > test > > > > > > suite. > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> > Tom > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > > > > > > >> -- > > > > > > > > > > > > > >> *Lewis* > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > *Lewis* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > *Lewis* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > *Lewis* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > *Lewis* > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > *Lewis* > > > > > > > > > > > > > > > -- > > *Lewis* > > > -- *Lewis*
