gitlab-org--gitlab-foss/spec
Rémy Coutable 5bd4bee00f Merge branch 'remove-callback' into 'master'
This MR removes AR before_validation callback in favor setter.

## Why was this MR needed?

Because setters is good practice to normalize model attributes instead AR callbacks. Because new object should be valid right after initialization. 
If it MR interested I can try to find other places where we can use setters.

See merge request !6763
2016-10-24 10:58:03 +00:00
..
config
controllers Fix status code expectation 2016-10-22 13:46:23 +02:00
factories
features
finders
fixtures
helpers
initializers
javascripts
lib
mailers Move spec/mailers/shared/notify.rb to spec/support 2016-10-22 17:51:50 -05:00
models Merge branch 'remove-callback' into 'master' 2016-10-24 10:58:03 +00:00
policies
requests Fix wrong endpoint in api/users documentation, fix same typo in spec describe blocks 2016-10-21 18:05:36 +03:00
routing
services
support Move spec/mailers/shared/notify.rb to spec/support 2016-10-22 17:51:50 -05:00
tasks/gitlab
uploaders
views
workers Re-organize queues to use for Sidekiq 2016-10-21 18:17:07 +02:00
factories_spec.rb
rails_helper.rb
simplecov_env.rb
spec_helper.rb
teaspoon_env.rb