Merge lp:~elopio/snapcraft/symlinks-2 into lp:~snappy-dev/snapcraft/core

Proposed by Leo Arias
Status: Rejected
Rejected by: Leo Arias
Proposed branch: lp:~elopio/snapcraft/symlinks-2
Merge into: lp:~snappy-dev/snapcraft/core
Prerequisite: lp:~elopio/snapcraft/symlinks
Diff against target: 18 lines (+4/-2)
1 file modified
snapcraft/tests/test_ubuntu_plugin.py (+4/-2)
To merge this branch: bzr merge lp:~elopio/snapcraft/symlinks-2
Reviewer Review Type Date Requested Status
Leo Arias (community) Disapprove
Review via email: mp+267240@code.launchpad.net

Commit message

Fixed the symlinks tests to check dir links.

To post a comment you must log in.
lp:~elopio/snapcraft/symlinks-2 updated
134. By Leo Arias

Cleaned the test.

Revision history for this message
Leo Arias (elopio) wrote :

From sergion on the prerequisite:

I'm going to reject this as it is no longer as relevant with filesets.

review: Disapprove

Unmerged revisions

134. By Leo Arias

Cleaned the test.

133. By Leo Arias

Fixed the symlinks tests to check dir links.

132. By Leo Arias

Merged with parent.

131. By Leo Arias

Use scenarios for symlinks.

130. By Leo Arias

Joined the ubuntu plugin unit tests.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'snapcraft/tests/test_ubuntu_plugin.py'
2--- snapcraft/tests/test_ubuntu_plugin.py 2015-08-06 18:26:49 +0000
3+++ snapcraft/tests/test_ubuntu_plugin.py 2015-08-06 18:26:49 +0000
4@@ -73,10 +73,12 @@
5 'type_': 'dir',
6 'expected_link': 'test_source_dir'}),
7 ('absolute_dir_path', {
8- 'source_name': 'test_source_dir',
9+ # The name of the absolute dir must match the name of an existing
10+ # directory, otherwise the link will be a file, not a dir.
11+ 'source_name': 'tmp',
12 'abs_source': True,
13 'type_': 'dir',
14- 'expected_link': 'test_source_dir'}),
15+ 'expected_link': 'tmp'}),
16 ('absolute_unexisting_path', {
17 'source_name': 'unexisting_test_source',
18 'abs_source': True,

Subscribers

People subscribed via source and target branches

to all changes: