datacite-mapping 0.2.5 → 0.3.0

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # ------------------------------------------------------------
 # SimpleCov setup
 

data/spec/unit/datacite/mapping/alternate_identifier_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/contributor_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/creator_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/date_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/date_value_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/description_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/funding_reference_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite
@@ -56,7 +58,7 @@ module Datacite
                          <funderIdentifier funderIdentifierType="Crossref Funder ID">http://doi.org/10.13039/501100000780</funderIdentifier>
                          <awardNumber awardURI="http://cordis.europa.eu/project/rcn/100180_en.html">282625</awardNumber>
                          <awardTitle>MOTivational strength of ecosystem services and alternative ways to express the value of BIOdiversity</awardTitle>
-                       </fundingReference>'.freeze
+                       </fundingReference>'
 
           @fref = FundingReference.parse_xml(fref_xml)
         end

data/spec/unit/datacite/mapping/geo_location_box_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/geo_location_point_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/geo_location_polygon_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite
@@ -6,17 +8,17 @@ module Datacite
       describe '#==' do
         it 'reports equal values as equal' do
           polygon1 = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(33.45, -122.33),
-            GeoLocationPoint.new(33.45, 122.33),
-            GeoLocationPoint.new(-33.45, -122.33)
-          ])
+                                              GeoLocationPoint.new(-33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, 122.33),
+                                              GeoLocationPoint.new(-33.45, -122.33)
+                                            ])
           polygon2 = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(33.45, -122.33),
-            GeoLocationPoint.new(33.45, 122.33),
-            GeoLocationPoint.new(-33.45, -122.33)
-          ])
+                                              GeoLocationPoint.new(-33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, 122.33),
+                                              GeoLocationPoint.new(-33.45, -122.33)
+                                            ])
           expect(polygon1).to eq(polygon2)
           expect(polygon1.hash).to eq(polygon2.hash)
           expect(polygon2).to eq(polygon1)
@@ -25,17 +27,17 @@ module Datacite
 
         it 'reports unequal values as unequal' do
           polygon1 = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(33.45, -122.33),
-            GeoLocationPoint.new(33.45, 122.33),
-            GeoLocationPoint.new(-33.45, -122.33)
-          ])
+                                              GeoLocationPoint.new(-33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, 122.33),
+                                              GeoLocationPoint.new(-33.45, -122.33)
+                                            ])
           polygon2 = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(33.45, 122.33),
-            GeoLocationPoint.new(-33.45, 122.33),
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(33.45, 122.33)
-          ])
+                                              GeoLocationPoint.new(33.45, 122.33),
+                                              GeoLocationPoint.new(-33.45, 122.33),
+                                              GeoLocationPoint.new(-33.45, -122.33),
+                                              GeoLocationPoint.new(33.45, 122.33)
+                                            ])
           expect(polygon1).not_to eq(polygon2)
           expect(polygon1.hash).not_to eq(polygon2.hash)
           expect(polygon2).not_to eq(polygon1)

data/spec/unit/datacite/mapping/geo_location_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite
@@ -45,11 +47,11 @@ module Datacite
         end
         it 'accepts a polygon' do
           polygon = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(47.61, -122.33),
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(47.61, -70.67),
-            GeoLocationPoint.new(47.61, -122.33)
-          ])
+                                             GeoLocationPoint.new(47.61, -122.33),
+                                             GeoLocationPoint.new(-33.45, -122.33),
+                                             GeoLocationPoint.new(47.61, -70.67),
+                                             GeoLocationPoint.new(47.61, -122.33)
+                                           ])
           loc = GeoLocation.new(polygon: polygon)
           expect(loc.polygon).to eq(polygon)
         end
@@ -106,11 +108,11 @@ module Datacite
       describe '#polygon=' do
         it 'sets the polygon' do
           polygon = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(47.61, -122.33),
