scripts/unit-test: Support running tests only

By default we will still run all of the checks so that our CI coverage
is still sane. However, this will make it possible for developers to
trivially ru

scripts/unit-test: Support running tests only

By default we will still run all of the checks so that our CI coverage
is still sane. However, this will make it possible for developers to
trivially run just the test cases of the project for faster turnaround
time for developing. Just set TEST_ONLY=1 in your environment to skip
all of the slow checks like valgrind / coverage / clang-tidy.

Tested:
Ran under an autotools build and meson build to verify that the
behavior is still as expected.

Change-Id: I0b420e1c3e779863b90288ce72fc056400226734
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

Support branch option in unit-test script

Building on the support of the branch option within the
docker container, add it to unit-test for any dependent
repositories that do not get built into the

Support branch option in unit-test script

Building on the support of the branch option within the
docker container, add it to unit-test for any dependent
repositories that do not get built into the docker
container

Testing:
- Verified with no branch input, worked as expected
- Verified with invalid branch, master was still used

Change-Id: I4f57d6165287ffe3b4e2f0377292128474199ca3
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Improve documentation for run-unit-test-docker.sh

Add a bit more detail to what this script does and which
parameters are required

Change-Id: Iebfb5408ff6da9c826cb5c57735b099ec9d55678
Signed-off-by

Improve documentation for run-unit-test-docker.sh

Add a bit more detail to what this script does and which
parameters are required

Change-Id: Iebfb5408ff6da9c826cb5c57735b099ec9d55678
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Clarify required parameters

UNIT_TEST_PKG and WORKSPACE are required so document this
and don't make a temporary directory if WORKSPACE is missing
because later steps just fail anyway.

Change-Id: I

Clarify required parameters

UNIT_TEST_PKG and WORKSPACE are required so document this
and don't make a temporary directory if WORKSPACE is missing
because later steps just fail anyway.

Change-Id: Ia3ff6a726b553d7ea7ce65dc1687c595565a094a
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Support branch input to unit test docker script

With the upcoming OpenBMC release, it will be the first time
an official branch is done to all of the repos. The CI
infrastructure needs to support bu

Support branch input to unit test docker script

With the upcoming OpenBMC release, it will be the first time
an official branch is done to all of the repos. The CI
infrastructure needs to support building against this
new branch (as well as master).

Testing:
- Verified with no branch input it behaves as it always did
- Verified with a non-existent branch it behaves as it always did
- Verified with a valid branch in a single repo that it used it
and defaulted to master for the rest

Change-Id: Ic71153483eb407f596afea1b748387b23bbc81fd
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

unit-test: Support running as non-root

This is nice to have so that all of the generated files in our workspace
are owned by our user so we don't have to deal with ownership issues
after the build h

unit-test: Support running as non-root

This is nice to have so that all of the generated files in our workspace
are owned by our user so we don't have to deal with ownership issues
after the build has completed.

Tested:
Built all of the openbmc packages locally and found no regressions.

Change-Id: I0811ac4e356a023a1238e291f8184dfe01eb003e
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

run-unit-test-docker: Allow overriding workdir

There are some cases where we want to be able to override the location
of the build workspace inside the container and decouple the location
from the h

run-unit-test-docker: Allow overriding workdir

There are some cases where we want to be able to override the location
of the build workspace inside the container and decouple the location
from the host location. This preserves the current behavior if
DOCKER_WORKDIR is not set.

Change-Id: I885f132e9da06ce9f3c1ddd9dfc7c69b9fb08cd4
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

unit-test CI: switch to clang-format-6.0

clang-format-5.0 doesn't support the IncludeBlocks configuration option.

Change-Id: I5668c46e502b64f3302754a29383f3137e321014
Signed-off-by: Patrick Venture

unit-test CI: switch to clang-format-6.0

clang-format-5.0 doesn't support the IncludeBlocks configuration option.

Change-Id: I5668c46e502b64f3302754a29383f3137e321014
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

Add Cmake Support

Added initial cmake suport to unit-test.py. Used
bmcweb as repo to test with so added missing dependencies
of boost and xxd as well.

Tested-by: Built and ran unit tests for bmcweb

Add Cmake Support

Added initial cmake suport to unit-test.py. Used
bmcweb as repo to test with so added missing dependencies
of boost and xxd as well.

Tested-by: Built and ran unit tests for bmcweb

Change-Id: If06b8484a96b9a711ef6aa42fc63138138077c30
Signed-off-by: James Feist <james.feist@linux.intel.com>

show more ...

run-unit-test-docker.sh: Pass MAKEFLAGS into container

This allows us to set options like "-j$(nproc)" or "V=1" as required.

Tested: Exported MAKEFLAGS and ran run-unit-test-docker.sh, confirmed
ve

run-unit-test-docker.sh: Pass MAKEFLAGS into container

This allows us to set options like "-j$(nproc)" or "V=1" as required.

