relaxed implementation when comparing symbolic references against other items which don't have a path. Fixed test_refs to work in all cases - it was previously dependent on the order of items returned by the file system

Byron · Byron · commit a5a75ab7533d · 2011-06-08T11:24:10.000+02:00
diff --git a/git/refs/symbolic.py b/git/refs/symbolic.py
@@ -46,7 +46,9 @@ def __repr__(self):
 		return '<git.%s "%s">' % (self.__class__.__name__, self.path)
 		
 	def __eq__(self, other):
-		return self.path == other.path
+		if hasattr(other, 'path'):
+			return self.path == other.path
+		return False
 		
 	def __ne__(self, other):
 		return not ( self == other )
diff --git a/git/test/test_refs.py b/git/test/test_refs.py
@@ -290,6 +290,15 @@ def test_head_reset(self, rw_repo):
 		assert remotes
 		for remote in remotes:
 			refs = remote.refs
+			
+			# If a HEAD exists, it must be deleted first. Otherwise it might
+			# end up pointing to an invalid ref it the ref was deleted before.
+			remote_head_name = "HEAD"
+			if remote_head_name in refs:
+				RemoteReference.delete(rw_repo, refs[remote_head_name])
+				del(refs[remote_head_name])
+			#END handle HEAD deletion
+			
 			RemoteReference.delete(rw_repo, *refs)
 			remote_refs_so_far += len(refs)
 			for ref in refs:
