Merge pull request #4952 from branfosj/fix_iter_cnt

do not determine number of iterations in easyblock constructor (which is too early), but in `run_all_steps`

Author: boegel

easybuild/framework/easyblock.py

@@ -312,7 +312,7 @@ def __init__(self, ec, logfile=None):
         self._init_log()
 
         # number of iterations
-        self.iter_cnt = self.det_iter_cnt()
+        self.iter_cnt = -1
 
         # try and use the specified group (if any)
         group_name = build_option('group')
@@ -4793,6 +4793,7 @@ def run_all_steps(self, run_test_cases):
         if self.cfg['stop'] == 'cfg':
             return True
 
+        self.iter_cnt = self.det_iter_cnt()
         steps = self.get_steps(run_test_cases=run_test_cases, iteration_count=self.iter_cnt)
 
         # figure out how many steps will actually be run (not be skipped)

test/framework/easyblock.py

@@ -1173,9 +1173,11 @@ def test_handle_iterate_opts(self):
 
         ec = process_easyconfig(test_ec)[0]
         eb = get_easyblock_instance(ec)
+        eb.iter_cnt = eb.det_iter_cnt()
 
         # check initial state
         self.assertEqual(eb.iter_idx, 0)
+        self.assertEqual(eb.iter_cnt, 3)
         self.assertEqual(eb.iter_opts, {})
         self.assertEqual(eb.cfg.iterating, False)
         self.assertEqual(eb.cfg.iterate_options, [])