Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] support/tests: use relative test_dir
@ 2017-09-05 16:21 Yann E. MORIN
  2017-09-05 19:09 ` Thomas Petazzoni
  2017-09-05 19:12 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Yann E. MORIN @ 2017-09-05 16:21 UTC (permalink / raw)
  To: buildroot

When we list the available tests, we use test_dir, which is set from
the path of the script. However, when we run the tests, we use the
hard-coded path.

Ditto to find the config file.

For consistency, always use test_dir.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com>
---
 support/testing/run-tests | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/support/testing/run-tests b/support/testing/run-tests
index 95c1565f72..ae0bd336b5 100755
--- a/support/testing/run-tests
+++ b/support/testing/run-tests
@@ -108,8 +108,8 @@ def main():
 
     nose2_args = ["-v",
                   "-N", str(args.testcases),
-                  "-s", "support/testing",
-                  "-c", "support/testing/conf/unittest.cfg"]
+                  "-s", test_dir,
+                  "-c", os.path.join(test_dir, "conf/unittest.cfg")]
 
     if len(args.testname) != 0:
         nose2_args += args.testname
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] support/tests: use relative test_dir
  2017-09-05 16:21 [Buildroot] [PATCH] support/tests: use relative test_dir Yann E. MORIN
@ 2017-09-05 19:09 ` Thomas Petazzoni
  2017-09-05 19:12 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2017-09-05 19:09 UTC (permalink / raw)
  To: buildroot

Hello,

On Tue,  5 Sep 2017 18:21:37 +0200, Yann E. MORIN wrote:
> When we list the available tests, we use test_dir, which is set from
> the path of the script. However, when we run the tests, we use the
> hard-coded path.
> 
> Ditto to find the config file.
> 
> For consistency, always use test_dir.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com>
> ---
>  support/testing/run-tests | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] support/tests: use relative test_dir
  2017-09-05 16:21 [Buildroot] [PATCH] support/tests: use relative test_dir Yann E. MORIN
  2017-09-05 19:09 ` Thomas Petazzoni
@ 2017-09-05 19:12 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2017-09-05 19:12 UTC (permalink / raw)
  To: buildroot

Hello,

On Tue,  5 Sep 2017 18:21:37 +0200, Yann E. MORIN wrote:
> When we list the available tests, we use test_dir, which is set from
> the path of the script. However, when we run the tests, we use the
> hard-coded path.
> 
> Ditto to find the config file.
> 
> For consistency, always use test_dir.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com>
> ---
>  support/testing/run-tests | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-09-05 19:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-05 16:21 [Buildroot] [PATCH] support/tests: use relative test_dir Yann E. MORIN
2017-09-05 19:09 ` Thomas Petazzoni
2017-09-05 19:12 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox