[jbosstools-dev] AS Test Suite failure fix?
Denis Golovin
dgolovin at exadel.com
Tue Mar 19 14:27:09 EDT 2013
Hudson is actually running build agaains minimum TP, because it has no
-Dmaximum switch in maven command line.
Denis
On 03/19/2013 02:10 AM, Rob Stryker wrote:
> https://github.com/jbosstools/jbosstools-server/commit/b3672b46e4df710741c39ff77733246485547c8c
>
>
> Denis:
>
> I realize the error in the test suite was that the .project file is
> now included, whereas it was not included in the past. Why do you
> think the proper solution to this change in behavior was to simply
> change the expected count? That seems like a very bad fix.
>
> If something upstream has changed, we should discover what it is
> before simply changing the expected count in the unit test. Clearly
> these tests used to pass, and started failing after the update to m5.
> Shouldn't we try to discover what exactly changed here?
>
> - Rob
More information about the jbosstools-dev
mailing list