TEST: added test for setter of the WeightedGraphPartition object

Author: CindeeM

brainx/tests/test_weighted_modularity.py

@@ -50,6 +50,12 @@ def test_init(self):
         # generated communities
         comm = [set([node]) for node in self.graph.nodes()]
         self.assertEqual(part.communities, comm)
+        # test communities cannot be replaced by garbage
+        with self.assertRaises(TypeError):
+            part.communities = 11
+        # but we can pass a valid community partition
+        part.communities = comm
+        self.assertEqual(part.communities, comm)
 
     def test_communities_degree(self):
         ## if no community, method will raise error

brainx/weighted_modularity.py

@@ -44,7 +44,7 @@ def communities(self):
 
     @communities.setter
     def communities(self, value):
-        self._communities = self.set_communities(value)
+        self.set_communities(value)
 
     def _init_communities_from_nodes(self):
         """ creates a new communities with one node per community