Put test-run in Makefile

Bug #720666 reported by Olaf van der Spek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Drizzle
Invalid
Undecided
Unassigned
7.0
Invalid
Undecided
Unassigned

Bug Description

Doc says: cd tests && ./test-run && cd ..
Shouldn't this just be make check?

http://wiki.drizzle.org/Contributing_Code

Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

well. "make test" will do the job in that context. Feel free to edit the wiki page to say that.

Revision history for this message
Olaf van der Spek (olafvdspek) wrote :

The two are not equivalent.
test-run:
Spent 84.348 of 152 seconds executing testcases
real 2m31.799s
user 1m1.928s
sys 1m13.589s

make test:
Spent 247.900 of 429 seconds executing testcases
real 8m16.049s
user 3m34.433s
sys 2m17.705s

Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

That is because test-run only runs the main test suite (by design). "make test" or "make check" (both do the same) should be used since they run the other test suites too.

Revision history for this message
Andrew Hutchings (linuxjedi) wrote :

Updated wiki, closing as invalid since there are no code changes needed.

Revision history for this message
Olaf van der Spek (olafvdspek) wrote : Re: [Bug 720666] Re: Put test-run in Makefile

On Fri, Feb 18, 2011 at 12:19 PM, Andrew Hutchings
<email address hidden> wrote:
> Updated wiki, closing as invalid since there are no code changes needed.

Fix Released seems better than Invalid.

--
Olaf

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.