annotations_editing_spec.rb

# frozen_string_literal: true

require 'rails_helper'

RSpec.feature 'Annotations::Editing', type: :feature do
  let!(:funder) { create(:org, :funder) }

  let!(:org) { create(:org, :school, :organisation) }

  let!(:template) { create(:template, :published, :publicly_visible, org: funder) }

  let!(:phase) { create(:phase, template: template) }

  let!(:section) { create(:section, phase: phase) }

  let!(:question) { create(:question, section: section) }

  let!(:annotation) do
    create(:annotation, question: question, org: org,
                        text: 'Foo bar', type: 'example_answer')
  end

  let!(:user) { create(:user, org: org) }

  before do
    create(:template, :default, :published)
    user.perms << create(:perm, :modify_templates)
    user.perms << create(:perm, :add_organisations)
    sign_in user
    visit org_admin_templates_path
  end

  scenario 'Admin changes an Annotation of a draft Template', :js do
    click_link 'Customisable Templates'
    within("#template_#{template.id}") do
      click_button 'Actions'
    end
    expect do
      click_link 'Customise'
      # `org_admin_template_path(Template.last)` would be preferred over %r{#{org_admin_templates_path}/\d+}
      # However, the test is currently evaluating Template.count prior to the new Template being created
      expect(page).to have_current_path(%r{#{org_admin_templates_path}/\d+})
    end.to change { Template.count }.by(1)

    # New Template created
    template = Template.last
    click_link 'Customise phase'

    click_link section.title

    within("fieldset#fields_annotation_#{template.annotation_ids.last}") do
      id = "question_annotations_attributes_annotation_#{template.annotation_ids.last}_text"
      tinymce_fill_in(id, with: 'Noo bar')
    end

    # NOTE: This is question 2, since Annotation was copied upon clicking "Customise"
    within("#edit_question_#{template.question_ids.last}") do
      # Expect it to destroy the newly cleared Annotation
      expect do
        click_button 'Save'
        current_path = org_admin_template_phase_path(template,
                                                     template.phases.first) +
                       "?section=#{template.phases.first.sections.first.id}"
        expect(page).to have_current_path(current_path)
      end.not_to change { Annotation.count }
    end
    expect(annotation.text).to eql('Foo bar')
    expect(Annotation.order('created_at').last.text).to eql('<p>Noo bar</p>')
    expect(page).not_to have_errors
  end

  scenario "Admin sets a Template's question annotation to blank string", :js do
    click_link 'Customisable Templates'
    within("#template_#{template.id}") do
      click_button 'Actions'
    end
    expect do
      click_link 'Customise'
      # `org_admin_template_path(Template.last)` would be preferred over %r{#{org_admin_templates_path}/\d+}
      # However, the test is currently evaluating Template.count prior to the new Template being created
      expect(page).to have_current_path(%r{#{org_admin_templates_path}/\d+})
    end.to change { Template.count }.by(1)
    template = Template.last
    click_link 'Customise phase'
    click_link section.title
    # NOTE: This is annotation 2, since Annotation was copied upon clicking "Customise"
    within("fieldset#fields_annotation_#{template.annotation_ids.last}") do
      id = "question_annotations_attributes_annotation_#{template.annotation_ids.last}_text"
      tinymce_fill_in(:"#{id}", with: ' ')
    end
    # NOTE: This is question 2, since Annotation was copied upon clicking "Customise"
    within("#edit_question_#{template.question_ids.last}") do
      # Expect it to destroy the newly cleared Annotation
      expect do
        click_button 'Save'
        current_path = org_admin_template_phase_path(template,
                                                     template.phases.first) +
                       "?section=#{template.phases.first.sections.first.id}"
        expect(page).to have_current_path(current_path)
      end.to change { Annotation.count }.by(-1)
    end
    expect(page).not_to have_errors
  end
end