wip-Development

working on tests next
I don't agree. Here "the thing" the function does is actually call get_admin(...). The purpose of this get_admin_id() function is to provide an easy way to unit test the underlying logic of validating a user log in attempt. We should change the name of the function to reflect this function's purpose.