Don't use FFaker in factories, use sequences instead

Signed-off-by: Rémy Coutable <remy@rymai.me>

db/fixtures/development/18_abuse_reports.rb

 module Db
   module Fixtures
-    class Development
+    module Development
+      class AbuseReport
         def self.seed
           Gitlab::Seeder.quiet do
-          (AbuseReport.default_per_page + 3).times do
+            (::AbuseReport.default_per_page + 3).times do
               reported_user =
-              User.create!(
+                ::User.create!(
                   username: FFaker::Internet.user_name,
                   name: FFaker::Name.name,
                   email: FFaker::Internet.email,
@@ -13,13 +14,14 @@ module Db
                   password: '12345678'
                 )
 
-            AbuseReport.create(reporter: User.take, user: reported_user, message: 'User sends spam')
+              ::AbuseReport.create(reporter: ::User.take, user: reported_user, message: 'User sends spam')
               print '.'
             end
           end
         end
       end
     end
+  end
 end
 
-Db::Fixtures::Development.seed
+Db::Fixtures::Development::AbuseReport.seed

spec/factories/labels.rb

 FactoryGirl.define do
-  factory :label, class: ProjectLabel do
-    title { generate(:label) }
+  trait :base_label do
+    title { generate(:label_title) }
     color "#990000"
+  end
+
+  factory :label, traits: [:base_label], class: ProjectLabel do
     project factory: :empty_project
 
     transient do
@@ -15,9 +18,7 @@ FactoryGirl.define do
     end
   end
 
-  factory :group_label, class: GroupLabel do
-    title { generate(:label) }
-    color "#990000"
+  factory :group_label, traits: [:base_label] do
     group
   end
 end

spec/factories/sequences.rb

@@ -6,7 +6,7 @@ FactoryGirl.define do
   sequence(:title) { |n| "My title #{n}" }
   sequence(:filename) { |n| "filename-#{n}.rb" }
   sequence(:url) { |n| "http://example#{n}.org" }
-  sequence(:label) { |n| "label#{n}" }
+  sequence(:label_title) { |n| "label#{n}" }
   sequence(:branch) { |n| "my-branch-#{n}" }
-  sequence(:issue_created_at) { |n| 4.hours.ago + (2 * n).seconds }
+  sequence(:past_time) { |n| 4.hours.ago + (2 * n).seconds }
 end

spec/requests/api/issues_spec.rb

@@ -19,7 +19,7 @@ describe API::Issues, api: true  do
            project: project,
            state: :closed,
            milestone: milestone,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 3.hours.ago
   end
   let!(:confidential_issue) do
@@ -28,7 +28,7 @@ describe API::Issues, api: true  do
            project: project,
            author: author,
            assignee: assignee,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 2.hours.ago
   end
   let!(:issue) do
@@ -37,7 +37,7 @@ describe API::Issues, api: true  do
            assignee: user,
            project: project,
            milestone: milestone,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 1.hour.ago
   end
   let!(:label) do

spec/requests/api/v3/issues_spec.rb

@@ -19,7 +19,7 @@ describe API::V3::Issues, api: true  do
            project: project,
            state: :closed,
            milestone: milestone,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 3.hours.ago
   end
   let!(:confidential_issue) do
@@ -28,7 +28,7 @@ describe API::V3::Issues, api: true  do
            project: project,
            author: author,
            assignee: assignee,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 2.hours.ago
   end
   let!(:issue) do
@@ -37,7 +37,7 @@ describe API::V3::Issues, api: true  do
            assignee: user,
            project: project,
            milestone: milestone,
-           created_at: generate(:issue_created_at),
+           created_at: generate(:past_time),
            updated_at: 1.hour.ago
   end
   let!(:label) do