imap 0.4.5 → 0.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78ec20b3a5517359526f4c5e0437a05b1ab97be444c1e1c741992f1ea595376e
-  data.tar.gz: d5d2ac27eb982ff25264eaf947ee71981fba45cfc645b34dcce8148617a4c790
+  metadata.gz: 26d3a42b29eb3be8775fc3ec60f3559b0834e8ef106e2e3458cd2b93bdee7364
+  data.tar.gz: 85aebe82c105a2d7db62c581ca61bd2c0630fcd7054dc777a02a86e84fd620ba
 SHA512:
-  metadata.gz: 6a470cbf282ff1c1b84e6283b8e6a882262e02379eb03a50a65657346624667f576c5f14fecc1471ac94be2d6fe83bdca7f0d57ba76d65f613330662527c8e50
-  data.tar.gz: 9754404cd62da6dfd26caa4e9c0ef3a7023704ff8f99da0864e57835a605b7c1dd04d4f992e35700184802ca1e6f075734ac89e3767234b1fd8cafb674f27315
+  metadata.gz: bb36b35b0cd5f9ddc6895aa0bfe9c0d644cd5386f13f4b26a66324d767560c3af41a8741be6a4fecf4f69fbbf6028dc823e1207e9aba50ab63a382b0a9750c50
+  data.tar.gz: 79add07e74eb8acc3105b3791be56d020cc50c89e80d47d3e47f8b46cf997b6e9c7fdd44e536292435f5560f380bd1bdac2d966c4bfc53bd548a010e77b93432

data/CHANGELOG

@@ -1,5 +1,13 @@
 # CHANGELOG
 
+## [0.4.6] - 2026-09-17, ~ Imap::Message#body: BODY.PEEK[TEXT]; ~ Imap::Search#to_imap_search_keys: raise upon a key which is neither
+- ~ Imap::Message#body: BODY.PEEK[TEXT] for BODY[TEXT]. Reading a body set \Seen, so anything which looked at one marked it read; PEEK is the whole of the difference and the response still answers to BODY[TEXT].
+- ~ Imap::Search#to_imap_search_keys: + UnknownSearchKey where the case matched neither operator. An unrecognised key was dropped and the search ran wider than it was asked for, saying nothing.
+- ~ Imap::Message#to: [] where the envelope carries no to, which raised NoMethodError upon nil.
+- ~ test/test_helper.rb: MockIMAP answers BODY.PEEK[TEXT] with BODY[TEXT] as a server does, records the attrs fetched, and gives message 99 an envelope with no to.
+- + a test apiece: that #body peeks, that #to is empty where there is no to, and that an unknown key raises.
+- ~ Imap::VERSION: /0.4.5/0.4.6/
+
 ## [0.4.5] - 2026-09-17, + Gemfile; ~ imap.gemspec: dependencies=, - spec.date, + metadata
 - + Gemfile: source and gemspec, as switches, eodhd and moby have it, so that the dependencies are declared once in the gemspec rather than restated here to drift. bitget and coinmarketcap restate theirs and are the minority. This was the only gem here with no Gemfile at all.
 - ~ imap.gemspec: + 'Gemfile' to spec.files, which every other gem here ships.

data/lib/Imap/Message.rb

@@ -29,7 +29,7 @@ class Imap
     attr_accessor :message_id
 
     def body
-      @body ||= fetch_data('BODY[TEXT]').first.attr['BODY[TEXT]']
+      @body ||= fetch_data('BODY.PEEK[TEXT]').first.attr['BODY[TEXT]']
     end
 
     def subject
@@ -41,7 +41,7 @@ class Imap
     end
 
     def to
-      @to ||= fetch_data('ENVELOPE').first.attr['ENVELOPE'].to.map{|addr| addr.mailbox + '@' + addr.host}
+      @to ||= (fetch_data('ENVELOPE').first.attr['ENVELOPE'].to || []).map{|addr| addr.mailbox + '@' + addr.host}
     end
 
     def urls

