[#571] exclude collection "/" from subcollections

"/" by definition cannot be a proper subcollection of any other
collection.

Author: d-w-moore

irods/collection.py

@@ -50,7 +50,7 @@ def metadata(self):
     def subcollections(self):
         query = self.manager.sess.query(Collection)\
             .filter(Collection.parent_name == self.path)
-        return [iRODSCollection(self.manager, row) for row in query]
+        return [iRODSCollection(self.manager, row) for row in query if row[Collection.name] != '/']
 
     @property
     def data_objects(self):

irods/test/collection_test.py

@@ -489,6 +489,15 @@ def test_touch_operation_ignores_unsupported_options__525(self):
             if collection:
                 user_session.collections.remove(path, recurse=True, force=True)
 
+    def test_subcollections_member_excludes_root_collection__571(self):
+
+        root_coll = self.sess.collections.get("/")
+
+        # Assert that none of the root collection's immediate children (as listed in the object's
+        # 'subcollections' property) point to the root subcollection.
+        self.assertEqual(root_coll.path, "/")
+        self.assertEqual([], [_ for _ in root_coll.subcollections if _.path == "/"])
+
 
 if __name__ == "__main__":
     # let the tests find the parent irods lib