diff --git a/pungi/util.py b/pungi/util.py index 9f080c37..97dd6eda 100644 --- a/pungi/util.py +++ b/pungi/util.py @@ -532,9 +532,6 @@ def find_old_compose( if not os.path.isdir(path): continue - if os.path.islink(path): - continue - status_path = os.path.join(path, "STATUS") if not os.path.isfile(status_path): continue diff --git a/tests/test_util.py b/tests/test_util.py index 055a36a1..a5fc894d 100644 --- a/tests/test_util.py +++ b/tests/test_util.py @@ -465,10 +465,14 @@ class TestFindOldCompose(unittest.TestCase): old = util.find_old_compose(self.tmp_dir + "/file", "Fedora", "Rawhide", "") self.assertIsNone(old) - def test_skips_symlink(self): - os.symlink(self.tmp_dir, self.tmp_dir + "/Fedora-Rawhide-20160229.0") + def test_do_not_skip_symlink(self): + touch(self.tmp_dir + "/Fedora-Rawhide-20160228.n.10/STATUS", "FINISHED") + os.symlink( + self.tmp_dir + "/Fedora-Rawhide-20160228.n.10", + self.tmp_dir + "/Fedora-Rawhide-20160229.n.0", + ) old = util.find_old_compose(self.tmp_dir, "Fedora", "Rawhide", "") - self.assertIsNone(old) + self.assertEqual(old, self.tmp_dir + "/Fedora-Rawhide-20160229.n.0") def test_finds_layered_product(self): touch(self.tmp_dir + "/Fedora-Rawhide-Base-1-20160229.0/STATUS", "FINISHED")