data/lib/Imap/Search.rb

@@ -11,6 +11,8 @@
 class Imap
   class Search
 
+    class UnknownSearchKey < ArgumentError; end
+
     ALL_SEARCH_KEYS = %w{
       ALL
       ANSWERED
@@ -112,6 +114,8 @@ class Imap
           m << general_to_imap_search_key(key, value)
         when boolean_operator?(key)
           m << boolean_to_imap_search_key(key, value)
+        else
+          raise UnknownSearchKey, "no such search key: #{key}"
         end
       end.flatten
     end

data/lib/Imap/VERSION.rb

@@ -1,3 +1,3 @@
 class Imap
-  VERSION = '0.4.5'
+  VERSION = '0.4.6'
 end

data/test/Imap/Message_test.rb

@@ -40,6 +40,11 @@ describe Imap::Message do
       second_call = imap_message.body
       _(first_call.object_id).must_equal second_call.object_id
     end
+
+    it 'peeks, so that reading a body does not mark it seen' do
+      imap_message.body
+      _(client.imap.fetched_attrs).must_include 'BODY.PEEK[TEXT]'
+    end
   end
 
   describe '#subject' do
@@ -74,6 +79,10 @@ describe Imap::Message do
       addresses = imap_message.to
       _(addresses.first).must_equal('user@example.com')
     end
+
+    it 'is empty where the envelope carries no to' do
+      _(Imap::Message.new(99, client).to).must_equal []
+    end
   end
 
   describe '#urls' do

data/test/Imap/Search_test.rb

@@ -33,6 +33,10 @@ describe Imap::Search do
       _(keys).must_include 'SEEN'
     end
 
+    it 'raises upon a key which is neither' do
+      _{Imap::Search.new(client, bogus: 'x').to_imap_search_keys}.must_raise Imap::Search::UnknownSearchKey
+    end
+
     it 'handles multiple criteria' do
       test_search = Imap::Search.new(client, from: 'test@example.com', seen: true)
       keys = test_search.to_imap_search_keys

data/test/test_helper.rb

@@ -31,7 +31,10 @@ class MockIMAP
     [1, 2, 3]
   end
 
+  attr_reader :fetched_attrs
+
   def fetch(message_id, attrs)
+    @fetched_attrs = attrs
     [OpenStruct.new(attr: mock_fetch_attrs(message_id, attrs))]
   end
 
@@ -58,7 +61,7 @@ class MockIMAP
     result = {}
     attrs.each do |attr|
       case attr
-      when 'BODY[TEXT]'
+      when 'BODY[TEXT]', 'BODY.PEEK[TEXT]'
         result['BODY[TEXT]'] = "Mock body for message #{message_id}"
       when 'BODY[HEADER.FIELDS (SUBJECT)]'
         result['BODY[HEADER.FIELDS (SUBJECT)]'] = "Subject: Mock Subject #{message_id}\r\n"
@@ -66,7 +69,7 @@ class MockIMAP
         result['BODY[HEADER.FIELDS (FROM)]'] = "From: sender@example.com\r\n"
       when 'ENVELOPE'
         result['ENVELOPE'] = OpenStruct.new(
-          to: [OpenStruct.new(mailbox: 'user', host: 'example.com')]
+          to: message_id == 99 ? nil : [OpenStruct.new(mailbox: 'user', host: 'example.com')]
         )
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: imap
 version: !ruby/object:Gem::Version
-  version: 0.4.5
+  version: 0.4.6
 platform: ruby
 authors:
 - thoran
@@ -85,7 +85,7 @@ files:
 - test/Imap/Message_test.rb
 - test/Imap/Search_test.rb
 - test/Imap/VERSION_test.rb
-- test/imap_test.rb
+- test/Imap_test.rb
 - test/test_helper.rb
 homepage: https://github.com/thoran/imap
 licenses: