Merge lp:~nskaggs/reminders-app/run-qml-tests-on-build into lp:reminders-app

Proposed by Nicholas Skaggs
Status: Rejected
Rejected by: Nicholas Skaggs
Proposed branch: lp:~nskaggs/reminders-app/run-qml-tests-on-build
Merge into: lp:reminders-app
Diff against target: 12 lines (+5/-0)
1 file modified
tests/qml/CMakeLists.txt (+5/-0)
To merge this branch: bzr merge lp:~nskaggs/reminders-app/run-qml-tests-on-build
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Disapprove
Review via email: mp+289063@code.launchpad.net

Commit message

Always run qml tests when building

Description of the change

Let's make sure qml tests run whenever we build.

To post a comment you must log in.
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Let's make this explicit in jenkins instead

review: Disapprove

Unmerged revisions

549. By Nicholas Skaggs

force qml tests to run on build

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/qml/CMakeLists.txt'
2--- tests/qml/CMakeLists.txt 2015-03-19 23:01:15 +0000
3+++ tests/qml/CMakeLists.txt 2016-03-15 15:44:15 +0000
4@@ -46,3 +46,8 @@
5 endif()
6
7 add_custom_target(tst_QmlFiles ALL SOURCES ${QML_TST_FILES})
8+add_custom_command(
9+ TARGET tst_QmlFiles
10+ POST_BUILD
11+ COMMAND ctest --output-on-failure
12+)

Subscribers

People subscribed via source and target branches