enju_seed 0.4.0.beta.1 → 0.4.0.beta.2

data/app/models/user_has_role.rb

@@ -2,9 +2,6 @@ class UserHasRole < ActiveRecord::Base
   belongs_to :user
   belongs_to :role
   accepts_nested_attributes_for :role
-
-  #  validates_uniqueness_of :role_id, scope: :user_id
-  #  validates_presence_of :role_id, :user_id
 end
 
 # == Schema Information
@@ -14,6 +11,6 @@ end
 #  id         :integer          not null, primary key
 #  user_id    :integer          not null
 #  role_id    :integer          not null
-#  created_at :datetime         not null
-#  updated_at :datetime         not null
+#  created_at :datetime
+#  updated_at :datetime
 #

data/app/views/my_accounts/_edit_credential.html.erb

@@ -36,9 +36,9 @@
       <%= user_form.label t('activerecord.models.role') -%><br />
       <%= user_form.fields_for :user_has_role do |u| %>
         <%- if current_user.has_role?('Administrator') and current_user != profile.user -%>
-          <%= u.select :role_id, @roles.collect{|r| [r.display_name.localize, r.id]} %>
+          <%= u.select :role_id, @roles.collect{|r| [r.display_name, r.id]} %>
         <% else %>
-          <%= current_user.role.display_name.localize -%>
+          <%= current_user.role.display_name -%>
         <% end %>
       <% end %>
     </div>
@@ -58,9 +58,9 @@
     <div class="field">
       <%= f.label t('activerecord.models.user_group') -%><br />
       <%- if current_user.has_role?('Librarian') -%>
-        <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name.localize, u.id]}) -%>
+        <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name, u.id]}) -%>
       <%- else -%>
-        <%= current_user.profile.user_group.display_name.localize if current_user.profile.user_group -%>
+        <%= current_user.profile.user_group.display_name if current_user.profile.user_group -%>
       <%- end -%>
     </div>
 
@@ -76,24 +76,24 @@
     <div class="field">
       <%= f.label t('activerecord.models.library') -%><br />
       <%- if current_user.has_role?('Librarian') -%>
-        <%= f.select(:library_id, @libraries.collect{|l| [l.display_name.localize, l.id]}) -%>
+        <%= f.select(:library_id, @libraries.collect{|l| [l.display_name, l.id]}) -%>
       <% else %>
-        <%= current_user.profile.library.display_name.localize if current_user.profile.library %>
+        <%= current_user.profile.library.display_name if current_user.profile.library %>
       <% end %>
     </div>
 
     <div class="field">
       <%= f.label t('role.required_role') -%><br />
       <%- if current_user.has_role?('Librarian') -%>
-        <%= f.select(:required_role_id, @roles.collect{|r| [r.display_name.localize, r.id]}) -%>
+        <%= f.select(:required_role_id, @roles.collect{|r| [r.display_name, r.id]}) -%>
       <%- else -%>
-        <%= current_user.role.display_name.localize -%>
+        <%= current_user.role.display_name -%>
       <%- end -%>
     </div>
 
     <div class="field">
       <%= f.label :locale -%><br />
-      <%= f.select(:locale, @available_languages.collect{|language| [language.display_name.localize, language.iso_639_1]}) -%>
+      <%= f.select(:locale, @available_languages.collect{|language| [language.display_name, language.iso_639_1]}) -%>
     </div>
   </div>
 <% end %>

data/app/views/profiles/_edit_credential.html.erb

@@ -34,9 +34,9 @@
       <%= user_form.label t('activerecord.models.role') -%><br />
       <%= user_form.fields_for :user_has_role do |u| %>
         <%- if current_user.has_role?('Administrator') and current_user != profile.user -%>
-          <%= u.select :role_id, @roles.select{|r| r.name != 'Guest' }.collect{|r| [r.display_name.localize, r.id]} %>
+          <%= u.select :role_id, @roles.select{|r| r.name != 'Guest' }.collect{|r| [r.display_name, r.id]} %>
         <% else %>
