Merge branch 'tests' into 'master'

Tests

See merge request !25

.gitlab-ci.yml

+shellcheck:
+  image: koalaman/shellcheck-alpine
+  script: 'shellcheck deploy/base/ros_entrypoint.bash scripts/*sh'
+
+# The ignored rules in hadolint checks are
+# DL3006 Always tag the version of an image explicitly
+#   One of the Dockerfiles uses a temporary image which isn't important to tag
+#   and the other uses a tagged image through an ARG, which apparently is not
+#   properly recognized.
+# DL3008 Pin versions in apt-get install
+#   A bit overkill
+dockerfiles:
+  image: hadolint/hadolint:latest-debian
+  script:
+    - RET=0
+    - hadolint --ignore DL3006 --ignore DL3008 deploy/base/Dockerfile || RET=1
+    - hadolint --ignore DL3006 --ignore DL3008 deploy/dev/Dockerfile || RET=1
+    - exit $RET
+
+yamllint:
+  image: sdesbure/yamllint
+  script: "yamllint $(find . \\( -name '*.yml' -o -name '*.yaml' \\))"

.yamllint

+extends: relaxed
+
+rules:
+  line-length: disable

deploy/base/Dockerfile

@@ -2,7 +2,7 @@ FROM industrial_ci_image
 
 RUN rm -rf /root/ici /root/src
 
-RUN apt-get update -qq && apt-get install -y -qq \
+RUN apt-get update -qq && apt-get install -y -qq --no-install-recommends \
   ros-kinetic-rosbash \
   && rm -rf /var/lib/apt/lists/*
 

deploy/base/ros_entrypoint.bash

@@ -2,5 +2,6 @@
 set -e
 
 # setup application environment
+# shellcheck disable=SC1091
 source "/root/catkin_ws/install/setup.bash"
 exec "$@"

deploy/dev/Dockerfile

 ARG APPLICATION_IMAGE
 FROM ${APPLICATION_IMAGE}
 
-RUN apt-get update -qq && apt-get install -y -qq \
+RUN apt-get update -qq && apt-get install -y -qq --no-install-recommends \
   apt-transport-https \
   curl \
-  libasound2
+  libasound2 \
+  && rm -rf /var/lib/apt/lists/*
 
+SHELL ["/bin/bash", "-o", "pipefail", "-c"]
 RUN curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.gpg \
   && mv microsoft.gpg /etc/apt/trusted.gpg.d/microsoft.gpg \
   && sh -c 'echo "deb [arch=amd64] https://packages.microsoft.com/repos/vscode stable main" > /etc/apt/sources.list.d/vscode.list'
 
-RUN apt-get update -qq && apt-get install -y -qq \
-  code
+RUN apt-get update -qq && apt-get install -y -qq --no-install-recommends \
+  code \
+  && rm -rf /var/lib/apt/lists/*
 
 RUN echo "alias code='code --user-data-dir /root'" >> /root/.bashrc
 

rosdistro/rosdep_tecnalia.yaml

@@ -76,10 +76,6 @@ rc_comm_utils:
   ubuntu:
     xenial: [ros-kinetic-rc-comm-utils]
 
-rc_genicam_api:
-  ubuntu:
-    xenial: [ros-kinetic-rc-genicam-api]
-
 rcimage:
   ubuntu:
     xenial: [rcimage]
@@ -112,17 +108,13 @@ rc_dynamics_api:
   ubuntu:
     xenial: [ros-kinetic-rc-dynamics-api]
 
-rc_dynamics_api:
-  ubuntu:
-    xenial: [rc-dynamics-api]
-
 rc_apps_msgs:
   ubuntu:
     xenial: [ros-kinetic-rc-apps-msgs]
 
 rc_genicam_api:
   ubuntu:
-    xenial: [rc-genicam-api]
+    xenial: [ros-kinetic-rc-genicam-api]
 
 rcobjectrendering:
   ubuntu:

scripts/ci_run_entry_points.sh

 #!/bin/bash
 
-echo Looking up and executing user entry scripts in: $1
+echo Looking up and executing user entry scripts in: "$1"
 
-find -L $1 -name ci_entry_script.bash -exec bash -c 'echo Running entry script from $(basename $(dirname {})) ; {} ' \;
+find -L "$1" -name ci_entry_script.bash -exec bash -c 'echo Running entry script from $(basename $(dirname $1)) ; $1 ' _ {} \;
 
 echo Finalized running the user entry scripts

scripts/roslaunch_test_generator.sh

 #!/usr/bin/env bash
 
-for foo in $(find $(pwd) -name "package.xml") ; do
-  package_name=$(basename $(realpath $(dirname $foo)))
-  echo Entering package: $package_name
-  cd $(realpath $(dirname $foo))
-  if [[ -n $(grep -i "roslaunch_add_file_check" CMakeLists.txt) ]] ; then
-    echo Package $package_name already has roslaunch_add_file_check - skipping
+# shellcheck disable=SC2044
+for foo in $(find "$(pwd)" -name "package.xml") ; do
+  package_name=$(basename "$(realpath "$(dirname "$foo")")")
+  echo Entering package: "$package_name"
+  cd "$(realpath "$(dirname "$foo")")" || exit
+  if grep -q -i "roslaunch_add_file_check" CMakeLists.txt; then
+    echo Package "$package_name" already has roslaunch_add_file_check - skipping
   else
     if [[ -n $(find . -type f -name "*.launch") ]] ; then
-      echo Adding roslaunch_add_file_check to $package_name
+      echo Adding roslaunch_add_file_check to "$package_name"
 cat <<EOT >> CMakeLists.txt
 
 ## WARNING: automatically generated code; can be (and probably is) very buggy