qti 2.5.1 → 2.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 80bd134de64054ac80ae1f3293bfc25cac5f1f0801f82767134759fafa1e50fa
-  data.tar.gz: '008ddea00fb490d5f58266ded09d4ce64bb8578a7ea6c2d48017f6084a094d3e'
+  metadata.gz: e121df522dfdc1e21c2944bfa04334fdb4226c8a288356ff4ad1ef84e7354a86
+  data.tar.gz: 01cea5632a13762bc2929b4f61cafbc47893c38fda88c1d14a5d51a5eb1b73aa
 SHA512:
-  metadata.gz: 0be90cee407e5119935bc98e9ba32c5a6202ac678fdd6d19e344f5a7133253cfb76b806e43d1362861d1766f54fb381fba3d2c0f8ea73040c38e17fabac16d07
-  data.tar.gz: ef3d1d4d84c6b1d9546b39d2326b2be9887ea4659a36c898b6f8120bce6d39e75b79007b8c9510c014ff60f5d0256f73d75318cc0e8d93d6de8f9a4cb44fb8f4
+  metadata.gz: 43eb6eee2a5e6073750023429875e267565ab4ff2a23a6bcf292231ccf95f5a09da886208d12ad1fc4db7c339616801e908b0c0a4e3aee0c72f4e651065eaec3
+  data.tar.gz: cfea73f7c7d3a86f41de834205c634a5f0789ea57519eb180491a8411c55e292c4648673ca76d5aa1b04653ee7e704de50a422b3399780f3f428301a897d2968

data/README.md

@@ -80,8 +80,8 @@ cached, making additional runs significantly faster.
 Individual spec runs can be started like so:
 
 ```bash
-docker-compose run --rm app /bin/bash -l -c \
-  "BUNDLE_GEMFILE=spec/gemfiles/rails-6.0.gemfile rvm-exec 2.5 rspec"
+docker-compose run --rm app bash -lc \
+  "BUNDLE_GEMFILE=spec/gemfiles/rails-6.1.gemfile rvm-exec 2.7 rspec"
 ```
 
 If you'd like to mount your git checkout within the docker container running

data/lib/qti/exporter.rb

@@ -7,7 +7,7 @@ module Qti
       @assessment_test = assessment_test
       @package_root_path = args[:package_root_path] || '.'
       @exported_file_path =
-        File.join(File.expand_path('..', package_root_path), File.basename(export_file_name)) + '.zip'
+        "#{File.join(File.expand_path('..', package_root_path), File.basename(export_file_name))}.zip"
     end
 
     def export
@@ -167,12 +167,10 @@ module Qti
 
     def add_all_files(zipfile, package_root_path)
       Dir["#{package_root_path}/**/**"].each do |file|
-        begin
-          entry = file.sub(package_root_path + '/', '')
-          zipfile.add(entry, file)
-        rescue Zip::EntryExistsError
-          logger.info("#{file} already exists")
-        end
+        entry = file.sub("#{package_root_path}/", '')
+        zipfile.add(entry, file)
+      rescue Zip::EntryExistsError
+        logger.info("#{file} already exists")
       end
     end
   end

data/lib/qti/models/base.rb

@@ -1,19 +1,22 @@
 module Qti
   class ParseError < StandardError; end
+
   class SpecificationViolation < StandardError; end
+
   class UnsupportedSchema < StandardError; end
 
   module Models
     class Base
       attr_reader :doc, :path, :package_root, :resource
       attr_accessor :manifest
+
       delegate :metadata, to: :@resource, allow_nil: true
 
       def sanitize_content!(html)
         sanitizer.clean(html)
       end
 
-      def self.from_path!(path, package_root = nil, resource = nil)
+      def self.from_path!(path, package_root: nil, resource: nil)
         new(path: path, package_root: package_root, resource: resource)
       end
 
@@ -82,7 +85,8 @@ module Qti
       def package_root=(package_root)
         @package_root = package_root
         return unless @package_root
-        @package_root = Pathname.new(@package_root).cleanpath.to_s + '/'
+
+        @package_root = "#{Pathname.new(@package_root).cleanpath}/"
       end
 
       def relative_path

