Adjusted tests to deal with API changes

Author: Byron

TODO

@@ -64,6 +64,8 @@ Refs
   a symbolic reference anymore. Currently, we cannot handle this that well 
   as we do not check for this case. This should be added though as it is 
   valid to have a detached head in some cases.
+* Allow to set SymbolicReferences to a specific Head Reference. This is like 
+  a git-reset --soft
 
 Remote
 ------

lib/git/refs.py

@@ -222,7 +222,7 @@ def reference(self):
 		"""
 		fp = open(os.path.join(self.repo.path, self.name), 'r')
 		try:
-			tokens = fp.readline().split(' ')
+			tokens = fp.readline().rstrip().split(' ')
 			if tokens[0] != 'ref:':
 				raise TypeError("%s is a detached symbolic reference as it points to %r" % tokens[0])
 			return Reference.from_path(self.repo, tokens[1])
@@ -298,10 +298,9 @@ def reset(self, commit='HEAD', index=True, working_tree = False,
 				raise ValueError( "Cannot reset the working tree if the index is not reset as well") 
 		# END working tree handling
 
-		repo.git.reset(mode, commit, paths, **kwargs)
+		self.repo.git.reset(mode, commit, paths, **kwargs)
 
-		# we always point to the active branch as it is the one changing
-		self
+		return self
 
 
 class Head(Reference):

test/git/test_index.py

@@ -116,8 +116,8 @@ def test_from_index_and_diff(self, rw_repo):
 		# test diff
 		# resetting the head will leave the index in a different state, and the 
 		# diff will yield a few changes
-		cur_head_commit = rw_repo.head.commit
-		ref = rw_repo.head.reset(rw_repo, 'HEAD~6', index=True, working_tree=False)
+		cur_head_commit = rw_repo.head.reference.commit
+		ref = rw_repo.head.reset('HEAD~6', index=True, working_tree=False)
 
 		# diff against same index is 0
 		diff = index.diff()

test/git/test_refs.py

@@ -72,13 +72,13 @@ def test_heads(self):
 	def test_head_reset(self, rw_repo):
 		cur_head = rw_repo.head
 		new_head_commit = cur_head.ref.commit.parents[0]
-		reset_head = Head.reset(rw_repo, new_head_commit, index=True)	# index only
-		assert reset_head.commit == new_head_commit
+		cur_head.reset(new_head_commit, index=True)	# index only
+		assert cur_head.reference.commit == new_head_commit
 
-		self.failUnlessRaises(ValueError, Head.reset, rw_repo, new_head_commit, index=False, working_tree=True)
+		self.failUnlessRaises(ValueError, cur_head.reset, new_head_commit, index=False, working_tree=True)
 		new_head_commit = new_head_commit.parents[0]
-		reset_head = Head.reset(rw_repo, new_head_commit, index=True, working_tree=True)	# index + wt
-		assert reset_head.commit == new_head_commit
+		cur_head.reset(new_head_commit, index=True, working_tree=True)	# index + wt
+		assert cur_head.reference.commit == new_head_commit
 
 		# paths
-		Head.reset(rw_repo, new_head_commit, paths = "lib")
+		cur_head.reset(new_head_commit, paths = "lib")

test/git/test_repo.py

@@ -168,17 +168,11 @@ def test_is_dirty(self):
 		self.rorepo._bare = True
 		assert self.rorepo.is_dirty == False
 
-	@patch_object(Git, '_call_process')
-	def test_active_branch(self, git):
-		git.return_value = 'refs/heads/major-refactoring'
-		assert_equal(self.rorepo.active_branch.name, 'major-refactoring')
-		assert_equal(git.call_args, (('symbolic_ref', 'HEAD'), {}))
-		
 	def test_head(self):
-		assert self.rorepo.head.object == self.rorepo.active_branch.object
+		assert self.rorepo.head.reference.object == self.rorepo.active_branch.object
 
 	def test_tag(self):
-		assert self.rorepo.tag('0.1.5').commit
+		assert self.rorepo.tag('refs/tags/0.1.5').commit
 
 	def test_archive(self):
 		tmpfile = os.tmpfile()