Add requests to requirements.txt, change _process_file to process file in rubyDataSource. Also avoid calling process_file twice

Signed-off-by: Shivam Sandbhor <shivam.sandbhor@gmail.com>

Author: sbs2001

requirements.txt

@@ -51,3 +51,4 @@ traitlets==4.3.3
 wcwidth==0.1.7
 whitenoise==5.0.1
 zipp==0.6.0
+requests==2.23.0

vulnerabilities/importers/ruby.py

@@ -25,11 +25,12 @@ def updated_advisories(self) -> Set[Advisory]:
         files = self._updated_files.union(self._added_files)
         advisories = []
         for f in files:
-            if self._process_file(f):
-                advisories.append(self._process_file(f))
+            processed_data = self.process_file(f)
+            if processed_data:
+                advisories.append(processed_data)
         return self.batch_advisories(advisories)
 
-    def _process_file(self, path) -> List[Advisory]:
+    def process_file(self, path) -> List[Advisory]:
         with open(path) as f:
             record = yaml.safe_load(f)
             package_name = record.get(

vulnerabilities/tests/test_ruby.py

@@ -97,7 +97,7 @@ def test_process_file(self, mock_write):
         found_advisories = set()
 
         for p in MOCK_ADDED_FILES:
-            found_advisories.add(self.data_src._process_file(p))
+            found_advisories.add(self.data_src.process_file(p))
         assert found_advisories == expected_advisories
 
     def test_categorize_versions(self):