Add finale tour examples and tests #558

Merged
merged 11 commits into from Jun 9, 2016

Conversation

Projects
None yet
4 participants
Contributor

didrocks commented Jun 9, 2016

  • Fix existing example by stamping 2.10 as a string.
  • Add finale tour examples and new tests for those.

They can be enabled as they are only pulling from npm and wiki parts, which are already tested elsewhere, so should work under CI jenkins firewall rules.

didrocks added some commits Jun 7, 2016

Enable installing snaps app in devmode while testing
Add an optional flag to install snaps in devmode through the testing
framework
Collaborator

sergiusens commented Jun 9, 2016

retest this please

Contributor

fgimenez commented Jun 9, 2016

retest this please

Contributor

fgimenez commented Jun 9, 2016

retest this please

Collaborator

sergiusens commented Jun 9, 2016

I approve of this. Just wish the tests passed.

@elopio or @fgimenez I hope you can Shepperd this in.

Collaborator

sergiusens commented Jun 9, 2016

retest this please

snaps_tests/__init__.py
@@ -160,9 +161,11 @@ def install_snap(self, snap_content_dir, snap_name, version):
self.addCleanup(
self.snappy_testbed.run_command,
['rm', snap_path_in_testbed])
+ cmd = ['sudo', 'snap', 'install', snap_path_in_testbed]
+ if devmode:
+ cmd.append("--devmode")
@elopio

elopio Jun 9, 2016

Member

single quotes :)
Thanks for the addition.

@didrocks

didrocks Jun 9, 2016

Contributor

rohhhhh :) ok, just pushed that change

@@ -25,6 +25,6 @@ class EasyStartTestCase(snaps_tests.SnapsTestCase):
def test_hello(self):
self.skipTest('Skip while the proxy rule is added by IS.')
@elopio

elopio Jun 9, 2016

Member

The rule for gnu.org was supposedly added already. I haven't tried it yet.

Member

elopio commented Jun 9, 2016

I approve too. I will fight with the tests after the meeting.

@sergiusens sergiusens merged commit 442b4f1 into snapcore:master Jun 9, 2016

4 checks passed

Examples tests Success
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.05%) to 95.719%
Details

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment