Skip to content

Commit

Permalink
tests: add spread suite for ament plugin (#2211)
Browse files Browse the repository at this point in the history
Move the snapd integration test for the Ament plugin into a Spread
suite.

LP: #1783827

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>
  • Loading branch information
Kyle Fazzari committed Aug 13, 2018
1 parent c1b0eaf commit bdaa8ea
Show file tree
Hide file tree
Showing 18 changed files with 31 additions and 51 deletions.
2 changes: 2 additions & 0 deletions spread.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,8 @@ prepare: |
restore: $SETUP_DIR/restore.sh

suites:
tests/spread/plugins/ament/:
summary: tests of snapcraft's Ament plugin
tests/spread/plugins/autotools/:
summary: tests of snapcraft's Autotools plugin
tests/spread/plugins/catkin/:
Expand Down
43 changes: 0 additions & 43 deletions tests/integration/snapd/test_ament_snap.py

This file was deleted.

21 changes: 21 additions & 0 deletions tests/spread/plugins/ament/run/task.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
summary: Build and run a basic ROS2 snap
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 40m
priority: 100 # Run this test early so we're not waiting for it

# The ament plugin only supports 16.04
systems: [ubuntu-16.04*]

environment:
SNAP_DIR: ../snaps/ros2-talker-listener

restore: |
cd "$SNAP_DIR"
snapcraft clean
rm -f ./*.snap
execute: |
cd "$SNAP_DIR"
snapcraft
sudo snap install ros2-talker-listener_*.snap --dangerous
ros2-talker-listener | MATCH "I heard: Hello world"
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: ros2-example
version: '0.1'
name: ros2-talker-listener
version: '1.0'
summary: ROS2 Example
description: |
A ROS2 workspace containing a talker and a listener.
Expand All @@ -8,7 +8,7 @@ grade: stable
confinement: strict

apps:
launch-project:
ros2-talker-listener:
command: launch "$(ros2 pkg prefix listener_py)/share/listener_py/launch/talk_and_listen.py"
plugs: [network]
environment:
Expand Down
2 changes: 1 addition & 1 deletion tests/spread/plugins/catkin/catkin-with-python/task.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
summary: Build a catkin snap that includes a python part
warn-timeout: 10m
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 20m
priority: 100 # Run this test early so we're not waiting for it

Expand Down
2 changes: 1 addition & 1 deletion tests/spread/plugins/catkin/content-sharing/task.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
summary: Build two catkin snaps using content sharing
warn-timeout: 10m
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 20m
priority: 100 # Run this test early so we're not waiting for it

Expand Down
2 changes: 1 addition & 1 deletion tests/spread/plugins/catkin/pip-support/task.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
summary: Build and run catkin snap using pip dependencies
warn-timeout: 10m
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 20m
priority: 100 # Run this test early so we're not waiting for it

Expand Down
2 changes: 1 addition & 1 deletion tests/spread/plugins/catkin/recursive-rosinstall/task.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
summary: Ensure that the catkin plugin can recursively use rosinstall
warn-timeout: 10m
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 20m
priority: 100 # Run this test early so we're not waiting for it

Expand Down
2 changes: 1 addition & 1 deletion tests/spread/plugins/catkin/run/task.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
summary: Build and run a basic catkin snap
warn-timeout: 10m
warn-timeout: 9m # Keep less than 10 minutes so Travis can't timeout
kill-timeout: 20m
priority: 100 # Run this test early so we're not waiting for it

Expand Down

0 comments on commit bdaa8ea

Please sign in to comment.