/snapcraft

Remove the unittests from the autopkgtest execution #600

Merged
merged 3 commits into from Jun 26, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@elopio @sergiusens @kyrofa
@elopio
Member

elopio commented Jun 24, 2016

The tests were being run twice, once when building the package and
then on the autopkg execution.

LP: #1596068
@elopio
Remove the unittests from the autopkgtest execution 
The tests were being run twice, once when building the package and
then on the autopkg execution.

LP: #1596068

All checks have passed

3 successful checks
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
continuous-integration/travis-ci/push — The Travis CI build passed
Details
coverage/coveralls — First build on bug/1596068/no_unit_autopkg at 96.175%
Details
09984db
@sergiusens
Collaborator

sergiusens commented Jun 24, 2016

I will wait for autopackage to work again before taking this one in ;-)
@elopio
Member

elopio commented Jun 24, 2016

what could possibly go wrong? ;)
it's almost ready.
@kyrofa
Member

kyrofa commented Jun 24, 2016 •
Edited 1 time

I seem to remember someone saying "30 minutes" about 5 hours ago or so ;) .
@elopio
Member

elopio commented Jun 24, 2016

http://devopsreactions.tumblr.com/post/139593622302/so-all-your-estimates-were-random
@kyrofa
Member

kyrofa commented Jun 24, 2016

😛

elopio added some commits Jun 24, 2016

@elopio
Merge branch 'master' into bug/1596068/no_unit_autopkg

Some checks were not successful

2 failing and 2 successful checks
@snappy-m-o
Examples tests
Details
@snappy-m-o
autopkgtest
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage remained the same at 96.186%
Details
6f712ea
@elopio
Merge branch 'master' into bug/1596068/no_unit_autopkg

All checks have passed

4 successful checks
@snappy-m-o
Examples tests — Success
Details
@snappy-m-o
autopkgtest — Success
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage remained the same at 96.207%
Details
3d45ce8
@sergiusens
Collaborator

sergiusens commented Jun 25, 2016

@elopio how about an update branch here? :-)
@elopio
Member

elopio commented Jun 25, 2016

retest this please
@elopio
Member

elopio commented Jun 26, 2016

All green. It took so long to execute because of https://bugs.launchpad.net/snappy/+bug/1596243

@sergiusens sergiusens merged commit 8a4d204 into snapcore:master Jun 26, 2016
4 checks passed

4 checks passed

Examples tests Success
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 96.207%
Details

@elopio elopio deleted the elopio:bug/1596068/no_unit_autopkg branch Jan 30, 2017

kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017

@elopio @sergiusens
Remove the unittests from the autopkgtest execution (#600) 
The tests were being run twice, once when building the package and
then on the autopkg execution.

LP: #1596068
997f444
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment