Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[travis] Only send test results to CDash on failure
  • Loading branch information
m-kuhn committed Sep 14, 2015
1 parent 9bd4276 commit 343cc54
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions qgis-test-travis.ctest
Expand Up @@ -71,7 +71,5 @@ IF(NOT ${TESTRES} EQUAL 0)
MESSAGE( FATAL_ERROR "Tests failed" )
ENDIF(NOT ${TESTRES} EQUAL 0)

ctest_submit (RETRY_COUNT 3 RETRY_DELAY 30)
MESSAGE("${Green}Test results submitted to${ColorReset}")
MESSAGE("${BoldGreen}http://dash.orfeo-toolbox.org/index.php?project=QGIS&filtercount=1&showfilters=1&field1=buildname/string&compare1=63&value1=$ENV{TRAVIS_COMMIT}${ColorReset}" )

MESSAGE("${BoldGreen}Success${ColorReset}")
MESSAGE("${Green}All tests passed successfully.${ColorReset}")

8 comments on commit 343cc54

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-kuhn any chance of reverting this? I find it handy to see the results from passing tests too...

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on 343cc54 Sep 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big chances if you give good arguments :)

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on 343cc54 Sep 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acceptable doc coverage: How about only send it for master builds but not for pull requests?

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on 343cc54 Sep 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or we print some important statistics and information directly to the travis output

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-kuhn if that's possible... Sure! Or the automated bot thing, if you want to do that ;)

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on 343cc54 Sep 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-kuhn
Copy link
Member Author

@m-kuhn m-kuhn commented on 343cc54 Sep 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will be shown if after_script is uncollapsed. Further information which should be shown can be appended to the file /tmp/cdash-important.log or from python test scripts by using printImportant(info) from utilities.py.

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-kuhn OK, sounds good to me

Please sign in to comment.