diff mbox series

[meta-oe,V4] bc: Add ptest.

Message ID 1666840152-10530-1-git-send-email-yanxk.fnst@fujitsu.com
State New
Headers show
Series [meta-oe,V4] bc: Add ptest. | expand

Commit Message

yanxk.fnst@fujitsu.com Oct. 27, 2022, 3:09 a.m. UTC 
From: Yan Xinkuan <yanxk.fnst@fujitsu.com>

Add package test function for OSS 'bc',
using bc in the system to do calculation
jobs according to the .b files from the
source code file of 'bc'.

Test example as below:
......
.00673400673400673400
“PASS: bc/div.b”
9983640860328357366034714556282968349582
7909199408566065153345558783\
9427595471.89114392327665123852
“PASS: bc/exp.b”
length(b)= 1406
“PASS: bc/fact.b”
......

If bc runs the .b files and does not crash,
it would 'PASS', otherwise it would 'FAIL'.

Tested in qemux86-64, with kvm enabled, test
cost 12 secs, so it should be a fast test.

Thanks to Ross Burton and Alexander Kanavin
for the professional guidance.

Signed-off-by: Yan Xinkuan <yanxk.fnst@fujitsu.com>
---
 meta/conf/distro/include/ptest-packagelists.inc | 1 +
 meta/recipes-extended/bc/bc/run-ptest           | 9 +++++++++
 meta/recipes-extended/bc/bc_1.07.1.bb           | 9 +++++++--
 3 files changed, 17 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-extended/bc/bc/run-ptest

Comments

Alexander Kanavin Oct. 31, 2022, 9:58 a.m. UTC | #1 
On Mon, 31 Oct 2022 at 10:55, Yan Xin Kuan <yanxk.fnst@fujitsu.com> wrote:
> @@ -69,6 +69,7 @@ PTESTS_FAST = "\
>      slang-ptest \
>      wayland-ptest \
>      zlib-ptest \
> +    bc-ptest \
>  "

This needs to be inserted in alphabetical order, otherwise looks good.

Alex
yanxk.fnst@fujitsu.com Oct. 31, 2022, 11:33 p.m. UTC | #2 
Got it.
Can be seen in v5.
diff mbox series

Patch

diff --git a/meta/conf/distro/include/ptest-packagelists.inc b/meta/conf/distro/include/ptest-packagelists.inc
index 56088e4e66..0a52a48ef1 100644
--- a/meta/conf/distro/include/ptest-packagelists.inc
+++ b/meta/conf/distro/include/ptest-packagelists.inc
@@ -69,6 +69,7 @@  PTESTS_FAST = "\
     slang-ptest \
     wayland-ptest \
     zlib-ptest \
+    bc-ptest \
 "
 PTESTS_FAST:append:libc-glibc = " glibc-tests-ptest"
 PTESTS_PROBLEMS:remove:libc-glibc = "glibc-tests-ptest"
diff --git a/meta/recipes-extended/bc/bc/run-ptest b/meta/recipes-extended/bc/bc/run-ptest
new file mode 100644
index 0000000000..da2156f8cd
--- /dev/null
+++ b/meta/recipes-extended/bc/bc/run-ptest
@@ -0,0 +1,9 @@ 
+#! /bin/sh
+
+for TEST in *.b; do
+	if bc -l $TEST </dev/null; then
+		echo “PASS: bc/$TEST”
+        else
+		echo “FAIL: bc/$TEST”
+        fi
+done
diff --git a/meta/recipes-extended/bc/bc_1.07.1.bb b/meta/recipes-extended/bc/bc_1.07.1.bb
index 1bec76bb2a..178588f11b 100644
--- a/meta/recipes-extended/bc/bc_1.07.1.bb
+++ b/meta/recipes-extended/bc/bc_1.07.1.bb
@@ -15,11 +15,12 @@  DEPENDS = "flex-native"
 SRC_URI = "${GNU_MIRROR}/${BPN}/${BP}.tar.gz \
            file://no-gen-libmath.patch \
            file://libmath.h \
-           file://0001-dc-fix-exit-code-of-q-command.patch"
+           file://0001-dc-fix-exit-code-of-q-command.patch \
+           file://run-ptest"
 SRC_URI[md5sum] = "cda93857418655ea43590736fc3ca9fc"
 SRC_URI[sha256sum] = "62adfca89b0a1c0164c2cdca59ca210c1d44c3ffc46daf9931cf4942664cb02a"
 
-inherit autotools texinfo update-alternatives
+inherit autotools texinfo update-alternatives ptest
 
 PACKAGECONFIG ??= "readline"
 PACKAGECONFIG[readline] = "--with-readline,--without-readline,readline"
@@ -29,6 +30,10 @@  do_compile:prepend() {
     cp -f ${WORKDIR}/libmath.h ${B}/bc/libmath.h
 }
 
+do_install_ptest() {
+        install ${S}/Test/*.b ${D}${PTEST_PATH}
+}
+
 ALTERNATIVE:${PN} = "bc dc"
 ALTERNATIVE_PRIORITY = "100"