Merge lp:~canonical-platform-qa/snappy-ecosystem-tests/upload-private-snap-logged-out into lp:snappy-ecosystem-tests

Proposed by Omer Akram
Status: Needs review
Proposed branch: lp:~canonical-platform-qa/snappy-ecosystem-tests/upload-private-snap-logged-out
Merge into: lp:snappy-ecosystem-tests
Diff against target: 39 lines (+21/-4)
1 file modified
snappy_ecosystem_tests/tests/test_install_private_snap.py (+21/-4)
To merge this branch: bzr merge lp:~canonical-platform-qa/snappy-ecosystem-tests/upload-private-snap-logged-out
Reviewer Review Type Date Requested Status
platform-qa-bot continuous-integration Approve
Snappy ecosystem tests developer Pending
Review via email: mp+321280@code.launchpad.net

Commit message

Test: try to install a private snap logged-out

Description of the change

Test: try to install a private snap logged-out

To post a comment you must log in.
Revision history for this message
Omer Akram (om26er) wrote :

results incoming, in a bit.

Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Omer Akram (om26er) wrote :

Unmerged revisions

50. By Omer Akram

try to install a private snap logged-out

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'snappy_ecosystem_tests/tests/test_install_private_snap.py'
2--- snappy_ecosystem_tests/tests/test_install_private_snap.py 2017-03-22 12:29:03 +0000
3+++ snappy_ecosystem_tests/tests/test_install_private_snap.py 2017-03-29 12:00:54 +0000
4@@ -33,10 +33,10 @@
5 super().setUp()
6 self.store = Store()
7 self.snapd = Snapd()
8- email, password = get_store_credentials()
9- self.assertTrue(self.snapd.login(email, password))
10- self.addCleanup(self.snapd.logout, email)
11- self.store.login(email, password)
12+ self.email, password = get_store_credentials()
13+ self.assertTrue(self.snapd.login(self.email, password))
14+ self.addCleanup(self.snapd.logout, self.email)
15+ self.store.login(self.email, password)
16 self.addCleanup(self.store.logout)
17 self.snap_builder = Snapcraft()
18
19@@ -58,3 +58,20 @@
20 self.snapd.install(snap_name)
21 self.addCleanup(self.snapd.remove, snap_name)
22 self.assertTrue(self.snapd.is_installed, snap_name)
23+
24+ def test_install_private_snap_logged_out(self):
25+ """Test cannot install private snap if not logged in."""
26+ snap_name = get_random_snap_name()
27+ channels = ['stable']
28+ revision = '1'
29+ self.assertTrue(self.store.register(snap_name, private=True),
30+ 'Unable to register snap')
31+ # Upload the snap and wait for its processing on server to finish.
32+ tracker = self.store.upload(
33+ snap_name, self.snap_builder.build_and_pull(snap_name, '0.1'))
34+ tracker.track()
35+ response = self.store.release(snap_name, revision, channels)
36+ self.assertEqual(channels, response.get('opened_channels', None))
37+ self.snapd.wait_for_publish(snap_name, revision, channels[0])
38+ self.snapd.logout(self.email)
39+ self.assertRaises(ValueError, self.snapd.install, snap_name)

Subscribers

People subscribed via source and target branches