bpo-42532: Check if NonCallableMock's spec_arg is not None instead of call its __bool__ function

Lib/unittest/mock.py

@@ -406,7 +406,7 @@ def __new__(cls, /, *args, **kw):
             # Check if spec is an async object or function
             bound_args = _MOCK_SIG.bind_partial(cls, *args, **kw).arguments
             spec_arg = bound_args.get('spec_set', bound_args.get('spec'))
-            if spec_arg and _is_async_obj(spec_arg):
+            if spec_arg is not None and _is_async_obj(spec_arg):
                 bases = (AsyncMockMixin, cls)
         new = type(cls.__name__, bases, {'__doc__': cls.__doc__})
         instance = _safe_super(NonCallableMock, cls).__new__(new)

Misc/NEWS.d/next/Library/2020-12-02-07-37-59.bpo-42532.ObNep_.rst

@@ -0,0 +1 @@
+Remove unexpected call of ``__bool__`` when passing a ``spec_arg`` argument to a Mock.