-            GeoLocationPoint.new(-33.45, -122.33),
-            GeoLocationPoint.new(47.61, -70.67),
-            GeoLocationPoint.new(47.61, -122.33)
-          ])
+                                             GeoLocationPoint.new(47.61, -122.33),
+                                             GeoLocationPoint.new(-33.45, -122.33),
+                                             GeoLocationPoint.new(47.61, -70.67),
+                                             GeoLocationPoint.new(47.61, -122.33)
+                                           ])
           loc = GeoLocation.new
           loc.polygon = polygon
           expect(loc.polygon).to eq(polygon)
@@ -168,11 +170,11 @@ module Datacite
           expect(loc.place).to eq('Atlantic Ocean')
           actual_polygon = loc.polygon
           expected_polygon = GeoLocationPolygon.new(points: [
-            GeoLocationPoint.new(31.233, -67.302),
-            GeoLocationPoint.new(-68.211, -71.032),
-            GeoLocationPoint.new(42.893, 41.09),
-            GeoLocationPoint.new(31.233, -67.302)
-          ])
+                                                      GeoLocationPoint.new(31.233, -67.302),
+                                                      GeoLocationPoint.new(-68.211, -71.032),
+                                                      GeoLocationPoint.new(42.893, 41.09),
+                                                      GeoLocationPoint.new(31.233, -67.302)
+                                                    ])
           expect(actual_polygon).to eq(expected_polygon)
         end
 
@@ -196,12 +198,12 @@ module Datacite
             point: GeoLocationPoint.new(31.233, -67.302),
             box: GeoLocationBox.new(41.09, -71.032, 42.893, -68.211),
             place: 'Atlantic Ocean',
-            polygon: (GeoLocationPolygon.new(points: [
-              GeoLocationPoint.new(-67.302, 31.233),
-              GeoLocationPoint.new(-71.032, -68.211),
-              GeoLocationPoint.new(41.09, 42.893),
-              GeoLocationPoint.new(-67.302, 31.233)
-            ]))
+            polygon: GeoLocationPolygon.new(points: [
+                                              GeoLocationPoint.new(-67.302, 31.233),
+                                              GeoLocationPoint.new(-71.032, -68.211),
+                                              GeoLocationPoint.new(41.09, 42.893),
+                                              GeoLocationPoint.new(-67.302, 31.233)
+                                            ])
           )
         end
 
@@ -257,7 +259,7 @@ module Datacite
 
           it 'writes DC3 in XSD-defined order: point, box, place' do
             actual_xml = loc.save_to_xml(mapping: :datacite_3)
-            expected_order = %w(geoLocationPoint geoLocationBox geoLocationPlace)
+            expected_order = %w[geoLocationPoint geoLocationBox geoLocationPlace]
             actual_order = actual_xml.children.map(&:name)
             expect(actual_order).to eq(expected_order)
           end

data/spec/unit/datacite/mapping/identifier_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite
@@ -21,13 +23,13 @@ module Datacite
         end
 
         it 'disallows bad DOIs' do
-          bad_dois = %w(
+          bad_dois = %w[
             20.14749/1407399495
             11.14749/1407399495
             10./1407399495
             10.14749\1407399495
             10.14749/
-          )
+          ]
           bad_dois.each do |doi|
             expect { Identifier.new(value: doi) }.to raise_error(ArgumentError)
           end
@@ -71,13 +73,13 @@ module Datacite
         end
 
         it 'raises ArgumentError if it is passed a bad DOI' do
-          bad_dois = %w(
+          bad_dois = %w[
             20.14749/1407399495
             11.14749/1407399495
             10./1407399495
             10.14749\1407399495
             10.14749/
-          )
+          ]
           bad_dois.each do |doi|
             expect { Identifier.from_doi(doi) }.to raise_error do |e|
               expect(e).to be_an(ArgumentError)
@@ -95,13 +97,13 @@ module Datacite
         end
         it 'disallows bad DOIs' do
           id = Identifier.allocate
-          bad_dois = %w(
+          bad_dois = %w[
             20.14749/1407399495
             11.14749/1407399495
             10./1407399495
             10.14749\1407399495
             10.14749/
-          )
+          ]
           bad_dois.each do |doi|
             expect { id.value = doi }.to raise_error(ArgumentError)
             expect(id.value).to be_nil

