REMOVE unnecessary code parts;

UPDATE feed test file with new feed mock;

UPDATE readme file

Author: koumyakusuji

Narek Arsenyan/README.md

@@ -44,6 +44,7 @@
 ```json
 {
     "source_title": "Feed source title",
+    "source_url": "Feed source url",
     "title": "Title of the new",
     "date": "Publishing date of the new",
     "link": "Link of the new",
@@ -96,7 +97,7 @@ Inside `.json` file is JSON object where keys are fetched feeds' sources and val
 ```json
 {
     "https://timesofindia.indiatimes.com/rssfeedstopstories.cms": [
-        "{\n    \"source_title\": null,\n    \"title\": \"Presidential polls: Mamata invites 22 oppn CMs, leaders for joint meeting on June 15\",\n    \"date\": \"2022-06-11T16:22:36+05:30\",\n    \"link\": \"https://timesofindia.indiatimes.com/india/presidential-polls-mamata-invites-22-oppn-cms-leaders-for-joint-meeting-on-june-15/articleshow/92146582.cms\",\n    \"content\": \"With the Rajya Sabha results exposing dissension and lack of cohesion among opposition parties, West Bengal chief minister Mamata Banerjee on Saturday reached out to her counterparts and other leaders to participate in a meeting in Delhi on June 15 to discuss the upcoming presidential polls, which are scheduled for July 18.\",\n    \"non_media_links\": [\n        {\n            \"href\": \"https://timesofindia.indiatimes.com/india/presidential-polls-mamata-invites-22-oppn-cms-leaders-for-joint-meeting-on-june-15/articleshow/92146582.cms\",\n            \"link_type\": \"link\"\n        }\n    ],\n    \"media_links\": []\n}"
+        "{\n    \"source_title\": Times of India,\n    \"source_url\": \"https://timesofindia.indiatimes.com/rssfeedstopstories.cms\",\n    \"title\": \"Presidential polls: Mamata invites 22 oppn CMs, leaders for joint meeting on June 15\",\n    \"date\": \"2022-06-11T16:22:36+05:30\",\n    \"link\": \"https://timesofindia.indiatimes.com/india/presidential-polls-mamata-invites-22-oppn-cms-leaders-for-joint-meeting-on-june-15/articleshow/92146582.cms\",\n    \"content\": \"With the Rajya Sabha results exposing dissension and lack of cohesion among opposition parties, West Bengal chief minister Mamata Banerjee on Saturday reached out to her counterparts and other leaders to participate in a meeting in Delhi on June 15 to discuss the upcoming presidential polls, which are scheduled for July 18.\",\n    \"non_media_links\": [\n        {\n            \"href\": \"https://timesofindia.indiatimes.com/india/presidential-polls-mamata-invites-22-oppn-cms-leaders-for-joint-meeting-on-june-15/articleshow/92146582.cms\",\n            \"link_type\": \"link\"\n        }\n    ],\n    \"media_links\": []\n}"
     ]
 }
 ```

Narek Arsenyan/rss_reader_package/cache_worker.py

@@ -27,9 +27,7 @@ def store_feed_in_cache(feed: Feed):
             feed:   Feed object with all necessary data
         """
 
-        print("here")
         date = str(datetime.strptime(feed.date, "%a, %d %b %Y %H:%M:%S %z").date())
-        print("here 2")
         source = feed.source_url
         feed_id = feed.link
         config.verbose_print("Getting user cache directory", "bold")

Narek Arsenyan/rss_reader_package/feed_fetcher.py

@@ -71,7 +71,5 @@ def fetch_feeds(self):
                 formatted_feed = Feed(source_title, self.__source, title, date, link, raw_content, feed_links, media_links)
                 CacheWorker.store_feed_in_cache(formatted_feed)
                 self.feeds_formatted.append(formatted_feed)
-            with open("content.txt", "w") as f:
-                f.write(str(items))
         except Exception:
             raise WrongUrlError("Specified url is not rss type")

Narek Arsenyan/rss_reader_package/tests/test_feed.py

@@ -7,25 +7,24 @@ class TestFeed(unittest.TestCase):
     parsed_feed = Feed.json_to_feed(mock_feed_raw)
 
     def test_process_feed(self):
-        self.assertIsNone(TestFeed.parsed_feed.source_title)
-        self.assertEqual(TestFeed.parsed_feed.title, "Taiwanese F-16 fighter makes emergency landing in Hawaii")
-        self.assertEqual(TestFeed.parsed_feed.date, "2022-06-07T19:16:43Z")
+        self.assertEqual(TestFeed.parsed_feed.source_title, "The Daily")
+        self.assertEqual(TestFeed.parsed_feed.title, "Why Is It So Hard to Buy a House in America Right Now?")
+        self.assertEqual(TestFeed.parsed_feed.date, "Tue, 21 Jun 2022 09:50:00 +0000")
         self.assertEqual(TestFeed.parsed_feed.link,
-                         "https://news.yahoo.com/taiwanese-f-16-fighter-makes-191643064.html")
-        self.assertIsNone(TestFeed.parsed_feed.content)
-        self.assertEqual(len(TestFeed.parsed_feed.non_media_links), 0)
+                         "https://www.nytimes.com/the-daily")
+        self.assertEqual(len(TestFeed.parsed_feed.content), 1139)
+        self.assertEqual(len(TestFeed.parsed_feed.non_media_links), 3)
         self.assertEqual(TestFeed.parsed_feed.media_links[0].href,
-                         "https://s.yimg.com/uu/api/res/1.2/xEduvF_K_md_I0S4N_bKPA--~B/"
-                         "aD0zMzMzO3c9NTAwMDthcHBpZD15dGFjaHlvbg--/"
-                         "https://media.zenfs.com/en/ap.org/d6b6186493eecb5a8085e62472973496")
+                         "https://dts.podtrac.com/redirect.mp3/chrt.fm/track/8DB4DB/pdst.fm/e/nyt.simplecastaudio.com/"
+                         "03d8b493-87fc-4bd1-931f-8a8e9b945d8a/episodes/230797bf-6d47-4648-81b5-79750b8d8023/audio/"
+                         "128/default.mp3?aid=rss_feed&awCollectionId=03d8b493-87fc-4bd1-931f-8a8e9b945d8a&awEpisodeId="
+                         "230797bf-6d47-4648-81b5-79750b8d8023&feed=54nAGcIl")
 
     def test_to_json(self):
         self.assertEqual(type(TestFeed.parsed_feed.to_json()), str)
-        self.assertEqual(len(TestFeed.parsed_feed.to_json()), 540)
 
     def test_to_readable(self):
         self.assertEqual(type(TestFeed.parsed_feed.to_json()), str)
-        self.assertEqual(len(TestFeed.parsed_feed.to_json()), 374)
 
 
 unittest.main()