Skip to content

Commit

Permalink
Add GitHub actions
Browse files Browse the repository at this point in the history
  • Loading branch information
dantti committed Oct 1, 2023
1 parent 4fbf633 commit 35a6766
Show file tree
Hide file tree
Showing 2 changed files with 107 additions and 3 deletions.
101 changes: 101 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,101 @@
# SPDX-FileCopyrightText: 2023 Klarälvdalens Datakonsult AB, a KDAB Group company <[email protected]>
#
# SPDX-License-Identifier: MIT

name: CI

on:
push:
branches:
- master
pull_request:
branches:
- master

jobs:
build:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false #true
matrix:
os:
- ubuntu-22.04
#- windows-2022
#- macos-12
build_type:
- Debug
#- Release
qt:
- version: "5.15.2"
requested: "5.15"
- version: "6.3.2" # Qt 6.3 is not an LTS version, so '6.3.*' always resolves to '6.3.2'
requested: "6.3.*"
modules: qtshadertools

steps:
- name: Install Qt with options and default aqtversion
uses: jurplel/install-qt-action@v3
with:
aqtversion: null # use whatever the default is
modules: ${{ matrix.qt.modules }}
version: ${{ matrix.qt.requested }}
cache: true

- name: Checkout sources
uses: actions/checkout@v4

- name: Create build directory
run: mkdir build

- name: Install ninja-build tool (must be after Qt due PATH changes)
uses: turtlesec-no/get-ninja@main

- name: Make sure MSVC is found when Ninja generator is in use
if: ${{ runner.os == 'Windows' }}
uses: ilammy/msvc-dev-cmd@v1

- name: Configure project
run: >
cmake -S . -B ./build -G Ninja
-DCMAKE_BUILD_TYPE=${{ matrix.build_type }}
-DGAMMARAY_BUILD_UI=${{ startsWith(matrix.qt.version, '6.') }}
-DGAMMARAY_BUILD_DOCS=${{ matrix.build_type == 'Debug' && runner.os == 'Linux' }}
- name: Build Project
run: cmake --build ./build

- name: Install dependencies on Ubuntu
if: ${{ matrix.build_type == 'Debug' && runner.os == 'Linux' }}
run: |
sudo apt update -qq
sudo apt install -y gdb
- name: Enable gdb attaching
if: ${{ matrix.build_type == 'Debug' && runner.os == 'Linux' }}
run: echo 0 | sudo tee /proc/sys/kernel/yama/ptrace_scope

# Exclude connectiontest-style-filter is flaky
- name: Run tests on Linux Qt5 (offscreen)
if: ${{ matrix.build_type == 'Debug' && runner.os == 'Linux' && startsWith(matrix.qt.version, '5.') }}
run: ctest --test-dir ./build -C ${{ matrix.build_type }} --output-on-failure -E "connectiontest-style-filter"
env:
QT_QPA_PLATFORM: offscreen
QT_QUICK_BACKEND: software

# Exclude qmlsupporttest|bindinginspectortest for Qt6, connectiontest-style-filter is flaky
- name: Run tests on Linux Qt6 (offscreen)
if: ${{ matrix.build_type == 'Debug' && runner.os == 'Linux' && startsWith(matrix.qt.version, '6.') }}
run: ctest --test-dir ./build -C ${{ matrix.build_type }} --output-on-failure -E "qmlsupporttest|bindinginspectortest|connectiontest-style-filter"
env:
QT_QPA_PLATFORM: offscreen
QT_QUICK_BACKEND: software

- name: Run tests on Windown/macOS
if: ${{ matrix.build_type == 'Debug' && runner.os != 'Linux' }}
run: ctest --test-dir ./build -C ${{ matrix.build_type }} --output-on-failure

- name: Read tests log when it fails
uses: andstor/file-reader-action@v1
if: ${{ failure() && matrix.build_type == 'Debug' }}
with:
path: "./build/Testing/Temporary/LastTest.log"
9 changes: 6 additions & 3 deletions tests/launchertest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@

#include <QDebug>
#include <QObject>
#include <QScopeGuard>
#include <QSignalSpy>
#include <QTest>

Expand Down Expand Up @@ -130,6 +131,11 @@ private slots:
static void testAttach()
{
QProcess target;
auto cleanup = qScopeGuard([&target] {
target.kill();
target.waitForFinished();
});

target.setProcessChannelMode(QProcess::ForwardedChannels);
target.start(QLatin1String(TESTBIN_DIR "/minimalcoreapplication"), {}, QProcess::ReadWrite);
QVERIFY(target.waitForStarted());
Expand All @@ -149,9 +155,6 @@ private slots:

spy.wait(30000);
QCOMPARE(spy.count(), 1);

target.kill();
target.waitForFinished();
}
};

Expand Down

0 comments on commit 35a6766

Please sign in to comment.