diff --git a/src/main/java/net/floodlightcontroller/core/IFloodlightProviderService.java b/src/main/java/net/floodlightcontroller/core/IFloodlightProviderService.java
index 6a92712c1f3401edca691e6366215d5e41ee3aa6..0674de9a513b7439c993952b281f21868f35ff5d 100644
--- a/src/main/java/net/floodlightcontroller/core/IFloodlightProviderService.java
+++ b/src/main/java/net/floodlightcontroller/core/IFloodlightProviderService.java
@@ -76,7 +76,7 @@ public interface IFloodlightProviderService extends IFloodlightService {
     public Map<OFType, List<IOFMessageListener>> getListeners();
 
     /**
-     * Returns a list of all actively connected OpenFlow switches. This doesn't
+     * Returns an unmodifiable map of all actively connected OpenFlow switches. This doesn't
      * contain switches that are connected but the controller's in the slave role.
      * @return the set of actively connected switches
      */
diff --git a/src/test/java/net/floodlightcontroller/core/internal/ControllerTest.java b/src/test/java/net/floodlightcontroller/core/internal/ControllerTest.java
index 9347295141680aa58c9c7bed1bc9044fe06da1e0..3ed1d8499fbfcfa34f886e2cf0d6ae451e072ba5 100644
--- a/src/test/java/net/floodlightcontroller/core/internal/ControllerTest.java
+++ b/src/test/java/net/floodlightcontroller/core/internal/ControllerTest.java
@@ -61,6 +61,7 @@ import net.floodlightcontroller.test.FloodlightTestCase;
 import net.floodlightcontroller.threadpool.IThreadPoolService;
 
 import org.easymock.Capture;
+import org.easymock.EasyMock;
 import org.jboss.netty.channel.Channel;
 import org.junit.Test;
 import org.openflow.protocol.OFError;
@@ -1150,5 +1151,24 @@ public class ControllerTest extends FloodlightTestCase {
                    controller.activeSwitches.isEmpty());
     }
     
-    
+    /**
+     * Tests that you can't remove a switch from the active
+     * switch list.
+     * @throws Exception
+     */
+    @Test
+    public void testRemoveActiveSwitch() {
+    	IOFSwitch sw = EasyMock.createNiceMock(IOFSwitch.class);
+    	boolean exceptionThrown = false;
+    	expect(sw.getId()).andReturn(1L).anyTimes();
+    	replay(sw);
+    	getController().activeSwitches.put(sw.getId(), sw);
+    	try {
+    		getController().getSwitches().remove(1L);
+    	} catch (UnsupportedOperationException e) {
+    		exceptionThrown = true;
+    	}
+    	assertTrue(exceptionThrown);
+    	verify(sw);
+    }
 }