At the end of new(), there is a test command, show-keychain-info.
On 10.11, this commands outputs
Keychain "<NULL>" no-timeout
because the keychain parameter is not given.
I don't see the reason to have the command here, so I would recommend just to remove it.
If there is a need, I would suggest using the
default-keychain parameter to obtain the default keychain.
Which one would you prefer? I can send a patch for either.