diff options
| author | GeorgeFI | 2023-06-10 10:41:37 +0200 |
|---|---|---|
| committer | GeorgeFI | 2023-06-10 10:41:37 +0200 |
| commit | d17f17b096089b8ea858fdd59c2756880c1ec54f (patch) | |
| tree | 48eb084ed70c9e8d8932ca36107ee377a146ca2d | |
| parent | d7692fdf295db769b407abfd537ea4ac21b086a9 (diff) | |
| download | sec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.tar.gz sec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.tar.zst sec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.zip | |
refactor: Light refactor of fips returns statements
| -rw-r--r-- | src/sec_certs/sample/fips.py | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/sec_certs/sample/fips.py b/src/sec_certs/sample/fips.py index d351c3ce..26510191 100644 --- a/src/sec_certs/sample/fips.py +++ b/src/sec_certs/sample/fips.py @@ -83,15 +83,13 @@ class FIPSHTMLParser: def _build_vendor_dict(vendor_div: Tag) -> dict[str, Any]: if not (link := vendor_div.find("a")): return {"vendor_url": None, "vendor": list(vendor_div.find("div", "panel-body").children)[0].strip()} - else: - return {"vendor_url": link.get("href"), "vendor": link.text.strip()} + return {"vendor_url": link.get("href"), "vendor": link.text.strip()} @staticmethod def _build_related_files_dict(related_files_div: Tag) -> dict[str, Any]: if cert_link := [x for x in related_files_div.find_all("a") if "Certificate" in x.text]: return {"certificate_pdf_url": constants.FIPS_BASE_URL + cert_link[0].get("href")} - else: - return {"certificate_pdf_url": None} + return {"certificate_pdf_url": None} @staticmethod def _build_validation_history_dict(validation_history_div: Tag) -> dict[str, Any]: @@ -433,8 +431,7 @@ class FIPSCertificate( fips_certlike = self.keywords["fips_certlike"].get("Certlike", {}) matches = {re.search(r"#\s{0,1}\d{1,4}", x) for x in fips_certlike} return {"".join([x for x in match.group() if x.isdigit()]) for match in matches if match} - else: - return set() + return set() @dataclass(eq=True) class Heuristics(BaseHeuristics, ComplexSerializableType): |
