imap_guard 1.0.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b9bad59e96b10c29a2a065cd6b5d866f10e8d7c8
-  data.tar.gz: d110224b73afef6abc0cc7805aefc906ee222b35
+SHA256:
+  metadata.gz: 99df3a5c8af846514e4270323a001ad4033f602f197965f6eec76dbf9cb999a0
+  data.tar.gz: 2be62429ce40e9b3e5edd27c8ff166fd5e62193c37cad995019958cf0997b7da
 SHA512:
-  metadata.gz: ef2d1979d3459bf83732861e5d6972af03cf698dd3178b8d9b7326eed9b0c44c4b17b3dab7c038a14425c5eb980c7f3255f6d412729af4c89810e66d597a42f9
-  data.tar.gz: f23003553803bf31aaf648d840f221a627387c147e326e399f8ecf02a320059325ef6a412161a00c35a96efea91fc65f1e2e2097309c18097045f8b932b02a0d
+  metadata.gz: 7d777c406347aff9d485111676a39ce14ea3348e8c17daa5ce343db7a91b1c6a6a4dd57504fecde6a4fa719501f4d1cd848119e6820bc085c3a7fd33382b9c5c
+  data.tar.gz: eda7c841a6b0b6e86d4c4d3ab62f5f7992bff53d0510cd303afd46b29d5780e00f8b3769f0386e0baf70908a53a048799104d46c11629d46eaa88c9b4f2f41cf

data/.rspec

@@ -1,3 +1,3 @@
 --color
 --format d
---order random
+--require spec_helper

data/.rubocop.yml

@@ -0,0 +1,26 @@
+AllCops:
+  TargetRubyVersion: 2.4
+
+Metrics:
+  Enabled: false
+
+Layout/AccessModifierIndentation:
+  EnforcedStyle: outdent
+
+Style/FormatStringToken:
+  Enabled: false
+
+Style/SafeNavigation:
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  Enabled: false
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
+Style/TrailingCommaInArguments:
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma

data/.travis.yml

@@ -3,13 +3,6 @@ after_script:
   - cane
   - yard stats --list-undoc
 rvm:
-  - 1.9.3
-  - 2.0.0
-  - 2.1
-  - 2.2
-  - ruby-head
-  - rbx
-matrix:
-  allow_failures:
-    - rvm: ruby-head
-    - rvm: rbx
+  - 2.6.0
+  - 2.5.3
+  - 2.4.5

data/Gemfile

@@ -1,4 +1,6 @@
-source 'https://rubygems.org'
+# frozen_string_literal: true
+
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in imap_guard.gemspec
 gemspec

data/Guardfile

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 # More info at https://github.com/guard/guard#readme
 
-guard 'rspec' do
+guard "rspec" do
   watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$})        { |m| "spec/lib/#{m[1]}_spec.rb" }
   watch(%r{^lib/(.+)/(.+)\.rb$})   { |m| "spec/#{m[1]}/#{m[2]}_spec.rb" }
-  watch('spec/spec_helper.rb')     { "spec" }
+  watch("spec/spec_helper.rb")     { "spec" }
   watch(%r{^spec/support/.+\.rb$}) { "spec" }
 end
-

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2013-2015 Cédric Félizard
+Copyright (c) 2013-2019 Cédric Félizard
 
 MIT License
 

data/README.md

@@ -1,13 +1,11 @@
-# ImapGuard [![Build Status](https://secure.travis-ci.org/infertux/imap_guard.png?branch=master)](https://travis-ci.org/infertux/imap_guard) [![Dependency Status](https://gemnasium.com/infertux/imap_guard.png)](https://gemnasium.com/infertux/imap_guard) [![Code Climate](https://codeclimate.com/github/infertux/imap_guard.png)](https://codeclimate.com/github/infertux/imap_guard)
+# ImapGuard
 
-**A guard for your IMAP mailboxes.**
+[![Gem Version](https://badge.fury.io/rb/imap_guard.svg)](https://badge.fury.io/rb/imap_guard)
+[![Build Status](https://secure.travis-ci.org/infertux/imap_guard.png?branch=master)](https://travis-ci.org/infertux/imap_guard)
+[![Code Climate](https://codeclimate.com/github/infertux/imap_guard.png)](https://codeclimate.com/github/infertux/imap_guard)
+[![Inline docs](http://inch-ci.org/github/infertux/imap_guard.svg)](http://inch-ci.org/github/infertux/imap_guard)
 
-  * [Source Code](https://github.com/infertux/imap_guard "Source Code on Github")
-  * [Bug Tracker](https://github.com/infertux/imap_guard/issues "Bug Tracker on Github")
-  * [API documentation](http://rubydoc.info/gems/imap_guard/frames "API Documentation on Rubydoc")
-  * [Rubygem](https://rubygems.org/gems/imap_guard "ImapGuard on Rubygems")
-  * [Continuous Integration](https://travis-ci.org/infertux/imap_guard "ImapGuard on Travis-CI")
-  * [Dependencies](https://gemnasium.com/infertux/imap_guard "ImapGuard dependencies on Gemnasium")
+**A guard for your IMAP mailboxes.**
 
 ImapGuard connects to your IMAP server and processes your emails.
 You can finely pick them thanks to advanced search queries and Ruby blocks.
@@ -70,7 +68,7 @@ guard.delete base_query.dup.before(7).subject("Logwatch for ") do |mail|
 end
 ```
 
-Finally, you can always forge your own raw IMAP search queries (the [RFC](http://tools.ietf.org/html/rfc3501#section-6.4.4) can help in that case):
+You can always forge your own raw IMAP search queries (the [RFC](http://tools.ietf.org/html/rfc3501#section-6.4.4) can help in that case):
 
 ```ruby
 query = 'SEEN SUBJECT "ALERT" FROM "root"'
@@ -79,19 +77,19 @@ guard.delete query do |mail|
 end
 ```
 
-Be aware that emails won't be touched until you `expunge` or `close` the mailbox:
+There is a `move` method as well:
 
 ```ruby
-guard.expunge # effectively delete emails marked as deleted
-guard.close # expunge then close the connection
+guard.move query, 'destination_folder' do |mail|
+  # and it can take a filter block like `delete`
+end
 ```
 
-Oh, and there is a `move` method as well:
+Finally, this should be handled automatically but you can explicitly expunge pending emails and close the connection:
 
 ```ruby
-guard.move query, 'destination_folder' do |mail|
-  # and it can take a filter block like `delete`
-end
+guard.expunge # effectively delete emails marked as deleted
+guard.close # expunge then close the connection
 ```
 
 ### Advanced features
@@ -134,4 +132,3 @@ MIT
 
 
 [Mail]: https://github.com/mikel/mail
-

data/Rakefile

@@ -1,9 +1,13 @@
+# frozen_string_literal: true
+
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
+require "rubocop/rake_task"
 
 RSpec::Core::RakeTask.new(:spec) do |t|
-  t.ruby_opts = '-w'
+  t.ruby_opts = "-w"
 end
 
-task default: :spec
+RuboCop::RakeTask.new(:rubocop)
 
+task default: %i[spec rubocop]

data/examples/example.rb

@@ -1,43 +1,45 @@
 #!/usr/bin/env ruby
 
-require 'imap_guard'
+# frozen_string_literal: true
+# rubocop:disable all
+
+require "imap_guard"
 
 SETTINGS = {
-  host: 'imap.googlemail.com',
+  host: "imap.googlemail.com",
   port: 993,
-  username: 'you@gmail.com',
-  password: 'your_pass',
-}
+  username: "you@gmail.com",
+  password: "your_pass",
+}.freeze
 
-settings = SETTINGS.merge({ read_only: false })
+settings = SETTINGS.merge(read_only: false)
 base_query = ImapGuard::Query.new.unflagged.unanswered.freeze
 guard = ImapGuard::Guard.new settings
 # guard.debug = ->(mail) { print "#{mail.subject}: " }
 guard.login
 
-guard.select 'INBOX'
+guard.select "INBOX"
 
 # Github
 %w(github.com notifications@travis-ci.org app@gemnasium.com).map do |from|
   base_query.dup.from(from)
 end.each do |query|
-  guard.move query, 'INBOX.Github'
+  guard.move query, "INBOX.Github"
 end
 
 # To Do
-guard.move base_query.dup.from("me").to("me"), 'INBOX.TODO'
+guard.move base_query.dup.from("me").to("me"), "INBOX.TODO"
 
 # Ops
-guard.select 'INBOX.Ops'
+guard.select "INBOX.Ops"
 query = base_query.dup.seen
 guard.delete query.dup.subject("monit alert -- ").before(7)
 guard.delete query.dup.subject("CRON-APT completed on ").before(3)
 guard.delete query.dup.subject("Logwatch for ").before(7)
-guard.select 'INBOX'
+guard.select "INBOX"
 
 # Uni
-guard.move base_query.dup.or.from("uni.tld").to("uni.tld"), 'INBOX.Uni'
+guard.move base_query.dup.or.from("uni.tld").to("uni.tld"), "INBOX.Uni"
 
 # Bye!
 guard.disconnect
-

data/imap_guard.gemspec

@@ -1,13 +1,14 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "imap_guard"
-  spec.version       = "1.0.0"
+  spec.version       = "1.2.0"
   spec.authors       = ["Cédric Félizard"]
   spec.email         = ["cedric@felizard.fr"]
-  spec.description   = %q{A guard for your IMAP server}
+  spec.description   = "A guard for your IMAP server"
   spec.summary       = spec.description
   spec.homepage      = "https://github.com/infertux/imap_guard"
   spec.license       = "MIT"
@@ -17,16 +18,15 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency 'mail', '>= 2.5.3'
-  spec.add_dependency 'term-ansicolor', '>= 1.2.2'
+  spec.add_dependency "mail", ">= 2.5.3"
+  spec.add_dependency "term-ansicolor", ">= 1.2.2"
 
-  spec.add_development_dependency 'bundler', '>= 1.3'
-  spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'simplecov'
-  spec.add_development_dependency 'rspec'
-  spec.add_development_dependency 'guard-rspec'
-  spec.add_development_dependency 'rb-inotify'
-  spec.add_development_dependency 'cane'
-  spec.add_development_dependency 'yard'
-  spec.add_development_dependency 'redcarpet' # for yardoc
+  spec.add_development_dependency "bundler", ">= 1.3"
+  spec.add_development_dependency "cane"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "redcarpet" # for yardoc
+  spec.add_development_dependency "rspec"
+  spec.add_development_dependency "rubocop"
+  spec.add_development_dependency "simplecov"
+  spec.add_development_dependency "yard"
 end

data/lib/imap_guard.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "imap_guard/guard"
 require "imap_guard/query"
 

data/lib/imap_guard/guard.rb

@@ -1,18 +1,21 @@
-require 'net/imap'
-require 'ostruct'
-require 'mail'
-require 'term/ansicolor'
+# frozen_string_literal: true
+
+require "net/imap"
+require "ostruct"
+require "mail"
+require "term/ansicolor"
+
 String.send(:include, Term::ANSIColor)
-Term::ANSIColor::coloring = STDOUT.isatty
+Term::ANSIColor.coloring = STDOUT.isatty
 
 module ImapGuard
   # Guard allows you to process your mailboxes.
   class Guard
     # List of required settings
-    REQUIRED_SETTINGS = [:host, :port, :username, :password]
+    REQUIRED_SETTINGS = %i[host port username password].freeze
 
     # List of optional settings
-    OPTIONAL_SETTINGS = [:read_only, :verbose]
+    OPTIONAL_SETTINGS = %i[read_only verbose].freeze
 
     # @return [Proc, nil] Matched emails are passed to this debug lambda if present
     attr_accessor :debug
@@ -24,7 +27,7 @@ module ImapGuard
     # @return [String, nil] Currently selected mailbox
     attr_reader :mailbox
 
-    def initialize settings
+    def initialize(settings)
       self.settings = settings
     end
 
@@ -39,7 +42,7 @@ module ImapGuard
 
     # Selects a mailbox (folder)
     # @return [void]
-    def select mailbox
+    def select(mailbox)
       if @settings.read_only
         @imap.examine(mailbox) # open in read-only
       else
@@ -53,15 +56,15 @@ module ImapGuard
     # @param mailbox Destination mailbox
     # @param filter Optional filter block
     # @return [void]
-    def move query, mailbox, &filter
-      operation = lambda { |message_id|
+    def move(query, mailbox, &filter)
+      operation = lambda do |message_id|
         unless @settings.read_only
           @imap.copy(message_id, mailbox)
           @imap.store(message_id, "+FLAGS", [Net::IMAP::DELETED])
         end
 
         "moved to #{mailbox}".yellow
-      }
+      end
       process query, operation, &filter
     end
 
@@ -69,14 +72,12 @@ module ImapGuard
     # @param query IMAP query
     # @param filter Optional filter block
     # @return [void]
-    def delete query, &filter
-      operation = lambda { |message_id|
-        unless @settings.read_only
-          @imap.store(message_id, "+FLAGS", [Net::IMAP::DELETED])
-        end
+    def delete(query, &filter)
+      operation = lambda do |message_id|
+        @imap.store(message_id, "+FLAGS", [Net::IMAP::DELETED]) unless @settings.read_only
 
-        'deleted'.red
-      }
+        "deleted".red
+      end
       process query, operation, &filter
     end
 
@@ -84,16 +85,16 @@ module ImapGuard
     # @param query IMAP query
     # @param opration Lambda to call on each message
     # @return [void]
-    def each query
-      operation = lambda { |message_id| yield message_id }
+    def each(query)
+      operation = ->(message_id) { yield message_id }
       process query, operation
     end
 
     # Fetches a message from its UID
     # @return [Mail]
     # @note We use "BODY.PEEK[]" to avoid setting the \Seen flag.
-    def fetch_mail message_id
-      msg = @imap.fetch(message_id, 'BODY.PEEK[]').first.attr['BODY[]']
+    def fetch_mail(message_id)
+      msg = @imap.fetch(message_id, "BODY.PEEK[]").first.attr["BODY[]"]
       Mail.read_from_string msg
     end
 
@@ -125,7 +126,7 @@ module ImapGuard
 
   private
 
-    def process query, operation
+    def process(query, operation)
       message_ids = search query
       count = message_ids.size
 
@@ -133,7 +134,7 @@ module ImapGuard
         print "Processing UID #{message_id} (#{index.succ}/#{count}): "
 
         result = true
-        if block_given? or debug
+        if block_given? || debug
           mail = fetch_mail message_id
 
           debug.call(mail) if debug
@@ -146,15 +147,12 @@ module ImapGuard
 
         puts result ? operation.call(message_id) : "ignored".green
       end
-
     ensure
       expunge
     end
 
-    def search query
-      unless [Array, String].any? { |type| query.is_a? type }
-        raise TypeError, "Query must be either a string holding the entire search string, or a single-dimension array of search keywords and arguments."
-      end
+    def search(query)
+      raise TypeError, "Query must be either a string holding the entire search string, or a single-dimension array of search keywords and arguments." unless [Array, String].any? { |type| query.is_a? type }
 
       messages = @imap.search query
       puts "Query on #{mailbox}: #{query.inspect}: #{messages.count} results".cyan
@@ -164,18 +162,16 @@ module ImapGuard
 
     def verbose
       @verbose ||= if @settings.verbose
-        $stdout
-      else
-        # anonymous null object
-        Class.new do
-          def method_missing(*)
-            nil
-          end
-        end.new
-      end
+                     $stdout
+                   else
+                     # anonymous null object
+                     # rubocop:disable all
+                     Class.new do def method_missing(*); nil end end.new
+                     # rubocop:enable all
+                   end
     end
 
-    def settings= settings
+    def settings=(settings)
       missing = REQUIRED_SETTINGS - settings.keys
       raise ArgumentError, "Missing settings: #{missing}" unless missing.empty?
 
@@ -187,4 +183,3 @@ module ImapGuard
     end
   end
 end
-

data/lib/imap_guard/query.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ImapGuard
   # Query is a neat DSL to help you generate IMAP search queries.
   # @note All methods return self so they can be chained.
@@ -5,31 +7,31 @@ module ImapGuard
     # Messages that have the +\Seen+ flag set.
     # @return [self]
     def seen
-      self << 'SEEN'
+      self << "SEEN"
     end
 
     # Messages that do not have the +\Seen+ flag set.
     # @return [self]
     def unseen
-      self << 'UNSEEN'
+      self << "UNSEEN"
     end
 
     # Messages that do not have the +\Answered+ flag set.
     # @return [self]
     def unanswered
-      self << 'UNANSWERED'
+      self << "UNANSWERED"
     end
 
     # Messages that do not have the +\Flagged+ flag set.
     # @return [self]
     def unflagged
-      self << 'UNFLAGGED'
+      self << "UNFLAGGED"
     end
 
     # Messages with the +\Deleted+ flag set.
     # @return [self]
     def deleted
-      self << 'DELETED'
+      self << "DELETED"
     end
 
     # Messages that match either search key.
@@ -41,13 +43,13 @@ module ImapGuard
     #   or.unanswered.unflagged     #=> ["OR", "UNANSWERED", "UNFLAGGED"]
     #   or(:unanswered, :unflagged) #=> ["OR", "UNANSWERED", "UNFLAGGED"]
     # @return [self]
-    def or search_key1 = nil, search_key2 = nil
-      self << 'OR'
+    def or(search_key1 = nil, search_key2 = nil)
+      self << "OR"
 
-      if search_key1 and search_key2
+      if search_key1 && search_key2
         send(search_key1)
         send(search_key2)
-      elsif search_key1 or search_key2
+      elsif search_key1 || search_key2
         raise ArgumentError, "You must give either zero or two arguments."
       end
 
@@ -60,8 +62,8 @@ module ImapGuard
     #   not.deleted   #=> ["NOT", "DELETED"]
     #   not(:deleted) #=> ["NOT", "DELETED"]
     # @return [self]
-    def not search_key = nil
-      self << 'NOT'
+    def not(search_key = nil)
+      self << "NOT"
       send(search_key) if search_key
       self
     end
@@ -69,29 +71,29 @@ module ImapGuard
     # Messages that contain the specified string in the envelope
     # structure's SUBJECT field.
     # @return [self]
-    def subject string
-      self << 'SUBJECT' << string
+    def subject(string)
+      self << "SUBJECT" << string
     end
 
     # Messages that contain the specified string in the envelope
     # structure's FROM field.
     # @return [self]
-    def from string
-      self << 'FROM' << string
+    def from(string)
+      self << "FROM" << string
     end
 
     # Messages that contain the specified string in the envelope
     # structure's TO field.
     # @return [self]
-    def to string
-      self << 'TO' << string
+    def to(string)
+      self << "TO" << string
     end
 
     # Messages that contain the specified string in the envelope
     # structure's CC field.
     # @return [self]
-    def cc string
-      self << 'CC' << string
+    def cc(string)
+      self << "CC" << string
     end
 
     # Messages whose internal date (disregarding time and timezone)
@@ -101,20 +103,19 @@ module ImapGuard
     #   - [Fixnum]: _n_ days before today
     #   - [Date]: uses this date
     # @return [self]
-    def before date
-      case date
-      when String
-        # noop, uses it as is
-      when Fixnum
-        date = (Date.today - date).strftime '%e-%b-%Y'
-      when Date
-        date = date.strftime '%e-%b-%Y'
-      else
-        raise ArgumentError, "#{date.inspect} is invalid."
-      end
+    def before(date)
+      date = case date
+             when String
+               date
+             when Integer
+               (Date.today - date).strftime "%e-%b-%Y"
+             when Date
+               date.strftime "%e-%b-%Y"
+             else
+               raise ArgumentError, "#{date.inspect} is invalid."
+             end
 
-      self << 'BEFORE' << date
+      self << "BEFORE" << date
     end
   end
 end
-

data/spec/imap_guard/guard_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 module ImapGuard
   describe Guard do
@@ -12,21 +12,21 @@ module ImapGuard
 
     let(:settings) do
       {
-        host: 'localhost',
+        host: "localhost",
         port: 993,
-        username: 'bob',
-        password: 'PASS',
+        username: "bob",
+        password: "PASS",
       }
     end
 
     let(:imap) do
-      double('Net::IMAP', search: [7, 28], expunge: nil, select: nil, list: [])
+      double("Net::IMAP", search: [7, 28], expunge: nil, select: nil, list: [])
     end
 
-    def guard_instance custom_settings = {}
+    def guard_instance(custom_settings = {})
       guard = Guard.new(settings.merge(custom_settings))
       guard.instance_variable_set(:@imap, imap)
-      guard.stub(:fetch_mail)
+      allow(guard).to receive(:fetch_mail)
       guard
     end
 
@@ -35,7 +35,7 @@ module ImapGuard
         let(:guard) { guard_instance(read_only: true) }
 
         it "opens the mailbox in read-only" do
-          imap.should_receive(:examine)
+          expect(imap).to receive(:examine)
           guard.select nil
         end
       end
@@ -44,7 +44,7 @@ module ImapGuard
         let(:guard) { guard_instance(read_only: false) }
 
         it "opens the mailbox in read-write" do
-          imap.should_receive(:select)
+          expect(imap).to receive(:select)
           guard.select nil
         end
       end
@@ -52,46 +52,46 @@ module ImapGuard
 
     describe "#mailbox" do
       it "returns nil when no mailbox has been selected" do
-        guard_instance.mailbox.should be_nil
+        expect(guard_instance.mailbox).to be_nil
       end
 
       it "returns the currently selected mailbox" do
         guard = guard_instance
 
-        guard.select 'Sent'
-        guard.mailbox.should eq 'Sent'
+        guard.select "Sent"
+        expect(guard.mailbox).to eq "Sent"
       end
     end
 
     describe "#list" do
       it "returns the list of mailboxes" do
-        imap.should_receive(:list)
-        guard_instance.list.should eq []
+        expect(imap).to receive(:list)
+        expect(guard_instance.list).to eq []
       end
     end
 
     describe "#search" do
       before do
-        imap.should_receive(:search) do |arg|
-          [13, 37] if [['ALL'], 'ALL'].include? arg
+        expect(imap).to receive(:search) do |arg|
+          [13, 37] if [["ALL"], "ALL"].include? arg
         end
       end
 
       it "accepts arrays" do
-        expect {
-          guard_instance.send(:search, ['ALL'])
-        }.to_not raise_error
+        expect do
+          guard_instance.send(:search, ["ALL"])
+        end.to_not raise_error
       end
 
       it "accepts strings" do
-        expect {
-          guard_instance.send(:search, 'ALL')
-        }.to_not raise_error
+        expect do
+          guard_instance.send(:search, "ALL")
+        end.to_not raise_error
       end
 
       it "returns results" do
-        messages = guard_instance.send(:search, 'ALL')
-        messages.should eq [13, 37]
+        messages = guard_instance.send(:search, "ALL")
+        expect(messages).to eq [13, 37]
       end
     end
 
@@ -101,49 +101,49 @@ module ImapGuard
 
       context "without a filter block" do
         it "does perform the operation" do
-          opeartion.should_receive(:call).with(7)
-          opeartion.should_receive(:call).with(28)
+          expect(opeartion).to receive(:call).with(7)
+          expect(opeartion).to receive(:call).with(28)
 
-          guard.send(:process, 'ALL', opeartion)
+          guard.send(:process, "ALL", opeartion)
         end
 
         it "does not execute the filter block" do
-          guard.should_not_receive(:fetch_mail)
+          expect(guard).not_to receive(:fetch_mail)
 
-          guard.send(:process, 'ALL', opeartion)
+          guard.send(:process, "ALL", opeartion)
         end
 
         context "with a debug proc" do
           it "calls the proc" do
             block = ->(mail) {}
             guard.debug = block
-            block.should_receive(:call).twice
+            expect(block).to receive(:call).twice
 
-            guard.send(:process, 'ALL', opeartion)
+            guard.send(:process, "ALL", opeartion)
           end
         end
       end
 
       context "with a filter block" do
         it "executes the filter block" do
-          guard.should_receive(:fetch_mail).twice
+          expect(guard).to receive(:fetch_mail).twice
 
-          guard.send(:process, 'ALL', opeartion) { }
+          guard.send(:process, "ALL", opeartion) {}
         end
 
         context "returning false" do
           it "does not perform the operation" do
-            opeartion.should_not_receive(:call)
+            expect(opeartion).not_to receive(:call)
 
-            guard.send(:process, 'ALL', opeartion) { false }
+            guard.send(:process, "ALL", opeartion) { false }
           end
         end
 
         context "returning true" do
           it "does perform the operation" do
-            opeartion.should_receive(:call).twice
+            expect(opeartion).to receive(:call).twice
 
-            guard.send(:process, 'ALL', opeartion) { true }
+            guard.send(:process, "ALL", opeartion) { true }
           end
         end
       end
@@ -151,29 +151,29 @@ module ImapGuard
 
     describe "#move" do
       it "copies emails before adding the :Deleted flag" do
-        imap.should_receive(:search)
-        imap.should_receive(:copy).with(7, 'destination').ordered
-        imap.should_receive(:store).with(7, '+FLAGS', [:Deleted]).ordered
-        imap.should_receive(:copy).with(28, 'destination').ordered
-        imap.should_receive(:store).with(28, '+FLAGS', [:Deleted]).ordered
+        expect(imap).to receive(:search)
+        expect(imap).to receive(:copy).with(7, "destination").ordered
+        expect(imap).to receive(:store).with(7, "+FLAGS", [:Deleted]).ordered
+        expect(imap).to receive(:copy).with(28, "destination").ordered
+        expect(imap).to receive(:store).with(28, "+FLAGS", [:Deleted]).ordered
 
-        guard_instance.move 'ALL', 'destination'
+        guard_instance.move "ALL", "destination"
       end
     end
 
     describe "#delete" do
       it "adds the :Deleted flag" do
-        imap.should_receive(:search)
-        imap.should_receive(:store).with(7, '+FLAGS', [:Deleted])
-        imap.should_receive(:store).with(28, '+FLAGS', [:Deleted])
+        expect(imap).to receive(:search)
+        expect(imap).to receive(:store).with(7, "+FLAGS", [:Deleted])
+        expect(imap).to receive(:store).with(28, "+FLAGS", [:Deleted])
 
-        guard_instance.delete 'ALL'
+        guard_instance.delete "ALL"
       end
     end
 
     describe "#each" do
       it "iterates over messages without errors" do
-        guard_instance.each 'ALL' do |message_id|
+        guard_instance.each "ALL" do |message_id|
           # noop
         end
       end
@@ -181,21 +181,21 @@ module ImapGuard
 
     describe "#expunge" do
       it "expunges the folder" do
-        imap.should_receive(:expunge)
+        expect(imap).to receive(:expunge)
         guard_instance.expunge
       end
     end
 
     describe "#close" do
       it "closes the IMAP connection" do
-        imap.should_receive(:close)
+        expect(imap).to receive(:close)
         guard_instance.close
       end
     end
 
     describe "#disconnect" do
       it "disconnects from the server" do
-        imap.should_receive(:disconnect)
+        expect(imap).to receive(:disconnect)
         guard_instance.disconnect
       end
     end
@@ -205,7 +205,7 @@ module ImapGuard
         let(:guard) { guard_instance(verbose: true) }
 
         it "does output to $stdout" do
-          $stdout.should_receive(:write).with("ham")
+          expect($stdout).to receive(:write).with("ham")
           guard.send(:verbose).print "ham"
         end
       end
@@ -214,7 +214,7 @@ module ImapGuard
         let(:guard) { guard_instance(verbose: false) }
 
         it "does not output to $stdout" do
-          $stdout.should_not_receive(:write)
+          expect($stdout).not_to receive(:write)
           guard.send(:verbose).print "ham"
         end
       end
@@ -226,23 +226,22 @@ module ImapGuard
 
         exception = (RUBY_VERSION >= "2.1.0" ? RuntimeError : TypeError)
 
-        expect {
-          guard.settings.host = 'example.net'
-        }.to raise_error(exception, /frozen/)
+        expect do
+          guard.settings.host = "example.net"
+        end.to raise_error(exception, /frozen/)
       end
 
       it "raises ArgumentError if any required key is missing" do
-        expect {
+        expect do
           Guard.new({})
-        }.to raise_error ArgumentError, /missing/i
+        end.to raise_error ArgumentError, /missing/i
       end
 
       it "raises ArgumentError if any key is unknown" do
-        expect {
+        expect do
           Guard.new(settings.merge(coffee: true))
-        }.to raise_error ArgumentError, /unknown/i
+        end.to raise_error ArgumentError, /unknown/i
       end
     end
   end
 end
-

data/spec/imap_guard/query_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 module ImapGuard
   describe Query do
@@ -9,35 +9,35 @@ module ImapGuard
     describe "#seen" do
       it "adds 'SEEN'" do
         subject.seen
-        subject.last.should eq 'SEEN'
+        expect(subject.last).to eq "SEEN"
       end
     end
 
     describe "#unseen" do
       it "adds 'UNSEEN'" do
         subject.unseen
-        subject.last.should eq 'UNSEEN'
+        expect(subject.last).to eq "UNSEEN"
       end
     end
 
     describe "#unanswered" do
       it "adds 'UNANSWERED'" do
         subject.unanswered
-        subject.last.should eq 'UNANSWERED'
+        expect(subject.last).to eq "UNANSWERED"
       end
     end
 
     describe "#unflagged" do
       it "adds 'UNFLAGGED'" do
         subject.unflagged
-        subject.last.should eq 'UNFLAGGED'
+        expect(subject.last).to eq "UNFLAGGED"
       end
     end
 
     describe "#deleted" do
       it "adds 'DELETED'" do
         subject.deleted
-        subject.last.should eq 'DELETED'
+        expect(subject.last).to eq "DELETED"
       end
     end
 
@@ -45,26 +45,26 @@ module ImapGuard
       context "without a search key" do
         it "adds 'OR'" do
           subject.or
-          subject.last.should eq 'OR'
+          expect(subject.last).to eq "OR"
         end
       end
 
       context "with search keys" do
         it "adds 'OR UNANSWERED UNFLAGGED '" do
           subject.or(:unanswered, :unflagged)
-          subject.last(3).should eq ["OR", "UNANSWERED", "UNFLAGGED"]
+          expect(subject.last(3)).to eq %w[OR UNANSWERED UNFLAGGED]
         end
       end
 
       context "with only one non-nil search key" do
         it "raises ArgumentError" do
-          expect {
+          expect do
             subject.or(:whatever)
-          }.to raise_error(ArgumentError)
+          end.to raise_error(ArgumentError)
 
-          expect {
+          expect do
             subject.or(nil, :whatever)
-          }.to raise_error(ArgumentError)
+          end.to raise_error(ArgumentError)
         end
       end
     end
@@ -72,28 +72,28 @@ module ImapGuard
     describe "#subject" do
       it "adds the search value" do
         subject.subject("Hey you")
-        subject.last.should eq "Hey you"
+        expect(subject.last).to eq "Hey you"
       end
     end
 
     describe "#from" do
       it "adds the search value" do
         subject.from("root@example.net")
-        subject.last.should eq "root@example.net"
+        expect(subject.last).to eq "root@example.net"
       end
     end
 
     describe "#to" do
       it "adds the search value" do
         subject.to("root@example.net")
-        subject.last.should eq "root@example.net"
+        expect(subject.last).to eq "root@example.net"
       end
     end
 
     describe "#cc" do
       it "adds the search value" do
         subject.cc("root@example.net")
-        subject.last.should eq "root@example.net"
+        expect(subject.last).to eq "root@example.net"
       end
     end
 
@@ -101,14 +101,14 @@ module ImapGuard
       context "without a search key" do
         it "adds 'NOT'" do
           subject.not.deleted
-          subject.last(2).should eq ["NOT", "DELETED"]
+          expect(subject.last(2)).to eq %w[NOT DELETED]
         end
       end
 
       context "with a search key" do
         it "adds 'NOT DELETED'" do
           subject.not(:deleted)
-          subject.last(2).should eq ["NOT", "DELETED"]
+          expect(subject.last(2)).to eq %w[NOT DELETED]
         end
       end
     end
@@ -116,40 +116,39 @@ module ImapGuard
     describe "#before" do
       context "when I pass 'nil' as an argument" do
         it "raises" do
-          expect {
+          expect do
             subject.before(nil)
-          }.to raise_error ArgumentError
+          end.to raise_error ArgumentError
         end
       end
 
       context "when I pass '1' as an argument" do
         it "returns yesterday" do
           subject.before(1)
-          Date.parse(subject.last).should eq Date.today.prev_day
+          expect(Date.parse(subject.last)).to eq Date.today.prev_day
         end
       end
 
       context "when I pass an integer" do
         it "uses it as a negative offset in days" do
           subject.before(3)
-          (Date.today - Date.parse(subject.last)).should eq 3
+          expect((Date.today - Date.parse(subject.last))).to eq 3
         end
       end
 
       context "when I pass '18-Mar-2013' as an argument" do
         it "uses it as is" do
-          subject.before('18-Mar-2013')
-          subject.last.should eq '18-Mar-2013'
+          subject.before("18-Mar-2013")
+          expect(subject.last).to eq "18-Mar-2013"
         end
       end
 
       context "when I pass an instance of Date as an argument" do
         it "extracts the date" do
           subject.before(Date.today)
-          Date.parse(subject.last).should eq Date.today
+          expect(Date.parse(subject.last)).to eq Date.today
         end
       end
     end
   end
 end
-

data/spec/spec_helper.rb

@@ -1,11 +1,12 @@
-require 'simplecov'
+# frozen_string_literal: true
+
+require "simplecov"
 SimpleCov.start do
-  minimum_coverage 90
+  minimum_coverage 95
   add_group "Sources", "lib"
   add_group "Tests", "spec"
 end
 
-require 'imap_guard'
+require "imap_guard"
 
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |file| require file }
-

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: imap_guard
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Cédric Félizard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-15 00:00:00.000000000 Z
+date: 2019-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mail
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: cane
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +95,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: guard-rspec
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rb-inotify
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: cane
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -150,20 +150,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: redcarpet
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: A guard for your IMAP server
 email:
 - cedric@felizard.fr
@@ -173,7 +159,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
-- ".ruby-version"
+- ".rubocop.yml"
 - ".travis.yml"
 - Gemfile
 - Guardfile
@@ -207,8 +193,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 3.0.1
 signing_key: 
 specification_version: 4
 summary: A guard for your IMAP server
@@ -216,4 +201,3 @@ test_files:
 - spec/imap_guard/guard_spec.rb
 - spec/imap_guard/query_spec.rb
 - spec/spec_helper.rb
-has_rdoc: 

data/.ruby-version

@@ -1 +0,0 @@
-2.2