feedjira 3.2.0 → 3.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db9d2eb01b3bf275ae7d0599df9bef74ea336ddfb9afdcc31998c4f79890cf51
-  data.tar.gz: 30de1f11b4054c3860ca79a27f8fb54b2244e8140bd26340d3f64f84c2d29884
+  metadata.gz: b6930aadfe8b2a06d65aa2a8dfaad0d8fcc4f34356a3c3646cf75ef70a753a8d
+  data.tar.gz: 00a921f06c7542ca84d552a6f680e56d99f4b0b3aa830113b75c0545f6fdbcf7
 SHA512:
-  metadata.gz: a83a5507737ed80d62515733dad8005f0f3d0bce59558d6b6f252a94084d459b664b7a56c5355d853cb706b62c1a7c804e5bd68df74f4cb9091bc12ce475d075
-  data.tar.gz: 8af825f5c5cde56ba80d69cbd7d55b1e36e4cb2c6c401f7092a9a8654757fcc133666dbd7ad38527b45f790523aa5449c29dd6886f43112dac49c328ccce816d
+  metadata.gz: 5970d946d3673ff856f95bef75e09b4cc0b6dbc3b8b9ba66a3ebade66d93fd894b23e784d8e9c3ab345eb6deae2094878e60be2164aae46623e2159bcf4530c0
+  data.tar.gz: ae1466452711a547ec9f2ad42ab34b11a8ce03a1886e6ddeb56f3a8723c3d1ab6f1a0b22461c339a0a5c490e64965a37da9329bcd701fd4d10117c8b9e28fbe1

data/.github/ISSUE_TEMPLATE/general-issue.md

@@ -0,0 +1,8 @@
+---
+name: Feed Parsing
+about: Anything else
+title: ''
+labels: ''
+assignees: ''
+
+---

data/.rubocop.yml

@@ -1,5 +1,10 @@
 inherit_from: .rubocop_todo.yml
 
+require:
+  - rubocop-rake
+  - rubocop-rspec
+  - rubocop-performance
+
 AllCops:
   TargetRubyVersion: 2.5
   NewCops: enable
@@ -32,3 +37,30 @@ Style/IfUnlessModifier:
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes
+
+RSpec/MultipleExpectations:
+  Max: 10
+
+RSpec/ExampleLength:
+  Max: 30
+
+RSpec/InstanceVariable:
+  Enabled: false
+
+RSpec/MessageSpies:
+  Enabled: false
+
+RSpec/NestedGroups:
+  Max: 5
+
+RSpec/MultipleMemoizedHelpers:
+  Max: 10
+
+RSpec/BeforeAfterAll:
+  Enabled: false
+
+RSpec/VerifiedDoubles:
+  Enabled: false
+
+RSpec/RepeatedExample:
+  Enabled: false

data/Gemfile

@@ -4,9 +4,6 @@ source "https://rubygems.org/"
 
 gemspec
 
-gem "pry"
-gem "rubocop", "~> 0.90"
-
 group :test do
   gem "oga"
   gem "ox", platforms: %i[mri rbx]

data/feedjira.gemspec

@@ -22,7 +22,8 @@ Gem::Specification.new do |s|
   s.metadata = {
     "homepage_uri" => "https://github.com/feedjira/feedjira",
     "source_code_uri" => "https://github.com/feedjira/feedjira",
-    "changelog_uri" => "https://github.com/feedjira/feedjira/blob/master/CHANGELOG.md"
+    "changelog_uri" => "https://github.com/feedjira/feedjira/blob/master/CHANGELOG.md",
+    "rubygems_mfa_required" => "true"
   }
 
   s.files         = `git ls-files`.split("\n")
@@ -35,7 +36,11 @@ Gem::Specification.new do |s|
   s.add_dependency "sax-machine",        ">= 1.0"
 
   s.add_development_dependency "faraday"
+  s.add_development_dependency "pry"
   s.add_development_dependency "rspec"
   s.add_development_dependency "rubocop"
+  s.add_development_dependency "rubocop-performance"
+  s.add_development_dependency "rubocop-rake"
+  s.add_development_dependency "rubocop-rspec"
   s.add_development_dependency "yard"
 end

