config: fixed incorrect handling of default value in get_value

remote.config: SectionConstraint now knows about set_value and get_value which are new to the GitConfigParser

Author: Byron

lib/git/config.py

@@ -360,7 +360,9 @@ def get_value(self, section, option, default = None):
 		try:
 			valuestr = self.get(section, option)
 		except Exception:
-			return default
+			if default is not None:
+				return default
+			raise
 
 		types = ( long, float )
 		for numtype in types:

lib/git/remote.py

@@ -21,7 +21,7 @@ class _SectionConstraint(object):
 	It supports all ConfigParser methods that operate on an option
 	"""
 	__slots__ = ("_config", "_section_name")
-	_valid_attrs_ = ("get", "set", "getint", "getfloat", "getboolean", "has_option")
+	_valid_attrs_ = ("get_value", "set_value", "get", "set", "getint", "getfloat", "getboolean", "has_option")
 
 	def __init__(self, config, section):
 		self._config = config
@@ -32,10 +32,10 @@ def __getattr__(self, attr):
 			return lambda *args: self._call_config(attr, *args)
 		return super(_SectionConstraint,self).__getattribute__(attr)
 
-	def _call_config(self, method, *args):
+	def _call_config(self, method, *args, **kwargs):
 		"""Call the configuration at the given method which must take a section name 
 		as first argument"""
-		return getattr(self._config, method)(self._section_name, *args)
+		return getattr(self._config, method)(self._section_name, *args, **kwargs)
 
 
 class PushProgress(object):

test/git/test_config.py

@@ -8,6 +8,7 @@
 from git import *
 import StringIO
 from copy import copy
+from ConfigParser import NoSectionError
 
 class TestBase(TestCase):
 
@@ -97,4 +98,7 @@ def test_base(self):
 		default = "my default value"
 		assert r_config.get_value("doesnt", "exist", default) == default
 
+		# it raises if there is no default though
+		self.failUnlessRaises(NoSectionError, r_config.get_value, "doesnt", "exist")
+		
 

test/git/test_remote.py

@@ -358,6 +358,7 @@ def test_base(self, rw_repo, remote_repo):
 				val = getattr(remote, opt)
 				reader = remote.config_reader
 				assert reader.get(opt) == val
+				assert reader.get_value(opt, None) == val
 
 				# unable to write with a reader
 				self.failUnlessRaises(IOError, reader.set, opt, "test")