Tested: Exported MAKEFLAGS and ran run-unit-test-docker.sh, confirmed
verbose output, concurrent jobs executed.

Change-Id: Ib640c22b034cdf673a1902e91f8a47395033fc98
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

run-unit-test-docker.sh: Update to ubuntu:artful

Move with the times and also avoid C++ standard library bugs that caused
patches to phosphor-mboxd to fail CI[0].

[0] https://github.com/openbmc/ope

run-unit-test-docker.sh: Update to ubuntu:artful

Move with the times and also avoid C++ standard library bugs that caused
patches to phosphor-mboxd to fail CI[0].

[0] https://github.com/openbmc/openbmc/issues/2986

Closes openbmc/openbmc#2986

Tested: Ran `DISTRO=ubuntu:artful ./test-distro.sh` with a set of
patches against repositories that were failing tests. The patches are
now in Gerrit. All tests passed.

Change-Id: I181d5f5e0fec7c070cd06cc60a54134e2ed1c877
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

run-unit-test-docker.sh: Don't force ubuntu:latest

Allow ubuntu docker images other than the 'latest' tag to run. 'latest'
points to Ubuntu 16.04 on Docker Hub[0], which isn't really what we're
afte

run-unit-test-docker.sh: Don't force ubuntu:latest

Allow ubuntu docker images other than the 'latest' tag to run. 'latest'
points to Ubuntu 16.04 on Docker Hub[0], which isn't really what we're
after.

This leaves in place forcing Fedora to be Ubuntu, but that should
probably change as well.

[0] https://hub.docker.com/_/ubuntu/

Tested: Ran `DISTRO=ubuntu:artful ./run-unit-test-docker.sh` and
verified the environment was Artful.

Change-Id: Ibdcab220db7388e0f1153337cbde01fc4b898883
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

run-unit-test-docker.sh: Use mktemp for temporary directories

Don't try to hand-roll the equivalent.

Tested: Ran /bin/bash -n on the script, parsed without failure.

Change-Id: Idb5bcd1300b4158e1cc

run-unit-test-docker.sh: Use mktemp for temporary directories

Don't try to hand-roll the equivalent.

Tested: Ran /bin/bash -n on the script, parsed without failure.

Change-Id: Idb5bcd1300b4158e1cc22964003792f9658de47e
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

run-unit-test-docker.sh: Make capitalisation of DISTRO consistent

Previously the script required $distro be set in the environment, but
used $DISTRO internally. This is confusing, so require the cap

run-unit-test-docker.sh: Make capitalisation of DISTRO consistent

Previously the script required $distro be set in the environment, but
used $DISTRO internally. This is confusing, so require the capitalised
version always.

Hopefully this won't break any external scripts.

Tested: Ran /bin/bash -n on the script, parsed without failure.

Change-Id: I64eea6638c3a9a28138495f7b849373486323d6f
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

Validate code formatting when CI job runs

This will execute the format-code.sh script from
within the docker container that runs the per repo
CI job

Change-Id: I50321ddad1d664fcae0f168ba973ab23cf3c

Validate code formatting when CI job runs

This will execute the format-code.sh script from
within the docker container that runs the per repo
CI job

Change-Id: I50321ddad1d664fcae0f168ba973ab23cf3c6f32
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Launch dbus session for docker unit tests

Fix for openbmc/openbmc#1137

Change-Id: I9e8a41820c6b85bff3625af4ab7b488a66b39c65
Signed-off-by: Leonel Gonzalez <lgonzalez@us.ibm.com>

Complete remaining TODOs in unit test script

Updated docker image unit test script, fixing the following TODOs:
- Replace os.system calls with subprocess.check_call
- Support possible configure.ac l

Complete remaining TODOs in unit test script

Updated docker image unit test script, fixing the following TODOs:
- Replace os.system calls with subprocess.check_call
- Support possible configure.ac line breaks ('\')
- Updated 'make check' call fails Jenkins

Also added helpful command prints for Jenkins console

Change-Id: I6ba3986165b8d313228bcaa611df17bd213447eb
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...

Unit test script that runs within a docker image

Recursively determine dependencies per package starting from the package
to be unit tested. This will then download, build, and install those
depende

Unit test script that runs within a docker image

Recursively determine dependencies per package starting from the package
to be unit tested. This will then download, build, and install those
dependencies returning up the call stack as they are installed with the
package to be unit tested last. A `make check` is issued to execute the
unit tests for that package after its installed.

Change-Id: I194ab0be2408dc0755e75029c7d1f54c5afcdba1
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...

Create run unit test docker script

Wrapper script for Jenkins that does everything necessary to build,
deploy, and run a docker image for unit tests.

Change-Id: I1436b84d078659c928804bb41aa0772176f

Create run unit test docker script

Wrapper script for Jenkins that does everything necessary to build,
deploy, and run a docker image for unit tests.

Change-Id: I1436b84d078659c928804bb41aa0772176f7f64e
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

show more ...