-          <%= u.select :role_id, @roles.select{|r| r.name != 'Guest' }.collect{|r| [r.display_name.localize, r.id]}, {}, {disabled: 'disabled'} %>
+          <%= u.select :role_id, @roles.select{|r| r.name != 'Guest' }.collect{|r| [r.display_name, r.id]}, {}, {disabled: 'disabled'} %>
         <% end %>
       <% end %>
     </div>
@@ -56,7 +56,7 @@
     <%- if current_user.has_role?('Librarian') -%>
       <div class="field">
         <%= f.label t('activerecord.models.user_group') -%><br />
-        <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name.localize, u.id]}) -%>
+        <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name, u.id]}) -%>
       </div>
     <%- end -%>
 
@@ -72,17 +72,17 @@
     <%- if current_user.has_role?('Librarian') -%>
       <div class="field">
         <%= f.label t('activerecord.models.library') -%><br />
-        <%= f.select(:library_id, @libraries.collect{|l| [l.display_name.localize, l.id]}) -%>
+        <%= f.select(:library_id, @libraries.collect{|l| [l.display_name, l.id]}) -%>
       </div>
 
       <div class="field">
         <%= f.label t('role.required_role') -%><br />
-        <%= f.select(:required_role_id, @roles.collect{|r| [r.display_name.localize, r.id]}) -%>
+        <%= f.select(:required_role_id, @roles.collect{|r| [r.display_name, r.id]}) -%>
       </div>
     <%- end -%>
     <div class="field">
       <%= f.label :locale -%><br />
-      <%= f.select(:locale, @available_languages.collect{|language| [language.display_name.localize, language.iso_639_1]}) -%>
+      <%= f.select(:locale, @available_languages.collect{|language| [language.display_name, language.iso_639_1]}) -%>
     </div>
   </div>
 <% end %>

data/app/views/profiles/new.html.erb

@@ -34,17 +34,17 @@
 
   <div class="field">
     <%= f.label :user_group_id -%>
-    <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name.localize, u.id]}) -%>
+    <%= f.select(:user_group_id, @user_groups.collect{|u| [u.display_name, u.id]}) -%>
   </div>
 
   <div class="field">
     <%= f.label t('activerecord.models.library') -%>
-    <%= f.select(:library_id, @libraries.collect{|l| [l.display_name.localize, l.id]}) -%>
+    <%= f.select(:library_id, @libraries.collect{|l| [l.display_name, l.id]}) -%>
   </div>
 
   <div class="field">
     <%= f.label :locale -%>
-    <%= f.select(:locale, @available_languages.collect{|language| [language.display_name.localize, language.iso_639_1]}) -%>
+    <%= f.select(:locale, @available_languages.collect{|language| [language.display_name, language.iso_639_1]}) -%>
   </div>
 
   <% if defined?(EnjuSearchLog) %>

data/app/views/profiles/show.html.erb

@@ -28,7 +28,7 @@
     <%= render 'submenu_checkout', profile: @profile %>
   <% end %>
   <ul>
-    <li><%= link_to t('page.back'), :back %></li>
+    <li><%= back_to_index(flash[:page_info]) -%></li>
     <% if user_signed_in? %>
       <% if policy(@profile).update? %>
         <li><%= link_to t('page.edit'), edit_profile_path(@profile) -%></li>

data/app/views/roles/index.html.erb

@@ -12,7 +12,7 @@
 <%- @roles.each do |role| -%>
   <tr class="line<%= cycle("0", "1") -%>">
     <td><%= link_to role.name, role -%></td>
-    <td><%= link_to role.display_name.localize, role -%></td>
+    <td><%= link_to role.display_name, role -%></td>
     <td>
       <%- if policy(role).update? -%>
         <%= link_to t('page.edit'), edit_role_path(role) -%>

data/app/views/roles/show.html.erb

@@ -10,12 +10,7 @@
 
 <p>
   <strong><%= t('activerecord.attributes.role.display_name') -%>:</strong>
-  <%= @role.display_name.localize -%>
-</p>
-
-<p>
-  <strong><%= t('activerecord.attributes.role.score') -%>:</strong>
-  <%= @role.score -%>
+  <%= @role.display_name -%>
 </p>
 
 <p>
