Commit916c45d

committed

refs: added constructor flag to allow refs to be instatiated from any path, including simple test

1 parente94df6a commit916c45dCopy full SHA for 916c45d

File tree

+11

-4

lines changed

+11

-4

lines changed

Lines changed: 6 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -22,15 +22,17 @@ class Reference(SymbolicReference, LazyMixin, Iterable):`
`22`	`22`	`_resolve_ref_on_create=True`
`23`	`23`	`_common_path_default="refs"`
`24`	`24`
`25`		`-def__init__(self,repo,path):`
	`25`	`+def__init__(self,repo,path,check_path=True):`
`26`	`26`	`"""Initialize this instance`
`27`	`27`	`:param repo: Our parent repository`
`28`	`28`
`29`	`29`	`:param path:`
`30`	`30`	`Path relative to the .git/ directory pointing to the ref in question, i.e.`
`31`		`-refs/heads/master"""`
`32`		`-ifnotpath.startswith(self._common_path_default+'/'):`
`33`		`-raiseValueError("Cannot instantiate %r from path %s"% (self.__class__.__name__,path ))`
	`31`	`+refs/heads/master`
	`32`	`+:param check_path: if False, you can provide any path. Otherwise the path must start with the`
	`33`	`+default path prefix of this type."""`
	`34`	`+ifcheck_pathandnotpath.startswith(self._common_path_default+'/'):`
	`35`	`+raiseValueError("Cannot instantiate %r from path %s"% (self.__class__.__name__,path))`
`34`	`36`	`super(Reference,self).__init__(repo,path)`
`35`	`37`
`36`	`38`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,11 @@ def test_from_path(self):`
`24`	`24`	`assertisinstance(instance,ref_type)`
`25`	`25`	`# END for each name`
`26`	`26`	`# END for each type`
	`27`	`+`
	`28`	`+# invalid path`
	`29`	`+self.failUnlessRaises(ValueError,TagReference,self.rorepo,"refs/invalid/tag")`
	`30`	`+# works without path check`
	`31`	`+TagReference(self.rorepo,"refs/invalid/tag",check_path=False)`
`27`	`32`
`28`	`33`	`deftest_tag_base(self):`
`29`	`34`	`tag_object_refs=list()`

Comments

(0)