diff mbox series

[1/2] runtest/smoketest: Add smoketest

Message ID 20200921114026.11184-1-chrubis@suse.cz
State Accepted
Headers show
Series [1/2] runtest/smoketest: Add smoketest | expand

Commit Message

Cyril Hrubis Sept. 21, 2020, 11:40 a.m. UTC
This runtest file is meant to be used for infrastructure checks i.e. to
test LTP integration with various frameworks. The testcases were choosen
so that the testrun is fast and also represents reasonable subset of
different LTP testcases, e.g. binary, shell, etc.

The primary reason we are adding this is that we want to get rid of
quickhit, which has been misused for this purpose.

Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
CC: "Bird, Tim" <Tim.Bird@sony.com>
CC: Petr Vorel <pvorel@suse.cz>
---
 runtest/smoketest | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
 create mode 100644 runtest/smoketest

Comments

Petr Vorel Sept. 21, 2020, 11:59 a.m. UTC | #1
Hi Cyril,

thanks for implementing this.
> This runtest file is meant to be used for infrastructure checks i.e. to
> test LTP integration with various frameworks. The testcases were choosen
> so that the testrun is fast and also represents reasonable subset of
> different LTP testcases, e.g. binary, shell, etc.

> The primary reason we are adding this is that we want to get rid of
> quickhit, which has been misused for this purpose.

> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> CC: "Bird, Tim" <Tim.Bird@sony.com>
> CC: Petr Vorel <pvorel@suse.cz>

Reviewed-by: Petr Vorel <pvorel@suse.cz>
Tested-by: Petr Vorel <pvorel@suse.cz>

Kind regards,
Petr
Bird, Tim Sept. 21, 2020, 5:40 p.m. UTC | #2
Sorry I didn't send mine in over the weekend.  I had planned
to send it in today.  But since you beat me to it I guess you
get naming rights ("smoketest" vs "ltp-selftest-quick"). :-)
That's fine with me.


> -----Original Message-----
> From: Cyril Hrubis <chrubis@suse.cz>
> 
> This runtest file is meant to be used for infrastructure checks i.e. to
> test LTP integration with various frameworks. The testcases were choosen
> so that the testrun is fast and also represents reasonable subset of
> different LTP testcases, e.g. binary, shell, etc.
> 
> The primary reason we are adding this is that we want to get rid of
> quickhit, which has been misused for this purpose.
> 
> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> CC: "Bird, Tim" <Tim.Bird@sony.com>
> CC: Petr Vorel <pvorel@suse.cz>
> ---
>  runtest/smoketest | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>  create mode 100644 runtest/smoketest
> 
> diff --git a/runtest/smoketest b/runtest/smoketest
> new file mode 100644
> index 000000000..218827a12
> --- /dev/null
> +++ b/runtest/smoketest
> @@ -0,0 +1,15 @@
> +# The purpose of this file is to have a quick integration test for various
> +# test frameworks that integrate LTP testsuite.
> +
> +access01 access01
> +chdir01 chdir01
> +fork01 fork01
> +time01 time01
> +wait02 wait02
> +write01 write01
> +symlink01 symlink01
> +stat04 symlink01 -T stat04
> +utime01A symlink01 -T utime01
> +rename01A symlink01 -T rename01
> +splice02 seq 1 20 | splice02
> +route4-change-dst route-change-dist.sh

this should be:
route4-change-dst route-change-dst.sh

$ find . -name "route-change-d*"
./testcases/network/stress/route/route-change-dst.sh

> --
> 2.26.2

Reviewed-by: Tim Bird <tim.bird@sony.com>
  -- Tim
Petr Vorel Sept. 22, 2020, 9:47 a.m. UTC | #3
Hi,

...
> > +route4-change-dst route-change-dist.sh

> this should be:
> route4-change-dst route-change-dst.sh
Good catch, thanks! Obviously typo.

Kind regards,
Petr

> $ find . -name "route-change-d*"
> ./testcases/network/stress/route/route-change-dst.sh
Cyril Hrubis Sept. 22, 2020, 11:28 a.m. UTC | #4
Hi!
> this should be:
> route4-change-dst route-change-dst.sh
> 
> $ find . -name "route-change-d*"
> ./testcases/network/stress/route/route-change-dst.sh

Sorry missed that one when applying, but fixed in follow up patch.
diff mbox series

Patch

diff --git a/runtest/smoketest b/runtest/smoketest
new file mode 100644
index 000000000..218827a12
--- /dev/null
+++ b/runtest/smoketest
@@ -0,0 +1,15 @@ 
+# The purpose of this file is to have a quick integration test for various
+# test frameworks that integrate LTP testsuite.
+
+access01 access01
+chdir01 chdir01
+fork01 fork01
+time01 time01
+wait02 wait02
+write01 write01
+symlink01 symlink01
+stat04 symlink01 -T stat04
+utime01A symlink01 -T utime01
+rename01A symlink01 -T rename01
+splice02 seq 1 20 | splice02
+route4-change-dst route-change-dist.sh