Can `extrapath` ever be None?  It doesn't seem so, the only real caller always passes a value there.  There are two calls to findModuleOfName in tests that don't supply an extrapath argument at all.  Maybe it's time to drop the `=None` default from the method signature (and fix the test code).