@@ -41,6 +36,6 @@
     <%- if policy(@role).update? -%>
       <li><%= link_to t('page.edit'), edit_role_path(@role) -%></li>
     <%- end -%>
-    <li><%= link_to t('page.back'), roles_path -%></li>
+    <li><%= back_to_index(flash[:page_info]) -%></li>
   </ul>
 </div>

data/db/migrate/041_create_roles.rb

@@ -1,10 +1,11 @@
-class CreateRoles < ActiveRecord::Migration[5.1]
+class CreateRoles < ActiveRecord::Migration[4.2]
   def change
-    create_table 'roles' do |t|
-      t.string :name, null: false
-      t.jsonb :display_name_translations
-      t.text :note
-      t.timestamps
+    create_table "roles" do |t|
+      t.column :name, :string, null: false
+      t.column :display_name, :string
+      t.column :note, :text
+      t.column :created_at, :datetime
+      t.column :updated_at, :datetime
       t.integer :score, default: 0, null: false
       t.integer :position
     end

data/db/migrate/20100606065209_create_user_has_roles.rb

@@ -1,8 +1,8 @@
-class CreateUserHasRoles < ActiveRecord::Migration[5.1]
+class CreateUserHasRoles < ActiveRecord::Migration[4.2]
   def change
     create_table :user_has_roles do |t|
-      t.references :user, foreign_key: {on_delete: :cascade}, null: false
-      t.references :role, foreign_key: true, null: false
+      t.references :user, index: true, foreign_key: true, null: false
+      t.references :role, index: true, foreign_key: true, null: false
 
       t.timestamps
     end

data/db/migrate/20130221154434_add_additional_attributes_to_user.rb

@@ -1,4 +1,4 @@
-class AddAdditionalAttributesToUser < ActiveRecord::Migration[5.1]
+class AddAdditionalAttributesToUser < ActiveRecord::Migration[4.2]
   def change
     add_column :users, :username, :string
     add_column :users, :deleted_at, :datetime
@@ -11,6 +11,6 @@ class AddAdditionalAttributesToUser < ActiveRecord::Migration[5.1]
     add_column :users, :confirmed_at, :datetime
 
     add_index :users, :username, unique: true
-    add_index :users, :unlock_token, unique: true
+    add_index :users, :unlock_token,         unique: true
   end
 end

data/db/migrate/20140122054321_create_profiles.rb

@@ -1,10 +1,11 @@
-class CreateProfiles < ActiveRecord::Migration[5.1]
+class CreateProfiles < ActiveRecord::Migration[4.2]
   def change
-    create_table :profiles, id: :uuid, default: 'gen_random_uuid()' do |t|
-      t.references :user_group, type: :uuid
-      t.references :library, type: :uuid
+    create_table :profiles do |t|
+      t.references :user, index: true, foreign_key: true
+      t.references :user_group, index: true
+      t.references :library, index: true
       t.string :locale
-      t.string :user_number, index: {unique: true}
+      t.string :user_number
       t.text :full_name
       t.text :note
       t.text :keyword_list
@@ -12,5 +13,7 @@ class CreateProfiles < ActiveRecord::Migration[5.1]
 
       t.timestamps
     end
+
+    add_index :profiles, :user_number, unique: true
   end
 end

data/db/migrate/20140610123439_drop_email_unique_constraint_enju_leaf_rc10.rb

@@ -0,0 +1,11 @@
+class DropEmailUniqueConstraintEnjuLeafRc10 < ActiveRecord::Migration[4.2]
+  def up
+    remove_index :users, :email
+    add_index :users, :email
+  end
+
+  def down
+    remove_index :users, :email
+    add_index :users, :email, unique: true
+  end
+end

data/db/migrate/20140811031145_add_expired_at_to_profile.rb

@@ -1,4 +1,4 @@
-class AddExpiredAtToProfile < ActiveRecord::Migration[5.1]
+class AddExpiredAtToProfile < ActiveRecord::Migration[4.2]
   def change
     add_column :profiles, :expired_at, :datetime
   end

data/db/migrate/20141003181336_add_full_name_transcription_to_profile.rb

