Skip to content

Commit

Permalink
Add MATLAB Tests CI
Browse files Browse the repository at this point in the history
  • Loading branch information
traversaro committed Mar 1, 2022
1 parent 446c845 commit a6f6ce9
Show file tree
Hide file tree
Showing 3 changed files with 149 additions and 1 deletion.
110 changes: 110 additions & 0 deletions .github/workflows/matlab.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,110 @@
name: MATLAB Tests CI

on:
push:
branches: [ main ]
pull_request:
branches: [ main ]
workflow_dispatch:
schedule:
# * is a special character in YAML so you have to quote this string
# Execute a "nightly" build at 2 AM UTC
- cron: '0 2 * * *'

jobs:
build-matlab-tests:
name: '[matlab:${{ matrix.matlab_version }}:${{ matrix.os }}]'
runs-on: ${{ matrix.os }}
defaults:
run:
shell: bash -l {0}
strategy:
fail-fast: false
matrix:
build_type: [Release]
os: [ubuntu-20.04, windows-2019, macos-10.15]
matlab_version: [R2020a, R2020b, R2021a, latest]
exclude:
# R2020* is not supported on Windows on GitHub Actions
- os: windows-2019
matlab_version: R2020a
build_type: Release
- os: windows-2019
matlab_version: R2020b
build_type: Release

steps:
- uses: actions/checkout@v2

- uses: conda-incubator/setup-miniconda@v2
with:
miniforge-variant: Mambaforge
miniforge-version: latest
channels: conda-forge,robotology

- name: Setup MATLAB
uses: matlab-actions/setup-matlab@v1
with:
release: ${{ matrix.matlab_version }}

# workaround for https://github.com/robotology/robotology-superbuild/issues/64
- name: Do not use MATLAB's stdc++ to avoid incompatibilities with other libraries
if: contains(matrix.os, 'ubuntu')
run:
echo 'LD_PRELOAD=/usr/lib/x86_64-linux-gnu/libstdc++.so.6' >> $GITHUB_ENV

- name: Dependencies
run: |
# Workaround for https://github.com/conda-incubator/setup-miniconda/issues/186
conda config --remove channels defaults
# Compilation related dependencies
mamba install cmake compilers make ninja pkg-config
# Actual dependencies
mamba install osqp
# Just a trick to make sure that the relevant installation path are on MATLABPATH
mamba install -c conda-forge -c robotology idyntree-matlab-bindings
# Additional dependencies useful only on Windows
- name: Dependencies [Conda/Windows]
if: contains(matrix.os, 'windows')
run: |
# Additional dependencies only useful on Windows
# See https://github.com/robotology/robotology-superbuild/issues/477
mamba install vs2019_win-64
- name: Print used environment [Conda]
shell: bash -l {0}
run: |
mamba list
env
- name: Configure [Conda - Linux or macOS]
if: contains(matrix.os, 'macos') || contains(matrix.os, 'ubuntu')
run: |
mkdir build
cd build
cmake -GNinja -DCMAKE_INSTALL_PREFIX=${CONDA_PREFIX} -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -DBUILD_TESTING:BOOL=ON ..
- name: Configure [Conda - Windows]
if: contains(matrix.os, 'windows')
run: |
mkdir build
cd build
cmake -G"Visual Studio 16 2019" -DCMAKE_INSTALL_PREFIX=${CONDA_PREFIX}/Library -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -DBUILD_TESTING:BOOL=ON ..
- name: Build
run: |
cd build
cmake --build . --config ${{ matrix.build_type }}
- name: Install
run: |
cd build
cmake --install . --config ${{ matrix.build_type }}
- name: Test
run: |
cd build
ctest --output-on-failure -C ${{ matrix.build_type }} -VV .
18 changes: 17 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ option(BUILD_SHARED_LIBS "Build libraries as shared as opposed to static" ON)
option(OSQP_MATLAB_USES_MATLAB "Do you want to create the MATLAB bindings" ON)
option(OSQP_MATLAB_USES_OCTAVE "Do you want to create the Octave bindings" OFF)

option(BUILD_TESTING "Create tests using CMake" OFF)
if(BUILD_TESTING)
enable_testing()
endif()

set(OSQP_MATLAB_INSTALL_MATLAB_LIBDIR "mex" CACHE
STRING "Location (relative to the install prefix) in which the Matlab mex libraries are installed.")
set(OSQP_MATLAB_INSTALL_MATLAB_MFILESDIR "mex" CACHE
Expand Down Expand Up @@ -73,7 +78,8 @@ set(M_FILES ${osqp-matlab_SOURCE_DIR}/osqp.m)
set(MEX_FILES ${osqp-matlab_SOURCE_DIR}/osqp_mex.hpp ${osqp-matlab_SOURCE_DIR}/osqp_mex.cpp)

if(OSQP_MATLAB_USES_MATLAB)
find_package(Matlab REQUIRED)
find_package(Matlab REQUIRED
COMPONENTS MAIN_PROGRAM)
matlab_add_mex(
NAME osqp_mex_matlab
OUTPUT_NAME osqp_mex
Expand All @@ -89,6 +95,15 @@ if(OSQP_MATLAB_USES_MATLAB)
install(
FILES ${M_FILES}
DESTINATION ${OSQP_MATLAB_INSTALL_MATLAB_MFILESDIR})

# Enable tests
if (BUILD_TESTING)
configure_file(${CMAKE_CURRENT_SOURCE_DIR}/run_osqp_tests_no_codegen.m.in ${CMAKE_CURRENT_BINARY_DIR}/run_osqp_tests_no_codegen.m @ONLY)
# Note: this only works if the project was installed in the correct location and can be found with MATLABPATH
add_test(NAME matlab_osqp_tests
COMMAND ${Matlab_MAIN_PROGRAM} -nodisplay -nodesktop -nojvm -batch "addpath('${CMAKE_CURRENT_BINARY_DIR}');run_osqp_tests_no_codegen;")

endif()
endif()

if(OSQP_MATLAB_USES_OCTAVE)
Expand Down Expand Up @@ -116,4 +131,5 @@ if(OSQP_MATLAB_USES_OCTAVE)
DESTINATION ${OSQP_MATLAB_INSTALL_OCTAVE_MFILESDIR})
endif()


include(AddUninstallTarget)
22 changes: 22 additions & 0 deletions run_osqp_tests_no_codegen.m.in
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
import matlab.unittest.TestSuite;
import matlab.unittest.selectors.HasSharedTestFixture;
import matlab.unittest.selectors.HasName;
import matlab.unittest.fixtures.PathFixture;
import matlab.unittest.constraints.EndsWithSubstring;
import matlab.unittest.constraints.ContainsSubstring;

osqp_src_path = "@osqp-matlab_SOURCE_DIR@";
unittest_dir = fullfile(osqp_src_path, 'unittests');
suiteFolder = TestSuite.fromFolder(unittest_dir);

% Exclude tests that contain codegen in the name
suiteNoCodegen = selectIf(suiteFolder,~HasName(ContainsSubstring('codegen','IgnoringCase',true)));

% Run all suite
result = run(suiteNoCodegen);

% Print test results
disp(table(result))

% Return error code if test failed
assertSuccess(result)

0 comments on commit a6f6ce9

Please sign in to comment.