Merge pull request #138 from dunn/idregexp

FileSystemFileResolver: teach #validate_identifier to handle ints

app/resolvers/riiif/file_system_file_resolver.rb

@@ -14,7 +14,7 @@ def pattern(id)
 
       # @return [Boolean] true if the id matches the regex
       def validate_identifier(id:, regex: identifier_regex)
-        return true if id =~ regex
+        return true if id.to_s =~ regex
         Rails.logger.warn "Invalid characters in id `#{id}`"
         false
       end

spec/models/riiif/file_system_file_resolver_spec.rb

@@ -78,5 +78,12 @@
         expect(subject).to be_nil
       end
     end
+
+    context 'as an integer' do
+      let(:id) { 1 }
+      it 'converts it to a string before parsing' do
+        expect { subject }.not_to raise_error
+      end
+    end
   end
 end