data/lib/feedjira/core_ext/date.rb

@@ -17,6 +17,6 @@ class Date
 
   def feed_utils_to_time(dest, method)
     Time.send(method, dest.year, dest.month, dest.day, dest.hour, dest.min,
-              dest.sec, dest.zone)
+              dest.sec, dest.offset)
   end
 end

data/lib/feedjira/core_ext/time.rb

@@ -29,7 +29,7 @@ class Time
   def self.parse_string_safely(string)
     return nil if string.empty?
 
-    if string =~ /\A\d{14}\z/
+    if /\A\d{14}\z/.match?(string)
       parse("#{string}Z", true)
     else
       parse(string).utc

data/lib/feedjira/date_time_utilities/date_time_language_parser.rb

@@ -15,7 +15,7 @@ module Feedjira
       def self.translate(string)
         MONTHS_SPANISH.each_with_index do |m, i|
           rgx = Regexp.new("\s#{m}\s", Regexp::IGNORECASE)
-          return string.gsub(rgx, MONTHS_ENGLISH[i]) if string =~ rgx
+          return string.gsub(rgx, MONTHS_ENGLISH[i]) if string&.match?(rgx)
         end
         raise "No translation found for #{string}"
       end

data/lib/feedjira/feed_utilities.rb

@@ -75,11 +75,11 @@ module Feedjira
       old_value = send(name)
       new_value = feed.send(name)
 
-      if old_value != new_value
+      if old_value == new_value
+        false
+      else
         send("#{name}=", new_value)
         true
-      else
-        false
       end
     end
 

data/lib/feedjira/parser/atom_feed_burner.rb

@@ -21,7 +21,7 @@ module Feedjira
       attr_writer :url, :feed_url
 
       def self.able_to_parse?(xml)
-        ((/<feed/ =~ xml) && (/Atom/ =~ xml) && (/feedburner/ =~ xml) && !(/<rss|<rdf/ =~ xml)) || false
+        (xml.include?("<feed") && xml.include?("Atom") && xml.include?("feedburner") && !(/<rss|<rdf/ =~ xml)) || false
       end
 
       # Feed url is <link> with type="text/html" if present,

data/lib/feedjira/parser/atom_google_alerts_entry.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "uri"
+
 module Feedjira
   module Parser
     # Parser for dealing with Feedburner Atom feed entries.

data/lib/feedjira/parser/atom_youtube.rb

@@ -14,8 +14,8 @@ module Feedjira
 
       elements :entry, as: :entries, class: AtomYoutubeEntry
 
-      def self.able_to_parse?(xml) #:nodoc:
-        %r{xmlns:yt="http://www.youtube.com/xml/schemas/2015"} =~ xml
+      def self.able_to_parse?(xml) # :nodoc:
+        xml.include?("xmlns:yt=\"http://www.youtube.com/xml/schemas/2015\"")
       end
     end
   end

data/lib/feedjira/parser/google_docs_atom.rb

@@ -17,7 +17,7 @@ module Feedjira
         @url ||= links.first
       end
 
