Commit3cfd229

authored and

committed

fix: ignore AutoInterrupt terminate errors during shutdown

1 parent82b7065 commit3cfd229Copy full SHA for 3cfd229

File tree

+41

-2

lines changed

+41

-2

lines changed

Lines changed: 6 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -368,8 +368,12 @@ def _terminate(self) -> None:`
`368`	`368`	`status=proc.wait()# Ensure the process goes away.`
`369`	`369`
`370`	`370`	`self.status=self._status_code_if_terminateorstatus`
`371`		`-exceptOSErrorasex:`
`372`		`-_logger.info("Ignored error after process had died: %r",ex)`
	`371`	`+except (OSError,AttributeError)asex:`
	`372`	`+# On interpreter shutdown (notably on Windows), parts of the stdlib used by`
	`373`	+# subprocess can already be torn down (e.g. `subprocess._winapi` becomes None),
	`374`	`+# which can cause AttributeError during terminate(). In that case, we prefer`
	`375`	`+# to silently ignore to avoid noisy "Exception ignored in: __del__" messages.`
	`376`	`+_logger.info("Ignored error while terminating process: %r",ex)`
`373`	`377`	`# END exception handling`
`374`	`378`
`375`	`379`	`def__del__(self)->None:`

Lines changed: 35 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,35 @@`
	`1`	`+importpytest`
	`2`	`+`
	`3`	`+fromgit.cmdimportGit`
	`4`	`+`
	`5`	`+`
	`6`	`+class_DummyProc:`
	`7`	`+"""Minimal stand-in for subprocess.Popen used to exercise AutoInterrupt.`
	`8`	`+`
	`9`	`+ We deliberately raise AttributeError from terminate() to simulate interpreter`
	`10`	`+ shutdown on Windows where subprocess internals (e.g. subprocess._winapi) may`
	`11`	`+ already be torn down.`
	`12`	`+ """`
	`13`	`+`
	`14`	`+stdin=None`
	`15`	`+stdout=None`
	`16`	`+stderr=None`
	`17`	`+`
	`18`	`+defpoll(self):`
	`19`	`+returnNone`
	`20`	`+`
	`21`	`+defterminate(self):`
	`22`	`+raiseAttributeError("TerminateProcess")`
	`23`	`+`
	`24`	`+defwait(self):# pragma: no cover - should not be reached in this test`
	`25`	`+raiseAssertionError("wait() should not be called if terminate() fails")`
	`26`	`+`
	`27`	`+`
	`28`	`+deftest_autointerrupt_terminate_ignores_attributeerror():`
	`29`	`+ai=Git.AutoInterrupt(_DummyProc(),args=["git","rev-list"])`
	`30`	`+`
	`31`	`+# Should not raise, even if terminate() triggers AttributeError.`
	`32`	`+ai._terminate()`
	`33`	`+`
	`34`	`+# Ensure the reference is cleared to avoid repeated attempts.`
	`35`	`+assertai.procisNone`

Comments

(0)