diff options
author | Richard Maw <richard.maw@codethink.co.uk> | 2015-06-17 21:04:47 +0000 |
---|---|---|
committer | straycat <richardipsum@fastmail.co.uk> | 2015-07-04 11:42:28 +0000 |
commit | 81451af4468ede41b703f1c0bf4b686ece8d0b03 (patch) | |
tree | 9af97be43b3e02fb3eb6bf8740f131162e1cb623 /morphlib | |
parent | e7d6489cce98c2e59b85c73609f3d1ea03939570 (diff) | |
download | morph-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
Diffstat (limited to 'morphlib')
-rw-r--r-- | morphlib/sourceresolver_tests.py | 9 |
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, |