-      def self.able_to_parse?(xml) #:nodoc:
+      def self.able_to_parse?(xml) # :nodoc:
         %r{<id>https?://docs\.google\.com/.*</id>} =~ xml
       end
 

data/lib/feedjira/parser/itunes_rss_category.rb

@@ -13,7 +13,7 @@ module Feedjira
                                    class: ITunesRSSCategory
 
       def each_subcategory(&block)
-        return to_enum(__method__) unless block_given?
+        return to_enum(__method__) unless block
 
         yield text
 
@@ -23,7 +23,7 @@ module Feedjira
       end
 
       def each_path(ancestors = [], &block)
-        return to_enum(__method__, ancestors) unless block_given?
+        return to_enum(__method__, ancestors) unless block
 
         category_hierarchy = ancestors + [text]
 

data/lib/feedjira/parser/podlove_chapter.rb

@@ -14,9 +14,9 @@ module Feedjira
         return unless start_ntp
 
         parts = start_ntp.split(":")
-        parts.reverse.to_enum.with_index.map do |part, index|
+        parts.reverse.to_enum.with_index.sum do |part, index|
           part.to_f * (60**index)
-        end.reduce(:+)
+        end
       end
     end
   end

data/lib/feedjira/parser/rss.rb

@@ -22,7 +22,7 @@ module Feedjira
       attr_accessor :feed_url
 
       def self.able_to_parse?(xml)
-        (/<rss|<rdf/ =~ xml) && !(/feedburner/ =~ xml)
+        (/<rss|<rdf/ =~ xml) && !xml.include?("feedburner")
       end
     end
   end

data/lib/feedjira/parser/rss_feed_burner.rb

@@ -15,8 +15,8 @@ module Feedjira
 
       attr_accessor :feed_url
 
-      def self.able_to_parse?(xml) #:nodoc:
-        (/<rss|<rdf/ =~ xml) && (/feedburner/ =~ xml)
+      def self.able_to_parse?(xml) # :nodoc:
+        (/<rss|<rdf/ =~ xml) && xml.include?("feedburner")
       end
     end
   end

data/lib/feedjira/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Feedjira
-  VERSION = "3.2.0"
+  VERSION = "3.2.1"
 end

data/lib/feedjira.rb

@@ -65,7 +65,7 @@ module Feedjira
 
     parser.parse(xml, &block)
   end
-  module_function :parse # rubocop:disable Style/AccessModifierDeclarations
+  module_function :parse
 
   # Find compatible parser for given XML
   #
@@ -77,5 +77,5 @@ module Feedjira
     start_of_doc = xml.slice(0, 2000)
     Feedjira.parsers.detect { |klass| klass.able_to_parse?(start_of_doc) }
   end
-  module_function :parser_for_xml # rubocop:disable Style/AccessModifierDeclarations
+  module_function :parser_for_xml
 end

data/spec/feedjira/feed_spec.rb

@@ -5,26 +5,27 @@ require "spec_helper"
 class Hell < StandardError; end
 
 class FailParser
-  def self.parse(_xml, &on_failure)
-    on_failure.call "this parser always fails."
+  def self.parse(_xml)
+    yield "this parser always fails."
+    # on_failure.call
   end
 end
 
 describe Feedjira::Feed do
   describe "#add_common_feed_element" do
     before(:all) do
-      Feedjira::Feed.add_common_feed_element("generator")
+      described_class.add_common_feed_element("generator")
     end
 
-    it "should parse the added element out of Atom feeds" do
+    it "parses the added element out of Atom feeds" do
       expect(Feedjira.parse(sample_wfw_feed).generator).to eq "TypePad"
     end
 
-    it "should parse the added element out of Atom Feedburner feeds" do
+    it "parses the added element out of Atom Feedburner feeds" do
       expect(Feedjira::Parser::Atom.new).to respond_to(:generator)
     end
 
-    it "should parse the added element out of RSS feeds" do
+    it "parses the added element out of RSS feeds" do
       expect(Feedjira::Parser::RSS.new).to respond_to(:generator)
     end
   end
@@ -32,19 +33,19 @@ describe Feedjira::Feed do
   describe "#add_common_feed_entry_element" do
     before(:all) do
       tag = "wfw:commentRss"
-      Feedjira::Feed.add_common_feed_entry_element tag, as: :comment_rss
+      described_class.add_common_feed_entry_element tag, as: :comment_rss
     end
 
-    it "should parse the added element out of Atom feeds entries" do
+    it "parses the added element out of Atom feeds entries" do
       entry = Feedjira.parse(sample_wfw_feed).entries.first
       expect(entry.comment_rss).to eq "this is the new val"
     end
 
-    it "should parse the added element out of Atom Feedburner feeds entries" do
+    it "parses the added element out of Atom Feedburner feeds entries" do
       expect(Feedjira::Parser::AtomEntry.new).to respond_to(:comment_rss)
     end
 
-    it "should parse the added element out of RSS feeds entries" do
+    it "parses the added element out of RSS feeds entries" do
       expect(Feedjira::Parser::RSSEntry.new).to respond_to(:comment_rss)
     end
   end

data/spec/feedjira/{date_time_utilities_spec.rb → feed_utilities_date_time_spec.rb}

@@ -3,44 +3,44 @@
 require "spec_helper"
 
 describe Feedjira::FeedUtilities do
-  before(:each) do
+  before do
     @klass = Class.new do
       include Feedjira::DateTimeUtilities
     end
   end
 
   describe "handling dates" do
-    it "should parse an ISO 8601 formatted datetime into Time" do
+    it "parses an ISO 8601 formatted datetime into Time" do
       time = @klass.new.parse_datetime("2008-02-20T8:05:00-010:00")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Feb 20 18:05:00 UTC 2008")
     end
 
-    it "should parse a ISO 8601 with milliseconds into Time" do
+    it "parses a ISO 8601 with milliseconds into Time" do
       time = @klass.new.parse_datetime("2013-09-17T08:20:13.931-04:00")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Tue Sep 17 12:20:13 UTC 2013")
     end
 
-    it "should parse a US Format into Time" do
+    it "parses a US Format into Time" do
       time = @klass.new.parse_datetime("8/23/2016 12:29:58 PM")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Aug 23 12:29:58 UTC 2016")
     end
 
-    it "should parse a Spanish Format into Time" do
+    it "parses a Spanish Format into Time" do
       time = @klass.new.parse_datetime("Wed, 31 Ago 2016 11:08:22 GMT")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Aug 31 11:08:22 UTC 2016")
     end
 
-    it "should parse Format with japanese symbols into Time" do
+    it "parses Format with japanese symbols into Time" do
       time = @klass.new.parse_datetime("水, 31 8 2016 07:37:00 PDT")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Aug 31 14:37:00 UTC 2016")
     end
 
-    it "should parse epoch into Time" do
+    it "parses epoch into Time" do
       time = @klass.new.parse_datetime("1472654220")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Aug 31 14:37:00 UTC 2016")

data/spec/feedjira/{feed_entry_utilities_spec.rb → feed_utilities_entry_spec.rb}

@@ -3,20 +3,20 @@
 require "spec_helper"
 
 describe Feedjira::FeedUtilities do
-  before(:each) do
+  before do
     @klass = Class.new do
       include Feedjira::FeedEntryUtilities
     end
   end
 
   describe "handling dates" do
-    it "should parse an ISO 8601 formatted datetime into Time" do
+    it "parses an ISO 8601 formatted datetime into Time" do
       time = @klass.new.parse_datetime("2008-02-20T8:05:00-010:00")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Wed Feb 20 18:05:00 UTC 2008")
     end
 
-    it "should parse a ISO 8601 with milliseconds into Time" do
+    it "parses a ISO 8601 with milliseconds into Time" do
       time = @klass.new.parse_datetime("2013-09-17T08:20:13.931-04:00")
       expect(time.class).to eq Time
       expect(time).to eq Time.parse_safely("Tue Sep 17 12:20:13 UTC 2013")
@@ -24,23 +24,23 @@ describe Feedjira::FeedUtilities do
   end
 
   describe "sanitizing" do
-    before(:each) do
+    before do
       @feed = Feedjira.parse(sample_atom_feed)
       @entry = @feed.entries.first
     end
 
     it "doesn't fail when no elements are defined on includer" do
-      expect { @klass.new.sanitize! }.to_not raise_error
+      expect { @klass.new.sanitize! }.not_to raise_error
     end
 
-    it "should provide a sanitized title" do
+    it "provides a sanitized title" do
       new_title = "<script>this is not safe</script>#{@entry.title}"
       @entry.title = new_title
       scrubbed_title = Loofah.scrub_fragment(new_title, :prune).to_s
       expect(@entry.title.sanitize).to eq scrubbed_title
     end
 
-    it "should sanitize content in place" do
+    it "sanitizes content in place" do
       new_content = "<script>#{@entry.content}"
       @entry.content = new_content.dup
 
@@ -50,7 +50,7 @@ describe Feedjira::FeedUtilities do
       expect(@entry.content).to eq scrubbed_content
     end
 
-    it "should sanitize things in place" do
+    it "sanitizes things in place" do
       @entry.title   += "<script>"
       @entry.author  += "<script>"
       @entry.content += "<script>"

data/spec/feedjira/feed_utilities_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 describe Feedjira::FeedUtilities do
-  before(:each) do
+  before do
     @klass = Class.new do
       include SAXMachine
       include Feedjira::FeedUtilities
@@ -14,7 +14,7 @@ describe Feedjira::FeedUtilities do
     context "when the flag is not set" do
       it "does not call the preprocessing method" do
         @klass.preprocess_xml = false
-        expect(@klass).to_not receive :preprocess
+        expect(@klass).not_to receive :preprocess
         @klass.parse sample_rss_feed
       end
     end
@@ -22,14 +22,14 @@ describe Feedjira::FeedUtilities do
     context "when the flag is set" do
       it "calls the preprocessing method" do
         @klass.preprocess_xml = true
-        expect(@klass).to receive(:preprocess).and_return sample_rss_feed
+        allow(@klass).to receive(:preprocess).and_return sample_rss_feed
         @klass.parse sample_rss_feed
       end
     end
   end
 
-  describe "strip whitespace" do
-    context "strip_whitespace config is true" do
+  describe "when configured to strip whitespace" do
+    context "when strip_whitespace config is true" do
       it "strips all XML whitespace" do
         Feedjira.configure { |config| config.strip_whitespace = true }
 
@@ -39,7 +39,7 @@ describe Feedjira::FeedUtilities do
       end
     end
 
-    context "strip_whitespace config is false" do
+    context "when strip_whitespace is configured false" do
       it "lstrips XML whitespace" do
         expect(@klass.strip_whitespace("\nfoobar\n")).to eq("foobar\n")
       end
@@ -47,27 +47,27 @@ describe Feedjira::FeedUtilities do
   end
 
   describe "instance methods" do
-    it "should provide an updated? accessor" do
+    it "provides an updated? accessor" do
       feed = @klass.new
-      expect(feed).to_not be_updated
+      expect(feed).not_to be_updated
       feed.updated = true
       expect(feed).to be_updated
     end
 
-    it "should provide a new_entries accessor" do
+    it "provides a new_entries accessor" do
       feed = @klass.new
       expect(feed.new_entries).to eq []
       feed.new_entries = [:foo]
       expect(feed.new_entries).to eq [:foo]
     end
 
-    it "should provide an etag accessor" do
+    it "provides an etag accessor" do
       feed = @klass.new
       feed.etag = "foo"
       expect(feed.etag).to eq "foo"
     end
 
-    it "should provide a last_modified accessor" do
+    it "provides a last_modified accessor" do
       feed = @klass.new
       time = Time.now
       feed.last_modified = time
@@ -75,13 +75,13 @@ describe Feedjira::FeedUtilities do
       expect(feed.last_modified.class).to eq Time
     end
 
-    it "should return new_entries? as true when entries are put into new_entries" do
+    it "returns new_entries? as true when entries are put into new_entries" do
       feed = @klass.new
       feed.new_entries << :foo
       expect(feed.new_entries?).to eq true
     end
 
-    it "should return a last_modified value from the entry with the most recent published date if the last_modified date hasn't been set" do
+    it "returns a last_modified value from the entry with the most recent published date if the last_modified date hasn't been set" do
       feed = Feedjira::Parser::Atom.new
       entry = Feedjira::Parser::AtomEntry.new
       entry.published = Time.now.to_s
@@ -89,7 +89,7 @@ describe Feedjira::FeedUtilities do
       expect(feed.last_modified).to eq entry.published
     end
 
-    it "should not throw an error if one of the entries has published date of nil" do
+    it "does not throw an error if one of the entries has published date of nil" do
       feed = Feedjira::Parser::Atom.new
       entry = Feedjira::Parser::AtomEntry.new
       entry.published = Time.now.to_s
@@ -101,7 +101,7 @@ describe Feedjira::FeedUtilities do
 
   describe "#update_from_feed" do
     describe "updating feed attributes" do
-      before(:each) do
+      before do
         # I'm using the Atom class when I know I should be using a different
         # one. However, this update_from_feed method would only be called
         # against a feed item.
@@ -113,44 +113,39 @@ describe Feedjira::FeedUtilities do
         @updated_feed = @feed.dup
       end
 
-      it "should update the title if changed" do
+      it "updates the title if changed" do
         @updated_feed.title = "new title"
         @feed.update_from_feed(@updated_feed)
         expect(@feed.title).to eq @updated_feed.title
         expect(@feed).to be_updated
       end
 
-      it "should not update the title if the same" do
+      it "does not update the title if the same" do
         @feed.update_from_feed(@updated_feed)
-        expect(@feed).to_not be_updated
+        expect(@feed).not_to be_updated
       end
 
-      it "should update the feed_url if changed" do
+      it "updates the feed_url if changed" do
         @updated_feed.feed_url = "a new feed url"
         @feed.update_from_feed(@updated_feed)
         expect(@feed.feed_url).to eq @updated_feed.feed_url
         expect(@feed).to be_updated
       end
 
-      it "should not update the feed_url if the same" do
-        @feed.update_from_feed(@updated_feed)
-        expect(@feed).to_not be_updated
-      end
-
-      it "should update the url if changed" do
+      it "updates the url if changed" do
         @updated_feed.url = "a new url"
         @feed.update_from_feed(@updated_feed)
         expect(@feed.url).to eq @updated_feed.url
       end
 
-      it "should not update the url if not changed" do
+      it "does not update the url if not changed" do
         @feed.update_from_feed(@updated_feed)
-        expect(@feed).to_not be_updated
+        expect(@feed).not_to be_updated
       end
     end
 
     describe "updating entries" do
-      before(:each) do
+      before do
         # I'm using the Atom class when I know I should be using a different
         # one. However, this update_from_feed method would only be called
         # against a feed item.
@@ -173,17 +168,17 @@ describe Feedjira::FeedUtilities do
         @updated_feed.entries << @old_entry
       end
 
-      it "should update last-modified from the latest entry date" do
+      it "updates last-modified from the latest entry date" do
         @feed.update_from_feed(@updated_feed)
         expect(@feed.last_modified).to eq @new_entry.published
       end
 
-      it "should put new entries into new_entries" do
+      it "puts new entries into new_entries" do
         @feed.update_from_feed(@updated_feed)
         expect(@feed.new_entries).to eq [@new_entry]
       end
 
-      it "should also put new entries into the entries collection" do
+      it "alsoes put new entries into the entries collection" do
         @feed.update_from_feed(@updated_feed)
         expect(@feed.entries).to include(@new_entry)
         expect(@feed.entries).to include(@old_entry)
@@ -194,7 +189,7 @@ describe Feedjira::FeedUtilities do
       let(:recent_entry_id) { "entry_id" }
       let(:old_entry_id) { nil }
 
-      before(:each) do
+      before do
         # I'm using the Atom class when I know I should be using a different
         # one. However, this update_from_feed method would only be called
         # against a feed item.
@@ -226,26 +221,26 @@ describe Feedjira::FeedUtilities do
         @updated_feed.entries << @old_entry
       end
 
-      context "changing the url of an existing entry" do
-        it "should not put the complete feed into new_entries" do
+      context "when changing the url of an existing entry" do
+        it "does not put the complete feed into new_entries" do
           @feed.update_from_feed(@updated_feed)
-          expect(@feed.new_entries).to_not include(@entry_changed_url)
-          expect(@feed.new_entries).to_not include(@old_entry)
+          expect(@feed.new_entries).not_to include(@entry_changed_url)
+          expect(@feed.new_entries).not_to include(@old_entry)
           expect(@feed.new_entries.size).to eq 0
-          expect(@feed.new_entries.size).to_not eq 2
+          expect(@feed.new_entries.size).not_to eq 2
         end
       end
 
-      context "feed not have entry id and only difference is a url" do
+      context "when feed does not have entry id and only difference is a url" do
         let(:recent_entry_id) { nil }
         let(:old_entry_id) { nil }
 
-        it "should put the complete feed into new_entries" do
+        it "puts the complete feed into new_entries" do
           @feed.update_from_feed(@updated_feed)
           expect(@feed.new_entries).to include(@entry_changed_url)
           expect(@feed.new_entries).to include(@old_entry)
           expect(@feed.new_entries.size).to eq 2
-          expect(@feed.new_entries.size).to_not eq 0
+          expect(@feed.new_entries.size).not_to eq 0
         end
       end
     end