maildir 0.5.0 → 0.6.0

data/Rakefile

@@ -30,3 +30,8 @@ desc "Run benchmarks"
 task :bench do
   load File.join(File.dirname(__FILE__), "benchmarks", "runner")
 end
+
+desc "Remove trailing whitespace"
+task :whitespace do
+  sh %{find . -name '*.rb' -exec sed -i '' 's/ *$//g' {} \\;}
+end

data/VERSION

@@ -1 +1 @@
-0.5.0
+0.6.0

data/lib/maildir/message.rb

@@ -122,7 +122,7 @@ class Maildir::Message
 
   # Returns an array of single letter flags applied to the message
   def flags
-    @info.sub(INFO,'').split('')
+    @info.sub(INFO,'').split(//)
   end
 
   # Sets the flags on a message.
@@ -180,10 +180,11 @@ class Maildir::Message
     guard { File.mtime(path) }
   end
 
-  # Deletes the message path and freezes the message object
+  # Deletes the message path and freezes the message object.
+  # Returns 1 if the file was destroyed, false if the file was missings.
   def destroy
-    guard { File.delete(path) }
     freeze
+    guard { File.delete(path) }
   end
 
   protected

data/lib/maildir/serializer/base.rb

@@ -17,4 +17,4 @@ module Maildir::Serializer
       File.open(path, "w") {|file| file.write(data)}
     end
   end
-end
+end

data/lib/maildir/serializer/json.rb

@@ -10,4 +10,4 @@ class Maildir::Serializer::JSON < Maildir::Serializer::Base
   def dump(data, path)
     super(data.to_json, path)
   end
-end
+end

data/lib/maildir/serializer/mail.rb

@@ -5,9 +5,9 @@ class Maildir::Serializer::Mail < Maildir::Serializer::Base
   def load(path)
     ::Mail.new(super(path))
   end
-  
+
   # Write data to path as a Mail message.
   def dump(data, path)
     super(data.to_s, path)
   end
-end
+end

data/lib/maildir/serializer/marshal.rb

@@ -9,4 +9,4 @@ class Maildir::Serializer::Marshal < Maildir::Serializer::Base
   def dump(data, path)
     super(::Marshal.dump(data), path)
   end
-end
+end

data/lib/maildir/serializer/yaml.rb

@@ -10,4 +10,4 @@ class Maildir::Serializer::YAML < Maildir::Serializer::Base
   def dump(data, path)
     super(data.to_yaml, path)
   end
-end
+end

data/lib/maildir/subdirs.rb

@@ -5,7 +5,7 @@ module Maildir::Subdirs
   DELIM = '.'
 
   def self.included(base)
-    base.instance_eval do 
+    base.instance_eval do
       alias_method :inspect_without_subdirs, :inspect
       alias_method :inspect, :inspect_with_subdirs
     end
@@ -39,7 +39,7 @@ module Maildir::Subdirs
       @subdirs ||= root.subdirs(false).select { |md| subdir_parts(File.basename(md.path))[0..-2] == my_parts }
     end
   end
-  
+
   # Friendly inspect method
   def inspect_with_subdirs
     "#<#{self.class} path=#{@path} mailbox_path=#{mailbox_path}>"

data/lib/maildir.rb

@@ -111,4 +111,4 @@ class Maildir
 end
 require 'maildir/unique_name'
 require 'maildir/serializer/base'
-require 'maildir/message'
+require 'maildir/message'

data/maildir.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{maildir}
-  s.version = "0.5.0"
+  s.version = "0.6.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Aaron Suggs", "Niklas E. Cathor"]
-  s.date = %q{2010-01-24}
+  s.date = %q{2010-01-28}
   s.description = %q{A ruby library for reading and writing arbitrary messages in DJB's maildir format}
   s.email = %q{aaron@ktheory.com}
   s.extra_rdoc_files = [

data/test/test_message.rb

@@ -129,18 +129,30 @@ class TestMessage < Test::Unit::TestCase
     end
   end
 
-  context "A destroyed message" do
+  context "Destroying a message" do
     setup do
       FakeFS::FileSystem.clear
       @message = Maildir::Message.create(temp_maildir, "foo")
-      @message.destroy
     end
-    should "be frozen" do
+
+    should "freeze it" do
+      @message.destroy
       assert @message.frozen?, "Message is not frozen"
     end
-    should "have a nonexistant path" do
+
+    should "delete the path" do
+      @message.destroy
       assert !File.exists?(@message.path), "Message path exists"
     end
+
+    should "return 1" do
+      assert_equal 1, @message.destroy
+    end
+
+    should "return false if the path doesn't exist" do
+      File.delete(@message.path)
+      assert_equal false, @message.destroy
+    end
   end
 
   context "A message with a bad path" do
@@ -213,14 +225,14 @@ class TestMessage < Test::Unit::TestCase
     end
 
     # atime not currently supported in FakeFS
-    # should "update the messages atime" do
-    #   @message = temp_maildir.add("")
-    #   time = Time.now - 60
-    #
-    #   @message.utime(time, time)
-    #
-    #   # Time should be within 1 second of each other
-    #   assert_in_delta time, @message.atime, 1
-    # end
+    should_eventually "update the messages atime" do
+      @message = temp_maildir.add("")
+      time = Time.now - 60
+
+      @message.utime(time, time)
+
+      # Time should be within 1 second of each other
+      assert_in_delta time, @message.atime, 1
+    end
   end
-end
+end

data/test/test_unique_name.rb

@@ -40,4 +40,4 @@ class TestUniqueName < Test::Unit::TestCase
       assert_equal value1+1, value2
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: maildir
 version: !ruby/object:Gem::Version 
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors: 
 - Aaron Suggs
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-01-24 00:00:00 -05:00
+date: 2010-01-28 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency