Fixed `columns_for_distinct` of postgresql adapter
add a test for existing mapper functionality

I'm not sure if this is actually used, but I'm adding a test to define
the behavior
set defaults at the top so we can avoid the ||= test
extract controller checks to methods
reduce blank? checks
reduce action.blank? calls
fewer blank? calls
invert logic to remove nil? and exclude? checks (use ruby rather than AS when possible
trade 2 is_a? checks for a nil check
return early if we have a valid controller name
swtich to returning early if to responds to call
we don't need the call to presence. that is my present, to you!
extract controller and action parsing to a method
only do one nil check against the controller
change to case / when on types
only one nil check on the action variable
translate action / controller to the desired object
move nil check to a method that yields to a block if the value is not nil
add a test for controllers without colons
only error handling between controller and action is the same