summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Maw <richard.maw@codethink.co.uk>2015-06-17 21:04:47 +0000
committerstraycat <richardipsum@fastmail.co.uk>2015-07-04 11:42:28 +0000
commit81451af4468ede41b703f1c0bf4b686ece8d0b03 (patch)
tree9af97be43b3e02fb3eb6bf8740f131162e1cb623
parente7d6489cce98c2e59b85c73609f3d1ea03939570 (diff)
downloadmorph-81451af4468ede41b703f1c0bf4b686ece8d0b03.tar.gz
sourceresolver_tests: Remove bogus test
This test, tests for a case that produces unhelpful results. It asserts that if we don't have the repository locally, and we try to fetch it remotely and the file doesn't exist, that we should have no morphology. What we'd actually like is for it to fetch the repository and try that locally, at which point it would succeed. The reason the flawed previous behaviour (that this test asserted to be correct) failed to cause problems for us, is that in the normal course of resolving refs, we would always either have found it remotely or cached it locally, when we were doing ls_tree or resolve_ref operations. Change-Id: I287eb1e5bc9c916b31f306b4f5f24b0a6d3f0559
-rw-r--r--morphlib/sourceresolver_tests.py9
1 files changed, 0 insertions, 9 deletions
diff --git a/morphlib/sourceresolver_tests.py b/morphlib/sourceresolver_tests.py
index 07428496..5985579c 100644
--- a/morphlib/sourceresolver_tests.py
+++ b/morphlib/sourceresolver_tests.py
@@ -273,15 +273,6 @@ class SourceResolverTests(unittest.TestCase):
'sha1', 'unreached.morph')
self.assertEqual(morph, None)
- def test_returns_none_when_fails_no_remote_morph(self):
- self.lrc.has_repo = self.doesnothaverepo
- self.rrc.cat_file = self.noremotefile
- morph = self.sr._get_morphology(
- {}, None, None, None,
- morphlib.morphloader.MorphologyLoader(), 'reponame',
- 'sha1', 'unreached.morph')
- self.assertEqual(morph, None)
-
def test_raises_error_when_repo_does_not_exist(self):
self.lrc.has_repo = self.doesnothaverepo
self.assertRaises(MorphologyNotFoundError,