diff tests/test_fetch_symlinks.py @ 741:b3128fec5d54

editor: handle property changes to links. Previously, property changes to links caused 'link ' to be prepended to the link destination. Removing a line that prepended it in Revision::set() appears to fix it. In these cases, the "file marked as link, but contains data" warning might be triggered. This should be safe, so it's lowered to a note and the language made less conclusive. In order to test this, extra revisions are added to the 'symlinks.svndump' fixture. As one of the new revisions add a link that points to 'link to this', a check that asserted that link destinations must not start with 'link ' was removed. This change is safe, as the test later on asserts exact equality with the contents of the 'links' dictionary.
author Dan Villiom Podlaski Christiansen <danchr@gmail.com>
date Tue, 09 Nov 2010 13:05:16 +0100
parents d2ef7220a079
children e9af7eba88db
line wrap: on
line diff
--- a/tests/test_fetch_symlinks.py
+++ b/tests/test_fetch_symlinks.py
@@ -9,14 +9,7 @@ class TestFetchSymlinks(test_util.TestBa
 
     def test_symlinks(self, stupid=False):
         repo = self._load_fixture_and_fetch('symlinks.svndump', stupid)
-        # Check no symlink contains the 'link ' prefix
-        for rev in repo:
-            r = repo[rev]
-            for f in r.manifest():
-                if 'l' not in r[f].flags():
-                    continue
-                self.assertFalse(r[f].data().startswith('link '))
-        # Check symlinks in tip
+        # Check symlinks throughout history
         links = {
             0: {
                 'linka': 'a',
@@ -35,6 +28,16 @@ class TestFetchSymlinks(test_util.TestBa
                 },
             3: {
                 },
+            4: {
+                'linka3': 'a',
+                },
+            5: {
+                'linka3': 'a',
+                },
+            6: {
+                'linka3': 'a',
+                'linka4': 'link to this',
+                },
             }
             
         for rev in repo: