On 7/15/2011 3:24 PM, Mark Diggory wrote: > My recommendation is just > > 1.) It'd be best to run tests by default if your going to build the > source release. If you do not need to run the tests because you didn't > alter the source, perhaps you shouldn't be building the source release > and instead be using the binary release.
I'm not sure we should be assuming that people who download the Source Release are always doing so to directly modify 'dspace-api' (which is the only place we have Unit Tests right now). People could also be downloading the Source Release so they have easy access to 'dspace-xmlui-api' or 'dspace-jspui-api' classes that they may want to Overlay (to apply a small tweak, or patch a bug, or similar). As unit tests are only run during the build of 'dspace-api', the running of them would be of no real benefit to people who are doing Maven Overlays. (Though I do see your point, Mark, that it'd be nice to run those Unit Tests for them if they just made a local dspace-api change that was problematic) Maybe this is not realistic (feel free to tell me if it's not), but I wonder if it'd actually be possible do to something like: 1. Default code in SVN to always run tests (maven.test.skip=false) 2. But, during the assembly of the Source Release we could flip it to "maven.test.skip=true' for that dspace-src-release-x.x.x build This has the dual benefits of forcing us (Committers & hardcore developers) to be developing with Unit Tests turned on (though I still wonder how many of us will immediately tweak our IDE to turn them off locally), while also not subjecting our source release users to the Unit Tests by default. It's just a thought -- again, not sure how feasible this is, or how strict everyone else thinks we should be around Unit Tests. Personally, with my NetBeans IDE, I like having the Unit Tests "off" by default (as they take a while to run, and I don't need them to run all the time). But, I can also just right click on a Module and select "Test" to immediately run all its Unit Tests (to be sure everything still passes before I do a commit). Obviously, I can have this same IDE setup no matter what we default "maven.test.skip" to, so it really doesn't matter for my personal development practices. But, I'd be curious to hear what other developers think. - Tim ------------------------------------------------------------------------------ AppSumo Presents a FREE Video for the SourceForge Community by Eric Ries, the creator of the Lean Startup Methodology on "Lean Startup Secrets Revealed." This video shows you how to validate your ideas, optimize your ideas and identify your business strategy. http://p.sf.net/sfu/appsumosfdev2dev _______________________________________________ Dspace-devel mailing list Dspace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-devel
