Compare commits

...

4 commits

Author SHA1 Message Date
Devin Howard
98081097b4 Revert "update rails dependencies"
Original commit broke the build, since some dependencies have breaking
changes.

This reverts commit f753392d49.
2018-11-09 07:42:26 -08:00
Devin Howard
f753392d49 update rails dependencies 2018-10-17 19:51:51 -07:00
Connor Turland
38a209a970 small bug fix (#1174)
if someone besides one of "us" tried to change their password, and their settings, it wouldn't work

in the typical case it would work fine
2018-03-11 21:57:23 -07:00
Devin Howard
bd7bf20810
factory_girl => factory_bot (#1172) 2018-03-10 08:10:09 -08:00
16 changed files with 23 additions and 23 deletions

View file

@ -38,7 +38,7 @@ gem 'uglifier'
group :test do group :test do
gem 'brakeman', require: false gem 'brakeman', require: false
gem 'factory_girl_rails' gem 'factory_bot_rails'
gem 'json-schema' gem 'json-schema'
gem 'rspec-rails' gem 'rspec-rails'
gem 'shoulda-matchers' gem 'shoulda-matchers'

View file

@ -105,10 +105,10 @@ GEM
actionmailer (>= 4.0, < 6) actionmailer (>= 4.0, < 6)
activesupport (>= 4.0, < 6) activesupport (>= 4.0, < 6)
execjs (2.7.0) execjs (2.7.0)
factory_girl (4.8.0) factory_bot (4.8.2)
activesupport (>= 3.0.0) activesupport (>= 3.0.0)
factory_girl_rails (4.8.0) factory_bot_rails (4.8.2)
factory_girl (~> 4.8.0) factory_bot (~> 4.8.2)
railties (>= 3.0.0) railties (>= 3.0.0)
faker (1.8.4) faker (1.8.4)
i18n (~> 0.5) i18n (~> 0.5)
@ -311,7 +311,7 @@ DEPENDENCIES
doorkeeper doorkeeper
dotenv-rails dotenv-rails
exception_notification exception_notification
factory_girl_rails factory_bot_rails
faker faker
httparty httparty
jquery-rails jquery-rails

View file

@ -42,7 +42,7 @@ class UsersController < ApplicationController
correct_pass = @user.valid_password?(params[:current_password]) correct_pass = @user.valid_password?(params[:current_password])
if correct_pass && @user.update_attributes(user_params) if correct_pass && @user.update_attributes(user_params)
update_follow_settings(@user, params[:settings]) if is_tester(@user) update_follow_settings(@user, params[:settings])
@user.image = nil if params[:remove_image] == '1' @user.image = nil if params[:remove_image] == '1'
@user.save @user.save
sign_in(@user, bypass: true) sign_in(@user, bypass: true)

View file

@ -26,13 +26,13 @@ At the time of writing, there are four directories in the spec folder. One,
`support`, is for helper functions. `rails_helper.rb` and `spec_helper.rb` are `support`, is for helper functions. `rails_helper.rb` and `spec_helper.rb` are
also for helper functions. also for helper functions.
`factories` is for a gem called [factory-girl][factory-girl]. This gem lets you `factories` is for a gem called [factory-bot][factory_bot]. This gem lets you
use the `create` and `build` functions to quickly create the simplest possible use the `create` and `build` functions to quickly create the simplest possible
valid version of a given model. For instance: valid version of a given model. For instance:
let(:map1) { create :map } let(:map1) { create :map }
let(:ronald) { create :user, name: "Ronald" } let(:alex) { create :user, name: "Alex" }
let(:map2) { create :map, user: ronald } let(:map2) { create :map, user: alex }
As you can see, you can also customize the factories. You can read the full As you can see, you can also customize the factories. You can read the full
documentation at the link above or check the existing specs to see how it works. documentation at the link above or check the existing specs to see how it works.
@ -53,5 +53,5 @@ the added code works. This will help in a few ways:
Happy testing! Happy testing!
[factory-girl]: https://github.com/thoughtbot/factory_girl [factory-bot]: https://github.com/thoughtbot/factory_bot
[rspec-docs]: http://rspec.info [rspec-docs]: http://rspec.info

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :access_request do factory :access_request do
map map
user user

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :mapping do factory :mapping do
xloc 0 xloc 0
yloc 0 yloc 0

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :map do factory :map do
sequence(:name) { |n| "Cool Map ##{n}" } sequence(:name) { |n| "Cool Map ##{n}" }
permission :commons permission :commons

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :message do factory :message do
association :resource, factory: :map association :resource, factory: :map
user user

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :metacode do factory :metacode do
sequence(:name) { |n| "Cool Metacode ##{n}" } sequence(:name) { |n| "Cool Metacode ##{n}" }
manual_icon 'https://images.com/image.png' manual_icon 'https://images.com/image.png'

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :star do factory :star do
end end
end end

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :synapse do factory :synapse do
sequence(:desc) { |n| "Cool synapse ##{n}" } sequence(:desc) { |n| "Cool synapse ##{n}" }
category :'from-to' category :'from-to'

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :token do factory :token do
user user
description '' description ''

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :topic do factory :topic do
user user
association :updated_by, factory: :user association :updated_by, factory: :user

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
FactoryGirl.define do FactoryBot.define do
factory :user_map do factory :user_map do
map map
user user

View file

@ -10,7 +10,7 @@
# have actual codes, you'll need to specify one simple_user and then you # have actual codes, you'll need to specify one simple_user and then you
# can specify other :code_user users based on the pre-existing user's code. # can specify other :code_user users based on the pre-existing user's code.
FactoryGirl.define do FactoryBot.define do
factory :code_user, class: User do factory :code_user, class: User do
sequence(:name) { |n| "Cool User ##{n}" } sequence(:name) { |n| "Cool User ##{n}" }
sequence(:email) { |n| "cooluser#{n}@cooldomain.com" } sequence(:email) { |n| "cooluser#{n}@cooldomain.com" }

View file

@ -1,6 +1,6 @@
# frozen_string_literal: true # frozen_string_literal: true
# lets you type create(:user) instead of FactoryGirl.create(:user) # lets you type create(:user) instead of FactoryBot.create(:user)
RSpec.configure do |config| RSpec.configure do |config|
config.include FactoryGirl::Syntax::Methods config.include FactoryBot::Syntax::Methods
end end