entrez 0.5.3 → 0.5.4

data/lib/entrez.rb

@@ -72,7 +72,12 @@ class Entrez
     def parse_ids_and_extend(response)
       response.instance_eval do
         def ids
-          @ids ||= self['eSearchResult']['IdList']['Id'].map(&:to_i)
+          return @ids if @ids
+          id_content = self['eSearchResult']['IdList']['Id']
+          # If there is only 1, Crack will parse it and return just the string.
+          # Need to always return array.
+          id_content = [id_content].flatten
+          @ids = id_content.map(&:to_i)
         rescue ::NoMethodError
           @ids = []
         end

data/lib/entrez/version.rb

@@ -1,3 +1,3 @@
 module Entrez
-  VERSION = "0.5.3"
+  VERSION = "0.5.4"
 end

data/spec/entrez_spec.rb

@@ -32,11 +32,17 @@ describe Entrez do
     response.ids.should == [60153, 29429, 28911, 48101, 59851, 59849, 59847, 59845, 59839, 59835, 59833, 59831, 51895, 59829, 59827, 60835, 59811, 60831, 60819, 33895]
   end
 
-  it 'ESearch returns empty array if nothing found' do
+  it '#ESearch returns empty array if nothing found' do
     response = Entrez.ESearch('genomeprj', {NON_EXISTENT_SEARCH_FIELD: 'does not exist even in oompaloompa land'}, retmode: :xml)
     response.ids.should be_empty
   end
 
+  it '#ESearch returns array even if only 1 id found' do
+    id = 60153
+    response = Entrez.ESearch('genomeprj', {uid: id}, retmode: :xml)
+    response.ids.should == [id]
+  end
+
   it '#ESearch accepts string as search_terms parameter' do
     response = Entrez.ESearch('genomeprj', 'hapmap[WORD]', retmode: :xml)
     response.ids.should include(60153)

metadata

@@ -2,7 +2,7 @@
 name: entrez
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.5.3
+  version: 0.5.4
 platform: ruby
 authors: 
 - Jared Ning