data/lib/qti/models/manifest.rb

@@ -24,7 +24,8 @@ module Qti
         builder = ASSESSMENT_CLASSES[version.split('/').first]
         raise_unsupported unless builder
         rsc = resource_for(identifier, version)
-        assessment = builder.from_path!(remap_href_path(asset_resource_for(rsc)), @package_root, rsc)
+        assessment = builder.from_path!(remap_href_path(asset_resource_for(rsc)), package_root: @package_root,
+                                                                                  resource: rsc)
         assessment.canvas_meta_data(rsc.canvas_metadata)
         assessment
       end
@@ -46,7 +47,7 @@ module Qti
       end
 
       def embedded_non_assessment
-        Qti::V2::Models::NonAssessmentTest.from_path!(@path, @package_root)
+        Qti::V2::Models::NonAssessmentTest.from_path!(@path, package_root: @package_root)
       end
     end
   end

data/lib/qti/models/resource.rb

@@ -91,7 +91,7 @@ module Qti
         Resource.new(resource_node(base_xpath), self)
       end
 
-      def assessment_identifiers(embedded_as_assessment = true)
+      def assessment_identifiers(embedded_as_assessment: true)
         id_list = identifier_list('/assessment')
         return id_list + [EMBEDDED_NON_ASSESSMENT_ID] if embedded_as_assessment && embedded_non_assessment?
         id_list
@@ -109,13 +109,16 @@ module Qti
           doc = load_asset_resource(rsc_path)
 
           # There are other types, but all we support right now are object banks...
-          Qti::V1::Models::ObjectBank.from_path!(rsc_path, @package_root, rsc) unless doc.search('objectbank').empty?
+          unless doc.search('objectbank').empty?
+            Qti::V1::Models::ObjectBank.from_path!(rsc_path, package_root: @package_root,
+                                                             resource: rsc)
+          end
         end.reject(&:nil?)
       end
 
       def objectbanks
         load_associated_content.select do |c|
-          c.class == Qti::V1::Models::ObjectBank
+          c.instance_of?(Qti::V1::Models::ObjectBank)
         end
       end
 

data/lib/qti/sanitizer.rb

@@ -37,6 +37,18 @@ module Qti
 
     private
 
+    def convert_canvas_math_images(env)
+      node = env[:node]
+      node_name = env[:node_name]
+      latex = node['data-equation-content']
+
+      return if env[:is_whitelisted] || !env[:node].element?
+      return unless node_name == 'img'
+      return unless latex
+
+      node.replace("\\(#{latex}\\)")
+    end
+
     def object_tag_transformer
       lambda do |env|
         return unless env[:node_name] == 'object'
@@ -73,6 +85,7 @@ module Qti
       transformers << src_transformers
       transformers << object_tag_transformer if import_objects
       transformers << remap_unknown_tags_transformer
+      transformers << method(:convert_canvas_math_images) if Qti.configuration.extract_latex_from_image_tags
       Sanitize::Config::RELAXED.merge transformers: transformers
     end
 
@@ -93,7 +106,7 @@ module Qti
       path = remap_href_path(node[:data])
       if path
         case node[:type]
-        when %r{^image\/}
+        when %r{^image/}
           return replace_with_image(node, node[:data])
         when 'text/html'
           return replace_with_html(node, path)

data/lib/qti/v1/models/assessment.rb

@@ -6,6 +6,10 @@ module Qti
 
         GROUP_ID = 'xmlns:section/xmlns:selection_ordering'.freeze
 
+        def identifier
+          @identifier ||= xpath_with_single_check('.//xmlns:assessment/@ident')&.content
+        end
+
         def title
           @title ||= xpath_with_single_check('.//xmlns:assessment/@title')&.content || File.basename(@path, '.xml')
         end

data/lib/qti/v1/models/assessment_item.rb

@@ -2,8 +2,8 @@ module Qti
   module V1
     module Models
       class AssessmentItem < Qti::V1::Models::Base
