Remove backward compatibility of factories.

6 files changed, 30 insertions, 30 deletions

diff --git a/spec/requests/api/issues_spec.rb b/spec/requests/api/issues_spec.rb
index 442e9c730dc..6ea7e9b5579 100644
--- a/spec/requests/api/issues_spec.rb
+++ b/spec/requests/api/issues_spec.rb
@@ -3,9 +3,9 @@ require 'spec_helper'
 describe Gitlab::API do
   include ApiHelpers
 
-  let(:user) { Factory :user }
-  let!(:project) { Factory :project, owner: user }
-  let!(:issue) { Factory :issue, author: user, assignee: user, project: project }
+  let(:user) { create(:user) }
+  let!(:project) { create(:project, owner: user) }
+  let!(:issue) { create(:issue, author: user, assignee: user, project: project) }
   before { project.add_access(user, :read) }
 
   describe "GET /issues" do
diff --git a/spec/requests/api/merge_requests_spec.rb b/spec/requests/api/merge_requests_spec.rb
index e1c7949f70c..e83f24671ed 100644
--- a/spec/requests/api/merge_requests_spec.rb
+++ b/spec/requests/api/merge_requests_spec.rb
@@ -1,11 +1,11 @@
 require "spec_helper"
 
-describe Gitlab::API do 
+describe Gitlab::API do
   include ApiHelpers
-  
-  let(:user) { Factory :user }
-  let!(:project) { Factory :project, owner: user }
-  let!(:merge_request) { Factory :merge_request, author: user, assignee: user, project: project, title: "Test" }
+
+  let(:user) { create(:user ) }
+  let!(:project) { create(:project, owner: user) }
+  let!(:merge_request) { create(:merge_request, author: user, assignee: user, project: project, title: "Test") }
   before { project.add_access(user, :read) }
 
   describe "GET /projects/:id/merge_requests" do
@@ -39,7 +39,7 @@ describe Gitlab::API do
       post api("/projects/#{project.code}/merge_requests", user),
         title: 'Test merge_request', source_branch: "stable", target_branch: "master", author: user
       response.status.should == 201
-      json_response['title'].should == 'Test merge_request' 
+      json_response['title'].should == 'Test merge_request'
     end
   end
 
@@ -47,7 +47,7 @@ describe Gitlab::API do
     it "should return merge_request" do
       put api("/projects/#{project.code}/merge_request/#{merge_request.id}", user), title: "New title"
       response.status.should == 200
-      json_response['title'].should == 'New title' 
+      json_response['title'].should == 'New title'
     end
   end
 
@@ -55,7 +55,7 @@ describe Gitlab::API do
     it "should return comment" do
       post api("/projects/#{project.code}/merge_request/#{merge_request.id}/comments", user), note: "My comment"
       response.status.should == 201
-      json_response['note'].should == 'My comment' 
+      json_response['note'].should == 'My comment'
     end
   end
 
diff --git a/spec/requests/api/milestones_spec.rb b/spec/requests/api/milestones_spec.rb
index cf5f65f068c..860825ab2db 100644
--- a/spec/requests/api/milestones_spec.rb
+++ b/spec/requests/api/milestones_spec.rb
@@ -3,9 +3,9 @@ require 'spec_helper'
 describe Gitlab::API do
   include ApiHelpers
 
-  let(:user) { Factory :user }
-  let!(:project) { Factory :project, owner: user }
-  let!(:milestone) { Factory :milestone, project: project }
+  let(:user) { create(:user) }
+  let!(:project) { create(:project, owner: user) }
+  let!(:milestone) { create(:milestone, project: project) }
 
   before { project.add_access(user, :read) }
 
diff --git a/spec/requests/api/projects_spec.rb b/spec/requests/api/projects_spec.rb
index 5f9a587d57a..d24ce43d3f2 100644
--- a/spec/requests/api/projects_spec.rb
+++ b/spec/requests/api/projects_spec.rb
@@ -3,14 +3,14 @@ require 'spec_helper'
 describe Gitlab::API do
   include ApiHelpers
 
-  let(:user) { Factory :user }
-  let(:user2) { Factory.create(:user) }
-  let(:user3) { Factory.create(:user) }
-  let!(:hook) { Factory :project_hook, project: project, url: "http://example.com" }
-  let!(:project) { Factory :project, owner: user }
-  let!(:snippet) { Factory :snippet, author: user, project: project, title: 'example' }
-  let!(:users_project) { Factory :users_project, user: user, project: project, project_access: UsersProject::MASTER  }
-  let!(:users_project2) { Factory :users_project, user: user3, project: project, project_access: UsersProject::DEVELOPER  }
+  let(:user) { create(:user) }
+  let(:user2) { create(:user) }
+  let(:user3) { create(:user) }
+  let!(:hook) { create(:project_hook, project: project, url: "http://example.com") }
+  let!(:project) { create(:project, owner: user ) }
+  let!(:snippet) { create(:snippet, author: user, project: project, title: 'example') }
+  let!(:users_project) { create(:users_project, user: user, project: project, project_access: UsersProject::MASTER) }
+  let!(:users_project2) { create(:users_project, user: user3, project: project, project_access: UsersProject::DEVELOPER) }
   before { project.add_access(user, :read) }
 
   describe "GET /projects" do
@@ -52,7 +52,7 @@ describe Gitlab::API do
     end
 
     it "should assign attributes to project" do
-      project = Factory.attributes(:project, {
+      project = attributes_for(:project, {
         path: 'path',
         code: 'code',
         description: Faker::Lorem.sentence,
diff --git a/spec/requests/api/session_spec.rb b/spec/requests/api/session_spec.rb
index f251f3921ac..afae8be8cbc 100644
--- a/spec/requests/api/session_spec.rb
+++ b/spec/requests/api/session_spec.rb
@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Gitlab::API do
   include ApiHelpers
 
-  let(:user) { Factory :user }
+  let(:user) { create(:user) }
 
   describe "POST /session" do
     context "when valid password" do
diff --git a/spec/requests/api/users_spec.rb b/spec/requests/api/users_spec.rb
index 4c2e6adaf7f..4cfb4884e7c 100644
--- a/spec/requests/api/users_spec.rb
+++ b/spec/requests/api/users_spec.rb
@@ -3,9 +3,9 @@ require 'spec_helper'
 describe Gitlab::API do
   include ApiHelpers
 
-  let(:user)  { Factory :user }
-  let(:admin) { Factory :admin }
-  let(:key)   { Factory :key, user: user }
+  let(:user)  { create(:user) }
+  let(:admin) { create(:admin) }
+  let(:key)   { create(:key, user: user) }
 
   describe "GET /users" do
     context "when unauthenticated" do
@@ -43,12 +43,12 @@ describe Gitlab::API do
 
     it "should create user" do
       expect {
-        post api("/users", admin), Factory.attributes(:user, projects_limit: 3)
+        post api("/users", admin), attributes_for(:user, projects_limit: 3)
       }.to change { User.count }.by(1)
     end
 
     it "shouldn't available for non admin users" do
-      post api("/users", user), Factory.attributes(:user)
+      post api("/users", user), attributes_for(:user)
       response.status.should == 403
     end
   end
@@ -103,7 +103,7 @@ describe Gitlab::API do
     end
 
     it "should create ssh key" do
-      key_attrs = Factory.attributes :key
+      key_attrs = attributes_for :key
       expect {
         post api("/user/keys", user), key_attrs
       }.to change{ user.keys.count }.by(1)