lideo 1.1.0 → 1.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b75bcd19ae0ea2074a456a66a458d1b305419628dbabb26c6cc4b2dd6324aa6c
-  data.tar.gz: 1022104a165fca7057ef73ceb62410634b234aaa1648d84ae3820534342250a0
+  metadata.gz: 68b144c6a100b8adb3707cc42190c4374f4ad7ceee6be1d1d92a95bec3fa328d
+  data.tar.gz: 38c378b27233565d043ff2e1e1638ea816146cc76c75769528a50c0ccb23ab09
 SHA512:
-  metadata.gz: d5df80edaba57a9e3f69acce1dc3bc7c8ea2e150397e4d6b265bedf71c9bf28fb3938610b7ac738a9dacd997f2cfb2acdae1f99acf49e211d1ea3e5fb9ab9277
-  data.tar.gz: 5b7bf6c7db7bb690ccdbd1bb18a356938d2c5cf964183e3333e7550f9c36e8a50fa8f4f141b37370aee6d857c8b0725af192038178f71e31184c1527b8ddfc96
+  metadata.gz: 5017a86f94e42c78df633950e761c10c0b7ba1007ba82c34a2cec6e421013446fc2958688fbc5c406e534e385a470096eddda491b030371b5162c8f547a1faa0
+  data.tar.gz: 6d95181d90ad47784dfec7601187969b00a82d85dbb5e97ca566d581768628643a97a000e24b371bc261a3ef4ff549cb7c19b6b9511850cf74113e02ec9f8eac

data/lib/cli.rb

@@ -25,6 +25,11 @@ class Cli < Thor
       return
     end
 
+    unless feed_works?(url)
+      puts 'A test fetch failed for the feed and as such it won\'t be added.'
+      return
+    end
+
     LideoController.new.add(url, group(options))
   end
 
@@ -65,6 +70,12 @@ class Cli < Thor
 
   private
 
+  def feed_works?(url)
+    Fetcher.new.fetch(Feed.new(url, ''))
+  rescue
+    false
+  end
+
   def list_feeds
     s = StringIO.new
     LideoController.new.feeds.map { |feed| s << "#{feed.to_s}\n" }

data/lib/fetcher.rb

@@ -5,15 +5,19 @@ require 'rss/atom'
 
 class Fetcher
   def fetch(feed)
-    open(feed.url) do |rss|
-      fetched = RSS::Parser.parse(rss)
-      fetched.items.map do |item|
-        title = fetched.is_a?(RSS::Atom::Feed) ? item.title.content : item.title
-        link = fetched.is_a?(RSS::Atom::Feed) ? item.link.href : item.link
-        channel = fetched.is_a?(RSS::Atom::Feed) ? fetched.author.name.content : fetched.channel.title
+    begin
+      open(feed.url) do |rss|
+        fetched = RSS::Parser.parse(rss)
+        fetched.items.map do |item|
+          title = fetched.is_a?(RSS::Atom::Feed) ? item.title.content : item.title
+          link = fetched.is_a?(RSS::Atom::Feed) ? item.link.href : item.link
+          channel = fetched.is_a?(RSS::Atom::Feed) ? fetched.author.name.content : fetched.channel.title
 
-        Headline.new(title, link, channel)
+          Headline.new(title, link, channel)
+        end
       end
+    rescue
+      puts "An error occurred while fetching #{feed.url}"
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lideo
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.5
 platform: ruby
 authors:
 - Gyowanny Queiroz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-12 00:00:00.000000000 Z
+date: 2020-03-13 00:00:00.000000000 Z
 dependencies: []
 description: A simple RSS aggregator CLI where you can read the headlines from your
   terminal