Merge pull request #729 from teojgo/bugfix/flex_alloc_access

[bugfix] Fix crash in flexible task allocation due to refactoring of `SystemPartition`

reframe/utility/__init__.py

@@ -273,6 +273,21 @@ def __len__(self, *args, **kwargs):
     def __reversed__(self, *args, **kwargs):
         return self.__container.__reversed__(*args, **kwargs)
 
+    def __add__(self, other):
+        if not isinstance(other, collections.abc.Sequence):
+            return NotImplemented
+
+        return SequenceView(self.__container + other)
+
+    def __iadd__(self, other):
+        return NotImplemented
+
+    def __eq__(self, other):
+        if not isinstance(other, collections.abc.Sequence):
+            return NotImplemented
+
+        return self.__container == other
+
 
 class MappingView(collections.abc.Mapping):
     """A read-only view of a mapping."""

unittests/test_utility.py

@@ -771,15 +771,23 @@ def test_sequence(self):
         self.assertEqual(2, l.count(2))
 
         # Assert immutability
+        m = l + [3, 4]
+        self.assertEqual([1, 2, 2, 3, 4], m)
+        self.assertIsInstance(m, util.SequenceView)
+
+        m = l
+        l += [3, 4]
+        self.assertIsNot(m, l)
+        self.assertEqual([1, 2, 2], m)
+        self.assertEqual([1, 2, 2, 3, 4], l)
+        self.assertIsInstance(l, util.SequenceView)
+
         with self.assertRaises(TypeError):
             l[1] = 3
 
         with self.assertRaises(TypeError):
             l[1:2] = [3]
 
-        with self.assertRaises(TypeError):
-            l += [3, 4]
-
         with self.assertRaises(TypeError):
             l *= 3