@@ -1,4 +1,4 @@
-class AddFullNameTranscriptionToProfile < ActiveRecord::Migration[5.1]
+class AddFullNameTranscriptionToProfile < ActiveRecord::Migration[4.2]
   def change
     add_column :profiles, :full_name_transcription, :text
   end

data/db/migrate/20141003182825_add_date_of_birth_to_profile.rb

@@ -1,4 +1,4 @@
-class AddDateOfBirthToProfile < ActiveRecord::Migration[5.1]
+class AddDateOfBirthToProfile < ActiveRecord::Migration[4.2]
   def change
     add_column :profiles, :date_of_birth, :datetime
   end

data/db/migrate/20190630151446_add_display_name_translations_to_role.rb

@@ -0,0 +1,5 @@
+class AddDisplayNameTranslationsToRole < ActiveRecord::Migration[5.2]
+  def change
+    add_column :roles, :display_name_translations, :jsonb, default: {}, null: false
+  end
+end

data/db/migrate/20190713114940_add_profile_id_to_user.rb

@@ -0,0 +1,5 @@
+class AddProfileIdToUser < ActiveRecord::Migration[5.2]
+  def change
+    add_reference :users, :profile, foreign_key: true
+  end
+end

data/db/migrate/20190713115451_add_full_name_translations_to_profile.rb

@@ -0,0 +1,5 @@
+class AddFullNameTranslationsToProfile < ActiveRecord::Migration[5.2]
+  def change
+    add_column :profiles, :full_name_translations, :jsonb, default: {}, null: false
+  end
+end

data/lib/enju_seed.rb

@@ -1,5 +1,5 @@
-require 'enju_seed/engine'
+require "enju_seed/engine"
+require "enju_seed/version"
 
 module EnjuSeed
-  # Your code goes here...
 end

data/lib/enju_seed/engine.rb

@@ -6,15 +6,15 @@ require 'strip_attributes'
 require 'friendly_id'
 require 'addressable/uri'
 require 'sunspot_rails'
+require 'sitemap_generator'
 require 'cocoon'
 require 'kramdown'
 require 'rails_autolink'
-require 'sitemap_generator'
 require 'browser'
 require 'simple_form'
 require 'validates_timeliness'
+require 'paperclip'
 require 'json_translate'
-require 'statesman'
 
 module EnjuSeed
   class Engine < ::Rails::Engine

data/lib/enju_seed/version.rb

@@ -1,3 +1,3 @@
 module EnjuSeed
-  VERSION = '0.4.0.beta.1'.freeze
+  VERSION = "0.4.0.beta.2"
 end

data/lib/generators/enju_seed/setup/setup_generator.rb

@@ -0,0 +1,9 @@
+class EnjuSeed::SetupGenerator < Rails::Generators::Base
+  source_root File.expand_path('../templates', __FILE__)
+  desc "Create a setup file for Next-L Enju"
+
+  def copy_setup_files
+    directory("db/fixtures", "db/fixtures/enju_seed")
+    rake("enju_seed_engine:install:migrations")
+  end
+end

data/lib/generators/enju_seed/setup/templates/db/fixtures/roles.yml

@@ -0,0 +1,38 @@
+--- 
+role_00001: 
+  name: Guest  
+  display_name_translations: {"en": "Guest", "ja": "ゲスト"}
+  id: 1
+  note: 
+  position: 1
+role_00002: 
+  name: User  
+  display_name_translations: {"en": "User", "ja": "一般利用者"}
+  id: 2
+  note: 
+  position: 2
+role_00003: 
+  name: Librarian  
+  display_name_translations: {"en": "Librarian", "ja": "図書館員"}
+  id: 3
+  note: 
+  position: 3
+role_00004: 
+  name: Administrator  
+  display_name_translations: {"en": "Administrator", "ja": "管理者"}
+  id: 4
+  note:
+  position: 4
+
+# == Schema Information
+#
+# Table name: roles
+#
+#  id                        :bigint           not null, primary key
+#  name                      :string           not null
+#  display_name_translations :jsonb            not null
+#  note                      :text
+#  position                  :integer          default(1), not null
+#  created_at                :datetime         not null
+#  updated_at                :datetime         not null
+#

data/lib/generators/enju_seed/update/update_generator.rb

