Commit b23025b6 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'rs-gems' into 'master'

Update ffaker gem

Adds a version requirement which it didn't have before, at 2.0.0. This version has breaking API changes in that the namespace is now `FFaker` instead of `Faker`.

Prior, if a new developer checked out the repo, ran `bundle install` and then tried `rake dev:setup`, seeding would fail due to having the latest version of ffaker without the API changes in this MR.

Also updates `spring` version and the binstubs, just for kicks.

See merge request !686
parents 08102ad7 11af5161
...@@ -239,7 +239,7 @@ group :development, :test do ...@@ -239,7 +239,7 @@ group :development, :test do
gem 'minitest', '~> 5.3.0' gem 'minitest', '~> 5.3.0'
# Generate Fake data # Generate Fake data
gem "ffaker" gem 'ffaker', '~> 2.0.0'
# Guard # Guard
gem 'guard-rspec' gem 'guard-rspec'
......
...@@ -176,7 +176,7 @@ GEM ...@@ -176,7 +176,7 @@ GEM
faraday_middleware (0.9.0) faraday_middleware (0.9.0)
faraday (>= 0.7.4, < 0.9) faraday (>= 0.7.4, < 0.9)
fastercsv (1.5.5) fastercsv (1.5.5)
ffaker (1.22.1) ffaker (2.0.0)
ffi (1.9.8) ffi (1.9.8)
fog (1.21.0) fog (1.21.0)
fog-brightbox fog-brightbox
...@@ -547,9 +547,9 @@ GEM ...@@ -547,9 +547,9 @@ GEM
sdoc (0.3.20) sdoc (0.3.20)
json (>= 1.1.3) json (>= 1.1.3)
rdoc (~> 3.10) rdoc (~> 3.10)
seed-fu (2.3.1) seed-fu (2.3.5)
activerecord (>= 3.1, < 4.2) activerecord (>= 3.1, < 4.3)
activesupport (>= 3.1, < 4.2) activesupport (>= 3.1, < 4.3)
select2-rails (3.5.2) select2-rails (3.5.2)
thor (~> 0.14) thor (~> 0.14)
settingslogic (2.0.9) settingslogic (2.0.9)
...@@ -589,7 +589,7 @@ GEM ...@@ -589,7 +589,7 @@ GEM
capybara (>= 2.0.0) capybara (>= 2.0.0)
railties (>= 3) railties (>= 3)
spinach (>= 0.4) spinach (>= 0.4)
spring (1.3.3) spring (1.3.6)
spring-commands-rspec (1.0.4) spring-commands-rspec (1.0.4)
spring (>= 0.9.1) spring (>= 0.9.1)
spring-commands-spinach (1.0.0) spring-commands-spinach (1.0.0)
...@@ -713,7 +713,7 @@ DEPENDENCIES ...@@ -713,7 +713,7 @@ DEPENDENCIES
email_spec email_spec
enumerize enumerize
factory_girl_rails factory_girl_rails
ffaker ffaker (~> 2.0.0)
fog (~> 1.14) fog (~> 1.14)
font-awesome-rails (~> 4.2) font-awesome-rails (~> 4.2)
foreman foreman
......
...@@ -3,6 +3,5 @@ begin ...@@ -3,6 +3,5 @@ begin
load File.expand_path("../spring", __FILE__) load File.expand_path("../spring", __FILE__)
rescue LoadError rescue LoadError
end end
require_relative '../config/boot' require 'bundler/setup'
require 'rake' load Gem.bin_path('rake', 'rake')
Rake.application.run
#!/usr/bin/env ruby #!/usr/bin/env ruby
# This file loads spring without using Bundler, in order to be fast # This file loads spring without using Bundler, in order to be fast.
# It gets overwritten when you run the `spring binstub` command # It gets overwritten when you run the `spring binstub` command.
unless defined?(Spring) unless defined?(Spring)
require "rubygems" require "rubygems"
require "bundler" require "bundler"
if match = Bundler.default_lockfile.read.match(/^GEM$.*?^ spring \((.*?)\)$.*?^$/m) if match = Bundler.default_lockfile.read.match(/^GEM$.*?^ (?: )*spring \((.*?)\)$.*?^$/m)
ENV["GEM_PATH"] = ([Bundler.bundle_path.to_s] + Gem.path).join(File::PATH_SEPARATOR) Gem.paths = { "GEM_PATH" => [Bundler.bundle_path.to_s, *Gem.path].uniq }
ENV["GEM_HOME"] = ""
Gem.paths = ENV
gem "spring", match[1] gem "spring", match[1]
require "spring/binstub" require "spring/binstub"
end end
......
...@@ -23,7 +23,7 @@ Sidekiq::Testing.inline! do ...@@ -23,7 +23,7 @@ Sidekiq::Testing.inline! do
name: group_path.titleize, name: group_path.titleize,
path: group_path path: group_path
) )
group.description = Faker::Lorem.sentence group.description = FFaker::Lorem.sentence
group.save group.save
group.add_owner(User.first) group.add_owner(User.first)
...@@ -35,7 +35,7 @@ Sidekiq::Testing.inline! do ...@@ -35,7 +35,7 @@ Sidekiq::Testing.inline! do
import_url: url, import_url: url,
namespace_id: group.id, namespace_id: group.id,
name: project_path.titleize, name: project_path.titleize,
description: Faker::Lorem.sentence, description: FFaker::Lorem.sentence,
visibility_level: Gitlab::VisibilityLevel.values.sample visibility_level: Gitlab::VisibilityLevel.values.sample
} }
......
...@@ -2,9 +2,9 @@ Gitlab::Seeder.quiet do ...@@ -2,9 +2,9 @@ Gitlab::Seeder.quiet do
(2..20).each do |i| (2..20).each do |i|
begin begin
User.create!( User.create!(
username: Faker::Internet.user_name, username: FFaker::Internet.user_name,
name: Faker::Name.name, name: FFaker::Name.name,
email: Faker::Internet.email, email: FFaker::Internet.email,
confirmed_at: DateTime.now, confirmed_at: DateTime.now,
password: '12345678' password: '12345678'
) )
......
...@@ -3,7 +3,7 @@ Gitlab::Seeder.quiet do ...@@ -3,7 +3,7 @@ Gitlab::Seeder.quiet do
(1..5).each do |i| (1..5).each do |i|
milestone_params = { milestone_params = {
title: "v#{i}.0", title: "v#{i}.0",
description: Faker::Lorem.sentence, description: FFaker::Lorem.sentence,
state: ['opened', 'closed'].sample, state: ['opened', 'closed'].sample,
} }
......
...@@ -2,8 +2,8 @@ Gitlab::Seeder.quiet do ...@@ -2,8 +2,8 @@ Gitlab::Seeder.quiet do
Project.all.each do |project| Project.all.each do |project|
(1..10).each do |i| (1..10).each do |i|
issue_params = { issue_params = {
title: Faker::Lorem.sentence(6), title: FFaker::Lorem.sentence(6),
description: Faker::Lorem.sentence, description: FFaker::Lorem.sentence,
state: ['opened', 'closed'].sample, state: ['opened', 'closed'].sample,
milestone: project.milestones.sample, milestone: project.milestones.sample,
assignee: project.team.users.sample assignee: project.team.users.sample
......
...@@ -10,8 +10,8 @@ Gitlab::Seeder.quiet do ...@@ -10,8 +10,8 @@ Gitlab::Seeder.quiet do
params = { params = {
source_branch: source_branch, source_branch: source_branch,
target_branch: target_branch, target_branch: target_branch,
title: Faker::Lorem.sentence(6), title: FFaker::Lorem.sentence(6),
description: Faker::Lorem.sentences(3).join(" "), description: FFaker::Lorem.sentences(3).join(" "),
milestone: project.milestones.sample, milestone: project.milestones.sample,
assignee: project.team.users.sample assignee: project.team.users.sample
} }
......
...@@ -28,8 +28,8 @@ eos ...@@ -28,8 +28,8 @@ eos
PersonalSnippet.seed(:id, [{ PersonalSnippet.seed(:id, [{
id: i, id: i,
author_id: user.id, author_id: user.id,
title: Faker::Lorem.sentence(3), title: FFaker::Lorem.sentence(3),
file_name: Faker::Internet.domain_word + '.rb', file_name: FFaker::Internet.domain_word + '.rb',
visibility_level: Gitlab::VisibilityLevel.values.sample, visibility_level: Gitlab::VisibilityLevel.values.sample,
content: content, content: content,
}]) }])
......
...@@ -6,7 +6,7 @@ Gitlab::Seeder.quiet do ...@@ -6,7 +6,7 @@ Gitlab::Seeder.quiet do
note_params = { note_params = {
noteable_type: 'Issue', noteable_type: 'Issue',
noteable_id: issue.id, noteable_id: issue.id,
note: Faker::Lorem.sentence, note: FFaker::Lorem.sentence,
} }
Notes::CreateService.new(project, user, note_params).execute Notes::CreateService.new(project, user, note_params).execute
...@@ -21,7 +21,7 @@ Gitlab::Seeder.quiet do ...@@ -21,7 +21,7 @@ Gitlab::Seeder.quiet do
note_params = { note_params = {
noteable_type: 'MergeRequest', noteable_type: 'MergeRequest',
noteable_id: mr.id, noteable_id: mr.id,
note: Faker::Lorem.sentence, note: FFaker::Lorem.sentence,
} }
Notes::CreateService.new(project, user, note_params).execute Notes::CreateService.new(project, user, note_params).execute
......
...@@ -23,7 +23,7 @@ class Spinach::Features::ProjectHooks < Spinach::FeatureSteps ...@@ -23,7 +23,7 @@ class Spinach::Features::ProjectHooks < Spinach::FeatureSteps
end end
step 'I submit new hook' do step 'I submit new hook' do
@url = Faker::Internet.uri("http") @url = FFaker::Internet.uri("http")
fill_in "hook_url", with: @url fill_in "hook_url", with: @url
expect { click_button "Add Web Hook" }.to change(ProjectHook, :count).by(1) expect { click_button "Add Web Hook" }.to change(ProjectHook, :count).by(1)
end end
......
...@@ -2,23 +2,23 @@ include ActionDispatch::TestProcess ...@@ -2,23 +2,23 @@ include ActionDispatch::TestProcess
FactoryGirl.define do FactoryGirl.define do
sequence :sentence, aliases: [:title, :content] do sequence :sentence, aliases: [:title, :content] do
Faker::Lorem.sentence FFaker::Lorem.sentence
end end
sequence :name do sequence :name do
Faker::Name.name FFaker::Name.name
end end
sequence :file_name do sequence :file_name do
Faker::Internet.user_name FFaker::Internet.user_name
end end
sequence(:url) { Faker::Internet.uri('http') } sequence(:url) { FFaker::Internet.uri('http') }
factory :user, aliases: [:author, :assignee, :owner, :creator] do factory :user, aliases: [:author, :assignee, :owner, :creator] do
email { Faker::Internet.email } email { FFaker::Internet.email }
name name
sequence(:username) { |n| "#{Faker::Internet.user_name}#{n}" } sequence(:username) { |n| "#{FFaker::Internet.user_name}#{n}" }
password "12345678" password "12345678"
confirmed_at { Time.now } confirmed_at { Time.now }
confirmation_token { nil } confirmation_token { nil }
...@@ -122,12 +122,12 @@ FactoryGirl.define do ...@@ -122,12 +122,12 @@ FactoryGirl.define do
factory :email do factory :email do
user user
email do email do
Faker::Internet.email('alias') FFaker::Internet.email('alias')
end end
factory :another_email do factory :another_email do
email do email do
Faker::Internet.email('another.alias') FFaker::Internet.email('another.alias')
end end
end end
end end
......
...@@ -26,7 +26,7 @@ describe "Admin::Hooks", feature: true do ...@@ -26,7 +26,7 @@ describe "Admin::Hooks", feature: true do
describe "New Hook" do describe "New Hook" do
before do before do
@url = Faker::Internet.uri("http") @url = FFaker::Internet.uri("http")
visit admin_hooks_path visit admin_hooks_path
fill_in "hook_url", with: @url fill_in "hook_url", with: @url
expect { click_button "Add System Hook" }.to change(SystemHook, :count).by(1) expect { click_button "Add System Hook" }.to change(SystemHook, :count).by(1)
......
...@@ -185,7 +185,7 @@ describe Notify do ...@@ -185,7 +185,7 @@ describe Notify do
context 'for issues' do context 'for issues' do
let(:issue) { create(:issue, author: current_user, assignee: assignee, project: project) } let(:issue) { create(:issue, author: current_user, assignee: assignee, project: project) }
let(:issue_with_description) { create(:issue, author: current_user, assignee: assignee, project: project, description: Faker::Lorem.sentence) } let(:issue_with_description) { create(:issue, author: current_user, assignee: assignee, project: project, description: FFaker::Lorem.sentence) }
describe 'that are new' do describe 'that are new' do
subject { Notify.new_issue_email(issue.assignee_id, issue.id) } subject { Notify.new_issue_email(issue.assignee_id, issue.id) }
...@@ -273,7 +273,7 @@ describe Notify do ...@@ -273,7 +273,7 @@ describe Notify do
context 'for merge requests' do context 'for merge requests' do
let(:merge_author) { create(:user) } let(:merge_author) { create(:user) }
let(:merge_request) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project) } let(:merge_request) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project) }
let(:merge_request_with_description) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project, description: Faker::Lorem.sentence) } let(:merge_request_with_description) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project, description: FFaker::Lorem.sentence) }
describe 'that are new' do describe 'that are new' do
subject { Notify.new_merge_request_email(merge_request.assignee_id, merge_request.id) } subject { Notify.new_merge_request_email(merge_request.assignee_id, merge_request.id) }
......
...@@ -165,7 +165,7 @@ describe API::API, api: true do ...@@ -165,7 +165,7 @@ describe API::API, api: true do
it "should assign attributes to project" do it "should assign attributes to project" do
project = attributes_for(:project, { project = attributes_for(:project, {
path: 'camelCasePath', path: 'camelCasePath',
description: Faker::Lorem.sentence, description: FFaker::Lorem.sentence,
issues_enabled: false, issues_enabled: false,
merge_requests_enabled: false, merge_requests_enabled: false,
wiki_enabled: false wiki_enabled: false
...@@ -274,7 +274,7 @@ describe API::API, api: true do ...@@ -274,7 +274,7 @@ describe API::API, api: true do
it 'should assign attributes to project' do it 'should assign attributes to project' do
project = attributes_for(:project, { project = attributes_for(:project, {
description: Faker::Lorem.sentence, description: FFaker::Lorem.sentence,
issues_enabled: false, issues_enabled: false,
merge_requests_enabled: false, merge_requests_enabled: false,
wiki_enabled: false wiki_enabled: false
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment