Subject edit (#77)

Author: kdimopulu

staff_features/shared/step_definitions.rb

@@ -79,11 +79,9 @@
 end
 
 When 'the user clicks on {string} in the dropdown menu' do |string|
-  dropdown_menu = all('.dropdown-menu').first
-
   dropdown_menus = all('.dropdown-menu')
 
-  within dropdown_menus.first do |dropdown_menu|
+  within dropdown_menus.first do
     elements = dropdown_menu.all(:xpath, ".//*[contains(text(), '#{string}')]")
 
     elements.each do |element|
@@ -245,12 +243,12 @@
 end
 
 When 'the user changes the {string} field to {string}' do |field, value|
-  element = find_field(field, match: :first)
+  field = find_field(field, match: :first)
 
-  if element.tag_name == 'select'
-    element.select value
+  if field.tag_name == 'select'
+    field.select value.strip
   else
-    element.fill_in with: value
+    field.fill_in with: value
   end
 end
 
@@ -272,6 +270,12 @@
   expect(find('.alert.alert-success.with-hide-alert').text).to match(/^#{string}.*updated$/i)
 end
 
+Then('the {string} saved message is displayed') do |string|
+  wait_for_ajax if current_url.include? 'resources'
+
+  expect(find('.alert.alert-success.with-hide-alert').text).to match(/^#{string}.*saved$/i)
+end
+
 Then('the {string} deleted message is displayed') do |string|
   expect(find('.alert.alert-success.with-hide-alert').text).to match(/^#{string}.*deleted$/i)
 end
@@ -384,3 +388,13 @@
 
   click_on 'Save'
 end
+
+When 'the user clears {string} in the {string} form' do |label, form_title|
+  section_title = find('h3', text: form_title)
+  section = section_title.ancestor('section')
+  expect(section[:id]).to_not eq nil
+
+  within section do
+    select '', from: label
+  end
+end

staff_features/subjects/step_definitions/subject_edit.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+Given 'the Subject appears in the search results list' do
+  visit "#{STAFF_URL}/subjects"
+
+  fill_in 'filter-text', with: "subject_term_#{@uuid}"
+
+  within '.search-filter' do
+    find('button').click
+  end
+
+  search_result_rows = all('#tabledSearchResults tbody tr')
+  expect(search_result_rows.length).to eq 1
+
+  within search_result_rows[0] do
+    element = find('a', text: 'Edit')
+
+    @subject_id = URI.decode_www_form_component(element[:href]).split('/').pop
+  end
+end
+
+Then 'the Subject is opened in the edit mode' do
+  uri = current_url.split('/')
+
+  action = uri.pop
+  subject_id = uri.pop
+
+  expect(action).to eq 'edit'
+  expect(subject_id).to eq @subject_id
+end
+
+Given 'the Subject is opened in the view mode' do
+  visit "#{STAFF_URL}/subjects/#{@subject_id}"
+end
+
+Given 'the Subject is opened in edit mode' do
+  visit "#{STAFF_URL}/subjects/#{@subject_id}/edit"
+end
+
+Given 'the Subject has one Metadata Rights Declarations' do
+  click_on 'Add Metadata Rights Declaration'
+
+  select 'This record is made available under an Universal 1.0 Public Domain Dedication Creative Commons license.', from: 'License'
+
+  click_on 'Save'
+
+  expect(find('.alert.alert-success.with-hide-alert').text).to eq 'Subject Saved'
+end
+
+Then 'the Subject does not have Metadata Rights Declarations' do
+  elements = all('#subject_metadata_rights_declarations_ li')
+
+  expect(elements.length).to eq 0
+end

staff_features/subjects/subject_edit.feature

@@ -0,0 +1,35 @@
+Feature: Subject Edit
+  Background:
+    Given an administrator user is logged in
+      And a Subject has been created
+  Scenario: Subject is opened in the edit mode from the browse menu
+    Given the Subject appears in the search results list
+     When the user clicks on 'Edit'
+     Then the Subject is opened in the edit mode
+  Scenario: Subject is opened in the edit mode from the view mode
+    Given the Subject is opened in the view mode
+     When the user clicks on 'Edit'
+     Then the Subject is opened in the edit mode
+  Scenario Outline: Subject is successfully updated
+    Given the Subject is opened in edit mode
+     When the user changes the '<Field>' field to '<NewValue>'
+      And the user clicks on 'Save'
+     Then the 'Subject' saved message is displayed
+     Then the field '<Field>' has value '<NewValue>'
+      Examples:
+       | Field        | NewValue        |
+       | Scope Note   | Test Scope Note |
+  Scenario: Subject is not updated
+    Given the Subject is opened in edit mode
+     When the user clears 'Source' in the 'Basic Information' form
+      And the user clicks on 'Save'
+     Then the following error messages are displayed
+       | Source - Property is required but was missing |
+  Scenario: Delete sub-record of a Subject
+    Given the Subject is opened in edit mode
+      And the Subject has one Metadata Rights Declarations
+     When the user clicks on remove icon in the 'Metadata Rights Declarations' form
+      And the user clicks on 'Confirm Removal'
+      And the user clicks on 'Save'
+     Then the 'Subject' saved message is displayed
+      And the Subject does not have Metadata Rights Declarations