Timothy Andrew 918e589c2b Implement a second round of review comments from @DouweM. ... - Don't use `TableReferences` - using `.arel_table` is shorter! - Move some database-related code to `Gitlab::Database` - Remove the `MergeRequest#issues_closed` and `Issue#closed_by_merge_requests` associations. They were either shadowing or were too similar to existing methods. They are not being used anywhere, so it's better to remove them to reduce confusion. - Use Rails 3-style validations - Index for `MergeRequest::Metrics#first_deployed_to_production_at` - Only include `CycleAnalyticsHelpers::TestGeneration` for specs that need it. - Other minor refactorings.		2016-09-21 00:47:37 +05:30
..
code_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
issue_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
plan_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
production_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
review_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
staging_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30
summary_spec.rb	Test the CycleAnalytics::Summary model.	2016-09-20 13:23:14 +05:30
test_spec.rb	Implement a second round of review comments from @DouweM.	2016-09-21 00:47:37 +05:30