Skip to content
This repository was archived by the owner on Oct 31, 2023. It is now read-only.

Commit 359dc52

Browse files
mdtanrikulumfranciszkiewicz
authored andcommitted
'last_update_time' won't be updated on restart case (#3092)
1 parent 5818178 commit 359dc52

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

golem/task/taskstate.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ def __init__(self):
2828

2929
def __setattr__(self, key, value):
3030
super().__setattr__(key, value)
31-
if key == 'status':
31+
# Set last update time when changing status to other than 'restarted'
32+
# (user interaction)
33+
if key == 'status' and value != TaskStatus.restarted:
3234
self.last_update_time = time.time()
3335

3436
def __repr__(self):

tests/golem/task/test_taskstate.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,12 @@ def test_last_update_time( # pylint: disable=no-self-argument
7979
self.assertEqual(ts.last_update_time, time.time())
8080

8181
frozen_time.tick() # pylint: disable=no-member
82+
83+
ts.status = TaskStatus.restarted
84+
self.assertNotEqual(ts.last_update_time, time.time())
85+
86+
frozen_time.tick() # pylint: disable=no-member
87+
8288
ts.status = TaskStatus.finished
8389
self.assertEqual(ts.last_update_time, time.time())
8490

0 commit comments

Comments
 (0)