I have checked in GEODE-487 into develop to remove integrationTests and distributedTests from the build task. The new task for running all tests is 'precheckin'. I have also updated the nightly build with this change.
--Mark On Thu, Oct 1, 2015 at 5:30 PM, Nitin Lamba <[email protected]> wrote: > +1 . > > Initial dev experience needs to be frictionless... :) > > ________________________________________ > From: Bruce Schuchardt <[email protected]> > Sent: Thursday, October 1, 2015 4:56 PM > To: [email protected] > Subject: Re: Propose changing check to only depend on UnitTest > > +1 > > Le 10/1/2015 2:21 PM, Darrel Schneider a écrit : > > +1 > > > > How about "precheckin" for the target that runs IntegrationTest and > > DistributionTest? Perhaps this target should start with a line informing > > the runner that it will take hours. > > > > > > On Thu, Oct 1, 2015 at 12:48 PM, Kirk Lund <[email protected]> wrote: > > > >> Proposal: > >> > >> 1) Remove IntegrationTest and DistributionTest from "check" target > >> > >> 2) Add a new target (name TBD) that executes IntegrationTest and > >> DistributionTest > >> > >> Reasons: > >> > >> 1) new people who clone and then build wait forever and complain that > build > >> hung resulting in bad first impression by users > >> > >> 2) Index-Geodein5minutes instructs user to skip tests but internet > wisdom > >> says never use -Dskip.tests=true > >> > >> > >> > https://cwiki.apache.org/confluence/display/GEODE/Index#Index-Geodein5minutes > >> > >> 3) non-developers don't need to run IntegrationTest and DistributedTest > -- > >> they just want to clone, build and try out basics to see if they want to > >> become a Geode User > >> > >> 4) anyone who decides they want to become a developer can easily learn > that > >> we invoke a target to run the long-running tests -- we can easily > document > >> this everywhere if people are truly worried about it > >> > >> 5) this change encourages us to improve our coverage with fast-running > >> UnitTests > >> > >> Anti-goals > >> > >> 1) I'm not suggesting that anyone can commit changes without running the > >> long-running tests > >> > >> -Kirk > >> > > -- Mark Bretl Software Build Engineer Pivotal 503-533-3869 www.pivotal.io
