panda_pal 4.1.0.beta2 → 5.0.0.beta.4

data/app/views/panda_pal/lti/iframe_cookie_fix.html.erb

@@ -1,12 +0,0 @@
-<script nonce=<%= content_security_policy_script_nonce %>>
-  const mainWindow = window.parent;
-  var url = window.location.href;
-  // Until PLAT-4836 is resolved, we need to make sure our url has a "?" in it.
-  if (!(url.indexOf("?") > -1)) {
-    url = url + "?dummy_param=1"
-  }
-  mainWindow.postMessage({
-    messageType: "requestFullWindowLaunch",
-    data: url
-  }, '*');
-</script>

data/spec/models/panda_pal/organization/settings_validation_spec.rb

@@ -1,175 +0,0 @@
-require 'rails_helper'
-
-module PandaPal
-  PandaPal::Organization
-  RSpec.describe OrganizationConcerns::SettingsValidation, type: :model do
-    let!(:org) { create :panda_pal_organization }
-
-    def set_test_settings_structure
-      PandaPal.lti_options = {
-        title: 'Test App',
-        settings_structure: structure,
-      }
-    end
-
-    RSpec.shared_examples "shared stuff" do
-      it 'does not perform any validations if settings is not defined' do
-        PandaPal.lti_options = {}
-        expect(org.valid?).to be_truthy
-      end
-
-      it 'does not perform any validations if options is not defined' do
-        PandaPal.lti_options = nil
-        expect(org.valid?).to be_truthy
-      end
-
-      it 'does perform validations if settings_structure is defined' do
-        set_test_settings_structure
-        org.valid?
-        expect(org.valid?).to be_falsey
-      end
-
-      it 'will fail if a required setting is not present' do
-        set_test_settings_structure
-        expect(org.valid?).to be_falsey
-        errors = org.errors.messages[:settings]
-        expect(errors[0]).to eq("Entry [:canvas] is required")
-      end
-
-      it 'will fail if a setting is supplied but data_type is wrong' do
-        set_test_settings_structure
-        org.settings = {canvas: "Dog", reports: {}}
-        expect(org.valid?).to be_falsey
-        errors = org.errors.messages[:settings]
-        expect(errors[0]).to eq("Expected [:canvas] to be a Hash. Was a String")
-      end
-
-      it 'will fail if a required subsetting is missing' do
-        set_test_settings_structure
-        org.settings = {canvas: {base_url: 'http://'}, reports: {}}
-        expect(org.valid?).to be_falsey
-        errors = org.errors.messages[:settings]
-        expect(errors[0]).to eq("Entry [:canvas][:api_token] is required")
-      end
-
-      it 'will fail if extra options are specified' do
-        set_test_settings_structure
-        org.settings = {canvas: {base_url: 'http://', api_token: 'TEST'}, reports: {}, unknown_option: "WHAT IS THIS?"}
-        expect(org.valid?).to be_falsey
-        errors = org.errors.messages[:settings]
-        expect(errors[0]).to eq("Did not expect [:] to contain [unknown_option]")
-      end
-
-      it 'will pass if all structure is maintained' do
-        set_test_settings_structure
-        org.settings = {canvas: {base_url: 'http://', api_token: 'TEST'}, reports: {submissions_report_time_length: 30.minutes, max_recheck_time: 10.hours}}
-        expect(org.valid?).to be_truthy
-      end
-    end
-
-    context 'new settings_structure' do
-      let!(:properties) do
-        {
-          canvas: {
-            required: true,
-            type: 'Hash',
-            properties: {
-              api_token: { type: 'String', required: true },
-              base_url: { type: 'String', required: true },
-            },
-          },
-          reports: {
-            required: true,
-            type: 'Hash',
-            allow_additional: true,
-          }
-        }
-      end
-
-      let!(:structure) do
-        {
-          properties: properties,
-        }
-      end
-
-      it_behaves_like("shared stuff")
-
-      describe ':allow_additional' do
-        it 'passes extra properties if allow_additional is a matching Hash' do
-          structure[:allow_additional] = { type: 'String' }
-          set_test_settings_structure
-          org.settings = {canvas: {base_url: 'http://', api_token: 'TEST'}, reports: {}, unknown_option: "WHAT IS THIS?"}
-          expect(org).to be_valid
-        end
-
-        it 'fails extra properties if allow_additional is an unmatching Hash' do
-          structure[:allow_additional] = { type: 'Hash' }
-          set_test_settings_structure
-          org.settings = {canvas: {base_url: 'http://', api_token: 'TEST'}, reports: {}, unknown_option: "WHAT IS THIS?"}
-          expect(org).not_to be_valid
-        end
-
-        it 'passes extra properties if allow_additional is a blank Hash' do
-          structure[:allow_additional] = { }
-          set_test_settings_structure
-          org.settings = {canvas: {base_url: 'http://', api_token: 'TEST'}, reports: {}, unknown_option: "WHAT IS THIS?"}
-          expect(org).to be_valid
-        end
-      end
-
-      describe ':validate' do
-        let!(:properties) do
-          {
-            blah: {
-              type: 'String',
-              validate: -> (v, *args) { v == 'blah' ? nil : '<path> failed validation' },
-            }
-          }
-        end
-
-        it 'supports a custom validation proc' do
-          set_test_settings_structure
-          org.settings = { blah: 'blah' }
-          expect(org).to be_valid
-        end
-
-        it 'replaces <path> with the human readable path' do
-          set_test_settings_structure
-          org.settings = { blah: '' }
-          expect(org.valid?).to be_falsey
-          errors = org.errors.messages[:settings]
-          expect(errors[0]).to eq("[:blah] failed validation")
-        end
-      end
-    end
-
-    context 'old settings_structure' do
-      let!(:structure) do
-        YAML.load("
-          canvas:
-            is_required: true
-            data_type: Hash
-            api_token:
-              is_required: true
-              data_type: String
-            base_url:
-              is_required: true
-              data_type: String
-          reports:
-            is_required: true
-            data_type: Hash
-            active_term_allowance:
-            submissions_report_time_length:
-              is_required: false
-              data_type: ActiveSupport::Duration
-            recheck_wait:
-              data_type: ActiveSupport::Duration
-            max_recheck_time:
-              is_required: false
-        ").deep_symbolize_keys
-      end
-
-      it_behaves_like("shared stuff")
-    end
-  end
-end

data/spec/models/panda_pal/organization/task_scheduling_spec.rb

@@ -1,134 +0,0 @@
-require 'rails_helper'
-
-module PandaPal
-  PandaPal::Organization
-
-  RSpec.describe OrganizationConcerns::TaskScheduling, type: :model do
-    let!(:organization) { create(:panda_pal_organization) }
-    let(:schedules) { {} }
-
-    before :each do
-      Organization.instance_variable_set(:@_schedule_descriptors, nil)
-      Organization.scheduled_task('0 0 0 * * *', :ident) { }
-
-      allow(Sidekiq).to receive(:remove_schedule)
-      allow(Sidekiq).to receive(:get_schedule) { schedules }
-    end
-
-    def descriptors
-      Organization.instance_variable_get(:@_schedule_descriptors)
-    end
-
-    def descriptor
-      descriptors[descriptors.keys[0]]
-    end
-
-    describe '.scheduled_task' do
-      it 'adds to the set of descriptors' do
-        expect(descriptors).to match(
-          'ident' => {
-            :key=>"ident",
-            :schedule=>"0 0 0 * * *",
-            :worker=>Proc,
-            :queue=>"default",
-          }
-        )
-      end
-    end
-
-    describe '.sync_schedules' do
-      it 'adds new schedules' do
-        expect(Sidekiq).to receive(:set_schedule).with(/org:\w+-ident/, anything)
-        Organization.sync_schedules
-      end
-
-      it 'removes schedules for deleted Orgs' do
-        schedules['org:deleted_org-schedule'] = {}
-        expect(Sidekiq).to receive(:remove_schedule).with('org:deleted_org-schedule')
-        Organization.sync_schedules
-      end
-
-      it 'keeps schedules created from other sources' do
-        schedules['other-schedule'] = {}
-        expect(Sidekiq).not_to receive(:remove_schedule).with('other-schedule')
-        Organization.sync_schedules
-      end
-    end
-
-    describe '#generate_schedule' do
-      it 'generates the expected schedule' do
-        expect(organization.generate_schedule).to eq({
-          "org:#{organization.name}-ident" => {
-            "cron"=>"0 0 0 * * *",
-            "queue"=>"default",
-            "class"=>"PandaPal::OrganizationConcerns::TaskScheduling::ScheduledTaskExecutor",
-            "args"=>[organization.name, "ident"],
-          }
-        })
-      end
-    end
-
-    describe '#schedule_task_cron_time' do
-      before :each do
-        organization.settings = { timezone: 'America/Denver' }
-      end
-
-      it 'includes timezone if in settings' do
-        expect(organization.send(:schedule_task_cron_time, descriptor)).to eq '0 0 0 * * * America/Denver'
-      end
-
-      it 'does not re-append timezone if already present' do
-        descriptor[:schedule] = '1 1 1 * * * America/Chicago'
-        expect(organization.send(:schedule_task_cron_time, descriptor)).to eq '1 1 1 * * * America/Chicago'
-      end
-    end
-
-    describe '#sync_schedule' do
-      it 'adds new schedules' do
-        expect(Sidekiq).to receive(:set_schedule).with(/org:\w+-ident/, anything)
-        organization.sync_schedule
-      end
-
-      it 'removes old jobs' do
-        schedules["org:#{organization.name}-old_schedule"] = {}
-        expect(Sidekiq).to receive(:remove_schedule).with("org:#{organization.name}-old_schedule")
-        organization.sync_schedule
-      end
-    end
-
-    describe 'SettingsValidation' do
-      before :each do
-        organization.settings = {}
-      end
-
-      it 'allows [:task_schedules] entry to be missing' do
-        expect(organization).to be_valid
-      end
-
-      it 'does not require [:task_schedules] sub-entries' do
-        organization.settings[:task_schedules] = {}
-        expect(organization).to be_valid
-      end
-
-      it 'allows task entry to be false' do
-        organization.settings[:task_schedules] = { ident: false }
-        expect(organization).to be_valid
-      end
-
-      it 'allows task entry to be a valid cron string' do
-        organization.settings[:task_schedules] = { ident: '0 0 0 * * * America/Denver' }
-        expect(organization).to be_valid
-      end
-
-      it 'does not allow task entry to be an invalid cron string' do
-        organization.settings[:task_schedules] = { ident: 'blort' }
-        expect(organization).not_to be_valid
-      end
-
-      it 'does not allow sub-entries for unknown tasks' do
-        organization.settings[:task_schedules] = { missing_ident: '0 0 0 * * * America/Denver' }
-        expect(organization).not_to be_valid
-      end
-    end
-  end
-end