[U-Boot] [PATCH 1/2] test/py: don't import pexpect
From: Stephen Warren swarren@nvidia.com
The code replaced pexpect with custom code long ago. Don't import the unused module so it doesn't need to be installed.
Signed-off-by: Stephen Warren swarren@nvidia.com --- test/py/conftest.py | 1 - 1 file changed, 1 deletion(-)
diff --git a/test/py/conftest.py b/test/py/conftest.py index 7b97cbaa0eed..1ec096e24182 100644 --- a/test/py/conftest.py +++ b/test/py/conftest.py @@ -17,7 +17,6 @@ import atexit import errno import os import os.path -import pexpect import pytest from _pytest.runner import runtestprotocol import ConfigParser
From: Stephen Warren swarren@nvidia.com
This provides runtime test coverage in Travis, in addition to the existing build coverage.
Signed-off-by: Stephen Warren swarren@nvidia.com --- .travis.yml | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/.travis.yml b/.travis.yml index 8caaeb34553e..18bf2ed4fcdb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,6 +16,8 @@ addons: - bc - build-essential - libsdl1.2-dev + - python + - python-virtualenv
cache: - apt @@ -38,6 +40,9 @@ install: - echo -e "\n\n[toolchain-alias]\n${BUILDMAN_ALIAS} i386\n" >> ~/.buildman - echo -e "${BUILDMAN_ALIAS_ARM} armv5te\n" >> ~/.buildman - cat ~/.buildman + - virtualenv /tmp/venv + - . /tmp/venv/bin/activate + - pip install pytest
env: global: @@ -177,5 +182,8 @@ matrix: # some statistics about the code base - env: - TEST_CMD="sloccount ." + # test/py + - env: + - TEST_CMD="./test/py/test.py --bd sandbox --build"
# TODO make it perfect ;-r
Thanks Stephen
-----Original Message----- From: Stephen Warren [mailto:swarren@wwwdotorg.org] Sent: Dienstag, 9. Februar 2016 02:24 To: Tom Rini trini@konsulko.com Cc: Simon Glass sjg@chromium.org; u-boot@lists.denx.de; Meier, Roger r.meier@siemens.com; Daniel Schwierzeck daniel.schwierzeck@gmail.com; Heiko Schocher hs@denx.de; Stephen Warren swarren@nvidia.com Subject: [PATCH 2/2] Enable test/py for sandbox in Travis CI
From: Stephen Warren swarren@nvidia.com
This provides runtime test coverage in Travis, in addition to the existing build coverage.
Signed-off-by: Stephen Warren swarren@nvidia.com
.travis.yml | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/.travis.yml b/.travis.yml index 8caaeb34553e..18bf2ed4fcdb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,6 +16,8 @@ addons: - bc - build-essential - libsdl1.2-dev
- python
- python-virtualenv
cache:
- apt
@@ -38,6 +40,9 @@ install:
- echo -e "\n\n[toolchain-alias]\n${BUILDMAN_ALIAS} i386\n" >>
~/.buildman
- echo -e "${BUILDMAN_ALIAS_ARM} armv5te\n" >> ~/.buildman
- cat ~/.buildman
- virtualenv /tmp/venv
- . /tmp/venv/bin/activate
- pip install pytest
env: global: @@ -177,5 +182,8 @@ matrix: # some statistics about the code base - env: - TEST_CMD="sloccount ."
- # test/py
- env:
- TEST_CMD="./test/py/test.py --bd sandbox --build"# TODO make it perfect ;-r
2.7.0
Reviewed-by: Roger Meier r.meier@siemens.com
Best Regards Roger
Hello Stephen,
Am 09.02.2016 um 02:23 schrieb Stephen Warren:
From: Stephen Warren swarren@nvidia.com
This provides runtime test coverage in Travis, in addition to the existing build coverage.
Signed-off-by: Stephen Warren swarren@nvidia.com
.travis.yml | 8 ++++++++ 1 file changed, 8 insertions(+)
Very nice, thanks!
Reviewed-by: Heiko Schocher hs@denx.de
bye, Heiko
diff --git a/.travis.yml b/.travis.yml index 8caaeb34553e..18bf2ed4fcdb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,6 +16,8 @@ addons: - bc - build-essential - libsdl1.2-dev
- python
- python-virtualenv
cache:
- apt
@@ -38,6 +40,9 @@ install:
- echo -e "\n\n[toolchain-alias]\n${BUILDMAN_ALIAS} i386\n" >> ~/.buildman
- echo -e "${BUILDMAN_ALIAS_ARM} armv5te\n" >> ~/.buildman
- cat ~/.buildman
- virtualenv /tmp/venv
- . /tmp/venv/bin/activate
- pip install pytest
env: global:
@@ -177,5 +182,8 @@ matrix: # some statistics about the code base - env: - TEST_CMD="sloccount ."
# test/py
- env:
- TEST_CMD="./test/py/test.py --bd sandbox --build"# TODO make it perfect ;-r
On 10 February 2016 at 22:41, Heiko Schocher hs@denx.de wrote:
Hello Stephen,
Am 09.02.2016 um 02:23 schrieb Stephen Warren:
From: Stephen Warren swarren@nvidia.com
This provides runtime test coverage in Travis, in addition to the existing build coverage.
Signed-off-by: Stephen Warren swarren@nvidia.com
.travis.yml | 8 ++++++++ 1 file changed, 8 insertions(+)
Very nice, thanks!
Reviewed-by: Heiko Schocher hs@denx.de
bye, Heiko
diff --git a/.travis.yml b/.travis.yml index 8caaeb34553e..18bf2ed4fcdb 100644 --- a/.travis.yml +++ b/.travis.yml @@ -16,6 +16,8 @@ addons: - bc - build-essential - libsdl1.2-dev
- python
- python-virtualenv
cache:
- apt
@@ -38,6 +40,9 @@ install:
- echo -e "\n\n[toolchain-alias]\n${BUILDMAN_ALIAS} i386\n" >>
~/.buildman
- echo -e "${BUILDMAN_ALIAS_ARM} armv5te\n" >> ~/.buildman
- cat ~/.buildman
- virtualenv /tmp/venv
- . /tmp/venv/bin/activate
- pip install pytest
env: global:
@@ -177,5 +182,8 @@ matrix: # some statistics about the code base - env: - TEST_CMD="sloccount ."
# test/py
- env:
- TEST_CMD="./test/py/test.py --bd sandbox --build"# TODO make it perfect ;-r
Acked-by: Simon Glass sjg@chromium.org
On Mon, Feb 08, 2016 at 06:23:35PM -0700, Stephen Warren wrote:
From: Stephen Warren swarren@nvidia.com
This provides runtime test coverage in Travis, in addition to the existing build coverage.
Signed-off-by: Stephen Warren swarren@nvidia.com Reviewed-by: Roger Meier r.meier@siemens.com Reviewed-by: Heiko Schocher hs@denx.de Acked-by: Simon Glass sjg@chromium.org
Applied to u-boot/master, thanks!
On 8 February 2016 at 18:23, Stephen Warren swarren@wwwdotorg.org wrote:
From: Stephen Warren swarren@nvidia.com
The code replaced pexpect with custom code long ago. Don't import the unused module so it doesn't need to be installed.
Signed-off-by: Stephen Warren swarren@nvidia.com
test/py/conftest.py | 1 - 1 file changed, 1 deletion(-)
Acked-by: Simon Glass sjg@chromium.org
On Mon, Feb 08, 2016 at 06:23:34PM -0700, Stephen Warren wrote:
From: Stephen Warren swarren@nvidia.com
The code replaced pexpect with custom code long ago. Don't import the unused module so it doesn't need to be installed.
Signed-off-by: Stephen Warren swarren@nvidia.com Acked-by: Simon Glass sjg@chromium.org
Applied to u-boot/master, thanks!
participants (5)
-
Heiko Schocher -
Meier, Roger -
Simon Glass -
Stephen Warren -
Tom Rini