Remove ability to ignore coverage violations (#86597)

* Remove ability to ignore coverage violations

* Adjust codecov for required platforms

* Update codecov.yml
This commit is contained in:
epenet
2023-01-25 14:27:16 +01:00
committed by GitHub
parent 0ccab19d2c
commit b2004e62b1
3 changed files with 28 additions and 94 deletions

View File

@@ -18,53 +18,6 @@ DONT_IGNORE = (
"scene.py",
)
# They were violating when we introduced this check
# Need to be fixed in a future PR.
ALLOWED_IGNORE_VIOLATIONS = {
("advantage_air", "diagnostics.py"),
("androidtv", "diagnostics.py"),
("asuswrt", "diagnostics.py"),
("aussie_broadband", "diagnostics.py"),
("doorbird", "logbook.py"),
("ecowitt", "diagnostics.py"),
("elkm1", "scene.py"),
("fibaro", "scene.py"),
("hunterdouglas_powerview", "diagnostics.py"),
("hunterdouglas_powerview", "scene.py"),
("jellyfin", "media_source.py"),
("launch_library", "diagnostics.py"),
("lcn", "scene.py"),
("lifx_cloud", "scene.py"),
("lutron", "scene.py"),
("lutron_caseta", "scene.py"),
("nanoleaf", "diagnostics.py"),
("nanoleaf", "device_trigger.py"),
("nut", "diagnostics.py"),
("open_meteo", "diagnostics.py"),
("overkiz", "diagnostics.py"),
("overkiz", "scene.py"),
("philips_js", "diagnostics.py"),
("radio_browser", "media_source.py"),
("rfxtrx", "diagnostics.py"),
("screenlogic", "diagnostics.py"),
("sonos", "diagnostics.py"),
("stookalert", "diagnostics.py"),
("stookwijzer", "diagnostics.py"),
("synology_dsm", "diagnostics.py"),
("system_bridge", "media_source.py"),
("tractive", "diagnostics.py"),
("tuya", "diagnostics.py"),
("tuya", "scene.py"),
("upb", "scene.py"),
("velbus", "diagnostics.py"),
("velux", "scene.py"),
("verisure", "diagnostics.py"),
("vicare", "diagnostics.py"),
("xbox", "media_source.py"),
("xiaomi_miio", "diagnostics.py"),
("yale_smart_alarm", "diagnostics.py"),
}
def validate(integrations: dict[str, Integration], config: Config) -> None:
"""Validate coverage."""
@@ -120,9 +73,6 @@ def validate(integrations: dict[str, Integration], config: Config) -> None:
if path.parts[-1] not in {"*", check}:
continue
if (integration_path.name, check) in ALLOWED_IGNORE_VIOLATIONS:
continue
if (integration_path / check).exists():
integration.add_error(
"coverage",