tests: Fix --without-vcsserver parameter

Ensuring now that we always disable the vcsserver auto start for the test suite,
since the test suite bootstraps its own vcsserver process.

Also enhanced TestINI.

rhodecode/tests/fixture.py

@@ -76,8 +76,8 @@ class TestINI(object):
 
         for data in self.ini_params:
             section, ini_params = data.items()[0]
-            key, val = ini_params.items()[0]
-            config[section][key] = val
+            for key, val in ini_params.items():
+                config[section][key] = val
         with tempfile.NamedTemporaryFile(
                 prefix=self.new_path_prefix, suffix='.ini', dir=self._dir,
                 delete=False) as new_ini_file:

rhodecode/tests/pylons_plugin.py

@@ -283,7 +283,14 @@ def pylons_config(request, tmpdir_factory, rcserver_port, vcsserver_port):
 
     overrides = [
         {'server:main': {'port': rcserver_port}},
-        {'app:main': {'vcs.server': 'localhost:%s' % vcsserver_port}}]
+        {'app:main': {
+            'vcs.server': 'localhost:%s' % vcsserver_port,
+            # johbo: We will always start the VCSServer on our own based on the
+            # fixtures of the test cases. For the test run it must always be
+            # off in the INI file.
+            'vcs.start_server': 'false',
+        }},
+    ]
     if _use_vcs_http_server(request.config):
         overrides.append({'app:main': {'vcs.server.protocol': 'http'}})