luggage 1.2.1 → 1.2.2

data/lib/luggage/factory.rb

@@ -57,9 +57,9 @@ module Luggage
       Message.new_local(connection, mailbox, args, &block)
     end
 
-    def mailboxes(*args)
+    def mailboxes(*args, &block)
       array = MailboxArray.new(connection)
-      args.empty? ? array : array[*args]
+      args.empty? ? array : array[*args, &block]
     end
 
     def inspect

data/lib/luggage/message.rb

@@ -42,7 +42,7 @@ module Luggage
     def initialize(connection, mailbox, args = {}, &block)
       @connection = connection
       @mailbox = mailbox.kind_of?(Mailbox) ? mailbox : Mailbox.new(connection, mailbox)
-      @flags = []
+      @flags = args.has_key?(:flags) ? Array(args[:flags]) : []
       @date = args[:date] || Time.now
       @template = args[:template]
       @message_id = args[:message_id] || "<#{UUIDTools::UUID.random_create}@test.oib.com>"
@@ -72,7 +72,7 @@ module Luggage
     #
     def save!
       mailbox.select!
-      connection.append(mailbox.name, raw_message, flags.map {|f| f.to_sym.upcase}, date)
+      connection.append(mailbox.name, raw_message, flags, date)
     end
 
     # Uses IMAP's COPY command to copy the message into the named mailbox
@@ -94,7 +94,7 @@ module Luggage
     #
     def exists?
       mailbox.select!
-      connection.uid_search("HEADER Message-ID #{message_id}")
+      connection.uid_search("HEADER.PEEK Message-ID #{message_id}")
     end
 
     # Proxy all other methods to this instance's Mail::Message

data/lib/luggage/version.rb

@@ -1,3 +1,3 @@
 module Luggage
-  VERSION = "1.2.1"
+  VERSION = "1.2.2"
 end

data/spec/luggage/factory_spec.rb

@@ -84,5 +84,14 @@ describe Luggage::Factory do
         expect(factory.mailboxes[1..2].map(&:name)).to eq(["Mailbox_2", "Mailbox_3"])
       end
     end
+
+    context "with a string argument (function-call syntax) and a block" do
+      it "executes the block in the mailbox's scope" do
+        Luggage::Mailbox.any_instance.should_receive(:message)
+        factory.mailboxes("Mailbox_1") do
+          message
+        end
+      end
+    end
   end
 end

data/spec/luggage/message_spec.rb

@@ -39,6 +39,20 @@ describe Luggage::Message do
         Luggage::Message.new_local(connection, :mailbox, :template =>"base")
       end
     end
+
+    context "with flags passed as argument" do
+      it "sets flags" do
+        message = Luggage::Message.new_local(connection, :mailbox, :flags => :Seen)
+        expect(message.flags).to include(:Seen)
+      end
+    end
+
+    context "with flags array passed as argument" do
+      it "sets flags" do
+        message = Luggage::Message.new_local(connection, :mailbox, :flags => [:Seen])
+        expect(message.flags).to include(:Seen)
+      end
+    end
   end
 
   describe "::new" do
@@ -89,7 +103,7 @@ describe Luggage::Message do
 
     it "fetches raw email" do
       connection.should_receive(:uid_fetch).
-        with([1], ["FLAGS", "INTERNALDATE", "BODY.PEEK[]"]).
+        with([1], ["FLAGS", "INTERNALDATE.PEEK", "BODY.PEEK[]"]).
         and_return( [{:attr => {"BODY[]" => "raw_body", "FLAGS" => [], "INTERNALDATE" => (Time.now - 60 * 60 * 24).to_s}}]  )
 
       message.reload
@@ -97,7 +111,7 @@ describe Luggage::Message do
 
     it "fetches flags" do
       connection.should_receive(:uid_fetch).
-        with([1], ["FLAGS", "INTERNALDATE", "BODY.PEEK[]"]).
+        with([1], ["FLAGS", "INTERNALDATE.PEEK", "BODY.PEEK[]"]).
         and_return( [{:attr => {"BODY[]" => "raw_body", "FLAGS" => [], "INTERNALDATE" => (Time.now - 60 * 60 * 24).to_s}}]  )
 
       message.reload
@@ -165,7 +179,7 @@ describe Luggage::Message do
 
     it "searches for message with message_id" do
       message.stub(:message_id).and_return("<foo@example.com>")
-      connection.should_receive(:uid_search).with("HEADER Message-ID <foo@example.com>").and_return([1])
+      connection.should_receive(:uid_search).with("HEADER.PEEK Message-ID <foo@example.com>").and_return([1])
 
       message.exists?
     end

metadata

@@ -1,7 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: luggage
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
+  prerelease: 
 platform: ruby
 authors:
 - Ryan Michael
@@ -9,11 +10,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-15 00:00:00.000000000 Z
+date: 2013-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mail
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -21,6 +23,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -28,6 +31,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: uuidtools
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -35,6 +39,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -42,6 +47,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -49,6 +55,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -56,6 +63,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -63,6 +71,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -70,6 +79,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -77,6 +87,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -84,6 +95,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -91,6 +103,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -98,6 +111,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -105,6 +119,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -112,6 +127,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: fuubar
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -119,6 +135,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -126,6 +143,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -133,6 +151,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -169,26 +188,27 @@ files:
 homepage: https://github.com/otherinbox/luggage
 licenses:
 - MIT
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 1.8.23
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Net::IMAP DSL
 test_files:
 - spec/luggage/factory_spec.rb

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 56d02cbdf3e91765d32a9e9e31d1044f3bc2205f
-  data.tar.gz: 39548bc74936a200a7b63c8c8ddaad8b59e1119a
-SHA512:
-  metadata.gz: 84ee01e2b3bd73119fddf71f3cc4b8ef0ea3a7cc0dbf5935867e7128b5974af1363627c5c24dcd497c9f9de5f3f0fa40220797fbaa0baf4dd2429c981c24b690
-  data.tar.gz: 1eb4a73055793e3b0fd091586d5887ac628e1165cef0b0cd4c0bb5e409bc744fb5af27cc7fe6b21b39f076c9b62ce9cf1f42692e64768605314b1c28b97d882a