RangeSetND: fix padding info when slicing using __getitem__() (#429)

Padding info may be lost when in RangeSetND.__getitem__() because we only checked the padding of the elements in the first tuple. Use existing pads() method to get the maximum padding length of each dimension instead. Closes #429. Change-Id: Ic3df8dd887f2d4f728ec4a6b568681c148af31ff

Author: thiell

lib/ClusterShell/RangeSet.py

@@ -956,8 +956,7 @@ def __getitem__(self, index):
  for rgvec in self._veclist:
  iveclist += product(*rgvec)
  assert(len(iveclist) == len(self))
- rnd = RangeSetND(iveclist[index],
- pads=[rg.padding for rg in self._veclist[0]],
+ rnd = RangeSetND(iveclist[index], pads=self.pads(),
  autostep=self.autostep)
  return rnd
 

tests/RangeSetNDTest.py

@@ -432,6 +432,9 @@ def test_getitem_slices(self):
  # steps
  self.assertEqual(str(rn1[0:12:2]), "0-3; 1\n10; 10,12\n")
  self.assertEqual(str(rn1[1:12:2]), "0-3; 2\n10; 11,13\n")
+ # GitHub #429
+ rn1 = RangeSetND([["110", "15-16"], ["107", "06"]])
+ self.assertEqual(str(rn1[0:3:2]), "107; 06\n110; 15\n")
 
  def test_contiguous(self):
  rn0 = RangeSetND()