@@ -0,0 +1,9 @@
+class EnjuSeed::UpdateGenerator < Rails::Generators::Base
+  source_root File.expand_path('../templates', __FILE__)
+  desc "Create files for updating Next-L Enju"
+
+  def copy_migration_files
+    generate('statesman:add_constraints_to_most_recent', 'UserImportFile', 'UserImportFileTransition')
+    generate('statesman:add_constraints_to_most_recent', 'UserExportFile', 'UserExportFileTransition')
+  end
+end

data/lib/tasks/enju_seed_tasks.rake

@@ -1,4 +1,8 @@
-# desc "Explaining what the task does"
-# task :enju_seed do
-#   # Task goes here
-# end
+namespace :enju_seed do
+  desc "create initial records for enju_seed"
+  task setup: :environment do
+    Dir.glob(Rails.root.to_s + '/db/fixtures/enju_seed/*.yml').each do |file|
+      ActiveRecord::FixtureSet.create_fixtures('db/fixtures/enju_seed', File.basename(file, '.*'))
+    end
+  end
+end

data/spec/controllers/my_accounts_controller_spec.rb

@@ -3,20 +3,20 @@ require 'rails_helper'
 describe MyAccountsController do
   fixtures :all
 
-  describe 'GET show' do
-    describe 'When logged in as Administrator' do
+  describe "GET show" do
+    describe "When logged in as Administrator" do
       before(:each) do
         sign_in User.friendly.find('enjuadmin')
       end
 
-      it 'assigns the requested user as @user' do
+      it "assigns the requested user as @user" do
         get :show, params: { id: 'admin' }
         expect(response).to be_successful
       end
     end
 
-    describe 'When not logged in' do
-      it 'assigns the requested user as @user' do
+    describe "When not logged in" do
+      it "assigns the requested user as @user" do
         get :show, params: { id: 'admin' }
         expect(assigns(:user)).to be_nil
         expect(response).to redirect_to(new_user_session_url)
@@ -24,46 +24,52 @@ describe MyAccountsController do
     end
   end
 
-  describe 'GET edit' do
-    describe 'When logged in as Administrator' do
+  describe "GET edit" do
+    describe "When logged in as Administrator" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:admin)
+        @user.profile = profile
         sign_in @user
       end
 
-      it 'assigns the requested user as @user' do
+      it "assigns the requested user as @user" do
         get :edit
         expect(assigns(:profile)).to eq(@user.profile)
       end
     end
 
-    describe 'When logged in as Librarian' do
+    describe "When logged in as Librarian" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:librarian)
+        @user.profile = profile
         sign_in @user
       end
 
-      it 'should assign the requested user as @user' do
+      it "should assign the requested user as @user" do
         get :edit
         expect(assigns(:profile)).to eq(@user.profile)
       end
     end
 
-    describe 'When logged in as User' do
+    describe "When logged in as User" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:user)
+        @user.profile = profile
         sign_in @user
       end
 
-      it 'should assign the requested user as @user' do
+      it "should assign the requested user as @user" do
         get :edit
         expect(assigns(:profile)).to eq(@user.profile)
         expect(response).to be_successful
       end
     end
 
-    describe 'When not logged in' do
-      it 'should not assign the requested user as @user' do
+    describe "When not logged in" do
+      it "should not assign the requested user as @user" do
         get :edit
         expect(assigns(:user)).to be_nil
         expect(response).to redirect_to(new_user_session_url)
@@ -71,32 +77,34 @@ describe MyAccountsController do
     end
   end
 
-  describe 'PUT update' do
+  describe "PUT update" do
     before(:each) do
       @attrs = { user_attributes: { email: 'newaddress@example.jp', current_password: 'password' }, locale: 'en' }
       @invalid_attrs = { user_attributes: { username: '' }, user_number: '日本語' }
       @invalid_passwd_attrs = { user_attributes: { current_password: '' } }
     end
 
-    describe 'When logged in as Administrator' do
+    describe "When logged in as Administrator" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:admin, password: 'password', password_confirmation: 'password')
+        @user.profile = profile
         sign_in @user
       end
 
-      describe 'with valid params' do
-        it 'updates the requested user' do
+      describe "with valid params" do
+        it "updates the requested user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
         end
 
