[Fuego] [PATCH 0/8] Fuego-Release-Test fixes

Tim.Bird at sony.com Tim.Bird at sony.com
Wed Apr 4 17:58:04 UTC 2018



> -----Original Message-----
> From: Guilherme Campos Camargo
> Hello, everyone.
> 
> This series of patches implements Tim's suggestions on top of the
> original patches that have been sent last week and integrated into
> fuego-core/master.

Thanks.  I did a very brief review, and things looked OK.

All patches applied and pushed.  I did have one question, which I'll ask on a related
patch.
 -- Tim

> Guilherme Campos Camargo (8):
>   Rename Functional.fuegotest to Functional.fuego_release_test
>   Add underscores to variables in spec.json
>   Add specs for 'test' and 'next' branches
>   Replace single-quote with double-quotes on long strings
>   Use '+' on long wrapped strings to show contatenation explicitly
>   Avoid reordering conditional logic to improve readability
>   Keep the output of the test in TAP format
>   Use atexit to remove the docker container after the test
> 
>  .../Functional.fuego_release_test/fuego_test.sh    |  35 +++++
>  .../tests/Functional.fuego_release_test/spec.json  |  23 ++++
>  .../test_run.py                                    | 146 +++++++++++----------
>  engine/tests/Functional.fuegotest/fuego_test.sh    |  36 -----
>  engine/tests/Functional.fuegotest/spec.json        |  11 --
>  5 files changed, 136 insertions(+), 115 deletions(-)
>  create mode 100755
> engine/tests/Functional.fuego_release_test/fuego_test.sh
>  create mode 100644 engine/tests/Functional.fuego_release_test/spec.json
>  rename engine/tests/{Functional.fuegotest =>
> Functional.fuego_release_test}/test_run.py (78%)
>  delete mode 100755 engine/tests/Functional.fuegotest/fuego_test.sh
>  delete mode 100644 engine/tests/Functional.fuegotest/spec.json
> 
> --
> 2.16.3
> 
> _______________________________________________
> Fuego mailing list
> Fuego at lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego


More information about the Fuego mailing list