# [JGIT PATCH 2/2] Extended test cases for RepositoryConfigTest 2 messages from 2008-07-27 to 2008-07-27. Participants: Shawn O. Pearce. Thread: https://gitlist.dev/t/14698 ## Shawn O. Pearce, 2008-07-27 02:11 Subject: [JGIT PATCH 1/2] Fix RepositoryConfig.setStringList to not corrupt internal state Message-ID: <1217124716-12063-1-git-send-email-spearce@spearce.org> URL: https://gitlist.dev/e/1217124716-12063-1-git-send-email-spearce%40spearce.org ``` Calling setStringList was incorrectly storing the String objects directly in byName. The byName table should have either an Entry or a List stored within its value position. Storing the String (or List) directly confused our get code as it did not find the object type it expected. This caused the get code to fallback to the base configuration (e.g. ~/.gitconfig) or just return null (claiming the value was never set). A test case for this appears in the next commit. Noticed-by: Marek Zawirski Signed-off-by: Shawn O. Pearce --- .../src/org/spearce/jgit/lib/RepositoryConfig.java | 23 ++++++++++++++++--- 1 files changed, 19 insertions(+), 4 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/RepositoryConfig.java b/org.spearce.jgit/src/org/spearce/jgit/lib/RepositoryConfig.java index 1431f1f..d1cd5fc 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/lib/RepositoryConfig.java +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/RepositoryConfig.java @@ -381,10 +381,25 @@ public class RepositoryConfig { key += "." + name.toLowerCase(); if (values.size() == 0) byName.remove(key); - else if (values.size() == 1) - byName.put(key, values.get(0)); - else - byName.put(key, new ArrayList(values)); + else if (values.size() == 1) { + final Entry e = new Entry(); + e.base = section; + e.extendedBase = subsection; + e.name = name; + e.value = values.get(0); + byName.put(key, e); + } else { + final ArrayList eList = new ArrayList(values.size()); + for (final String v : values) { + final Entry e = new Entry(); + e.base = section; + e.extendedBase = subsection; + e.name = name; + e.value = v; + eList.add(e); + } + byName.put(key, eList); + } int entryIndex = 0; int valueIndex = 0; -- 1.6.0.rc0.182.gb96c7 ``` ## Shawn O. Pearce, 2008-07-27 02:11 Subject: [JGIT PATCH 2/2] Extended test cases for RepositoryConfigTest Message-ID: <1217124716-12063-2-git-send-email-spearce@spearce.org> URL: https://gitlist.dev/e/1217124716-12063-2-git-send-email-spearce%40spearce.org In-Reply-To: <1217124716-12063-1-git-send-email-spearce@spearce.org> ``` From: Marek Zawirski Add cases for getting values after setting them. It's something different from reading config file from scratch or writing it out after modifications. It's case for getting values after modifications in object model. Signed-off-by: Marek Zawirski Signed-off-by: Shawn O. Pearce --- .../org/spearce/jgit/lib/RepositoryConfigTest.java | 20 +++++++++++++++++++- 1 files changed, 19 insertions(+), 1 deletions(-) diff --git a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/RepositoryConfigTest.java b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/RepositoryConfigTest.java index 8c55fe8..da7e704 100644 --- a/org.spearce.jgit.test/tst/org/spearce/jgit/lib/RepositoryConfigTest.java +++ b/org.spearce.jgit.test/tst/org/spearce/jgit/lib/RepositoryConfigTest.java @@ -2,6 +2,7 @@ * Copyright (C) 2007, Dave Watson * Copyright (C) 2008, Robin Rosenberg * Copyright (C) 2008, Shawn O. Pearce + * Copyright (C) 2008, Marek Zawirski * * All rights reserved. * @@ -41,6 +42,8 @@ package org.spearce.jgit.lib; import java.io.File; import java.io.IOException; +import java.util.Arrays; +import java.util.LinkedList; /** * Test reading of git config @@ -84,11 +87,26 @@ public class RepositoryConfigTest extends RepositoryTestCase { checkFile(cfgFile, "[sec \"ext\"]\n\tname = value\n\tname2 = value2\n"); } - public void test004_PutSimple() throws IOException { + public void test004_PutGetSimple() throws IOException { File cfgFile = writeTrashFile("config_004", ""); RepositoryConfig repositoryConfig = new RepositoryConfig(null, cfgFile); repositoryConfig.setString("my", null, "somename", "false"); repositoryConfig.save(); checkFile(cfgFile, "[my]\n\tsomename = false\n"); + assertEquals("false", repositoryConfig + .getString("my", null, "somename")); + } + + public void test005_PutGetStringList() throws IOException { + File cfgFile = writeTrashFile("config_005", ""); + RepositoryConfig repositoryConfig = new RepositoryConfig(null, cfgFile); + final LinkedList values = new LinkedList(); + values.add("value1"); + values.add("value2"); + repositoryConfig.setStringList("my", null, "somename", values); + repositoryConfig.save(); + assertTrue(Arrays.equals(values.toArray(), repositoryConfig + .getStringList("my", null, "somename"))); + checkFile(cfgFile, "[my]\n\tsomename = value1\n\tsomename = value2\n"); } } -- 1.6.0.rc0.182.gb96c7 ```