Remove useless `VALID_AUTOMATIC_INVERSE_MACROS` checking
Remove useless `VALID_AUTOMATIC_INVERSE_MACROS` checking

I suspect the `VALID_AUTOMATIC_INVERSE_MACROS` checking is to avoid
`has_and_belongs_to_many` association, but `has_and_belongs_to_many` is
defined as `has_many` with implicit through association, so all possible
macro variants are `:has_many`, `:has_one`, and `:belongs_to`, therefore
`VALID_AUTOMATIC_INVERSE_MACROS.include?(macro)` is always true, and in
that through association case it is already covered by `options[:through]`
checking.

https://github.com/rails/rails/blob/master/activerecord/lib/active_record/associations.rb#L1955-L1963

esparta/rails,esparta/rails,Stellenticket/rails,rails/rails,Vasfed/rails,Erol/rails,Envek/rails,MSP-Greg/rails,bogdanvlviv/rails,georgeclaghorn/rails,tgxworld/rails,kddeisz/rails,tgxworld/rails,tjschuck/rails,lcreid/rails,MSP-Greg/rails,yawboakye/rails,kddeisz/rails,eileencodes/rails,rails/rails,eileencodes/rails,tjschuck/rails,Envek/rails,MSP-Greg/rails,BlakeWilliams/rails,Erol/rails,tjschuck/rails,Vasfed/rails,Vasfed/rails,eileencodes/rails,Stellenticket/rails,georgeclaghorn/rails,lcreid/rails,Envek/rails,joonyou/rails,Erol/rails,georgeclaghorn/rails,eileencodes/rails,Edouard-chin/rails,joonyou/rails,kddeisz/rails,Envek/rails,yawboakye/rails,yawboakye/rails,Erol/rails,betesh/rails,kddeisz/rails,EmmaB/rails-1,rails/rails,Edouard-chin/rails,Vasfed/rails,kmcphillips/rails,esparta/rails,Stellenticket/rails,BlakeWilliams/rails,bogdanvlviv/rails,BlakeWilliams/rails,esparta/rails,tgxworld/rails,betesh/rails,kmcphillips/rails,Stellenticket/rails,Edouard-chin/rails,EmmaB/rails-1,betesh/rails,yahonda/rails,bogdanvlviv/rails,BlakeWilliams/rails,rails/rails,tgxworld/rails,yahonda/rails,yahonda/rails,MSP-Greg/rails,kmcphillips/rails,georgeclaghorn/rails,tjschuck/rails,lcreid/rails,lcreid/rails,bogdanvlviv/rails,yahonda/rails,Edouard-chin/rails,kmcphillips/rails,joonyou/rails,EmmaB/rails-1,betesh/rails,yawboakye/rails,joonyou/rails