data/spec/unit/datacite/mapping/name_identifier_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/related_identifier_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite

data/spec/unit/datacite/mapping/resource_spec.rb

@@ -1,5 +1,8 @@
-# coding: utf-8
+
+# frozen_string_literal: true
+
 require 'spec_helper'
+require 'tmpdir'
 
 module Datacite
   module Mapping
@@ -119,9 +122,9 @@ module Datacite
                 resource = Resource.new(args)
                 expect(resource.creator_affiliations)
                   .to eq([
-                    ['United Artists', 'Metro-Goldwyn-Mayer'],
-                    ['Gaumont Buena Vista International', '20th Century Fox']
-                  ])
+                           ['United Artists', 'Metro-Goldwyn-Mayer'],
+                           ['Gaumont Buena Vista International', '20th Century Fox']
+                         ])
               end
             end
           end
@@ -228,7 +231,7 @@ module Datacite
           it 'converts strings to integers' do
             new_pub_year = 1963
             resource = Resource.new(args)
-            resource.publication_year = "#{new_pub_year}"
+            resource.publication_year = new_pub_year.to_s
             expect(resource.publication_year).to eq(new_pub_year)
           end
         end
@@ -426,7 +429,8 @@ module Datacite
                   scheme_uri: URI('http://iatistandard.org/201/codelists/OrganisationIdentifier/'),
                   value: 'GR-9¾'
                 ),
-                type: ContributorType::FUNDER)
+                type: ContributorType::FUNDER
+              )
               @resource.contributors << @funder
             end
 
@@ -465,7 +469,7 @@ module Datacite
           describe 'dates:' do
             it 'can be initialized' do
               dates = [
-                Date.new(value: DateTime.new(1914, 8, 4, 11, 01, 6.0123, '+1'), type: DateType::AVAILABLE),
+                Date.new(value: DateTime.new(1914, 8, 4, 11, 0o1, 6.0123, '+1'), type: DateType::AVAILABLE),
                 Date.new(value: '1914-08-04T11:01:06.0123+01:00', type: DateType::AVAILABLE)
               ]
               args[:dates] = dates
@@ -483,7 +487,7 @@ module Datacite
             it 'can be set' do
               resource = Resource.new(args)
               dates = [
-                Date.new(value: DateTime.new(1914, 8, 4, 11, 01, 6.0123, '+1'), type: DateType::AVAILABLE),
+                Date.new(value: DateTime.new(1914, 8, 4, 11, 0o1, 6.0123, '+1'), type: DateType::AVAILABLE),
                 Date.new(value: '1914-08-04T11:01:06.0123+01:00', type: DateType::AVAILABLE)
               ]
               resource.dates = dates
@@ -625,7 +629,7 @@ module Datacite
 
           describe 'sizes:' do
             it 'can be initialized' do
-              sizes = %w(48K 128K)
+              sizes = %w[48K 128K]
               args[:sizes] = sizes
               resource = Resource.new(args)
               expect(resource.sizes).to eq(sizes)
@@ -639,7 +643,7 @@ module Datacite
 
           describe '#sizes=' do
             it 'can be set' do
-              sizes = %w(48K 128K)
+              sizes = %w[48K 128K]
               resource = Resource.new(args)
               resource.sizes = sizes
               expect(resource.sizes).to eq(sizes)
@@ -660,7 +664,7 @@ module Datacite
 
           describe 'formats:' do
             it 'can be initialized' do
-              formats = %w(D64 DSK)
+              formats = %w[D64 DSK]
               args[:formats] = formats
               resource = Resource.new(args)
               expect(resource.formats).to eq(formats)
@@ -674,7 +678,7 @@ module Datacite
 
           describe '#formats=' do
             it 'can be set' do
-              formats = %w(D64 DSK)
+              formats = %w[D64 DSK]
               resource = Resource.new(args)
               resource.formats = formats
               expect(resource.formats).to eq(formats)
