Array version of allow_value

[douglascamata]

Sometimes you need to see if some values are allowed for a defined field and using the current features you would have to use many allow_value().for() constructs. It would be nice to have a array version, like allow_values('foo', 'bar').for().

[mcmire]

You can say that already:

# will assert that both "foo" and "bar" are valid values for :attr
allow_value("foo", "bar").for(:attr)

The only case where this breaks down is in the should_not case, where you do need to have separate assertions.

Read more: http://thoughtbot.github.io/shoulda-matchers/v2.6.2/Shoulda/Matchers/ActiveModel.html#allow_value-instance_method

[douglascamata]

@mcmire hum, that's cool but not intuitive. I thought allow_values would do something like this, not the singular one. Although, do you think it's possible to implement it to the negative assertion case?

[mcmire]

@douglascamata It is possible but it would break backward compatibility, so we plan on doing that in 3.0.

As for allow_values, I agree that the name doesn't express that this is possible. If you open a PR to make an alias for this (with a test for the alias, like this) then I will happily merge it in.

[douglascamata]

@mcmire I'll happy implement that as soon as I get home 👍

[mcmire]

Closed by #692 -- the alias is now in master.