fat_free_crm 0.17.3 → 0.19.2
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of fat_free_crm might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/.rubocop.yml +1 -1
- data/.rubocop_todo.yml +71 -148
- data/.travis.yml +35 -14
- data/CHANGELOG.md +92 -6
- data/CONTRIBUTORS.md +96 -53
- data/Dockerfile +45 -14
- data/Gemfile +23 -13
- data/Gemfile.lock +256 -248
- data/Procfile +1 -1
- data/README.md +9 -6
- data/Rakefile +1 -1
- data/app/assets/javascripts/crm.js.coffee +3 -3
- data/app/assets/javascripts/crm_select2.js.coffee +15 -14
- data/app/assets/stylesheets/common.scss +1 -1
- data/app/controllers/admin/application_controller.rb +1 -1
- data/app/controllers/admin/field_groups_controller.rb +9 -4
- data/app/controllers/admin/fields_controller.rb +4 -4
- data/app/controllers/admin/groups_controller.rb +1 -1
- data/app/controllers/admin/tags_controller.rb +2 -4
- data/app/controllers/admin/users_controller.rb +5 -8
- data/app/controllers/application_controller.rb +22 -45
- data/app/controllers/comments_controller.rb +16 -11
- data/{config/initializers/authlogic.rb → app/controllers/confirmations_controller.rb} +4 -2
- data/app/controllers/emails_controller.rb +0 -2
- data/app/controllers/entities/accounts_controller.rb +1 -3
- data/app/controllers/entities/campaigns_controller.rb +8 -5
- data/app/controllers/entities/contacts_controller.rb +4 -24
- data/app/controllers/entities/leads_controller.rb +16 -12
- data/app/controllers/entities/opportunities_controller.rb +17 -16
- data/app/controllers/entities_controller.rb +31 -12
- data/app/controllers/home_controller.rb +2 -4
- data/app/controllers/lists_controller.rb +5 -1
- data/app/controllers/passwords_controller.rb +3 -59
- data/{spec/features/support/maintain_sessions.rb → app/controllers/registrations_controller.rb} +12 -5
- data/{lib/development_tasks/gem.rake → app/controllers/sessions_controller.rb} +6 -6
- data/app/controllers/tasks_controller.rb +22 -17
- data/app/controllers/users_controller.rb +8 -29
- data/app/helpers/accounts_helper.rb +1 -1
- data/app/helpers/admin/users_helper.rb +1 -1
- data/app/helpers/application_helper.rb +28 -33
- data/app/helpers/campaigns_helper.rb +1 -1
- data/app/helpers/contacts_helper.rb +1 -3
- data/app/helpers/leads_helper.rb +1 -1
- data/app/helpers/opportunities_helper.rb +48 -3
- data/app/helpers/tasks_helper.rb +1 -1
- data/app/helpers/users_helper.rb +1 -3
- data/{config/initializers/paper_trail.rb → app/mailers/devise_mailer.rb} +5 -1
- data/app/mailers/user_mailer.rb +0 -9
- data/app/models/entities/account.rb +10 -10
- data/app/models/entities/campaign.rb +4 -6
- data/app/models/entities/contact.rb +24 -12
- data/app/models/entities/lead.rb +7 -14
- data/app/models/entities/opportunity.rb +10 -11
- data/app/models/fields/custom_field.rb +1 -0
- data/app/models/fields/custom_field_date_pair.rb +2 -0
- data/app/models/fields/field.rb +1 -3
- data/app/models/list.rb +1 -1
- data/app/models/observers/entity_observer.rb +3 -7
- data/app/models/observers/lead_observer.rb +2 -4
- data/app/models/observers/opportunity_observer.rb +5 -7
- data/app/models/observers/task_observer.rb +1 -1
- data/app/models/polymorphic/email.rb +2 -2
- data/app/models/polymorphic/task.rb +13 -9
- data/app/models/polymorphic/version.rb +3 -2
- data/app/models/setting.rb +2 -0
- data/app/models/users/ability.rb +3 -4
- data/app/models/users/permission.rb +3 -3
- data/app/models/users/preference.rb +2 -1
- data/app/models/users/user.rb +67 -42
- data/app/views/accounts/_top_section.html.haml +1 -1
- data/app/views/accounts/edit.js.haml +1 -1
- data/app/views/accounts/update.js.haml +2 -2
- data/app/views/admin/users/_user.html.haml +4 -4
- data/app/views/campaigns/_metrics.html.haml +3 -3
- data/app/views/contacts/_index_brief.html.haml +1 -1
- data/app/views/contacts/_index_full.html.haml +1 -1
- data/app/views/contacts/_index_long.html.haml +1 -1
- data/app/views/devise/confirmations/new.html.haml +9 -0
- data/app/views/devise/mailer/confirmation_instructions.html.haml +4 -0
- data/app/views/devise/mailer/password_change.html.haml +3 -0
- data/app/views/devise/mailer/reset_password_instructions.html.haml +6 -0
- data/app/views/devise/passwords/edit.html.haml +18 -0
- data/app/views/devise/passwords/new.html.haml +10 -0
- data/app/views/devise/registrations/new.html.haml +21 -0
- data/app/views/devise/sessions/new.html.haml +32 -0
- data/app/views/home/_opportunity.html.haml +4 -19
- data/app/views/layouts/_about.html.haml +5 -5
- data/app/views/layouts/_header.html.haml +3 -3
- data/app/views/layouts/admin/_header.html.haml +1 -1
- data/app/views/opportunities/_index_long.html.haml +1 -24
- data/app/views/opportunities/_sidebar_show.html.haml +3 -3
- data/app/views/opportunities/_top_section.html.haml +1 -1
- data/app/views/shared/_address.html.haml +5 -5
- data/app/views/shared/_empty.html.haml +1 -1
- data/app/views/shared/_paginate_with_per_page.html.haml +1 -0
- data/app/views/users/_avatar.html.haml +1 -1
- data/bin/bundle +1 -1
- data/bin/rails +1 -1
- data/bin/setup +38 -0
- data/bin/update +33 -0
- data/bin/yarn +13 -0
- data/config/application.rb +8 -6
- data/config/boot.rb +1 -1
- data/config/brakeman.ignore +2 -2
- data/config/database.postgres.docker.yml +5 -5
- data/config/environment.rb +1 -1
- data/config/environments/development.rb +1 -0
- data/config/environments/test.rb +7 -0
- data/config/initializers/action_mailer.rb +1 -3
- data/config/initializers/application_controller_renderer.rb +9 -0
- data/config/initializers/assets.rb +6 -11
- data/config/initializers/backtrace_silencers.rb +0 -6
- data/config/initializers/content_security_policy.rb +26 -0
- data/config/initializers/cookies_serializer.rb +3 -6
- data/config/initializers/devise.rb +289 -0
- data/config/initializers/filter_parameter_logging.rb +0 -5
- data/config/initializers/gravatar.rb +0 -1
- data/config/initializers/inflections.rb +0 -6
- data/config/initializers/mime_types.rb +1 -9
- data/config/initializers/new_framework_defaults_5_2.rb +40 -0
- data/config/initializers/relative_url_root.rb +1 -3
- data/config/initializers/session_store.rb +1 -3
- data/config/initializers/wrap_parameters.rb +4 -9
- data/config/locales/fat_free_crm.en-GB.yml +5 -5
- data/config/locales/fat_free_crm.en-US.yml +5 -5
- data/config/locales/fat_free_crm.fr.yml +1 -1
- data/config/locales/fat_free_crm.ru.yml +1 -0
- data/config/routes.rb +20 -9
- data/db/demo/users.yml +62 -81
- data/db/migrate/20100928030620_remove_uuid.rb +1 -2
- data/db/migrate/20120316045804_activities_to_versions.rb +1 -0
- data/db/migrate/20120510025219_add_not_null_constraints_for_timestamp_columns.rb +1 -0
- data/db/migrate/20180107082701_authlogic_to_devise.rb +58 -0
- data/db/schema.rb +48 -46
- data/docker-compose.yml +10 -0
- data/fat_free_crm.gemspec +12 -14
- data/lib/development_tasks/license.rake +2 -2
- data/lib/fat_free_crm/callback.rb +2 -2
- data/lib/fat_free_crm/comment_extensions.rb +2 -4
- data/lib/fat_free_crm/core_ext/string.rb +1 -1
- data/lib/fat_free_crm/engine.rb +2 -2
- data/lib/fat_free_crm/errors.rb +1 -1
- data/lib/fat_free_crm/export_csv.rb +1 -0
- data/lib/fat_free_crm/exportable.rb +1 -1
- data/lib/fat_free_crm/fields.rb +2 -2
- data/lib/fat_free_crm/gem_dependencies.rb +1 -1
- data/lib/fat_free_crm/gem_ext/simple_form/action_view_extensions/form_helper.rb +1 -3
- data/lib/fat_free_crm/i18n.rb +2 -2
- data/lib/fat_free_crm/mail_processor/base.rb +4 -10
- data/lib/fat_free_crm/mail_processor/dropbox.rb +5 -15
- data/lib/fat_free_crm/permissions.rb +7 -18
- data/lib/fat_free_crm/sortable.rb +1 -1
- data/lib/fat_free_crm/tabs.rb +2 -2
- data/lib/fat_free_crm/version.rb +2 -2
- data/lib/gravatar_image_tag.rb +7 -8
- data/lib/missing_translation_detector.rb +1 -0
- data/lib/tasks/ffcrm/missing_translations.rake +1 -0
- data/lib/tasks/ffcrm/setup.rake +13 -4
- data/lib/tasks/ffcrm/update_data.rake +2 -2
- data/script/rails +2 -2
- data/spec/controllers/admin/users_controller_spec.rb +25 -81
- data/spec/controllers/comments_controller_spec.rb +19 -19
- data/spec/controllers/emails_controller_spec.rb +2 -2
- data/spec/controllers/entities/accounts_controller_spec.rb +56 -56
- data/spec/controllers/entities/campaigns_controller_spec.rb +66 -66
- data/spec/controllers/entities/contacts_controller_spec.rb +69 -68
- data/spec/controllers/entities/leads_controller_spec.rb +126 -126
- data/spec/controllers/entities/opportunities_controller_spec.rb +101 -101
- data/spec/controllers/entities_controller_spec.rb +5 -0
- data/spec/controllers/home_controller_spec.rb +30 -30
- data/spec/controllers/tasks_controller_spec.rb +42 -40
- data/spec/controllers/users_controller_spec.rb +43 -113
- data/spec/factories/account_factories.rb +13 -13
- data/spec/factories/campaign_factories.rb +8 -8
- data/spec/factories/contact_factories.rb +18 -18
- data/spec/factories/field_factories.rb +11 -10
- data/spec/factories/lead_factories.rb +13 -13
- data/spec/factories/list_factories.rb +3 -3
- data/spec/factories/opportunity_factories.rb +9 -9
- data/spec/factories/sequences.rb +1 -1
- data/spec/factories/setting_factories.rb +5 -5
- data/spec/factories/shared_factories.rb +25 -23
- data/spec/factories/subscription_factories.rb +1 -1
- data/spec/factories/tag_factories.rb +1 -1
- data/spec/factories/task_factories.rb +11 -11
- data/spec/factories/user_factories.rb +27 -30
- data/spec/features/accounts_spec.rb +17 -4
- data/spec/features/admin/groups_spec.rb +2 -2
- data/spec/features/admin/users_spec.rb +4 -2
- data/spec/features/campaigns_spec.rb +5 -5
- data/spec/features/contacts_spec.rb +11 -5
- data/spec/features/dashboard_spec.rb +8 -8
- data/spec/features/devise/sign_in_spec.rb +58 -0
- data/spec/features/devise/sign_up_spec.rb +36 -0
- data/spec/features/leads_spec.rb +5 -5
- data/spec/features/opportunities_overview_spec.rb +16 -16
- data/spec/features/opportunities_spec.rb +35 -9
- data/spec/features/support/autocomlete_helper.rb +17 -0
- data/spec/features/support/browser.rb +5 -9
- data/spec/features/tasks_spec.rb +5 -5
- data/spec/helpers/admin/field_groups_helper_spec.rb +1 -1
- data/spec/helpers/application_helper_spec.rb +1 -1
- data/spec/helpers/tasks_helper_spec.rb +1 -1
- data/spec/helpers/users_helper_spec.rb +7 -7
- data/spec/lib/comment_extensions_spec.rb +11 -5
- data/spec/lib/errors_spec.rb +2 -2
- data/spec/lib/mail_processor/base_spec.rb +3 -3
- data/spec/lib/mail_processor/comment_replies_spec.rb +3 -3
- data/spec/lib/mail_processor/dropbox_spec.rb +17 -17
- data/spec/lib/mail_processor/sample_emails/dropbox.rb +8 -8
- data/spec/lib/permissions_spec.rb +15 -28
- data/spec/mailers/devise_mailer_spec.rb +35 -0
- data/spec/mailers/user_mailer_spec.rb +6 -32
- data/spec/models/entities/account_spec.rb +58 -32
- data/spec/models/entities/campaign_spec.rb +18 -25
- data/spec/models/entities/contact_spec.rb +113 -21
- data/spec/models/entities/lead_spec.rb +9 -11
- data/spec/models/entities/opportunity_spec.rb +45 -45
- data/spec/models/fields/custom_field_date_pair_spec.rb +4 -2
- data/spec/models/fields/custom_field_spec.rb +21 -19
- data/spec/models/list_spec.rb +2 -2
- data/spec/models/observers/entity_observer_spec.rb +7 -7
- data/spec/models/polymorphic/address_spec.rb +1 -1
- data/spec/models/polymorphic/avatar_spec.rb +5 -5
- data/spec/models/polymorphic/comment_spec.rb +5 -5
- data/spec/models/polymorphic/task_spec.rb +65 -58
- data/spec/models/polymorphic/version_spec.rb +31 -31
- data/spec/models/setting_spec.rb +2 -2
- data/spec/models/users/preference_spec.rb +6 -6
- data/spec/models/users/user_spec.rb +46 -50
- data/spec/routing/users_routing_spec.rb +30 -8
- data/spec/shared/controllers.rb +3 -9
- data/spec/shared/models.rb +22 -22
- data/spec/spec_helper.rb +12 -4
- data/spec/support/assert_select.rb +1 -0
- data/spec/support/devise_helpers.rb +28 -0
- data/spec/{features/support/helpers.rb → support/feature_helpers.rb} +11 -11
- data/spec/support/macros.rb +7 -4
- data/spec/views/accounts/_edit.haml_spec.rb +1 -1
- data/spec/views/accounts/create.js.haml_spec.rb +3 -3
- data/spec/views/accounts/destroy.js.haml_spec.rb +1 -1
- data/spec/views/accounts/edit.js.haml_spec.rb +2 -2
- data/spec/views/accounts/index.haml_spec.rb +2 -2
- data/spec/views/accounts/index.js.haml_spec.rb +1 -1
- data/spec/views/accounts/show.haml_spec.rb +4 -4
- data/spec/views/accounts/update.js.haml_spec.rb +1 -1
- data/spec/views/admin/field_groups/create.js.haml_spec.rb +1 -1
- data/spec/views/admin/field_groups/destroy.js.haml_spec.rb +1 -1
- data/spec/views/admin/field_groups/edit.js.haml_spec.rb +1 -1
- data/spec/views/admin/field_groups/new.js.haml_spec.rb +1 -1
- data/spec/views/admin/field_groups/update.js.haml_spec.rb +1 -1
- data/spec/views/admin/users/create.js.haml_spec.rb +2 -2
- data/spec/views/admin/users/destroy.js.haml_spec.rb +2 -2
- data/spec/views/admin/users/edit.js.haml_spec.rb +2 -2
- data/spec/views/admin/users/index.haml_spec.rb +1 -1
- data/spec/views/admin/users/index.js.haml_spec.rb +2 -2
- data/spec/views/admin/users/reactivate.js.haml_spec.rb +1 -1
- data/spec/views/admin/users/suspend.js.haml_spec.rb +1 -1
- data/spec/views/admin/users/update.js.haml_spec.rb +1 -1
- data/spec/views/application/auto_complete.haml_spec.rb +3 -3
- data/spec/views/campaigns/_edit.haml_spec.rb +1 -1
- data/spec/views/campaigns/create.js.haml_spec.rb +4 -4
- data/spec/views/campaigns/destroy.js.haml_spec.rb +1 -1
- data/spec/views/campaigns/edit.js.haml_spec.rb +4 -4
- data/spec/views/campaigns/index.haml_spec.rb +1 -1
- data/spec/views/campaigns/index.js.haml_spec.rb +2 -2
- data/spec/views/campaigns/show.haml_spec.rb +4 -4
- data/spec/views/campaigns/update.js.haml_spec.rb +2 -2
- data/spec/views/contacts/_edit.haml_spec.rb +7 -7
- data/spec/views/contacts/_new.haml_spec.rb +1 -1
- data/spec/views/contacts/create.js.haml_spec.rb +5 -5
- data/spec/views/contacts/destroy.js.haml_spec.rb +1 -1
- data/spec/views/contacts/edit.js.haml_spec.rb +4 -4
- data/spec/views/contacts/index.haml_spec.rb +1 -1
- data/spec/views/contacts/index.js.html_spec.rb +2 -2
- data/spec/views/contacts/new.js.haml_spec.rb +1 -1
- data/spec/views/contacts/show.haml_spec.rb +3 -3
- data/spec/views/contacts/update.js.haml_spec.rb +5 -5
- data/spec/views/home/index.haml_spec.rb +1 -1
- data/spec/views/home/index.js.haml_spec.rb +1 -1
- data/spec/views/home/options.js.haml_spec.rb +2 -2
- data/spec/views/leads/_convert.haml_spec.rb +3 -3
- data/spec/views/leads/_edit.haml_spec.rb +2 -2
- data/spec/views/leads/_new.haml_spec.rb +2 -2
- data/spec/views/leads/_sidebar_show.haml_spec.rb +5 -5
- data/spec/views/leads/convert.js.haml_spec.rb +4 -4
- data/spec/views/leads/create.js.haml_spec.rb +5 -5
- data/spec/views/leads/destroy.js.haml_spec.rb +2 -2
- data/spec/views/leads/edit.js.haml_spec.rb +4 -4
- data/spec/views/leads/index.haml_spec.rb +1 -1
- data/spec/views/leads/index.js.haml_spec.rb +1 -1
- data/spec/views/leads/new.js.haml_spec.rb +1 -1
- data/spec/views/leads/promote.js.haml_spec.rb +7 -7
- data/spec/views/leads/reject.js.haml_spec.rb +2 -2
- data/spec/views/leads/show.haml_spec.rb +2 -2
- data/spec/views/leads/update.js.haml_spec.rb +4 -4
- data/spec/views/opportunities/_edit.haml_spec.rb +7 -7
- data/spec/views/opportunities/_new.haml_spec.rb +2 -2
- data/spec/views/opportunities/create.js.haml_spec.rb +6 -6
- data/spec/views/opportunities/destroy.js.haml_spec.rb +3 -3
- data/spec/views/opportunities/edit.js.haml_spec.rb +3 -3
- data/spec/views/opportunities/index.haml_spec.rb +1 -1
- data/spec/views/opportunities/index.js.haml_spec.rb +1 -1
- data/spec/views/opportunities/new.js.haml_spec.rb +1 -1
- data/spec/views/opportunities/show.haml_spec.rb +3 -3
- data/spec/views/opportunities/update.js.haml_spec.rb +5 -5
- data/spec/views/tasks/_edit.haml_spec.rb +1 -1
- data/spec/views/tasks/complete.js.haml_spec.rb +4 -4
- data/spec/views/tasks/create.js.haml_spec.rb +6 -6
- data/spec/views/tasks/destroy.js.haml_spec.rb +2 -2
- data/spec/views/tasks/index.haml_spec.rb +4 -4
- data/spec/views/tasks/new.js.haml_spec.rb +1 -1
- data/spec/views/tasks/uncomplete.js.haml_spec.rb +2 -2
- data/spec/views/tasks/update.js.haml_spec.rb +18 -18
- data/spec/views/users/upload_avatar.js.haml_spec.rb +2 -2
- metadata +49 -71
- data/app/controllers/authentications_controller.rb +0 -53
- data/app/models/users/authentication.rb +0 -56
- data/app/views/authentications/new.html.haml +0 -19
- data/app/views/passwords/edit.html.haml +0 -15
- data/app/views/passwords/new.html.haml +0 -10
- data/app/views/user_mailer/password_reset_instructions.html.haml +0 -6
- data/app/views/users/new.html.haml +0 -19
- data/spec/controllers/authentications_controller_spec.rb +0 -150
- data/spec/controllers/passwords_controller_spec.rb +0 -32
- data/spec/models/users/authentication_spec.rb +0 -19
- data/spec/support/auth_macros.rb +0 -49
- data/spec/views/authentications/new.haml_spec.rb +0 -31
data/spec/models/setting_spec.rb
CHANGED
@@ -24,7 +24,7 @@ describe Setting do
|
|
24
24
|
end
|
25
25
|
|
26
26
|
it "should find existing setting by its name using [] or method notations, and cache settings" do
|
27
|
-
@setting =
|
27
|
+
@setting = create(:setting, name: "thingymabob", value: "magoody")
|
28
28
|
expect(Setting.cache.key?("thingymabob")).to eq(false)
|
29
29
|
expect(Setting[:thingymabob]).to eq("magoody")
|
30
30
|
expect(Setting.cache.key?("thingymabob")).to eq(true)
|
@@ -32,7 +32,7 @@ describe Setting do
|
|
32
32
|
end
|
33
33
|
|
34
34
|
it "should use value from YAML if setting is missing from database" do
|
35
|
-
@setting =
|
35
|
+
@setting = create(:setting, name: "magoody", value: nil)
|
36
36
|
Setting.yaml_settings[:magoody] = "thingymabob"
|
37
37
|
expect(Setting[:magoody]).to eq("thingymabob")
|
38
38
|
expect(Setting.magoody).to eq("thingymabob")
|
@@ -21,7 +21,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
|
|
21
21
|
|
22
22
|
describe Preference do
|
23
23
|
before(:each) do
|
24
|
-
@user =
|
24
|
+
@user = create(:user)
|
25
25
|
@magoody = Base64.encode64(Marshal.dump("magoody"))
|
26
26
|
end
|
27
27
|
|
@@ -31,7 +31,7 @@ describe Preference do
|
|
31
31
|
|
32
32
|
describe "get user preference" do
|
33
33
|
it "should find and decode existing user preference by its name" do
|
34
|
-
@preference =
|
34
|
+
@preference = create(:preference, user: @user, name: "thingymabob", value: @magoody)
|
35
35
|
expect(@user.preference[:thingymabob]).to eq("magoody")
|
36
36
|
expect(@user.preference["thingymabob"]).to eq("magoody")
|
37
37
|
end
|
@@ -41,17 +41,17 @@ describe Preference do
|
|
41
41
|
end
|
42
42
|
|
43
43
|
it "should return correct user_id" do
|
44
|
-
@preference =
|
44
|
+
@preference = create(:preference, user: @user, name: "thingymabob", value: @magoody)
|
45
45
|
expect(@user.preference[:user_id]).to eq(@user.id)
|
46
46
|
end
|
47
47
|
|
48
48
|
it "should disregard other user's preference with the same name" do
|
49
|
-
@preference =
|
49
|
+
@preference = create(:preference, user: create(:user), name: "thingymabob", value: @magoody)
|
50
50
|
expect(@user.preference[:thingymabob]).to eq(nil)
|
51
51
|
end
|
52
52
|
|
53
53
|
it "should not fail is user is nil" do
|
54
|
-
@preference =
|
54
|
+
@preference = create(:preference, user: nil, name: "thingymabob", value: @magoody)
|
55
55
|
expect(@preference[:thingymabob]).to eq(nil)
|
56
56
|
end
|
57
57
|
end
|
@@ -63,7 +63,7 @@ describe Preference do
|
|
63
63
|
end
|
64
64
|
|
65
65
|
it "should update existing user preference" do
|
66
|
-
@preference =
|
66
|
+
@preference = create(:preference, user: @user, name: "thingymabob", value: @magoody)
|
67
67
|
@user.preference[:thingymabob] = "thingy"
|
68
68
|
expect(@user.reload.preference[:thingymabob]).to eq("thingy")
|
69
69
|
end
|
@@ -11,7 +11,7 @@
|
|
11
11
|
#
|
12
12
|
# id :integer not null, primary key
|
13
13
|
# username :string(32) default(""), not null
|
14
|
-
# email :string(
|
14
|
+
# email :string(254) default(""), not null
|
15
15
|
# first_name :string(32)
|
16
16
|
# last_name :string(32)
|
17
17
|
# title :string(64)
|
@@ -23,21 +23,27 @@
|
|
23
23
|
# yahoo :string(32)
|
24
24
|
# google :string(32)
|
25
25
|
# skype :string(32)
|
26
|
-
#
|
26
|
+
# encrypted_password :string(255) default(""), not null
|
27
27
|
# password_salt :string(255) default(""), not null
|
28
|
-
#
|
29
|
-
#
|
30
|
-
#
|
31
|
-
#
|
32
|
-
#
|
33
|
-
# current_login_ip :string(255)
|
34
|
-
# login_count :integer default(0), not null
|
28
|
+
# last_sign_in_at :datetime
|
29
|
+
# current_sign_in_at :datetime
|
30
|
+
# last_sign_in_ip :string(255)
|
31
|
+
# current_sign_in_ip :string(255)
|
32
|
+
# sign_in_count :integer default(0), not null
|
35
33
|
# deleted_at :datetime
|
36
34
|
# created_at :datetime
|
37
35
|
# updated_at :datetime
|
38
36
|
# admin :boolean default(FALSE), not null
|
39
37
|
# suspended_at :datetime
|
40
|
-
#
|
38
|
+
# unconfirmed_email :string(254) default(""), not null
|
39
|
+
# reset_password_token :string(255)
|
40
|
+
# reset_password_sent_at :datetime
|
41
|
+
# remember_token :string(255)
|
42
|
+
# remember_created_at :datetime
|
43
|
+
# authentication_token :string(255)
|
44
|
+
# confirmation_token :string(255)
|
45
|
+
# confirmed_at :datetime
|
46
|
+
# confirmation_sent_at :datetime
|
41
47
|
#
|
42
48
|
|
43
49
|
require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
|
@@ -53,28 +59,34 @@ describe User do
|
|
53
59
|
end
|
54
60
|
|
55
61
|
it "should have a valid factory" do
|
56
|
-
expect(
|
62
|
+
expect(build(:user)).to be_valid
|
57
63
|
end
|
58
64
|
|
59
65
|
describe '#destroyable?' do
|
60
66
|
describe "Destroying users with and without related assets" do
|
61
67
|
before do
|
62
|
-
@user =
|
63
|
-
@current_user =
|
68
|
+
@user = build(:user)
|
69
|
+
@current_user = build(:user)
|
64
70
|
end
|
65
71
|
|
66
72
|
%w[account campaign lead contact opportunity].each do |asset|
|
67
73
|
it "should not destroy the user if she owns #{asset}" do
|
68
|
-
|
74
|
+
create(asset, user: @user)
|
69
75
|
expect(@user.destroyable?(@current_user)).to eq(false)
|
70
76
|
end
|
71
77
|
|
72
78
|
it "should not destroy the user if she has #{asset} assigned" do
|
73
|
-
|
79
|
+
create(asset, assignee: @user)
|
74
80
|
expect(@user.destroyable?(@current_user)).to eq(false)
|
75
81
|
end
|
76
82
|
end
|
77
83
|
|
84
|
+
it "should not destroy the user if she owns a comment" do
|
85
|
+
account = build(:account, user: @current_user)
|
86
|
+
FactoryBot.create(:comment, user: @user, commentable: account)
|
87
|
+
expect(@user.destroyable?(@current_user)).to eq(false)
|
88
|
+
end
|
89
|
+
|
78
90
|
it "should not destroy the current user" do
|
79
91
|
expect(@user.destroyable?(@user)).to eq(false)
|
80
92
|
end
|
@@ -87,19 +99,19 @@ describe User do
|
|
87
99
|
|
88
100
|
describe '#destroy' do
|
89
101
|
before do
|
90
|
-
@user =
|
102
|
+
@user = create(:user)
|
91
103
|
end
|
92
104
|
it "once the user gets deleted all her permissions must be deleted too" do
|
93
|
-
|
94
|
-
|
105
|
+
create(:permission, user: @user, asset: create(:account))
|
106
|
+
create(:permission, user: @user, asset: create(:contact))
|
95
107
|
expect(@user.permissions.count).to eq(2)
|
96
108
|
@user.destroy
|
97
109
|
expect(@user.permissions.count).to eq(0)
|
98
110
|
end
|
99
111
|
|
100
112
|
it "once the user gets deleted all her preferences must be deleted too" do
|
101
|
-
|
102
|
-
|
113
|
+
create(:preference, user: @user, name: "Hello", value: "World")
|
114
|
+
create(:preference, user: @user, name: "World", value: "Hello")
|
103
115
|
expect(@user.preferences.count).to eq(2)
|
104
116
|
@user.destroy
|
105
117
|
expect(@user.preferences.count).to eq(0)
|
@@ -109,7 +121,7 @@ describe User do
|
|
109
121
|
describe '#suspend_if_needs_approval' do
|
110
122
|
it "should set suspended timestamp upon creation if signups need approval and the user is not an admin" do
|
111
123
|
allow(Setting).to receive(:user_signup).and_return(:needs_approval)
|
112
|
-
@user =
|
124
|
+
@user = build(:user, suspended_at: nil)
|
113
125
|
|
114
126
|
@user.suspend_if_needs_approval
|
115
127
|
|
@@ -118,7 +130,7 @@ describe User do
|
|
118
130
|
|
119
131
|
it "should not set suspended timestamp upon creation if signups need approval and the user is an admin" do
|
120
132
|
allow(Setting).to receive(:user_signup).and_return(:needs_approval)
|
121
|
-
@user =
|
133
|
+
@user = build(:user, admin: true, suspended_at: nil)
|
122
134
|
|
123
135
|
@user.suspend_if_needs_approval
|
124
136
|
|
@@ -129,16 +141,16 @@ describe User do
|
|
129
141
|
context "scopes" do
|
130
142
|
describe "have_assigned_opportunities" do
|
131
143
|
before do
|
132
|
-
@user1 =
|
133
|
-
|
144
|
+
@user1 = create(:user)
|
145
|
+
create(:opportunity, assignee: @user1, stage: 'analysis', account: nil, campaign: nil, user: nil)
|
134
146
|
|
135
|
-
@user2 =
|
147
|
+
@user2 = create(:user)
|
136
148
|
|
137
|
-
@user3 =
|
138
|
-
|
149
|
+
@user3 = create(:user)
|
150
|
+
create(:opportunity, assignee: @user3, stage: 'won', account: nil, campaign: nil, user: nil)
|
139
151
|
|
140
|
-
@user4 =
|
141
|
-
|
152
|
+
@user4 = create(:user)
|
153
|
+
create(:opportunity, assignee: @user4, stage: 'lost', account: nil, campaign: nil, user: nil)
|
142
154
|
|
143
155
|
@result = User.have_assigned_opportunities
|
144
156
|
end
|
@@ -161,10 +173,10 @@ describe User do
|
|
161
173
|
context "instance methods" do
|
162
174
|
describe "assigned_opportunities" do
|
163
175
|
before do
|
164
|
-
@user =
|
176
|
+
@user = create(:user)
|
165
177
|
|
166
|
-
@opportunity1 =
|
167
|
-
@opportunity2 =
|
178
|
+
@opportunity1 = create(:opportunity, assignee: @user, account: nil, campaign: nil, user: nil)
|
179
|
+
@opportunity2 = create(:opportunity, assignee: create(:user), account: nil, campaign: nil, user: nil)
|
168
180
|
|
169
181
|
@result = @user.assigned_opportunities
|
170
182
|
end
|
@@ -181,7 +193,7 @@ describe User do
|
|
181
193
|
|
182
194
|
describe "Setting I18n.locale" do
|
183
195
|
before do
|
184
|
-
@user =
|
196
|
+
@user = build(:user)
|
185
197
|
@locale = I18n.locale
|
186
198
|
end
|
187
199
|
|
@@ -202,24 +214,8 @@ describe User do
|
|
202
214
|
end
|
203
215
|
end
|
204
216
|
|
205
|
-
describe "Setting single access token" do
|
206
|
-
it "should update single_access_token attribute if it is not set already" do
|
207
|
-
@user = FactoryGirl.build(:user, single_access_token: nil)
|
208
|
-
|
209
|
-
@user.set_single_access_token
|
210
|
-
expect(@user.single_access_token).not_to eq(nil)
|
211
|
-
end
|
212
|
-
|
213
|
-
it "should not update single_access_token attribute if it is set already" do
|
214
|
-
@user = FactoryGirl.build(:user, single_access_token: "token")
|
215
|
-
|
216
|
-
@user.set_single_access_token
|
217
|
-
expect(@user.single_access_token).to eq("token")
|
218
|
-
end
|
219
|
-
end
|
220
|
-
|
221
217
|
describe "serialization" do
|
222
|
-
let(:user) {
|
218
|
+
let(:user) { build(:user) }
|
223
219
|
|
224
220
|
it "to json" do
|
225
221
|
expect(user.to_json).to eql([user.name].to_json)
|
@@ -9,12 +9,38 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
|
|
9
9
|
|
10
10
|
describe UsersController do
|
11
11
|
describe "routing" do
|
12
|
-
it "
|
13
|
-
expect(get: "/users").
|
12
|
+
it "recognizes #index" do
|
13
|
+
expect(get: "/users").to route_to(controller: "users", action: "index")
|
14
14
|
end
|
15
15
|
|
16
|
-
it "recognizes and generates
|
17
|
-
expect(get: "/
|
16
|
+
it "recognizes and generates Devise registrations routes" do
|
17
|
+
expect(get: "/users/sign_up").to route_to(controller: "registrations", action: "new")
|
18
|
+
expect(get: "/users/edit").to route_to(controller: "registrations", action: "edit")
|
19
|
+
expect(get: "/users/cancel").to route_to(controller: "registrations", action: "cancel")
|
20
|
+
expect(post: "/users").to route_to(controller: "registrations", action: "create")
|
21
|
+
expect(put: "/users").to route_to(controller: "registrations", action: "update")
|
22
|
+
expect(patch: "/users").to route_to(controller: "registrations", action: "update")
|
23
|
+
expect(delete: "/users").to route_to(controller: "registrations", action: "destroy")
|
24
|
+
end
|
25
|
+
|
26
|
+
it "recognizes and generates Devise sessions routes" do
|
27
|
+
expect(get: "/users/sign_in").to route_to(controller: "sessions", action: "new")
|
28
|
+
expect(post: "/users/sign_in").to route_to(controller: "sessions", action: "create")
|
29
|
+
expect(delete: "/users/sign_out").to route_to(controller: "sessions", action: "destroy")
|
30
|
+
end
|
31
|
+
|
32
|
+
it "recognizes and generates Devise passwords routes" do
|
33
|
+
expect(get: "/users/password/new").to route_to(controller: "passwords", action: "new")
|
34
|
+
expect(get: "/users/password/edit").to route_to(controller: "passwords", action: "edit")
|
35
|
+
expect(post: "/users/password").to route_to(controller: "passwords", action: "create")
|
36
|
+
expect(put: "/users/password").to route_to(controller: "passwords", action: "update")
|
37
|
+
expect(patch: "/users/password").to route_to(controller: "passwords", action: "update")
|
38
|
+
end
|
39
|
+
|
40
|
+
it "recognizes and generates Devise confirmations routes" do
|
41
|
+
expect(get: "/users/confirmation/new").to route_to(controller: "confirmations", action: "new")
|
42
|
+
expect(get: "/users/confirmation").to route_to(controller: "confirmations", action: "show")
|
43
|
+
expect(post: "/users/confirmation").to route_to(controller: "confirmations", action: "create")
|
18
44
|
end
|
19
45
|
|
20
46
|
it "recognizes and generates #show as /profile" do
|
@@ -29,10 +55,6 @@ describe UsersController do
|
|
29
55
|
expect(get: "/users/aaron/edit").not_to be_routable
|
30
56
|
end
|
31
57
|
|
32
|
-
it "recognizes and generates #create" do
|
33
|
-
expect(post: "/users").to route_to(controller: "users", action: "create")
|
34
|
-
end
|
35
|
-
|
36
58
|
it "recognizes and generates #update" do
|
37
59
|
expect(put: "/users/1").to route_to(controller: "users", action: "update", id: "1")
|
38
60
|
end
|
data/spec/shared/controllers.rb
CHANGED
@@ -22,9 +22,7 @@ shared_examples "auto complete" do
|
|
22
22
|
|
23
23
|
# We don't save Admin/Users autocomplete controller in a session since Users are not
|
24
24
|
# exposed through the Jumpbox.
|
25
|
-
unless controller.class.to_s.starts_with?("Admin::")
|
26
|
-
expect(session[:auto_complete]).to eq(@controller.controller_name.to_sym)
|
27
|
-
end
|
25
|
+
expect(session[:auto_complete]).to eq(@controller.controller_name.to_sym) unless controller.class.to_s.starts_with?("Admin::")
|
28
26
|
end
|
29
27
|
|
30
28
|
it "should render application/_auto_complete template" do
|
@@ -39,12 +37,8 @@ shared_examples "attach" do
|
|
39
37
|
expect(@model.send(@attachment.class.name.tableize)).to include(@attachment)
|
40
38
|
expect(assigns[:attachment]).to eq(@attachment)
|
41
39
|
expect(assigns[:attached]).to eq([@attachment])
|
42
|
-
if @model.is_a?(Campaign) && (@attachment.is_a?(Lead) || @attachment.is_a?(Opportunity))
|
43
|
-
|
44
|
-
end
|
45
|
-
if @model.is_a?(Account) && @attachment.respond_to?(:account) # Skip Tasks...
|
46
|
-
expect(assigns[:account]).to eq(@attachment.reload.account)
|
47
|
-
end
|
40
|
+
expect(assigns[:campaign]).to eq(@attachment.reload.campaign) if @model.is_a?(Campaign) && (@attachment.is_a?(Lead) || @attachment.is_a?(Opportunity))
|
41
|
+
expect(assigns[:account]).to eq(@attachment.reload.account) if @model.is_a?(Account) && @attachment.respond_to?(:account) # Skip Tasks...
|
48
42
|
expect(response).to render_template("entities/attach")
|
49
43
|
end
|
50
44
|
|
data/spec/shared/models.rb
CHANGED
@@ -47,7 +47,7 @@ end
|
|
47
47
|
shared_examples Ability do |klass|
|
48
48
|
subject { ability }
|
49
49
|
let(:ability) { Ability.new(user) }
|
50
|
-
let(:user) {
|
50
|
+
let(:user) { create(:user) }
|
51
51
|
let(:factory) { klass.model_name.to_s.underscore }
|
52
52
|
|
53
53
|
context "create" do
|
@@ -55,86 +55,86 @@ shared_examples Ability do |klass|
|
|
55
55
|
end
|
56
56
|
|
57
57
|
context "when public access" do
|
58
|
-
let!(:asset) {
|
58
|
+
let!(:asset) { create(factory, access: 'Public') }
|
59
59
|
|
60
60
|
it { is_expected.to be_able_to(:manage, asset) }
|
61
61
|
end
|
62
62
|
|
63
63
|
context "when private access owner" do
|
64
|
-
let!(:asset) {
|
64
|
+
let!(:asset) { create(factory, access: 'Private', user_id: user.id) }
|
65
65
|
|
66
66
|
it { is_expected.to be_able_to(:manage, asset) }
|
67
67
|
end
|
68
68
|
|
69
69
|
context "when private access administrator" do
|
70
|
-
let!(:asset) {
|
71
|
-
let(:user) {
|
70
|
+
let!(:asset) { create(factory, access: 'Private') }
|
71
|
+
let(:user) { create(:user, admin: true) }
|
72
72
|
|
73
73
|
it { is_expected.to be_able_to(:manage, asset) }
|
74
74
|
end
|
75
75
|
|
76
76
|
context "when private access not owner" do
|
77
|
-
let!(:asset) {
|
77
|
+
let!(:asset) { create(factory, access: 'Private') }
|
78
78
|
|
79
79
|
it { is_expected.not_to be_able_to(:manage, asset) }
|
80
80
|
end
|
81
81
|
|
82
82
|
context "when private access not owner but is assigned" do
|
83
|
-
let!(:asset) {
|
83
|
+
let!(:asset) { create(factory, access: 'Private', assigned_to: user.id) }
|
84
84
|
|
85
85
|
it { is_expected.to be_able_to(:manage, asset) }
|
86
86
|
end
|
87
87
|
|
88
88
|
context "when shared access with permission" do
|
89
|
-
let!(:asset) {
|
89
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission]) }
|
90
90
|
let(:permission) { Permission.new(user: user) }
|
91
91
|
|
92
92
|
it { is_expected.to be_able_to(:manage, asset) }
|
93
93
|
end
|
94
94
|
|
95
95
|
context "when shared access with no permission" do
|
96
|
-
let!(:asset) {
|
97
|
-
let(:permission) { Permission.new(user:
|
96
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission]) }
|
97
|
+
let(:permission) { Permission.new(user: create(:user)) }
|
98
98
|
|
99
99
|
it { is_expected.not_to be_able_to(:manage, asset) }
|
100
100
|
end
|
101
101
|
|
102
102
|
context "when shared access with no permission but administrator" do
|
103
|
-
let!(:asset) {
|
104
|
-
let(:permission) { Permission.new(user:
|
105
|
-
let(:user) {
|
103
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission]) }
|
104
|
+
let(:permission) { Permission.new(user: create(:user)) }
|
105
|
+
let(:user) { create(:user, admin: true) }
|
106
106
|
|
107
107
|
it { is_expected.to be_able_to(:manage, asset) }
|
108
108
|
end
|
109
109
|
|
110
110
|
context "when shared access with no permission but assigned" do
|
111
|
-
let!(:asset) {
|
112
|
-
let(:permission) { Permission.new(user:
|
111
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission], assigned_to: user.id) }
|
112
|
+
let(:permission) { Permission.new(user: create(:user)) }
|
113
113
|
|
114
114
|
it { is_expected.to be_able_to(:manage, asset) }
|
115
115
|
end
|
116
116
|
|
117
117
|
context "when shared access with group permission" do
|
118
|
-
let!(:asset) {
|
118
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission]) }
|
119
119
|
let(:permission) { Permission.new(group: group) }
|
120
|
-
let(:group) {
|
120
|
+
let(:group) { create(:group, users: [user]) }
|
121
121
|
|
122
122
|
it { is_expected.to be_able_to(:manage, asset) }
|
123
123
|
end
|
124
124
|
|
125
125
|
context "when shared access with several group permissions" do
|
126
|
-
let!(:asset) {
|
126
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: permissions) }
|
127
127
|
let(:permissions) { [Permission.new(group: group1), Permission.new(group: group2)] }
|
128
|
-
let(:group1) {
|
129
|
-
let(:group2) {
|
128
|
+
let(:group1) { create(:group, users: [user]) }
|
129
|
+
let(:group2) { create(:group, users: [user]) }
|
130
130
|
|
131
131
|
it { is_expected.to be_able_to(:manage, asset) }
|
132
132
|
end
|
133
133
|
|
134
134
|
context "when shared access with no group permission" do
|
135
|
-
let!(:asset) {
|
135
|
+
let!(:asset) { create(factory, access: 'Shared', permissions: [permission]) }
|
136
136
|
let(:permission) { Permission.new(group: group) }
|
137
|
-
let(:group) {
|
137
|
+
let(:group) { create(:group) }
|
138
138
|
|
139
139
|
it { is_expected.not_to be_able_to(:manage, asset) }
|
140
140
|
end
|