@@ -940,7 +944,7 @@ module Datacite
           r1 = r0.gsub(%r{&lt;br\s+/&gt;}, '<br/>') # entity-de-escape <br/> tags
           # r2 = r1.gsub(%r{<(?!br)[^>]+/>}, '') # remove empty tags
           r2 = r1
-          r3 = r2.gsub(/<resource (xmlns:xsi="[^"]+")\s+(xsi:schemaLocation="[^"]+")>/, "<resource \\2 \\1 xmlns=\"http://datacite.org/schema/kernel-3\">") # fix missing namespace
+          r3 = r2.gsub(/<resource (xmlns:xsi="[^"]+")\s+(xsi:schemaLocation="[^"]+")>/, '<resource \\2 \\1 xmlns="http://datacite.org/schema/kernel-3">') # fix missing namespace
           r4 = r3.gsub(%r{(<identifier[^>]+>)\s*([^ ]+)\s*(</identifier>)}, '\\1\\2\\3') # trim identifiers
           r5 = r4.gsub(%r{<([^>]+tude)>([0-9.-]+?)(0?)0+</\1>}, '<\\1>\\2\\3</\\1>') # strip trailing coordinate zeroes
           r6 = r5.gsub(%r{<(geoLocation[^>]+)>[^<]+</\1>}) { |loc| loc.gsub(/([0-9\-]+\.[0-9]+?)0+([^0-9])/, '\\1\\2') } # strip trailing coordinate zeroes
@@ -1017,7 +1021,7 @@ module Datacite
             matches = warnings_including(substring)
             found_count = matches.size
             msg = "expected #{count} warnings including '#{substring}', found #{found_count}"
-            msg << ": #{matches}" if include_matches
+            msg += ": #{matches}" if include_matches
             expect(found_count).to eq(count), msg
           end
 
@@ -1025,7 +1029,7 @@ module Datacite
             matches = REXML::XPath.match(rexml, xpath)
             found_count = matches.size
             msg = "expected #{count} matches for XPath '#{xpath}', found #{found_count}"
-            msg << ": #{matches}" if include_matches
+            msg += ": #{matches}" if include_matches
             expect(found_count).to eq(count), msg
           end
 
@@ -1033,7 +1037,7 @@ module Datacite
             @warnings = []
             allow(ReadOnlyNodes).to receive(:warn) do |w|
               warnings << w
-              Kernel.warn(w) # for debugging
+              # Kernel.warn(w) # for debugging
             end
 
             xml = File.read('spec/data/datacite-4-synthetic.xml')
@@ -1051,7 +1055,7 @@ module Datacite
           end
 
           it 'warns about givenNames and familyNames' do
-            name_tags = %w(givenName familyName)
+            name_tags = %w[givenName familyName]
             name_tags.each do |tag|
               expect_matches("//#{tag}", 0, true)
               expect_warning(tag, 1)
@@ -1151,7 +1155,7 @@ module Datacite
 
       describe '#save_to_file' do
         it 'saves to a file' do
-          xml_text =  File.read('spec/data/datacite4/datacite-example-full-v4.0.xml')
+          xml_text = File.read('spec/data/datacite4/datacite-example-full-v4.0.xml')
           resource = Resource.parse_xml(xml_text)
           Dir.mktmpdir('resource_spec') do |dir|
             path = "#{dir}/resource.xml"

data/spec/unit/datacite/mapping/rights_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module Datacite
@@ -80,7 +82,7 @@ module Datacite
           expected_xml = '<rights rightsURI="https://creativecommons.org/licenses/by/4.0/">Creative Commons Attribution 4.0 International (CC BY 4.0)</rights>'
           expect(rights.save_to_xml).to be_xml(expected_xml)
 
-          [:CC_ZERO, :CC_BY].each do |c|
+          %i[CC_ZERO CC_BY].each do |c|
             r = Rights.const_get(c)
             puts "#{c.to_s.downcase.gsub('_zero', '0')}:"
             puts "  uri: #{r.uri}"