-        it 'assigns the requested user as @user' do
+        it "assigns the requested user as @user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
           expect(assigns(:profile)).to eq(@user.profile)
         end
 
-        it 'redirects to the user' do
+        it "redirects to the user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
           expect(assigns(:profile)).to eq(@user.profile)
@@ -104,15 +112,15 @@ describe MyAccountsController do
         end
       end
 
-      describe 'with invalid params' do
-        it 'assigns the requested user as @user' do
+      describe "with invalid params" do
+        it "assigns the requested user as @user" do
           put :update, params: { profile: @invalid_attrs }
           expect(assigns(:profile)).to eq(@user.profile)
         end
 
         it "re-renders the 'edit' template" do
           put :update, params: { profile: @invalid_attrs }
-          expect(response).to render_template('edit')
+          expect(response).to render_template("edit")
         end
       end
 
@@ -124,25 +132,27 @@ describe MyAccountsController do
       # end
     end
 
-    describe 'When logged in as Librarian' do
+    describe "When logged in as Librarian" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:librarian, password: 'password', password_confirmation: 'password')
+        @user.profile = profile
         sign_in @user
       end
 
-      describe 'with valid params' do
-        it 'updates the requested user' do
+      describe "with valid params" do
+        it "updates the requested user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
         end
 
-        it 'assigns the requested user as @user' do
+        it "assigns the requested user as @user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
           expect(assigns(:profile)).to eq(@user.profile)
         end
 
-        it 'redirects to the user' do
+        it "redirects to the user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
           expect(assigns(:profile)).to eq(@user.profile)
@@ -150,34 +160,36 @@ describe MyAccountsController do
         end
       end
 
-      describe 'with invalid params' do
-        it 'assigns the user as @user' do
+      describe "with invalid params" do
+        it "assigns the user as @user" do
           put :update, params: { profile: @invalid_attrs }
           expect(assigns(:profile)).to_not be_valid
           expect(response).to be_successful
         end
 
-        it 'should ignore username' do
+        it "should ignore username" do
           put :update, params: { profile: @invalid_attrs }
-          expect(response).to render_template('edit')
+          expect(response).to render_template("edit")
           expect(assigns(:profile).changed?).to be_truthy
         end
       end
     end
 
-    describe 'When logged in as User' do
+    describe "When logged in as User" do
       before(:each) do
+        profile = FactoryBot.create(:profile)
         @user = FactoryBot.create(:user, password: 'password', password_confirmation: 'password')
+        @user.profile = profile
         sign_in @user
       end
 
-      describe 'with valid params' do
-        it 'updates the requested user' do
+      describe "with valid params" do
+        it "updates the requested user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
         end
 
-        it 'assigns the requested user as @user' do
+        it "assigns the requested user as @user" do
           @attrs[:user_attributes][:id] = @user.id
           put :update, params: { profile: @attrs }
           expect(assigns(:profile)).to eq(@user.profile)
@@ -185,30 +197,29 @@ describe MyAccountsController do
         end
       end
 
-      describe 'with invalid params' do
-        it 'assigns the requested user as @user' do
+      describe "with invalid params" do
+        it "assigns the requested user as @user" do
           put :update, params: { profile: @invalid_attrs }
-          #expect(response).to be_successful
           expect(assigns(:profile).user.username).to eq(@user.username)
           expect(response).to redirect_to(my_account_url)
         end
       end
     end
 
-    describe 'When not logged in' do
-      describe 'with valid params' do
-        it 'updates the requested user' do
+    describe "When not logged in" do
+      describe "with valid params" do
+        it "updates the requested user" do
           put :update, params: { profile: @attrs }
         end
 
-        it 'should be forbidden' do
+        it "should be forbidden" do
           put :update, params: { profile: @attrs }
           expect(response).to redirect_to(new_user_session_url)
         end
       end
 
-      describe 'with invalid params' do
-        it 'assigns the requested user as @user' do
+      describe "with invalid params" do
+        it "assigns the requested user as @user" do
           put :update, params: { profile: @invalid_attrs }
           expect(response).to redirect_to(new_user_session_url)
         end