On Thu, Jul 24, 2014 at 12:25 AM, Giuseppe Scrivano <[email protected]> wrote: > Darshit Shah <[email protected]> writes: > >> Hi, >> >> I've attached a patch which enables all the tests in our test suite to >> run through valgrind This behaviour can be invoked by setting the >> environment variable: VALGRIND_TESTS. >> >> Simultaneously, I also propose that we merge the new python based test >> suite into origin/master, to ensure that all new tests are created for >> the new test suite. In case a feature isn't implemented, we can add it >> ASAP. > > have you already tried to run it against master? If it just works, we > can do that without postponing it anymore. > Yes. It runs perfectly against master without any changes at all. I'll generate a set of patches for merging the testenv to master. >> >> -- >> Thanking You, >> Darshit Shah >> >> From 6140b1b6e3831a35fd2cf4536fda741a4dcc9019 Mon Sep 17 00:00:00 2001 >> From: Darshit Shah <[email protected]> >> Date: Wed, 23 Jul 2014 18:42:43 +0530 >> Subject: [PATCH] Support running tests through valgrind >> >> --- >> testenv/ChangeLog | 8 ++++++++ >> testenv/README | 6 +++++- >> testenv/conf/expected_ret_code.py | 11 +++++++---- >> testenv/test/base_test.py | 6 +++++- >> 4 files changed, 25 insertions(+), 6 deletions(-) > > ACK. > > Giuseppe
-- Thanking You, Darshit Shah
