Reverting change in ConnectionManager, sourceList() will only return …

…the instantiated datasources

Fixing remaining tests failing due to contamination of loaded plugin is app

lib/Cake/Model/ConnectionManager.php

@@ -106,14 +106,16 @@ public static function getDataSource($name) {
 
 /**
  * Gets the list of available DataSource connections
+ * This will only return the datasources instantiated by this manager
+ * It differs from enumConnectionObjects, since the latter will return all configured connections
  *
  * @return array List of available connections
  */
 	public static function sourceList() {
 		if (empty(self::$_init)) {
 			self::init();
 		}
-		return array_keys((array)self::$config);
+		return array_keys(self::$_dataSources);
 	}
 
 /**

lib/Cake/Test/Case/Controller/ScaffoldTest.php

@@ -366,7 +366,7 @@ function testGetViewFilename() {
 			'action' => 'admin_add',
 			'admin' => true
 		));
-		$Controller->plugin = 'test_plugin';
+		$Controller->plugin = 'TestPlugin';
 
 		$ScaffoldView = new TestScaffoldView($Controller);
 		$result = $ScaffoldView->testGetFilename('admin_add');
@@ -826,6 +826,7 @@ function testScaffoldFlashMessages() {
  * @return void
  */
 	function testHabtmFieldAdditionWithScaffoldForm() {
+		CakePlugin::unload();
 		$params = array(
 			'plugin' => null,
 			'pass' => array(1),

lib/Cake/Test/Case/Model/ConnectionManagerTest.php

@@ -185,6 +185,7 @@ function testGetPluginDataSourceAndLocalDriver() {
  * @return void
  */
 	function testSourceList() {
+		ConnectionManager::getDataSource('test');
 		$sources = ConnectionManager::sourceList();
 		$this->assertTrue(count($sources) >= 1);
 		$this->assertTrue(in_array('test', array_keys($sources)));