Add clone functionality in Importer

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>

Author: TG1999

vulnerabilities/importer.py

@@ -294,6 +294,7 @@ class Importer:
     spdx_license_expression = ""
     license_url = ""
     notice = ""
+    vcs_response = None
 
     def __init__(self):
         if not self.spdx_license_expression:
@@ -319,6 +320,14 @@ def advisory_data(self) -> Iterable[AdvisoryData]:
         """
         raise NotImplementedError
 
+    def clone(self, repo_url):
+        try:
+            self.vcs_response = fetch_via_vcs(repo_url)
+        except Exception as e:
+            msg = f"Failed to fetch {repo_url} via vcs: {e}"
+            logger.error(msg)
+            raise ForkError(msg) from e
+
 
 class ForkError(Exception):
     pass

vulnerabilities/importers/npm.py

@@ -20,7 +20,7 @@
 
 from vulnerabilities.importer import AdvisoryData
 from vulnerabilities.importer import AffectedPackage
-from vulnerabilities.importer import GitImporter
+from vulnerabilities.importer import Importer
 from vulnerabilities.importer import Reference
 from vulnerabilities.importer import VulnerabilitySeverity
 from vulnerabilities.severity_systems import CVSSV2
@@ -29,16 +29,14 @@
 from vulnerabilities.utils import load_json
 
 
-class NpmImporter(GitImporter):
+class NpmImporter(Importer):
     spdx_license_expression = "MIT"
     license_url = "https://github.com/nodejs/security-wg/blob/main/LICENSE.md"
-
-    def __init__(self):
-        super().__init__(repo_url="git+https://github.com/nodejs/security-wg")
+    repo_url = "git+https://github.com/nodejs/security-wg"
 
     def advisory_data(self) -> Iterable[AdvisoryData]:
         try:
-            self.clone()
+            self.clone(self.repo_url)
             path = Path(self.vcs_response.dest_dir)
 
             vuln = path / "vuln"