From c0dca7a44cd12492ec34ad9c7cf2a7aa5ab0d116 Mon Sep 17 00:00:00 2001
From: Nicola Soranzo <nicola.soranzo@earlham.ac.uk>
Date: Mon, 4 Feb 2019 23:53:45 +0000
Subject: [PATCH] Respect `section=False` parameter in
 `BaseToolBoxTestCase._setup_two_versions_in_config()`

Broken in commit fd1c34b4c9fec55286c3d33e2802062ef393de82
---
 test/unit/tools/test_toolbox.py | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/test/unit/tools/test_toolbox.py b/test/unit/tools/test_toolbox.py
index 94b0a4e446d..50fb2722e54 100644
--- a/test/unit/tools/test_toolbox.py
+++ b/test/unit/tools/test_toolbox.py
@@ -112,21 +112,17 @@ class BaseToolBoxTestCase(unittest.TestCase, UsesApp, UsesTools):
     def _setup_two_versions_in_config(self, section=False):
         if section:
             template = """<toolbox tool_path="%s">
-<section id="tid" name="TID" version="">
-    %s
-</section>
-<section id="tid" name="TID" version="">
-    %s
-</section>
+    <section id="tid" name="TID" version="">
+        %s
+    </section>
+    <section id="tid" name="TID" version="">
+        %s
+    </section>
 </toolbox>"""
         else:
             template = """<toolbox tool_path="%s">
-<section id="tid" name="TID" version="">
     %s
-</section>
-<section id="tid" name="TID" version="">
     %s
-</section>
 </toolbox>"""
         self._add_config(template % (self.test_directory, CONFIG_TEST_TOOL_VERSION_1, CONFIG_TEST_TOOL_VERSION_2))