From 46a7de94fda317931cbc59e899444ebda493245b Mon Sep 17 00:00:00 2001 From: Yashna Parikh Date: Thu, 13 Aug 2026 10:03:48 -0400 Subject: [PATCH 1/2] Increase sleep timeout --- src/extension/tests/Test_ExtOutputStatusHandler.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/extension/tests/Test_ExtOutputStatusHandler.py b/src/extension/tests/Test_ExtOutputStatusHandler.py index 5f5df1b4..49048734 100644 --- a/src/extension/tests/Test_ExtOutputStatusHandler.py +++ b/src/extension/tests/Test_ExtOutputStatusHandler.py @@ -88,7 +88,7 @@ def test_update_file(self): modified_time = stat_file_name.st_mtime self.assertEqual(prev_modified_time, modified_time) - time.sleep(0.03) # ensure filesystem mtime granularity is exceeded + time.sleep(0.08) # ensure filesystem mtime granularity is exceeded ext_status_handler.update_file(file_name) stat_file_name = os.stat(os.path.join(dir_path, file_name + ".status")) modified_time = stat_file_name.st_mtime From f0d6e9d515a58309977e8fc07c5c2ddeb26de240 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 13 Aug 2026 15:24:35 +0000 Subject: [PATCH 2/2] Stabilize update_file test without mtime timing Co-authored-by: yashnap <267671466+yashnap@users.noreply.github.com> --- .../tests/Test_ExtOutputStatusHandler.py | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/src/extension/tests/Test_ExtOutputStatusHandler.py b/src/extension/tests/Test_ExtOutputStatusHandler.py index 49048734..f58b95c1 100644 --- a/src/extension/tests/Test_ExtOutputStatusHandler.py +++ b/src/extension/tests/Test_ExtOutputStatusHandler.py @@ -15,10 +15,8 @@ # Requires Python 2.7+ import json -import os import shutil import tempfile -import time import unittest from extension.src.Constants import Constants from extension.src.file_handlers.ExtOutputStatusHandler import ExtOutputStatusHandler @@ -79,21 +77,15 @@ def test_update_file(self): ext_status_handler = ExtOutputStatusHandler(self.logger, self.utility, self.json_file_handler, dir_path) ext_status_handler.write_status_file(operation, file_name, self.status.Success.lower()) - stat_file_name = os.stat(os.path.join(dir_path, file_name + ".status")) - prev_modified_time = stat_file_name.st_mtime + original_status_json = ext_status_handler.read_file(file_name) - time.sleep(0.02) ext_status_handler.update_file("test1") - stat_file_name = os.stat(os.path.join(dir_path, file_name + ".status")) - modified_time = stat_file_name.st_mtime - self.assertEqual(prev_modified_time, modified_time) + status_json_after_different_seq_update = ext_status_handler.read_file(file_name) + self.assertEqual(original_status_json, status_json_after_different_seq_update) - time.sleep(0.08) # ensure filesystem mtime granularity is exceeded ext_status_handler.update_file(file_name) - stat_file_name = os.stat(os.path.join(dir_path, file_name + ".status")) - modified_time = stat_file_name.st_mtime - self.assertNotEqual(prev_modified_time, modified_time) # Fails here on GitHub updated_status_json = ext_status_handler.read_file(file_name) + self.assertNotEqual(original_status_json, updated_status_json) self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_status], self.status.Transitioning.lower()) self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_name], "Azure Patch Management") self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_operation], "Assessment") @@ -101,9 +93,6 @@ def test_update_file(self): self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_formatted_message][self.status_file_fields.status_formatted_message_message], "") ext_status_handler.update_file(file_name, Constants.Status.Success.lower(), Constants.ExitCode.Okay, "Test message") - stat_file_name = os.stat(os.path.join(dir_path, file_name + ".status")) - modified_time = stat_file_name.st_mtime - self.assertNotEqual(prev_modified_time, modified_time) updated_status_json = ext_status_handler.read_file(file_name) self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_status], self.status.Success.lower()) self.assertEqual(updated_status_json[0][self.status_file_fields.status][self.status_file_fields.status_name], "Azure Patch Management")