-        attr_reader :doc
-        attr_reader :resource
+        attr_reader :doc, :resource
+
         delegate :metadata, to: :@resource
 
         def initialize(item, package_root = nil, resource = nil)

data/lib/qti/v1/models/interactions/base_fill_blank_interaction.rb

@@ -3,7 +3,7 @@ module Qti
     module Models
       module Interactions
         class BaseFillBlankInteraction < BaseInteraction
-          CANVAS_REGEX ||= /(\[.+?\])/.freeze
+          CANVAS_REGEX ||= /(\[[A-Za-z0-9_\-.]+\])/.freeze
 
           def canvas_stem_items(item_prompt)
             item_prompt.split(CANVAS_REGEX).map.with_index do |stem_item, index|

data/lib/qti/v1/models/interactions.rb

@@ -56,11 +56,10 @@ module Qti
         end
 
         def self.get_matches(node, parent, classlist)
-          matches = classlist.each_with_object([]) do |interaction_class, result|
+          classlist.each_with_object([]) do |interaction_class, result|
             match = interaction_class.matches(node, parent)
             result << match if match
           end
-          matches
         end
       end
     end

data/lib/qti/v1/models/numerics/exact_match.rb

@@ -3,10 +3,6 @@ module Qti
     module Models
       module Numerics
         class ExactMatch < ScoringBase
