]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
GH-91054: Reset static events counts in code watchers tests (#99978)
authorItamar Ostricher <itamarost@gmail.com>
Sun, 4 Dec 2022 12:38:21 +0000 (04:38 -0800)
committerGitHub <noreply@github.com>
Sun, 4 Dec 2022 12:38:21 +0000 (12:38 +0000)
Lib/test/test_capi/test_watchers.py
Modules/_testcapi/watchers.c

index ebe7d2783189a3c49081ed520250ab3898bce32a..1922614ef60558ab5f73ee5179edb8ff1802308d 100644 (file)
@@ -383,11 +383,11 @@ class TestCodeObjectWatchers(unittest.TestCase):
                 del co3
                 self.assert_event_counts(2, 2, 1, 1)
 
-        # verify counts remain as they were after both watchers are cleared
+        # verify counts are reset and don't change after both watchers are cleared
         co4 = _testcapi.code_newempty("test_watchers", "dummy4", 0)
-        self.assert_event_counts(2, 2, 1, 1)
+        self.assert_event_counts(0, 0, 0, 0)
         del co4
-        self.assert_event_counts(2, 2, 1, 1)
+        self.assert_event_counts(0, 0, 0, 0)
 
     def test_clear_out_of_range_watcher_id(self):
         with self.assertRaisesRegex(ValueError, r"Invalid code watcher ID -1"):
index f0e51fd462e70e65d1618df01f473a2904fc701d..1d91c206f630927dc120129d244aba7c0e442432 100644 (file)
@@ -325,9 +325,13 @@ add_code_watcher(PyObject *self, PyObject *which_watcher)
     long which_l = PyLong_AsLong(which_watcher);
     if (which_l == 0) {
         watcher_id = PyCode_AddWatcher(first_code_object_callback);
+        num_code_object_created_events[0] = 0;
+        num_code_object_destroyed_events[0] = 0;
     }
     else if (which_l == 1) {
         watcher_id = PyCode_AddWatcher(second_code_object_callback);
+        num_code_object_created_events[1] = 0;
+        num_code_object_destroyed_events[1] = 0;
     }
     else {
         return NULL;
@@ -346,6 +350,11 @@ clear_code_watcher(PyObject *self, PyObject *watcher_id)
     if (PyCode_ClearWatcher(watcher_id_l) < 0) {
         return NULL;
     }
+    // reset static events counters
+    if (watcher_id_l >= 0 && watcher_id_l < NUM_CODE_WATCHERS) {
+        num_code_object_created_events[watcher_id_l] = 0;
+        num_code_object_destroyed_events[watcher_id_l] = 0;
+    }
     Py_RETURN_NONE;
 }