[Buildroot] [PATCH 1/4] support/testing: make treq tests to ignore warning

[Buildroot] [PATCH 1/4] support/testing: make treq tests to ignore warning

[Ricardo Martincoski]

Newer versions of python-treq display a warning at runtime when
service_identity is not installed:
"Without the service_identity module, Twisted can perform only
rudimentary TLS client hostname verification."

This warning message confuses the test that looks for another string in
stdout. Make the test ignore other messages while still expecting
"Connection refused".

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139449
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139450

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Yegor Yefremov <yegorslists@googlemail.com>
---
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022950
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022951
---
 support/testing/tests/package/test_python_treq.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/support/testing/tests/package/test_python_treq.py b/support/testing/tests/package/test_python_treq.py
index 7108b95696..e24ae845be 100644
--- a/support/testing/tests/package/test_python_treq.py
+++ b/support/testing/tests/package/test_python_treq.py
@@ -7,7 +7,8 @@ class TestPythonTreq(TestPythonPackageBase):
     def run_sample_scripts(self):
         cmd = self.interpreter + " sample_python_treq.py"
         output, exit_code = self.emulator.run(cmd, timeout=20)
-        self.assertIn("Connection refused", output[0])
+        refuse_msgs = [1 for l in output if "Connection refused" in l]
+        self.assertGreater(sum(refuse_msgs), 0)
         self.assertEqual(exit_code, 0)
 
 
-- 
2.17.1

[Buildroot] [PATCH 2/4] support/testing: make crossbar to use umsgpack

[Ricardo Martincoski]

Since version 19.3.3, python-autobahn assumes that when CPython is used
msgpack will be used as well [1]. But it still allows the user to
override this behavior by setting an environment variable [2] to use
umsgpack.

Make the test to explicitly use umsgpack since it is part of the minimal
config (python-crossbar selects python-u-msgpack).

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139430

[1] https://github.com/crossbario/autobahn-python/commit/ea019b80427c9f6d43938631f8143bf7c993b4e7
[2] https://github.com/crossbario/autobahn-python/commit/fe70ceebe04849470b0c81b26ff6e65e709b0d33

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Yegor Yefremov <yegorslists@googlemail.com>
---
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022934
---
 support/testing/tests/package/sample_python_crossbar.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/support/testing/tests/package/sample_python_crossbar.py b/support/testing/tests/package/sample_python_crossbar.py
index 3695fe92c0..374cccd837 100644
--- a/support/testing/tests/package/sample_python_crossbar.py
+++ b/support/testing/tests/package/sample_python_crossbar.py
@@ -1,3 +1,5 @@
+import os
 import crossbar
 
+os.environ["AUTOBAHN_USE_UMSGPACK"] = "1"
 crossbar.run(["version"])
-- 
2.17.1

[Buildroot] [PATCH 3/4] support/testing: provide entropy to lua tests

[Ricardo Martincoski]

Newer versions of lua-http require entropy.
Switch to use armv5 builtin kernel that already provides entropy for all
lua tests.

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139374
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139376

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Cc: Francois Perrad <francois.perrad@gadz.org>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022884
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022885

full run:
https://gitlab.com/RicardoMartincoski/buildroot/pipelines/75770319
---
 support/testing/tests/package/test_lua.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/testing/tests/package/test_lua.py b/support/testing/tests/package/test_lua.py
index 77358ba138..66ae85c999 100644
--- a/support/testing/tests/package/test_lua.py
+++ b/support/testing/tests/package/test_lua.py
@@ -12,7 +12,7 @@ class TestLuaBase(infra.basetest.BRTest):
 
     def login(self):
         cpio_file = os.path.join(self.builddir, "images", "rootfs.cpio")
-        self.emulator.boot(arch="armv7",
+        self.emulator.boot(arch="armv5",
                            kernel="builtin",
                            options=["-initrd", cpio_file])
         self.emulator.login()
-- 
2.17.1

[Buildroot] [PATCH 4/4] support/testing: provide entropy to perl tests

[Ricardo Martincoski]

Newer versions of perl-io-socket-ssl require entropy.
Switch to use armv5 builtin kernel that already provides entropy for all
perl tests.

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/269139402

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Cc: Francois Perrad <francois.perrad@gadz.org>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022909

full run:
https://gitlab.com/RicardoMartincoski/buildroot/pipelines/75770319
---
 support/testing/tests/package/test_perl.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/testing/tests/package/test_perl.py b/support/testing/tests/package/test_perl.py
index 033b7cf347..37e7390a4b 100644
--- a/support/testing/tests/package/test_perl.py
+++ b/support/testing/tests/package/test_perl.py
@@ -12,7 +12,7 @@ class TestPerlBase(infra.basetest.BRTest):
 
     def login(self):
         cpio_file = os.path.join(self.builddir, "images", "rootfs.cpio")
-        self.emulator.boot(arch="armv7",
+        self.emulator.boot(arch="armv5",
                            kernel="builtin",
                            options=["-initrd", cpio_file])
         self.emulator.login()
-- 
2.17.1

[Buildroot] [PATCH 1/4] support/testing: make treq tests to ignore warning

[Thomas Petazzoni]

Hello Ricardo,

On Sat, 10 Aug 2019 22:35:35 -0300
Ricardo Martincoski <ricardo.martincoski@gmail.com> wrote:

> Newer versions of python-treq display a warning at runtime when
> service_identity is not installed:
> "Without the service_identity module, Twisted can perform only
> rudimentary TLS client hostname verification."
> 
> This warning message confuses the test that looks for another string in
> stdout. Make the test ignore other messages while still expecting
> "Connection refused".
> 
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/269139449
> https://gitlab.com/buildroot.org/buildroot/-/jobs/269139450
> 
> Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Cc: Yegor Yefremov <yegorslists@googlemail.com>
> ---
> https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022950
> https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/270022951
> ---
>  support/testing/tests/package/test_python_treq.py | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Series applied, thanks! Thanks a lot for coming back so quickly with
fixes for those runtime test issues!

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com