Skip tests that require plugin unless plugin installed

fixes CNVS-36355

Test plan:

- docker-compose build --pull
- docker-compose run --rm web bundle exec rspec \
  spec/{apis,controllers,helpers,initializers,integration,lib,message,\
  middleware,migrations,models,observers,presenters,serializers,views}\
  /**/*_spec.rb

Change-Id: I4220a314dbc2e343dae26e6e522806f4f2666f85
Reviewed-on: https://gerrit.instructure.com/120671
Reviewed-by: Cody Cutrer <[email protected]>
Tested-by: Jenkins
Product-Review: Cody Cutrer <[email protected]>
QA-Review: Cody Cutrer <[email protected]>
Product-Review: Jon Jensen <[email protected]>
QA-Review: Jon Jensen <[email protected]>

gems/attachment_fu/lib/attachment_fu/backends/s3_backend.rb

@@ -142,9 +142,7 @@ class ConfigFileNotFoundError < StandardError; end
       def self.included(base) #:nodoc:
         require 'aws-sdk-s3'
 
-        s3_config_path = base.attachment_options[:s3_config_path] || (Rails.root + 'config/amazon_s3.yml')
-        s3_config = YAML.load(ERB.new(File.read(s3_config_path)).result)[Rails.env].symbolize_keys
-
+        s3_config = load_s3_config(base.attachment_options[:s3_config_path])
         bucket_name = s3_config.delete(:bucket_name)
 
         s3 = Aws::S3::Resource.new(Canvas::AWS.validate_v2_config(s3_config, 'amazon_s3.yml'))
@@ -153,6 +151,11 @@ def self.included(base) #:nodoc:
         base.before_update :rename_file
       end
 
+      def self.load_s3_config(path = nil)
+        s3_config_path = path || (Rails.root + 'config/amazon_s3.yml')
+        YAML.load(ERB.new(File.read(s3_config_path)).result)[Rails.env].symbolize_keys
+      end
+
       # Overwrites the base filename writer in order to store the old filename
       def filename=(value)
         @old_filename = filename unless filename.nil? || @old_filename

gems/plugins/moodle_importer/spec_canvas/moodle2_converter_spec.rb

@@ -105,6 +105,8 @@
 
   context "modules" do
     it "should convert modules and module items" do
+      skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
       expect(@course.context_modules.count).to eq 8
       expect(@course.context_module_tags.where(:content_type => "Assignment", :title => "Assignment Name")).to be_exists
       expect(@course.context_module_tags.where(:content_type => "WikiPage", :title => "My Sample Page")).to be_exists

spec/apis/v1/master_courses/master_templates_api_spec.rb

@@ -427,6 +427,8 @@ def run_master_migration
     end
 
     it "returns change information from the minion side" do
+      skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
       minion = @minions.first
       minion_migration = minion.content_migrations.last
       minion_page = minion.wiki.wiki_pages.where(migration_id: @template.migration_id_for(@page)).first

spec/lib/config_file_spec.rb

@@ -20,6 +20,8 @@
 
 describe ConfigFile do
   describe ".cache_object" do
+    before { ConfigFile.unstub }
+
     it "caches objects" do
       expect(File).to receive(:exist?).and_return(true)
       expect(File).to receive(:read).and_return('test: {}')

spec/models/content_migration/course_copy_dates_spec.rb

@@ -433,6 +433,8 @@ def copy_assignment(options = {})
     end
 
     it "should not break link resolution in quiz_data" do
+      skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
       topic = @copy_from.discussion_topics.create!(:title => "some topic", :message => "<p>some text</p>")
 
       html = "<a href='/courses/#{@copy_from.id}/discussion_topics/#{topic.id}'>link</a>"

spec/models/content_migration/course_copy_external_content_spec.rb

@@ -55,6 +55,8 @@ def self.send_imported_content(course, imported_content)
     end
 
     it "should translate ids for copied course content" do
+      skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
       assmt = @copy_from.assignments.create!
       topic = @copy_from.discussion_topics.create!(:message => "hi", :title => "discussion title")
       ann = @copy_from.announcements.create!(:message => "goodbye")

spec/models/content_migration/course_copy_unpublished_items_spec.rb

@@ -125,6 +125,8 @@
     end
 
     it "should not re-unpublish module items on re-copy" do
+      skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
       mod = @copy_from.context_modules.create!(:name => "some module")
       tags = []
 

spec/models/content_migration_spec.rb

@@ -433,6 +433,8 @@ def test_zip_import(context)
   end
 
   it "should correclty handle media comment resolution in quizzes" do
+    skip 'Requires QtiMigrationTool' unless Qti.qti_enabled?
+
     course_with_teacher
     cm = ContentMigration.new(:context => @course, :user => @user)
     cm.migration_type = 'canvas_cartridge_importer'

spec/models/user_observer_spec.rb

@@ -119,6 +119,10 @@
   end
 
   it "should not enroll the observer in institutions where they lack a login" do
+    unless has_sharding?
+      skip 'Sharding specs fail without additional support from a multi-tenancy plugin'
+    end
+
     a1 = account_model
     c1 = course_factory(account: a1, active_all: true)
     e1 = student_in_course(course: c1, user: student, active_all: true)

spec/sharding_spec_helper.rb

@@ -20,13 +20,23 @@
 require_relative 'spec_helper'
 require_relative 'support/onceler/sharding'
 
+def has_sharding?
+  User.instance_method(:associated_shards).owner != User
+end
+
 def specs_require_sharding
-  include Switchman::RSpecHelper
-  include Onceler::Sharding
+  if has_sharding?
+    include Switchman::RSpecHelper
+    include Onceler::Sharding
 
-  before :all do
-    Shard.with_each_shard do
-      Role.ensure_built_in_roles!
+    before :all do
+      Shard.with_each_shard do
+        Role.ensure_built_in_roles!
+      end
+    end
+  else
+    before(:once) do
+      skip 'Sharding specs fail without additional support from a multi-tenancy plugin'
     end
   end
 end

spec/spec_helper.rb

@@ -48,6 +48,7 @@ def self.included(other)
 end
 
 Dir[Rails.root.join("spec/support/**/*.rb")].each { |f| require f }
+require_relative 'sharding_spec_helper'
 
 # nuke the db (say, if `rake db:migrate RAILS_ENV=test` created records),
 # and then ensure people aren't creating records outside the rspec
@@ -398,6 +399,10 @@ def using_transactions_properly?
     Timecop.safe_mode = true
   end
 
+  config.before do
+    allow(AttachmentFu::Backends::S3Backend).to receive(:load_s3_config) { StubS3::AWS_CONFIG.dup }
+  end
+
   # this runs on post-merge builds to capture dependencies of each spec;
   # we then use that data to run just the bare minimum subset of selenium
   # specs on the patchset builds
@@ -661,21 +666,20 @@ def as(ancestor)
   end
 
   module StubS3
+    AWS_CONFIG = {
+      access_key_id: 'stub_id',
+      secret_access_key: 'stub_key',
+      region: 'us-east-1',
+      stub_responses: true,
+      bucket_name: 'no-bucket'
+    }.freeze
+
     def self.stubbed?
       false
     end
 
     def load(file, *args)
-      if StubS3.stubbed? && file == 'amazon_s3'
-        return {
-          access_key_id: 'stub_id',
-          secret_access_key: 'stub_key',
-          region: 'us-east-1',
-          stub_responses: true,
-          bucket_name: 'no-bucket'
-        }
-      end
-
+      return AWS_CONFIG.dup if StubS3.stubbed? && file == 'amazon_s3'
       super
     end
   end