-          def initialize(scoring_node)
-            super(scoring_node)
-          end
-
           def scoring_data
             return unless valid?
             Struct.new(

data/lib/qti/v1/models/numerics/margin_error.rb

@@ -3,10 +3,6 @@ module Qti
     module Models
       module Numerics
         class MarginError < ScoringBase
-          def initialize(scoring_node)
-            super(scoring_node)
-          end
-
           def scoring_data
             return unless valid?
             Struct.new(

data/lib/qti/v1/models/numerics/precision.rb

@@ -5,10 +5,6 @@ module Qti
         class Precision < ScoringBase
           include ActionView::Helpers::NumberHelper
 
-          def initialize(scoring_node)
-            super(scoring_node)
-          end
-
           def scoring_data
             return unless equal_node && gt_node && lte_node
             Struct.new(

data/lib/qti/v1/models/numerics/scoring_data.rb

@@ -4,6 +4,7 @@ module Qti
       module Numerics
         class ScoringData
           class UnsupportedNumreicType < StandardError; end
+
           def initialize(node)
             @scoring_node = ScoringNode.new(node)
           end

data/lib/qti/v1/models/numerics/within_range.rb

@@ -3,10 +3,6 @@ module Qti
     module Models
       module Numerics
         class WithinRange < ScoringBase
-          def initialize(scoring_node)
-            super(scoring_node)
-          end
-
           def scoring_data
             return unless valid?
             Struct.new(

data/lib/qti/v1/models/object_bank.rb

@@ -8,6 +8,17 @@ module Qti
             './/xmlns:qtimetadatafield/xmlns:fieldlabel[text()="bank_title"]/../xmlns:fieldentry'
           )&.content || File.basename(@path, '.xml')
         end
+
+        def identifier
+          @identifier ||= xpath_with_single_check('.//xmlns:objectbank/@ident')&.content
+        end
+
+        # tells us whether the bank was an account or course bank
+        def bank_type
+          @bank_type ||= xpath_with_single_check(
+            './/xmlns:qtimetadatafield/xmlns:fieldlabel[text()="bank_type"]/../xmlns:fieldentry'
+          )&.content
+        end
       end
     end
   end

data/lib/qti/v1/models/question_group.rb

@@ -34,6 +34,14 @@ module Qti
           selection&.xpath('xmlns:selection_number')&.text&.to_i
         end
 
+        def sourcebank_ref
+          selection&.xpath('xmlns:sourcebank_ref')&.text
+        end
+
+        def sourcebank_export_id
+          selection&.xpath('xmlns:sourcebank_export_id')&.text
+        end
+
         def points_per_item
           selection.xpath('xmlns:selection_extension/xmlns:points_per_item')&.text&.to_f
         end

data/lib/qti/v2/models/assessment_test.rb

@@ -5,6 +5,10 @@ module Qti
         include Qti::Models::AssessmentMetaBase
         include Qti::XPathHelpers
 
+        def identifier
+          @identifier ||= xpath_with_single_check('//xmlns:assessmentTest/@identifier')&.content
+        end
+
         def title
           @title ||= xpath_with_single_check('//xmlns:assessmentTest/@title')&.content || File.basename(@path, '.xml')
         end
@@ -27,7 +31,8 @@ module Qti
         end
 
         def create_assessment_item(ref)
-          item = Qti::V2::Models::AssessmentItem.from_path!(ref[:path], @package_root, ref[:resource])
+          item = Qti::V2::Models::AssessmentItem.from_path!(ref[:path], package_root: @package_root,
+                                                                        resource: ref[:resource])
           item.manifest = manifest
           item
         end

data/lib/qti/v2/models/interactions/match_interaction.rb

@@ -6,7 +6,7 @@ module Qti
     module Models
       module Interactions
         class MatchInteraction < BaseInteraction
-          extend Forwardable
+          extend ::Forwardable
 
           attr_reader :implementation
 

data/lib/qti/version.rb

@@ -1,3 +1,3 @@
 module Qti
-  VERSION = '2.5.1'.freeze
+  VERSION = '2.8.0'.freeze
 end

data/lib/qti.rb

@@ -4,7 +4,9 @@ require 'active_support/core_ext/string'
 require 'active_support/core_ext/hash/except'
 require 'active_support/core_ext/module/delegation'
 require 'dry-struct'
+require 'ostruct'
 require 'find'
+require 'forwardable'
 require 'mathml2latex'
 require 'nokogiri'
 require 'pathname'
@@ -14,8 +16,8 @@ require 'zip'
 
 module Qti
   class Importer
-    attr_reader :package_root
-    attr_reader :assessment_id
+    attr_reader :package_root, :assessment_id
+
     delegate :assessment_identifiers, to: :@manifest
     delegate :question_bank_identifiers, to: :@manifest
 
@@ -39,7 +41,7 @@ module Qti
     def self.manifest(path)
       mpath = manifest_path(path)
       package_root = File.dirname(mpath)
-      manifest = Qti::Models::Manifest.from_path!(mpath, package_root = package_root)
+      manifest = Qti::Models::Manifest.from_path!(mpath, package_root: package_root)
       [mpath, package_root, manifest]
     end
 
@@ -67,6 +69,14 @@ module Qti
       @import.create_question_group(question_group_ref)
     end
   end
+
+  def self.configuration
+    @configuration ||= OpenStruct.new
+  end
+
+  def self.configure
+    yield(configuration)
+  end
 end
 
 # The load order of all of these is important.

data/spec/fixtures/items_1.2/question_group.xml

@@ -15,6 +15,8 @@
             <selection_extension>
               <points_per_item>1.0</points_per_item>
             </selection_extension>
+            <sourcebank_ref>sourcebank_reference_uuid</sourcebank_ref>
+            <sourcebank_export_id>sourcebank_export_id_test</sourcebank_export_id>
           </selection>
         </selection_ordering>
         <item ident="i09d4a5492a38a5b4d1533b8fdbdf7376" title="Question">

data/spec/fixtures/items_1.2/true_false.xml

@@ -5,7 +5,7 @@
       <item title="Grading - specific - 3 pt score" ident="QUE_1003">
         <presentation>
           <material>
-            <mattext texttype="text/html"><![CDATA[If I get a 3, I must have done something wrong. <img align="bottom" alt="image.png" src="org0/images/image.png" border="0"/>]]></mattext>
+            <mattext texttype="text/html"><![CDATA[If I get a 3, I must have done something wrong. <img data-equation-content="sample equation" script="alert('bad')" align="bottom" alt="image.png" src="org0/images/image.png" border="0"/> <img script="alert('bad')" align="bottom" alt="image.png" src="org0/images/image.png" border="0"/>]]></mattext>
           </material>
           <response_lid ident="QUE_1004_RL" rcardinality="Single" rtiming="No">
             <render_choice shuffle="Yes">

data/spec/gemfiles/{rails-5.1.gemfile → rails-6.1.gemfile}

@@ -2,4 +2,4 @@ source 'https://rubygems.org'
 
 gemspec path: '../../'
 
-gem 'rails', '~> 5.1.4'
+gem 'rails', '~> 6.1.0'

data/spec/lib/qti/models/base_spec.rb

@@ -118,7 +118,7 @@ describe Qti::Models::Base do
       context 'with explicit package root' do
         let(:package_root) { File.join('spec', 'fixtures', 'test_qti_2.2') }
         let(:item_path) { File.join(package_root, 'true-false', 'true-false.xml') }
-        let(:item) { described_class.from_path!(item_path, package_root) }
+        let(:item) { described_class.from_path!(item_path, package_root: package_root) }
 
         it 'allows safe .. hrefs' do
           expect do
@@ -135,7 +135,7 @@ describe Qti::Models::Base do
 
       context 'with nil package root' do
         let(:item_path) { File.join('spec', 'fixtures', 'test_qti_2.2', 'true-false', 'true-false.xml') }
-        let(:item) { described_class.from_path!(item_path, nil) }
+        let(:item) { described_class.from_path!(item_path, package_root: nil) }
 
         it 'rejects .. hrefs' do
           expect do

data/spec/lib/qti/v1/models/assessment_item_spec.rb

@@ -29,6 +29,18 @@ describe Qti::V1::Models::AssessmentItem do
     it 'has sanitized item_body' do
       expect(loaded_class.item_body).to include '<img'
       expect(loaded_class.item_body).to include 'If I get a 3, I must have done something wrong.'
+      expect(loaded_class.item_body).not_to include 'script="alert(\'bad\')"'
+    end
+
+    it 'transforms canvas math content when conversion is enabled' do
+      expect(loaded_class.item_body).to include '\(sample equation\)'
+    end
+
+    it 'does not transform math content when conversion is Disabled' do
+      Qti.configure do |config|
+        config.extract_latex_from_image_tags = false
+      end
+      expect(loaded_class.item_body).not_to include '"sample equation"'
     end
 
     describe '#points_possible' do

data/spec/lib/qti/v1/models/assessment_spec.rb

@@ -19,6 +19,12 @@ describe Qti::V1::Models::Assessment do
           expect(loaded_class.assessment_items.count).to eq(expected_item_count)
         end
       end
+
+      describe '#identifier' do
+        it 'has the identifier' do
+          expect(loaded_class.identifier).to eq(expected_identifier)
+        end
+      end
     end
 
     shared_examples_for 'verify quiz items' do
@@ -36,6 +42,7 @@ describe Qti::V1::Models::Assessment do
       let(:path) { File.join(fixtures_path, 'test_qti_1.2', 'quiz.xml') }
       let(:loaded_class) { described_class.from_path!(path) }
       let(:expected_title) { '1.2 Import Quiz' }
+      let(:expected_identifier) { 'A1001' }
       let(:expected_item_count) { 5 }
 
       include_examples('basic quiz fields')
@@ -45,6 +52,7 @@ describe Qti::V1::Models::Assessment do
       let(:path) { File.join(fixtures_path, 'feedback_quiz_1.2.xml') }
       let(:loaded_class) { described_class.from_path!(path) }
       let(:expected_title) { 'I Can Haz Feedback' }
+      let(:expected_identifier) { 'i618e88580f76f70a1ed28804f497df9c' }
       let(:expected_item_count) { 3 }
       let(:expected_item_data) do
         [
@@ -74,6 +82,7 @@ describe Qti::V1::Models::Assessment do
       let(:path) { File.join(fixtures_path, 'all_canvas_simple_1.2.xml') }
       let(:loaded_class) { described_class.from_path!(path) }
       let(:expected_title) { 'Every Canvas Interaction' }
+      let(:expected_identifier) { 'ie09bc528e6ecd1cb9ebfafd940c20215' }
       let(:expected_item_count) { 10 }
       let(:expected_item_data) do
         [

data/spec/lib/qti/v1/models/interactions/base_fill_blank_interaction_spec.rb

@@ -10,6 +10,31 @@ describe Qti::V1::Models::Interactions::BaseFillBlankInteraction do
     end
   end
 
+  context 'canvas_stem_items' do
+    let(:file_path) { File.join(fixtures_path, 'canvas_multiple_fib_as_single.xml') }
+    let(:simple_prompt) { 'fill in the [blank]' }
+    let(:simple_expected) do
+      [
+        { id: 'stem_0', position: 1, type: 'text', value: 'fill in the ' },
+        { id: 'stem_1', position: 2, type: 'text', value: '[blank]' }
+      ]
+    end
+    let(:embedded_prompt) { '[[embedded] [groups]]' }
+    let(:embedded_expected) do
+      [
+        { id: 'stem_0', position: 1, type: 'text', value: '[' },
+        { id: 'stem_1', position: 2, type: 'text', value: '[embedded]' },
+        { id: 'stem_2', position: 3, type: 'text', value: ' ' },
+        { id: 'stem_3', position: 4, type: 'text', value: '[groups]' },
+        { id: 'stem_4', position: 5, type: 'text', value: ']' }
+      ]
+    end
+    it 'decomposes item prompts' do
+      expect(loaded_class.canvas_stem_items(simple_prompt)).to eq(simple_expected)
+      expect(loaded_class.canvas_stem_items(embedded_prompt)).to eq(embedded_expected)
+    end
+  end
+
   context 'canvas_multiple_dropdowns.xml' do
     let(:file_path) { File.join(fixtures_path, 'canvas_multiple_dropdowns.xml') }
     let(:expected_blanks) do

data/spec/lib/qti/v1/models/object_bank_spec.rb

@@ -1,4 +1,24 @@
 describe Qti::V1::Models::ObjectBank do
+  let(:doc) do
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8"?>
+      <questestinterop xmlns="http://www.imsglobal.org/xsd/ims_qtiasiv1p2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.imsglobal.org/xsd/ims_qtiasiv1p2 http://www.imsglobal.org/xsd/ims_qtiasiv1p2p1.xsd">
+        <objectbank ident="gooblegobble12345">
+          <qtimetadata>
+            <qtimetadatafield>
+              <fieldlabel>not_a_bank_title</fieldlabel>
+              <fieldentry>A different metadata entry</fieldentry>
+            </qtimetadatafield>
+            <qtimetadatafield>
+              <fieldlabel>bank_type</fieldlabel>
+              <fieldentry>account</fieldentry>
+            </qtimetadatafield>
+          </qtimetadata>
+        </objectbank>
+      </questestinterop>
+    XML
+  end
+
   describe 'bank loading' do
     [
       'gf3edf8167be16b3a65a00ca923132b07.xml.qti',
@@ -19,21 +39,25 @@ describe Qti::V1::Models::ObjectBank do
 
   describe '#title' do
     it 'missing bank_title defaults to filename' do
-      allow(File).to receive(:read).and_return(<<~XML)
-        <?xml version="1.0" encoding="UTF-8"?>
-        <questestinterop xmlns="http://www.imsglobal.org/xsd/ims_qtiasiv1p2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.imsglobal.org/xsd/ims_qtiasiv1p2 http://www.imsglobal.org/xsd/ims_qtiasiv1p2p1.xsd">
-          <objectbank ident="gooblegobble12345">
-            <qtimetadata>
-              <qtimetadatafield>
-                <fieldlabel>not_a_bank_title</fieldlabel>
-                <fieldentry>A different metadata entry</fieldentry>
-              </qtimetadatafield>
-            </qtimetadata>
-          </objectbank>
-        </questestinterop>
-      XML
+      allow(File).to receive(:read).and_return(doc)
       objectbank = described_class.new path: '/etc/FakeBank007.xml'
       expect(objectbank.title).to eq 'FakeBank007'
     end
   end
+
+  describe '#identifier' do
+    it 'has the identifier attribute' do
+      allow(File).to receive(:read).and_return(doc)
+      objectbank = described_class.new path: '/etc/FakeBank008.xml'
+      expect(objectbank.identifier).to eq 'gooblegobble12345'
+    end
+  end
+
+  describe '#bank_type' do
+    it 'has the bank_type attribute' do
+      allow(File).to receive(:read).and_return(doc)
+      objectbank = described_class.new path: '/etc/FakeBank008.xml'
+      expect(objectbank.bank_type).to eq 'account'
+    end
+  end
 end

data/spec/lib/qti/v1/models/question_group_spec.rb

@@ -7,6 +7,8 @@ describe Qti::V1::Models::QuestionGroup do
   it 'configures a group correctly' do
     expect(loaded_class.title).to eq('Group 1 (1/4)')
     expect(loaded_class.identifier).to eq('i4663897607358cfba8636ed6127b9466')
+    expect(loaded_class.sourcebank_ref).to eq('sourcebank_reference_uuid')
+    expect(loaded_class.sourcebank_export_id).to eq('sourcebank_export_id_test')
     expect(loaded_class.items.count).to eq(4)
     expect(loaded_class.selection_number).to eq(1)
     expect(loaded_class.points_per_item).to eq(1)

data/spec/lib/qti/v2/models/assessment_item_spec.rb

@@ -84,9 +84,9 @@ describe Qti::V2::Models::AssessmentItem do
 
   #   it 'removes the passage from the item_body' do
   #     item = test_object.create_assessment_item(item_ref)
-  # rubocop:disable AsciiComments
+  # rubocop:disable Style/AsciiComments
   #     expect(item.item_body).not_to include '¡El equipo de hockey te necesita!'
-  # rubocop:enable AsciiComments
+  # rubocop:enable Style/AsciiComments
   #   end
   # end
 end

data/spec/lib/qti/v2/models/assessment_test_spec.rb

@@ -15,6 +15,10 @@ describe Qti::V2::Models::AssessmentTest do
       expect(loaded_class.title).to eq 'Simple Feedback Test'
     end
 
+    it 'has the identifier' do
+      expect(loaded_class.identifier).to eq 'SPECTATUS-GENERATED-TEST'
+    end
+
     it 'gets dependency file refs' do
       refs = loaded_class.assessment_items
       expect(refs.all? { |ref| File.extname(ref[:path]) == '.xml' })
@@ -83,9 +87,9 @@ describe Qti::V2::Models::AssessmentTest do
   #       'passages', '0cfd5cf7-2c91-4b35-a57a-9f5d1709f68f.html'
   #     )
   #     stimulus = loaded_class.create_stimulus(stimulus_path)
-  # rubocop:disable AsciiComments
+  # rubocop:disable Style/AsciiComments
   #     expect(stimulus.title).to eq '¡El equipo de hockey te necesita!'
-  # rubocop:enable AsciiComments
+  # rubocop:enable Style/AsciiComments
   #   end
   # end
 end

data/spec/lib/qti/v2/models/stimulus_item_spec.rb

@@ -11,17 +11,17 @@
 #   end
 
 #   it 'has the title' do
-# rubocop:disable AsciiComments
+# rubocop:disable Style/AsciiComments
 #     expect(loaded_class.title).to eq '¡El equipo de hockey te necesita!'
-# rubocop:enable AsciiComments
+# rubocop:enable Style/AsciiComments
 #   end
 
 #   it 'has sanitized item_body' do
 #     expect(loaded_class.body).to include '<div'
 #     expect(loaded_class.body).to include 'Listen to the audio passage.'
-# rubocop:disable AsciiComments
+# rubocop:disable Style/AsciiComments
 #     expect(loaded_class.body).to include '¡'
-# rubocop:enable AsciiComments
+# rubocop:enable Style/AsciiComments
 #   end
 
 #   it 'has the identifier used to identify it in manifest file' do

data/spec/lib/qti_spec.rb

@@ -40,8 +40,8 @@ describe Qti::Importer do
 
       it 'sets the path and package root properly' do
         item = importer.create_assessment_item(importer.assessment_item_refs.first)
-        expect(item.path).to eq file_path + '/quiz.xml'
-        expect(item.package_root).to eq file_path + '/'
+        expect(item.path).to eq "#{file_path}/quiz.xml"
+        expect(item.package_root).to eq "#{file_path}/"
       end
     end
 
@@ -64,7 +64,7 @@ describe Qti::Importer do
         ref = importer.assessment_item_refs.first
         item = importer.create_assessment_item(ref)
         expect(item.path).to eq ref[:path]
-        expect(item.package_root).to eq file_path + '/'
+        expect(item.package_root).to eq "#{file_path}/"
         expect(item.manifest).not_to be_nil
       end
     end

data/spec/spec_helper.rb

@@ -1,7 +1,7 @@
 require 'byebug'
 
 # Limit coverage reporting to one build:
-if /^2\.6/ =~ RUBY_VERSION && /rails-6\.0/ =~ ENV['BUNDLE_GEMFILE']
+if /^2\.7/ =~ RUBY_VERSION && /rails-6\.1/ =~ ENV['BUNDLE_GEMFILE']
   require 'simplecov'
 
   SimpleCov.start do
@@ -16,7 +16,11 @@ end
 $LOAD_PATH.unshift File.expand_path('../lib', __dir__)
 require 'qti'
 
-Dir['./spec/support/**/*.rb'].each { |f| require f }
+Qti.configure do |config|
+  config.extract_latex_from_image_tags = true
+end
+
+Dir['./spec/support/**/*.rb'].sort.each { |f| require f }
 
 RSpec.configure do |config|
   config.expect_with :rspec do |expectations|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: qti
 version: !ruby/object:Gem::Version
-  version: 2.5.1
+  version: 2.8.0
 platform: ruby
 authors:
 - Hannah Bottalla
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-25 00:00:00.000000000 Z
+date: 2022-01-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionview
@@ -17,40 +17,40 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
 - !ruby/object:Gem::Dependency
   name: dry-struct
   requirement: !ruby/object:Gem::Requirement
@@ -141,14 +141,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.17'
+        version: '2.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.17'
+        version: '2.1'
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
@@ -183,14 +183,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -225,14 +225,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.74.0
+        version: 1.8.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.74.0
+        version: 1.8.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.9.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.9.1
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -613,9 +627,9 @@ files:
 - spec/fixtures/with_banks/non_cc_assessments/g9a20cf3af178b54e4792cbe992f65790.xml.qti
 - spec/fixtures/with_banks/non_cc_assessments/gab22de457404cb5cf022078f1e4da75e.xml.qti
 - spec/fixtures/with_banks/non_cc_assessments/gf3edf8167be16b3a65a00ca923132b07.xml.qti
-- spec/gemfiles/rails-5.1.gemfile
 - spec/gemfiles/rails-5.2.gemfile
 - spec/gemfiles/rails-6.0.gemfile
+- spec/gemfiles/rails-6.1.gemfile
 - spec/lib/qti/assessment_item_exporter_spec.rb
 - spec/lib/qti/exporter_spec.rb
 - spec/lib/qti/models/assessment_meta_spec.rb
@@ -678,7 +692,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -965,9 +979,9 @@ test_files:
 - spec/fixtures/with_banks/non_cc_assessments/g9a20cf3af178b54e4792cbe992f65790.xml.qti
 - spec/fixtures/with_banks/non_cc_assessments/gab22de457404cb5cf022078f1e4da75e.xml.qti
 - spec/fixtures/with_banks/non_cc_assessments/gf3edf8167be16b3a65a00ca923132b07.xml.qti
-- spec/gemfiles/rails-5.1.gemfile
 - spec/gemfiles/rails-5.2.gemfile
 - spec/gemfiles/rails-6.0.gemfile
+- spec/gemfiles/rails-6.1.gemfile
 - spec/lib/qti/assessment_item_exporter_spec.rb
 - spec/lib/qti/exporter_spec.rb
 - spec/lib/qti/models/assessment_meta_spec.rb