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

Proposed by Leo Arias on 2015-08-06
Status: Rejected
Rejected by: Leo Arias on 2015-09-13
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 Disapprove on 2015-09-13
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 on 2015-08-06
134. By Leo Arias on 2015-08-06

Cleaned the test.

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 on 2015-08-06

Cleaned the test.

133. By Leo Arias on 2015-08-06

Fixed the symlinks tests to check dir links.

132. By Leo Arias on 2015-08-06

Merged with parent.

131. By Leo Arias on 2015-08-06

Use scenarios for symlinks.

130. By Leo Arias on 2015-08-06

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: