qti 0.9.0 → 0.9.1

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

@@ -54,37 +54,37 @@ describe Qti::Models::Base do
       end
 
       it 'uses an implicit package root' do
-        expect {
+        expect do
           loaded_class.remap_href_path('../sneaky.txt')
-        }.to raise_error(Qti::ParseError)
+        end.to raise_error(Qti::ParseError)
       end
 
-      context "with explicit package root" 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) }
 
         it 'allows safe .. hrefs' do
-          expect {
+          expect do
             item.remap_href_path('../okay.txt')
-          }.not_to raise_error
+          end.not_to raise_error
         end
 
         it 'rejects attempts to escape the package' do
-          expect {
+          expect do
             item.remap_href_path('../../bad.txt')
-          }.to raise_error(Qti::ParseError)
+          end.to raise_error(Qti::ParseError)
         end
       end
 
-      context "with nil package root" 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) }
 
         it 'rejects .. hrefs' do
-          expect {
+          expect do
             item.remap_href_path('../no_longer_okay.txt')
-          }.to raise_error(Qti::ParseError)
+          end.to raise_error(Qti::ParseError)
         end
       end
     end

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

@@ -41,5 +41,4 @@ describe Qti::Models::Manifest do
     file = File.join(fixtures_path, 'items_2.1/imsmanifest.xml')
     expect { described_class.from_path!(file).assessment_test }.to raise_error('Unsupported QTI version')
   end
-
 end

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

@@ -21,14 +21,13 @@ describe Qti::V1::Models::Assessment do
 
     context '<assessmentTest> has no title property' do
       it 'sets the title to the filename by default' do
-        empty_test_string =
-            <<-XML.strip_heredoc
-              <?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">
-                <assessment ident="A1001">
-                </assessment>
-              </questestinterop>
-            XML
+        empty_test_string = <<~XML.strip_heredoc
+          <?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">
+            <assessment ident="A1001">
+            </assessment>
+          </questestinterop>
+        XML
         allow(File).to receive(:read).and_return(empty_test_string)
         assessment_test = described_class.new path: '/a/b/c/Test123.xml'
         expect(assessment_test.title).to eq 'Test123'

data/spec/lib/qti/v1/models/choices/fill_blank_choice_spec.rb

@@ -9,7 +9,7 @@ describe Qti::V1::Models::Choices::FillBlankChoice do
     let(:choices) { nodes.map { |node| described_class.new(node, test_object) } }
 
     it 'returns the right identifier' do
-      expect(choices.map(&:identifier)).to eq %w(FIB_STR)
+      expect(choices.map(&:identifier)).to eq %w[FIB_STR]
     end
 
     it 'returns the item body text' do

data/spec/lib/qti/v1/models/choices/logical_identifier_choice_spec.rb

@@ -9,7 +9,7 @@ describe Qti::V1::Models::Choices::LogicalIdentifierChoice do
     let(:choices) { nodes.map { |node| described_class.new(node, test_object) } }
 
     it 'returns the right identifier' do
-      expect(choices.map(&:identifier)).to eq %w(QUE_1003 QUE_1007 QUE_1022)
+      expect(choices.map(&:identifier)).to eq %w[QUE_1003 QUE_1007 QUE_1022]
     end
 
     it 'returns the item body text' do

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

@@ -14,7 +14,7 @@ describe Qti::V1::Models::Interactions::FillBlankInteraction do
 
   shared_examples_for 'blanks' do
     it 'returns the blanks' do
-      expect(loaded_class.blanks).to eq [{ id: "FIB01" }, { id: "FIB02" }, { id: "FIB03" }]
+      expect(loaded_class.blanks).to eq [{ id: 'FIB01' }, { id: 'FIB02' }, { id: 'FIB03' }]
     end
   end
 
@@ -42,19 +42,21 @@ describe Qti::V1::Models::Interactions::FillBlankInteraction do
   context 'fib_str.xml' do
     let(:file_path) { File.join(fixtures_path, 'fib_str.xml') }
     let(:shuffle_value) { false }
-    let(:scoring_data_ids) { %w(FIB01 FIB02 FIB03) }
-    let(:scoring_data_values) { %w(Winter Summer York) }
-    let(:scoring_data_case) { %w(Yes Yes Yes) }
+    let(:scoring_data_ids) { %w[FIB01 FIB02 FIB03] }
+    let(:scoring_data_values) { %w[Winter Summer York] }
+    let(:scoring_data_case) { %w[Yes Yes Yes] }
     let(:answer_count) { 3 }
-    let(:expected_stem_items) {[
-      { id: "stem_0", position: 1, type: "text", value: "Fill-in-the blanks in this text from Richard III: " },
-      { id: "stem_1", position: 2, type: "text", value: "Now is the " },
-      { id: "stem_2", position: 3, type: "blank", blank_id: "FIB01" },
-      { id: "stem_3", position: 4, type: "text", value: " of our discontent made glorious " },
-      { id: "stem_4", position: 5, type: "blank", blank_id: "FIB02" },
-      { id: "stem_5", position: 6, type: "text", value: " by these sons of " },
-      { id: "stem_6", position: 7, type: "blank", blank_id: "FIB03" }
-    ]}
+    let(:expected_stem_items) do
+      [
+        { id: 'stem_0', position: 1, type: 'text', value: 'Fill-in-the blanks in this text from Richard III: ' },
+        { id: 'stem_1', position: 2, type: 'text', value: 'Now is the ' },
+        { id: 'stem_2', position: 3, type: 'blank', blank_id: 'FIB01' },
+        { id: 'stem_3', position: 4, type: 'text', value: ' of our discontent made glorious ' },
+        { id: 'stem_4', position: 5, type: 'blank', blank_id: 'FIB02' },
+        { id: 'stem_5', position: 6, type: 'text', value: ' by these sons of ' },
+        { id: 'stem_6', position: 7, type: 'blank', blank_id: 'FIB03' }
+      ]
+    end
 
     include_examples 'shuffled?'
     include_examples 'answers'

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

@@ -15,7 +15,7 @@ describe Qti::V1::Models::Interactions::OrderingInteraction do
   describe '#scoring_data_structs' do
     it 'grabs scoring data value for ordering questions' do
       assessment_item = described_class.new(assessment.assessment_items.first, assessment)
-      expect(assessment_item.scoring_data_structs.map(&:values)).to eq %w(A B E D C)
+      expect(assessment_item.scoring_data_structs.map(&:values)).to eq %w[A B E D C]
     end
   end
 end

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

@@ -67,9 +67,9 @@ describe Qti::V2::Models::AssessmentItem do
 
   context 'rubrics' do
     describe 'with a rubric' do
-    let(:fixtures_path) { File.join('spec', 'fixtures') }
-    let(:file_path) { File.join(fixtures_path, 'items_2.1', 'extended_text_rubric.xml') }
-    let(:loaded_class) { described_class.from_path!(file_path) }
+      let(:fixtures_path) { File.join('spec', 'fixtures') }
+      let(:file_path) { File.join(fixtures_path, 'items_2.1', 'extended_text_rubric.xml') }
+      let(:loaded_class) { described_class.from_path!(file_path) }
 
       it 'removes the rubric from the item_body' do
         expect(loaded_class.item_body).to include 'Read this postcard'

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

@@ -60,13 +60,12 @@ describe Qti::V2::Models::AssessmentTest do
 
   describe '#title' do
     it 'sets the title to the filename by default' do
-      empty_test_string =
-          <<-XML.strip_heredoc
-            <?xml version="1.0" encoding="UTF-8"?>
-            <assessmentTest xmlns="http://www.imsglobal.org/xsd/imsqti_v2p2p1" xmlns:xi="http://www.w3.org/2001/XInclude" xmlns:m="http://www.w3.org/1998/Math/MathML" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.imsglobal.org/xsd/imsqti_v2p2 http://www.imsglobal.org/xsd/qti/qtiv2p2/imsqti_v2p2.xsd">
-              <testPart identifier="Main" navigationMode="linear" submissionMode="individual"/>
-            </assessmentTest>
-          XML
+      empty_test_string = <<~XML.strip_heredoc
+        <?xml version="1.0" encoding="UTF-8"?>
+        <assessmentTest xmlns="http://www.imsglobal.org/xsd/imsqti_v2p2p1" xmlns:xi="http://www.w3.org/2001/XInclude" xmlns:m="http://www.w3.org/1998/Math/MathML" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.imsglobal.org/xsd/imsqti_v2p2 http://www.imsglobal.org/xsd/qti/qtiv2p2/imsqti_v2p2.xsd">
+          <testPart identifier="Main" navigationMode="linear" submissionMode="individual"/>
+        </assessmentTest>
+      XML
       allow(File).to receive(:read).and_return(empty_test_string)
       assessment_test = described_class.new path: '/a/b/c/Test123.xml'
       expect(assessment_test.title).to eq 'Test123'

data/spec/lib/qti/v2/models/choices/gap_match_choice_spec.rb

@@ -9,7 +9,7 @@ describe Qti::V2::Models::Choices::GapMatchChoice do
     let(:choices) { nodes.map { |node| described_class.new(node, item) } }
 
     it 'returns the right identifier' do
-      expect(choices.map(&:identifier)).to eq %w(W Sp Su A)
+      expect(choices.map(&:identifier)).to eq %w[W Sp Su A]
     end
 
     it 'returns the item body text' do

data/spec/lib/qti/v2/models/choices/simple_associable_choice_spec.rb

@@ -8,14 +8,14 @@ describe Qti::V2::Models::Choices::SimpleAssociableChoice do
 
   it 'returns the expected attributes' do
     expected_attrs = [
-      %w(Match28433682 1 1 Weißeritz),
-      %w(Match7191791 1 1 Mulde),
-      %w(Match20473010 1 1 Saale),
-      %w(Match6429655 1 1 Spree),
-      %w(Match2675678 1 1 Dresden),
-      %w(Match9372581 1 1 Leipzig),
-      %w(Match22744006 1 1 Halle),
-      %w(Match17943221 1 1 Bautzen)
+      %w[Match28433682 1 1 Weißeritz],
+      %w[Match7191791 1 1 Mulde],
+      %w[Match20473010 1 1 Saale],
+      %w[Match6429655 1 1 Spree],
+      %w[Match2675678 1 1 Dresden],
+      %w[Match9372581 1 1 Leipzig],
+      %w[Match22744006 1 1 Halle],
+      %w[Match17943221 1 1 Bautzen]
     ]
     expect(choices.map { |c| [c.identifier, c.match_min, c.match_max, c.item_body] }).to eq expected_attrs
   end

data/spec/lib/qti/v2/models/choices/simple_choice_spec.rb

@@ -9,7 +9,7 @@ describe Qti::V2::Models::Choices::SimpleChoice do
     let(:choices) { nodes.map { |node| described_class.new(node, item) } }
 
     it 'returns the right identifier' do
-      expect(choices.map(&:identifier)).to eq %w(true false)
+      expect(choices.map(&:identifier)).to eq %w[true false]
     end
 
     it 'returns the item body text' do

data/spec/lib/qti/v2/models/interactions/categorization_interaction_spec.rb

@@ -13,9 +13,9 @@ describe Qti::V2::Models::Interactions::CategorizationInteraction do
   describe '#scoring_data_structs' do
     it 'returns the scoring data for categorization questions' do
       expected_scoring_data_structs = [
-        Qti::V2::Models::ScoringData.new("A Midsummer-Night's Dream", "directedPair", id: "M", questions_ids: ["D", "L"]),
-        Qti::V2::Models::ScoringData.new('Romeo and Juliet', "directedPair", id: "R", questions_ids: ['C']),
-        Qti::V2::Models::ScoringData.new('The Tempest', "directedPair", id: "T", questions_ids: ['P']),
+        Qti::V2::Models::ScoringData.new("A Midsummer-Night's Dream", 'directedPair', id: 'M', questions_ids: %w[D L]),
+        Qti::V2::Models::ScoringData.new('Romeo and Juliet', 'directedPair', id: 'R', questions_ids: ['C']),
+        Qti::V2::Models::ScoringData.new('The Tempest', 'directedPair', id: 'T', questions_ids: ['P'])
       ]
       expect(subject.scoring_data_structs).to eq expected_scoring_data_structs
     end

data/spec/lib/qti/v2/models/interactions/gap_match_interaction_spec.rb

@@ -7,34 +7,36 @@ describe Qti::V2::Models::Interactions::GapMatchInteraction do
     let(:node) { Nokogiri::XML(io) }
     let(:loaded_class) { described_class.new(node, item) }
 
-    let(:expected_stem_items) {[
-      {type: "text",
-        value: "Identify the missing words in this famous quote from Shakespeare's Richard III.",
-        id: "stem_0", position: 1},
-      {type: 'text', value: 'Now is the ', id: 'stem_1', position: 2},
-      {type: 'blank', :blank_id=>'G1', id: 'stem_2', position: 3},
-      {type: 'text', value: ' of our discontent', id: 'stem_3', position: 4},
-      {type: 'text', value: ' ', id: 'stem_4', position: 5},
-      {type: 'text', value: ' Made glorious ', id: 'stem_5', position: 6},
-      {type: 'blank', :blank_id=>'G2', id: 'stem_6', position: 7},
-      {type: 'text', value: ' by this sun of York;', id: 'stem_7', position: 8},
-      {type: 'text', value: ' ', id: 'stem_8', position: 9},
-      {type: 'text', value: " And all the clouds that lour'd\n          upon our house",
-        id: 'stem_9', position: 10},
-      {type: 'text', value: ' ', id: 'stem_10', position: 11},
-      {type: 'text', value: ' In the deep bosom of the ocean buried.',
-        id: 'stem_11', position: 12}
-    ]}
-
-    let(:scoring_data_ids) { %w(G1 G2) }
-    let(:scoring_data_values) { %w(winter summer) }
+    let(:expected_stem_items) do
+      [
+        { type: 'text',
+          value: "Identify the missing words in this famous quote from Shakespeare's Richard III.",
+          id: 'stem_0', position: 1 },
+        { type: 'text', value: 'Now is the ', id: 'stem_1', position: 2 },
+        { type: 'blank', blank_id: 'G1', id: 'stem_2', position: 3 },
+        { type: 'text', value: ' of our discontent', id: 'stem_3', position: 4 },
+        { type: 'text', value: ' ', id: 'stem_4', position: 5 },
+        { type: 'text', value: ' Made glorious ', id: 'stem_5', position: 6 },
+        { type: 'blank', blank_id: 'G2', id: 'stem_6', position: 7 },
+        { type: 'text', value: ' by this sun of York;', id: 'stem_7', position: 8 },
+        { type: 'text', value: ' ', id: 'stem_8', position: 9 },
+        { type: 'text', value: " And all the clouds that lour'd\n          upon our house",
+          id: 'stem_9', position: 10 },
+        { type: 'text', value: ' ', id: 'stem_10', position: 11 },
+        { type: 'text', value: ' In the deep bosom of the ocean buried.',
+          id: 'stem_11', position: 12 }
+      ]
+    end
+
+    let(:scoring_data_ids) { %w[G1 G2] }
+    let(:scoring_data_values) { %w[winter summer] }
 
     it 'returns shuffle setting' do
       expect(loaded_class.shuffled?).to eq false
     end
 
     it 'returns the blanks' do
-      expect(loaded_class.blanks).to eq [{ id: "W" }, { id: "Sp" }, { id: "Su" }, { id: "A" }]
+      expect(loaded_class.blanks).to eq [{ id: 'W' }, { id: 'Sp' }, { id: 'Su' }, { id: 'A' }]
     end
 
     it 'returns the stem_items' do
@@ -46,7 +48,6 @@ describe Qti::V2::Models::Interactions::GapMatchInteraction do
       expect(loaded_class.answers.first).to be_an_instance_of(Qti::V2::Models::Choices::GapMatchChoice)
     end
 
-
     it 'returns the choices' do
       expect(loaded_class.choices.count).to eq 4
     end

data/spec/lib/qti/v2/models/interactions/ordering_interaction_spec.rb

@@ -13,7 +13,7 @@ describe Qti::V2::Models::Interactions::OrderingInteraction do
 
   describe '#scoring_data_structs' do
     it 'grabs scoring data value for ordering questions' do
-      expect(subject.scoring_data_structs.map(&:values)).to eq ['DriverC', 'DriverA', 'DriverB']
+      expect(subject.scoring_data_structs.map(&:values)).to eq %w[DriverC DriverA DriverB]
     end
   end
 end

data/spec/lib/qti/v2/models/interactions/short_text_interaction_spec.rb

@@ -16,7 +16,7 @@ describe Qti::V2::Models::Interactions::ShortTextInteraction do
         node = double('Nokogiri::XML::Document')
         matches = [double('Nokogiri::XML::Element'), double('Nokogiri::XML::Element')]
         allow(node).to receive(:xpath).with('.//xmlns:textEntryInteraction').and_return(matches)
-        expect{described_class.matches(node, assessment_item)}.to raise_error(Qti::UnsupportedSchema)
+        expect { described_class.matches(node, assessment_item) }.to raise_error(Qti::UnsupportedSchema)
       end
     end
   end

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

@@ -16,7 +16,7 @@ describe Qti::V2::Models::NonAssessmentTest do
     it 'should return the stimulus ref if it exists' do
       item = loaded_class.assessment_items[1]
       stimulus_ref = loaded_class.stimulus_ref(item)
-      expect(stimulus_ref).to eq File.join(File.dirname(path), "passages/0cfd5cf7-2c91-4b35-a57a-9f5d1709f68f.html")
+      expect(stimulus_ref).to eq File.join(File.dirname(path), 'passages/0cfd5cf7-2c91-4b35-a57a-9f5d1709f68f.html')
     end
 
     it 'should return nil if no stimulus ref' do

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

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Qti::V2::Models::AssessmentItem do
-  context "object element" do
+  context 'object element' do
     let(:path) { File.join('spec', 'fixtures', 'no_assessment_XML', '59ee7c9d-e6cc-4d0c-b545-258e20b1a244.xml') }
     let(:loaded_class) { described_class.from_path!(path) }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: qti
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
 platform: ruby
 authors:
 - Hannah Bottalla
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-25 00:00:00.000000000 Z
+date: 2017-09-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -494,12 +494,19 @@ files:
 - spec/fixtures/tests/tests/rtest26.xml
 - spec/fixtures/tests/tests/rtest27.xml
 - spec/gemfiles/nokogiri-1.6.gemfile
+- spec/gemfiles/nokogiri-1.6.gemfile.lock
 - spec/gemfiles/nokogiri-1.8.gemfile
+- spec/gemfiles/nokogiri-1.8.gemfile.lock
 - spec/gemfiles/rails-4.2.gemfile
+- spec/gemfiles/rails-4.2.gemfile.lock
 - spec/gemfiles/rails-5.0.gemfile
+- spec/gemfiles/rails-5.0.gemfile.lock
 - spec/gemfiles/rails-5.1.gemfile
+- spec/gemfiles/rails-5.1.gemfile.lock
 - spec/gemfiles/sanitize-4.2.gemfile
+- spec/gemfiles/sanitize-4.2.gemfile.lock
 - spec/gemfiles/sanitize-4.5.gemfile
+- spec/gemfiles/sanitize-4.5.gemfile.lock
 - spec/lib/qti/assessment_item_exporter_spec.rb
 - spec/lib/qti/exporter_spec.rb
 - spec/lib/qti/models/base_spec.rb
@@ -552,7 +559,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: QTI 1.2 and 2.1 import and export models
@@ -769,12 +776,19 @@ test_files:
 - spec/fixtures/tests/tests/rtest26.xml
 - spec/fixtures/tests/tests/rtest27.xml
 - spec/gemfiles/nokogiri-1.6.gemfile
+- spec/gemfiles/nokogiri-1.6.gemfile.lock
 - spec/gemfiles/nokogiri-1.8.gemfile
+- spec/gemfiles/nokogiri-1.8.gemfile.lock
 - spec/gemfiles/rails-4.2.gemfile
+- spec/gemfiles/rails-4.2.gemfile.lock
 - spec/gemfiles/rails-5.0.gemfile
+- spec/gemfiles/rails-5.0.gemfile.lock
 - spec/gemfiles/rails-5.1.gemfile
+- spec/gemfiles/rails-5.1.gemfile.lock
 - spec/gemfiles/sanitize-4.2.gemfile
+- spec/gemfiles/sanitize-4.2.gemfile.lock
 - spec/gemfiles/sanitize-4.5.gemfile
+- spec/gemfiles/sanitize-4.5.gemfile.lock
 - spec/lib/qti/assessment_item_exporter_spec.rb
 - spec/lib/qti/exporter_spec.rb
 - spec/lib/qti/models/base_spec.rb