Skip to contentSkip to conversation

test/py: don't import pexpect

PATCH New Stephen Warren 5 participants 8 messages Feb 9, 2016 – Feb 15, 2016

lore
Share & tools
lore

Stephen Warren

author
#0 lore raw

PATCH 1/2 test/py: don't import pexpect

From: Stephen Warren <[email protected]>

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 <[email protected]>
---
 test/py/conftest.py | 1 -
 1 file changed, 1 deletion(-)
test/py/conftest.py +0 -1
index 7b97cbaa0eed..1ec096e24182 100644
@@ -17,7 +17,6 @@ import atexit
 import errno
 import os
 import os.path
-import pexpect
 import pytest
 from _pytest.runner import runtestprotocol
 import ConfigParser
Signature · 1 line
2.7.0
author
#1 lore raw

PATCH 2/2 Enable test/py for sandbox in Travis CI

From: Stephen Warren <[email protected]>

This provides runtime test coverage in Travis, in addition to the existing build coverage.

Signed-off-by Stephen Warren <[email protected]>
---
 .travis.yml | 8 ++++++++
 1 file changed, 8 insertions(+)
.travis.yml +8 -0
index 8caaeb34553e..18bf2ed4fcdb 100644
@@ -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
Signature · 1 line
2.7.0
#2 lore raw

PATCH 2/2 Enable test/py for sandbox in Travis CI

Thanks Stephen

Quoted 54 lines

-----Original Message-----
From: Stephen Warren [mailto:swarren at wwwdotorg.org]
Sent: Dienstag, 9. Februar 2016 02:24
To: Tom Rini <[email protected]>
Cc: Simon Glass <[email protected]>; u-boot at lists.denx.de; Meier, Roger <[email protected]>; Daniel Schwierzeck <[email protected]>; Heiko Schocher <[email protected]>; Stephen Warren <[email protected]>
Subject: [PATCH 2/2] Enable test/py for sandbox in Travis CI

From: Stephen Warren <[email protected]>

This provides runtime test coverage in Travis, in addition to the existing build coverage.

Signed-off-by Stephen Warren <[email protected]>
---
 .travis.yml | 8 ++++++++
 1 file changed, 8 insertions(+)
.travis.yml +2 -0
index 8caaeb34553e..18bf2ed4fcdb 100644
@@ -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:

.travis.yml +3 -1
@@ -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
--
Quoted 1 line

2.7.0

Reviewed-by Roger Meier <[email protected]>

Best Regards
Roger

#3 lore raw

PATCH 2/2 Enable test/py for sandbox in Travis CI

In reply to Stephen Warren

Hello Stephen,

Am 09.02.2016 um 02:23 schrieb Stephen Warren:

Quoted 9 lines

From: Stephen Warren <[email protected]>

This provides runtime test coverage in Travis, in addition to the existing build coverage.

Signed-off-by Stephen Warren <[email protected]>
---
  .travis.yml | 8 ++++++++
  1 file changed, 8 insertions(+)

Very nice, thanks!

Reviewed-by Heiko Schocher <[email protected]>

bye,
Heiko

Quoted 34 lines
.travis.yml +8 -0
index 8caaeb34553e..18bf2ed4fcdb 100644
@@ -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
Signature · 2 lines
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#4 lore raw

PATCH 2/2 Enable test/py for sandbox in Travis CI

Quoted 55 lines · Heiko Schocher

Hello Stephen,

Am 09.02.2016 um 02:23 schrieb Stephen Warren:

Quoted 10 lines

From: Stephen Warren <[email protected]>

This provides runtime test coverage in Travis, in addition to the existing build coverage.

Signed-off-by Stephen Warren <[email protected]>
---
  .travis.yml | 8 ++++++++
  1 file changed, 8 insertions(+)

Very nice, thanks!

Reviewed-by Heiko Schocher <[email protected]>

bye,
Heiko

Quoted 33 lines
.travis.yml +2 -0
index 8caaeb34553e..18bf2ed4fcdb 100644
@@ -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:

.travis.yml +3 -0
@@ -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"
Quoted 2 lines

.travis.yml matrix: · line 189

Quoted 2 lines
.travis.yml +0 -0
  # TODO make it perfect ;-r
Acked-by Simon Glass <[email protected]>
#5 lore raw

PATCH 2/2 Enable test/py for sandbox in Travis CI

In reply to Stephen Warren
Quoted 9 lines · Stephen Warren

From: Stephen Warren <[email protected]>

This provides runtime test coverage in Travis, in addition to the existing build coverage.

Signed-off-by Stephen Warren <[email protected]>
Reviewed-by Roger Meier <[email protected]>
Reviewed-by Heiko Schocher <[email protected]>
Acked-by Simon Glass <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160215/7b897df0/attachment.sig>
#7 lore raw

PATCH 1/2 test/py: don't import pexpect

In reply to Stephen Warren
Quoted 7 lines · Stephen Warren

From: Stephen Warren <[email protected]>

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 <[email protected]>
Acked-by Simon Glass <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160215/79ac0506/attachment.sig>