The test failure looks interesting, seems to be here:

Maybe replacing it with pkgutil.resolve_name will fix it? (python/cpython#18544)

The test failure looks interesting, seems to be here:

bot/tests/_autospec.py

Lines 53 to 54 in 395e058

	if type(target) is str:
	target = unittest.mock._importer(target)

not surprising since it seems like that was intended to be a private method.
Maybe replacing it with pkgutil.resolve_name will fix it? (python/cpython#18544)

I had actually forgotten about this PR.
I'll try to look into the option you proposed.

one other option we had discussed the other day was monkey patching it by including it in our testing utils.

We'd likely want to avoid that if there's another way that can do this out of the box.

The test failure looks interesting, seems to be here:

bot/tests/_autospec.py

Lines 53 to 54 in 395e058

	if type(target) is str:
	target = unittest.mock._importer(target)

not surprising since it seems like that was intended to be a private method.

Maybe replacing it with pkgutil.resolve_name will fix it? (python/cpython#18544)

Yeah, this is intended to be like this. That's where I got the original code from, and it used to use that internal method.