From d78a82dc7cde20bb1010a4a4dbd6a48f6c1e37bb Mon Sep 17 00:00:00 2001 From: Julien Kauffmann Date: Wed, 8 Feb 2012 08:51:48 +0100 Subject: Added test for backslahed/quoted values --- git/test/fixtures/git_config_values | 4 ++++ git/test/test_config.py | 26 +++++++++++++++++++++++++- 2 files changed, 29 insertions(+), 1 deletion(-) create mode 100644 git/test/fixtures/git_config_values (limited to 'git/test') diff --git a/git/test/fixtures/git_config_values b/git/test/fixtures/git_config_values new file mode 100644 index 00000000..5ba039ef --- /dev/null +++ b/git/test/fixtures/git_config_values @@ -0,0 +1,4 @@ +[values] + backslash = some\\data + quote = this is a \"quoted value\" + quoted = "all" "your \"quotes\" a"re bel"ong to """"us" diff --git a/git/test/test_config.py b/git/test/test_config.py index d2e199e3..b37db290 100644 --- a/git/test/test_config.py +++ b/git/test/test_config.py @@ -101,4 +101,28 @@ class TestConfig(TestBase): # it raises if there is no default though self.failUnlessRaises(NoSectionError, r_config.get_value, "doesnt", "exist") - + def test_values(self): + file_obj = self._to_memcache(fixture_path("git_config_values")) + w_config = GitConfigParser(file_obj, read_only = False) + w_config.write() # enforce writing + orig_value = file_obj.getvalue() + + # Reading must unescape backslashes + backslash = w_config.get('values', 'backslash') + assert backslash == r'some\data' + + # Reading must unescape quotes + quote = w_config.get('values', 'quote') + assert quote == 'this is a "quoted value"' + + # Reading must remove surrounding quotes + quoted = w_config.get('values', 'quoted') + assert quoted == 'all your "quotes" are belong to us' + + # Writing must escape backslashes and quotes + w_config.set('values', 'backslash', backslash) + w_config.set('values', 'quote', quote) + w_config.write() # enforce writing + + # Contents shouldn't differ + assert file_obj.getvalue() == orig_value -- cgit v1.2.1