sentry-devenv 1.26.0__py3-none-any.whl → 1.27.0__py3-none-any.whl

devenv/bootstrap.py

@@ -54,9 +54,15 @@ def main(context: Context, argv: Sequence[str] | None = None) -> ExitCode:
         except RuntimeError:
             return "Failed to find git. Run xcode-select --install, then re-run bootstrap when done."
 
+    if constants.LINUX and not (
+        shutil.which("docker") and shutil.which("dockerd")
+    ):
+        raise SystemExit("docker engine not installed; required on linux")
+
     # even though this is called before colima starts,
     # better to try and potentially (although unlikely) fail earlier rather than later
-    rosetta.ensure()
+    if constants.DARWIN:
+        rosetta.ensure()
 
     github.add_to_known_hosts()
 
@@ -103,11 +109,15 @@ Updating global tools (at {constants.root}/bin).
 """
     )
     os.makedirs(f"{constants.root}/bin", exist_ok=True)
-    brew.install()
-    docker.install_global()
+
+    if constants.DARWIN:
+        # we only install brew and colima-related stuff on macos
+        brew.install()
+        docker.install_global()
+        colima.install_global()
+        limactl.install_global()
+
     direnv.install()
-    colima.install_global()
-    limactl.install_global()
 
     os.makedirs(context["code_root"], exist_ok=True)
 

devenv/checks/colimaSsh.py

@@ -8,7 +8,7 @@ from devenv.lib import proc
 from devenv.lib_check.types import checker
 from devenv.lib_check.types import fixer
 
-tags: set[str] = {"builtin"}
+tags: set[str] = {"builtin", "colima"}
 name = "colima ssh credentials should only be owner rw"
 
 

devenv/checks/diskfree.py

@@ -14,11 +14,11 @@ def check() -> tuple[bool, str]:
     disk_total, disk_used, disk_free = shutil.disk_usage("/")
     disk_gib_free = disk_free / (1024**3)
 
-    if disk_gib_free < 10000:
+    if disk_gib_free < 10:
         return (
             False,
             f"You have less than 10 GiB disk free ({disk_gib_free} GiB free). "
-            "You might start to encounter various problems when using colima.",
+            "You might start to encounter various problems when using docker.",
         )
 
     return True, ""

devenv/checks/dockerConfig.py

@@ -7,8 +7,8 @@ import shutil
 from devenv.lib_check.types import checker
 from devenv.lib_check.types import fixer
 
-tags: set[str] = {"builtin"}
-name = "correct docker configuration"
+tags: set[str] = {"builtin", "colima"}
+name = "correct docker configuration for colima"
 
 
 @checker

devenv/checks/dockerDesktop.py

@@ -6,8 +6,8 @@ from devenv.lib import proc
 from devenv.lib_check.types import checker
 from devenv.lib_check.types import fixer
 
-tags: set[str] = {"builtin"}
-name = "docker desktop shouldn't be running"
+tags: set[str] = {"builtin", "colima"}
+name = "docker desktop shouldn't be running if you're trying to use colima"
 
 
 def docker_desktop_is_running() -> bool:
@@ -20,7 +20,7 @@ def check() -> tuple[bool, str]:
     if docker_desktop_is_running():
         return (
             False,
-            "Docker Desktop is running. We don't support it, and it conflicts with colima.",
+            "Docker Desktop is running. It cannot be running at the same time as colima.",
         )
 
     return True, ""

devenv/checks/limaDns.py

@@ -7,7 +7,7 @@ from devenv.lib import proc
 from devenv.lib_check.types import checker
 from devenv.lib_check.types import fixer
 
-tags: set[str] = {"builtin"}
+tags: set[str] = {"builtin", "colima"}
 name = "colima's DNS isn't working"
 
 

devenv/constants.py

@@ -4,6 +4,7 @@ import importlib.metadata
 import os
 import platform
 import pwd
+import shutil
 import typing
 
 troubleshooting_help = """\
@@ -27,6 +28,8 @@ CI = os.getenv("CI")
 SYSTEM = platform.system().lower()
 MACHINE = platform.machine()
 DARWIN = SYSTEM == "darwin"
+# we only support apt-based linuxes
+LINUX = shutil.which("dpkg") is not None
 INTEL_MAC = DARWIN and (MACHINE == "x86_64")
 SHELL_UNSET = "(SHELL unset)"
 DEBUG = os.getenv("SNTY_DEVENV_DEBUG", os.getenv("DEBUG", ""))

devenv/doctor.py

@@ -14,6 +14,7 @@ from typing import Dict
 from typing import List
 
 import devenv.checks
+from devenv import constants
 from devenv.lib.context import Context
 from devenv.lib.modules import DevModuleInfo
 from devenv.lib.repository import Repository
@@ -78,7 +79,9 @@ class Check:
         super().__init__()
 
 
-def load_checks(repo: Repository, match_tags: set[str]) -> List[Check]:
+def load_checks(
+    repo: Repository, match_tags: set[str], exclude_tags: set[str]
+) -> List[Check]:
     """
     Load all checks from the checks directory.
     Optionally filter by tags.
@@ -104,11 +107,15 @@ def load_checks(repo: Repository, match_tags: set[str]) -> List[Check]:
             continue
         if match_tags and not check.tags.issuperset(match_tags):
             continue
+        if check.tags & exclude_tags:
+            continue
         checks.append(check)
     return checks
 
 
-def load_builtin_checks(match_tags: set[str]) -> List[Check]:
+def load_builtin_checks(
+    match_tags: set[str], exclude_tags: set[str]
+) -> List[Check]:
     """
     Loads builtin checks.
     Optionally filter by tags.
@@ -126,6 +133,8 @@ def load_builtin_checks(match_tags: set[str]) -> List[Check]:
             continue
         if match_tags and not check.tags.issuperset(match_tags):
             continue
+        if check.tags & exclude_tags:
+            continue
         checks.append(check)
     return checks
 
@@ -193,23 +202,33 @@ def main(context: Context, argv: Sequence[str] | None = None) -> int:
         action="append",
         help="Used to match a subset of checks.",
     )
+    parser.add_argument(
+        "--exclude-tag",
+        type=str,
+        action="append",
+        help="Used to unmatch checks.",
+    )
     parser.add_argument(
         "--check-only", action="store_true", help="Do not run fixers."
     )
     args = parser.parse_args(argv)
 
     match_tags: set[str] = set(args.tag if args.tag else ())
+    exclude_tags: set[str] = set(args.exclude_tag if args.exclude_tag else ())
+
+    if not constants.DARWIN:
+        exclude_tags.add("colima")
 
     # First, we load builtin checks. These are not repo specific.
-    checks = load_builtin_checks(match_tags)
+    checks = load_builtin_checks(match_tags, exclude_tags)
 
     # Then we load any repo specific checks if any.
     repo = context.get("repo")
     if repo is not None:
-        checks.extend(load_checks(repo, match_tags))
+        checks.extend(load_checks(repo, match_tags, exclude_tags))
 
     if not checks:
-        print(f"No checks found for tags: {args.tag}")
+        print("No checks found.")
         return 1
 
     # We run every check on a separate thread, aggregate the results,

devenv/fetch.py

@@ -6,10 +6,7 @@ import os
 import sys
 from collections.abc import Sequence
 
-from devenv.constants import CI
-from devenv.constants import DARWIN
-from devenv.constants import EXTERNAL_CONTRIBUTOR
-from devenv.constants import homebrew_bin
+from devenv import constants
 from devenv.lib import proc
 from devenv.lib.context import Context
 from devenv.lib.modules import DevModuleInfo
@@ -35,33 +32,16 @@ def main(context: Context, argv: Sequence[str] | None = None) -> ExitCode:
         "getsentry/sentry",
         "getsentry/getsentry",
     ]:
-        fetch(code_root, "getsentry/sentry", auth=CI is None, sync=False)
-
-        if DARWIN:
-            print("Installing sentry's brew dependencies...")
-            if CI:
-                # Installing everything from brew takes too much time,
-                # and chromedriver cask flakes occasionally. Really all we need to
-                # set up the devenv is colima and docker-cli.
-                # This is also required for arm64 macOS GHA runners.
-                # We manage colima, so just need to install docker + qemu here.
-                proc.run(("brew", "install", "docker", "qemu"))
-            else:
-                proc.run(
-                    (f"{homebrew_bin}/brew", "bundle"),
-                    cwd=f"{code_root}/sentry",
-                )
-        else:
-            print(
-                "Not on MacOS; assuming you have a docker cli and runtime installed."
-            )
+        fetch(
+            code_root, "getsentry/sentry", auth=constants.CI is None, sync=False
+        )
 
         proc.run(
             (sys.executable, "-P", "-m", "devenv", "sync"),
             cwd=f"{code_root}/sentry",
         )
 
-        if not CI and not EXTERNAL_CONTRIBUTOR:
+        if not constants.CI and not constants.EXTERNAL_CONTRIBUTOR:
             fetch(code_root, "getsentry/getsentry")
 
         print(
@@ -88,25 +68,24 @@ def fetch(
 
     if os.path.exists(reporoot):
         print(f"{reporoot} already exists")
-        return
-
-    print(f"fetching {repo} into {reporoot}")
-
-    additional_args = (
-        # git@ clones forces the use of cloning through SSH which is what we want,
-        # though CI must clone open source repos via https (no git authentication)
-        (f"git@github.com:{repo}",)
-        if auth
-        else (
-            "--depth",
-            "1",
-            "--single-branch",
-            f"--branch={os.environ['DEVENV_FETCH_BRANCH']}",
-            f"https://github.com/{repo}",
+    else:
+        print(f"fetching {repo} into {reporoot}")
+
+        additional_args = (
+            # git@ clones forces the use of cloning through SSH which is what we want,
+            # though CI must clone open source repos via https (no git authentication)
+            (f"git@github.com:{repo}",)
+            if auth
+            else (
+                "--depth",
+                "1",
+                "--single-branch",
+                f"--branch={os.environ['DEVENV_FETCH_BRANCH']}",
+                f"https://github.com/{repo}",
+            )
         )
-    )
 
-    proc.run(("git", "-C", coderoot, "clone", *additional_args), exit=True)
+        proc.run(("git", "-C", coderoot, "clone", *additional_args), exit=True)
 
     context_post_fetch = {
         "reporoot": reporoot,
@@ -117,6 +96,7 @@ def fetch(
     # optional post-fetch, meant for recommended but not required defaults
     fp = f"{reporoot}/devenv/post_fetch.py"
     if os.path.exists(fp):
+        print(f"running {fp}")
         spec = importlib.util.spec_from_file_location("post_fetch", fp)
 
         module = importlib.util.module_from_spec(spec)  # type: ignore
@@ -127,6 +107,7 @@ def fetch(
             print(f"warning! failed running {fp} (code {rc})")
 
     if sync:
+        print("running devenv sync")
         proc.run((sys.executable, "-P", "-m", "devenv", "sync"), cwd=reporoot)
 
 

devenv/lib/colima.py

@@ -63,8 +63,6 @@ def install_global() -> None:
         "darwin_x86_64_sha256": "791330c62c60389f70e5e1c33a56c35502a9e36e544a418daea0273e539acbf4",
         "darwin_arm64": f"https://github.com/abiosoft/colima/releases/download/{version}/colima-Darwin-arm64",
         "darwin_arm64_sha256": "c266fcb272b39221ef6152d2093bb02a1ebadc26042233ad359e1ae52d5d5922",
-        "linux_x86_64": f"https://github.com/abiosoft/colima/releases/download/{version}/colima-Linux-x86_64",
-        "linux_x86_64_sha256": "f2d6664a79ff3aa35f0718aac2ba9f6b531772e1464f3b096c1ac2aab404943e",
     }
 
     binroot = f"{root}/bin"

devenv/lib/docker.py

@@ -113,8 +113,6 @@ def install_global() -> None:
         "darwin_x86_64_sha256": "1b621d4c9a57ff361811cf29754aafb0c28bc113c70011927af8d73c2c162186",
         "darwin_arm64": f"https://download.docker.com/mac/static/stable/aarch64/docker-{version}.tgz",
         "darwin_arm64_sha256": "9dae125282116146b06eb777c2125ddda6c0468c0b9ad6c72a82edbc6783a77b",
-        "linux_x86_64": f"https://download.docker.com/linux/static/stable/x86_64/docker-{version}.tgz",
-        "linux_x86_64_sha256": "9b4f6fe406e50f9085ee474c451e2bb5adb119a03591f467922d3b4e2ddf31d3",
     }
 
     version_buildx = "v0.22.0"
@@ -123,8 +121,6 @@ def install_global() -> None:
         "darwin_x86_64_sha256": "5221ad6b8acd2283f8fbbeebc79ae4b657e83519ca1c1e4cfbb9405230b3d933",
         "darwin_arm64": f"https://github.com/docker/buildx/releases/download/{version_buildx}/buildx-{version_buildx}.darwin-arm64",
         "darwin_arm64_sha256": "5898c338abb1f673107bc087997dc3cb63b4ea66d304ce4223472f57bd8d616e",
-        "linux_x86_64": f"https://github.com/docker/buildx/releases/download/{version_buildx}/buildx-{version_buildx}.linux-amd64",
-        "linux_x86_64_sha256": "805195386fba0cea5a1487cf0d47da82a145ea0a792bd3fb477583e2dbcdcc2f",
     }
 
     binroot = f"{root}/bin"

devenv/lib/limactl.py

@@ -37,10 +37,7 @@ def _install(url: str, sha256: str, into: str) -> None:
         archive_file = archive.download(url, sha256, dest=f"{tmpd}/download")
 
         # the archive from homebrew has a lima/version prefix
-        if url.startswith("https://ghcr.io/v2/homebrew"):
-            archive.unpack_strip_n(archive_file, tmpd, n=2)
-        else:
-            archive.unpack(archive_file, tmpd)
+        archive.unpack_strip_n(archive_file, tmpd, n=2)
 
         # the archive was atomically placed into tmpd so
         # these are on the same fs and can be atomically moved too
@@ -75,10 +72,6 @@ def install_global() -> None:
         # arm64_sonoma
         "darwin_arm64": "https://ghcr.io/v2/homebrew/core/lima/blobs/sha256:8aeb0a3b7295f0c3e0c2a7a92a798a44397936e5bb732db825aee6da5e762d7a",
         "darwin_arm64_sha256": "8aeb0a3b7295f0c3e0c2a7a92a798a44397936e5bb732db825aee6da5e762d7a",
-        # on linux we use github releases since most people are probably not using
-        # linuxbrew and the go binary in homebrew links to linuxbrew's ld.so
-        "linux_x86_64": f"https://github.com/lima-vm/lima/releases/download/v{version}/lima-{version}-Linux-x86_64.tar.gz",
-        "linux_x86_64_sha256": "b109cac29569a4aacab01c588f922ea6c7e2ef06ce9260bbc4c382e475bc3b98",
     }
 
     binroot = f"{root}/bin"

devenv/update.py

@@ -36,11 +36,16 @@ Updating global tools (at {constants.root}/bin).
 """
         )
         os.makedirs(f"{constants.root}/bin", exist_ok=True)
-        brew.install()
-        docker.install_global()
+
+        if constants.DARWIN:
+            # we only install brew and colima-related stuff on macos
+            brew.install()
+            docker.install_global()
+            colima.install_global()
+            limactl.install_global()
+
         direnv.install()
-        colima.install_global()
-        limactl.install_global()
+
         return 0
 
     is_global_devenv = sys.executable.startswith(

{sentry_devenv-1.26.0.dist-info → sentry_devenv-1.27.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sentry_devenv
-Version: 1.26.0
+Version: 1.27.0
 Summary: Utilities for setting up a Sentry development environment
 Author-email: Joshua Li <joshua.li@sentry.io>, Ian Woodard <ian.woodard@sentry.io>, Buck Evan <buck.evan@sentry.io>
 Classifier: Programming Language :: Python :: 3
@@ -98,7 +98,7 @@ Everything devenv needs is in `~/.local/share/sentry-devenv`.
     - `direnv`
       - we currently rely on direnv and a minimal [`[reporoot]/.envrc`](#direnv) to add `[reporoot]/.devenv/bin` to PATH
       - see [examples](#examples) for .envrc suggestions
-    - global tools: `docker` (cli), `colima`
+    - global tools (macos only; you are otherwise expected to install docker yourself): `docker` (cli), `colima`
 
 
 ### runtime

{sentry_devenv-1.26.0.dist-info → sentry_devenv-1.27.0.dist-info}/RECORD

@@ -1,35 +1,35 @@
 ci/integration/repo/devenv/sync.py,sha256=mZhOcSeA10BQ5DkDBoSbpjc2rHeRVYEZAZ6zse1bo-Y,873
 devenv/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 devenv/__main__.py,sha256=O8ROZOx3emX7hhzMD608Ud_Pny_PCy5RsWXwBLhf8Q8,122
-devenv/bootstrap.py,sha256=VSlS1giUv8fE0K5pZgIO-QBKg5PsaWxirF6G9OPUUhE,4047
+devenv/bootstrap.py,sha256=pFDNhMFrD3keYYsELNVH_VSFF169NtzUDsyle7PHFJw,4359
 devenv/colima.py,sha256=Dk9t6eu2DEoqqkY8xLmgiB96r-6zi3aXcvON3P8h1JQ,1537
-devenv/constants.py,sha256=wl0sRfkNeJL-sML6MG5nS0zU-dwRo1mDPrhUSHHY1YQ,1498
-devenv/doctor.py,sha256=Qqytk_oYtyQTPqVvRCPHC3c-4_yDOuOqtO2AnmRG8-Y,8728
-devenv/fetch.py,sha256=adDMbqQF19Q_MmunQMMy6n3rBGpUh3ju1Thrt-5OA4I,4078
+devenv/constants.py,sha256=T9M0TulSc4BWBAtBunFNL9c3vsMPKscc7Naw6V3ERNQ,1589
+devenv/doctor.py,sha256=oZtTGSHDUhuibf0FUUlqMnCInowHBF5jn5uKWpcxZE4,9225
+devenv/fetch.py,sha256=-jFqlsM3I3h2krqacapYMjYdcioCvAjuo34X73OPtNY,3293
 devenv/main.py,sha256=grOHjy0kUJjeGW8STg-WMneVYlkTRVMoFjNdIgt7sjg,3267
 devenv/pin_gha.py,sha256=t7A5CV1bnYRa4SAikuoJeodHuwV6moIePvhQ1Zdc1eE,2187
 devenv/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 devenv/pythons.py,sha256=ZcjBrkfmQEkiMP217VyTG48cyteJdYG_hoQ6oFKx_CU,1252
 devenv/sync.py,sha256=B9bvAuI7K-4SqcfHD4_zwKpozAnHWCNOL-IhVOcc3Zg,1254
-devenv/update.py,sha256=hkapWwk5L_sHvGSLoXqs0JVRjNfHN80xf5o8W61tOjs,2710
+devenv/update.py,sha256=gPNiHo8YNJbTaoWX-3qNj0-c1nRQXIQEB9WneSCF4Bw,2827
 devenv/checks/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-devenv/checks/colimaSsh.py,sha256=FCF2y-JrCWWyKVDLDddwuOA1J4NQnKIirxBs8EIbRGk,1363
-devenv/checks/diskfree.py,sha256=JGVUqmEtgc75ZFUXjcJBO7MO_PxIKg0o9SMcmqm8CoU,861
-devenv/checks/dockerConfig.py,sha256=l1xhsN0z3my27_DseXyBq_iVcGSgWKHi6suYOyBoWvo,1929
-devenv/checks/dockerDesktop.py,sha256=ynt8nNQ2DDT2rJDGZ-g7xxc4HErHJV_iqBqYHQtvA7Q,1343
-devenv/checks/limaDns.py,sha256=cHef2czdYYLvQqiM4bxiYtzYw4UVTIXJPwrGW2Kjpxo,1885
+devenv/checks/colimaSsh.py,sha256=NJQ38F32lyAwt7miqP6ZhAKDjmpHjr5s9JYYibeNxmI,1373
+devenv/checks/diskfree.py,sha256=JYQW3WbFqVyQe4AMt53k0w6_2Pj6rk8uu8lFO07CG_E,858
+devenv/checks/dockerConfig.py,sha256=4Lgamx7gZz4LFkdv1gOjTyH6DK6wguG5mitX6-_AyPU,1950
+devenv/checks/dockerDesktop.py,sha256=UVPQ4uC5W4Cm_nCIcjILwpx1NAzu_a0bneIgeMSILt8,1382
+devenv/checks/limaDns.py,sha256=me7yq_qNfpOQIBZaLKar6VnWzUrL7MA94kBhe9hgIK0,1895
 devenv/lib/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 devenv/lib/archive.py,sha256=75MjlDuwHVaazbNUr8vJmqTyl4Jz6KRd_fUcpwvPwec,4710
 devenv/lib/brew.py,sha256=_7recvYh-LnUJC0eKnYK7F8AzGRL5bi_Khv0f_mgrlM,1795
-devenv/lib/colima.py,sha256=wsKC0B8zm1s7R4ZicqwH1mT7yd1Fzk26pSewY8Esvtk,7859
+devenv/lib/colima.py,sha256=G_vVlWg-8nonLtv8S9ymxkV1MUftwlTkGMIRShGX8cA,7649
 devenv/lib/config.py,sha256=5eXYwM44AJ3w2VfVbCDqchYOL1zAjazNcirtM6kxxGw,3344
 devenv/lib/context.py,sha256=4EDImK9pvTp3FqeNigevC9lbJsgdDO9D1t2_Y4NyPlc,207
 devenv/lib/direnv.py,sha256=HDRUaIR0PMAQYiiTaw3gEQuh6Tdb7BUJmV7dBZdPztM,1325
-devenv/lib/docker.py,sha256=d-jGdagfvxGpZtfoEG_m6IMcm8YvUEjJoHx0ACKNWKk,5688
+devenv/lib/docker.py,sha256=z9ifo-ZEp58XbsnQxaMe320gZl56drMe9s0hlYsD8HY,5254
 devenv/lib/fs.py,sha256=IBahQ9fB-o9cK9HipK_L0gso2OVinIDTxFWoeMte9ww,2764
 devenv/lib/gcloud.py,sha256=pJ28_L0gUS1lZna-gIHeJPo9bEfSHVXh63CoRQGyAw8,3552
 devenv/lib/github.py,sha256=AMtcz6gTVKy7qvCP1zTUdHQlnSM32TVaji0aOXJ_F9A,3463
-devenv/lib/limactl.py,sha256=wMA_Q4DRPjmBPe0JSj1xPwYesFa9MT_vVSdaBAAKn5w,4256
+devenv/lib/limactl.py,sha256=LWwJEEW0XLGcN7DJMrXX5gDEzoZbcEci-pqZW5F_mV8,3750
 devenv/lib/modules.py,sha256=shoqgja0b-xzXr601Ryz0JOy3Vxbw53I3cRYezVt77g,859
 devenv/lib/node.py,sha256=Rz5A5tep3YEfTfyVnrcAZEZCD6-rmdG4MdgpwRC033E,4119
 devenv/lib/proc.py,sha256=eU7DHIk_66wgkHRmdtKbBKZTa6q4tl9QJbrlnQOQCls,2743
@@ -52,7 +52,7 @@ tests/checks/test_dockerConfig.py,sha256=R9ed7DfYcCn37__98y3itCVG37BCDbJa0gLA9fQ
 tests/doctor/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/doctor/test_attempt_fix.py,sha256=cGUv7-xRKWCizC9cNNqiCp7aa95k3nHUXvRFFuLfJ34,1075
 tests/doctor/test_filter_failing_checks.py,sha256=5L5XdBSyeySwZtJeAR71lbktt3NxkJdUfncKBJPYdAc,1270
-tests/doctor/test_load_checks.py,sha256=C0uIdjftQZokZNYf003pNFfWuT9Gk1C7ALZzN_JnsWQ,2714
+tests/doctor/test_load_checks.py,sha256=xgBTfQP4D7fif5TI7C7DRPgbXhVCWvJUz7eRtKqz_cw,3691
 tests/doctor/test_prompt_for_fix.py,sha256=dfFcdZnbkboB2-4vk1iyAHj7sCcsvM4AkBarnU8xBs8,1035
 tests/doctor/test_run_checks.py,sha256=t03x6ICUPuceTcHUQKBMRTrr0VzRORv9_hZwcnm2v08,2320
 tests/doctor/devenv/checks/bad_check.py,sha256=uohSVSUMnZmj0eFXEwCOHBzCJN6cdjv2hm6_u9ZMEFw,336
@@ -74,8 +74,8 @@ tests/lib/test_node.py,sha256=0cmWS-YqTXtQ5p6dk2Hc5qTl8XQGt3CGGfVy1rrsE9k,1082
 tests/lib/test_proc.py,sha256=XH6OnxKPSSm3HvDjYHApputMKwgOE8lYqDuK2vK0Z0U,2626
 tests/lib/test_repository.py,sha256=gUi1lkY7bha5WwZ5xcnENOlFYboVV8TFW1lCESvS0VA,787
 tests/lib/test_venv.py,sha256=3XGpINFBkn6Yq4OycsF-Ki4xMWAINt2g69uOHtkDU6k,3266
-sentry_devenv-1.26.0.dist-info/METADATA,sha256=7HrK8HSN-Q1gmZ5ZcM_UBfZKM_Q3QesAa2zqa461Gfc,12319
-sentry_devenv-1.26.0.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-sentry_devenv-1.26.0.dist-info/entry_points.txt,sha256=StsIuNugcoEU8jsPy6H9ECjzoDzOVKXl1vUaYTIGbzM,44
-sentry_devenv-1.26.0.dist-info/top_level.txt,sha256=dOQExvIA0fj_EQjCrMTS7CCHNH7WZFHxEU0M7LlNcKQ,16
-sentry_devenv-1.26.0.dist-info/RECORD,,
+sentry_devenv-1.27.0.dist-info/METADATA,sha256=EQ2bo9kzVfE5K9QYiKD42gaR46QVFW3vhpAbaQmEdLg,12387
+sentry_devenv-1.27.0.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+sentry_devenv-1.27.0.dist-info/entry_points.txt,sha256=StsIuNugcoEU8jsPy6H9ECjzoDzOVKXl1vUaYTIGbzM,44
+sentry_devenv-1.27.0.dist-info/top_level.txt,sha256=dOQExvIA0fj_EQjCrMTS7CCHNH7WZFHxEU0M7LlNcKQ,16
+sentry_devenv-1.27.0.dist-info/RECORD,,

tests/doctor/test_load_checks.py

@@ -9,12 +9,15 @@ from devenv.lib.repository import Repository
 
 
 def test_load_checks_no_checks() -> None:
-    assert doctor.load_checks(Repository("not a real repository"), set()) == []
+    assert (
+        doctor.load_checks(Repository("not a real repository"), set(), set())
+        == []
+    )
 
 
 def test_load_checks_test_checks(capsys: pytest.CaptureFixture[str]) -> None:
     loaded_checks = doctor.load_checks(
-        Repository(os.path.join(os.path.dirname(__file__))), set()
+        Repository(os.path.join(os.path.dirname(__file__))), set(), set()
     )
     loaded_check_names = [check.name for check in loaded_checks]
     assert len(loaded_check_names) == 5
@@ -37,7 +40,7 @@ def test_load_checks_test_checks(capsys: pytest.CaptureFixture[str]) -> None:
 
 def test_load_checks_only_passing_tag() -> None:
     loaded_checks = doctor.load_checks(
-        Repository(os.path.join(os.path.dirname(__file__))), {"pass"}
+        Repository(os.path.join(os.path.dirname(__file__))), {"pass"}, set()
     )
     loaded_check_names = [check.name for check in loaded_checks]
     assert len(loaded_check_names) == 1
@@ -46,7 +49,7 @@ def test_load_checks_only_passing_tag() -> None:
 
 def test_load_checks_only_failing_tag() -> None:
     loaded_checks = doctor.load_checks(
-        Repository(os.path.join(os.path.dirname(__file__))), {"fail"}
+        Repository(os.path.join(os.path.dirname(__file__))), {"fail"}, set()
     )
     loaded_check_names = [check.name for check in loaded_checks]
     assert len(loaded_check_names) == 2
@@ -56,7 +59,9 @@ def test_load_checks_only_failing_tag() -> None:
 
 def test_load_checks_passing_and_failing_tag() -> None:
     loaded_checks = doctor.load_checks(
-        Repository(os.path.join(os.path.dirname(__file__))), {"pass", "fail"}
+        Repository(os.path.join(os.path.dirname(__file__))),
+        {"pass", "fail"},
+        set(),
     )
     loaded_check_names = [check.name for check in loaded_checks]
     assert len(loaded_check_names) == 0
@@ -64,7 +69,7 @@ def test_load_checks_passing_and_failing_tag() -> None:
 
 def test_load_checks_test_tag() -> None:
     loaded_checks = doctor.load_checks(
-        Repository(os.path.join(os.path.dirname(__file__))), {"test"}
+        Repository(os.path.join(os.path.dirname(__file__))), {"test"}, set()
     )
     loaded_check_names = [check.name for check in loaded_checks]
     assert len(loaded_check_names) == 5
@@ -73,3 +78,25 @@ def test_load_checks_test_tag() -> None:
     assert "failing check with msg" in loaded_check_names
     assert "broken check" in loaded_check_names
     assert "broken fix" in loaded_check_names
+
+
+def test_load_checks_exclude_passing_tag() -> None:
+    loaded_checks = doctor.load_checks(
+        Repository(os.path.join(os.path.dirname(__file__))), {"test"}, {"pass"}
+    )
+    loaded_check_names = [check.name for check in loaded_checks]
+    assert len(loaded_check_names) == 4
+    assert "passing check" not in loaded_check_names
+    assert "failing check" in loaded_check_names
+    assert "failing check with msg" in loaded_check_names
+
+
+def test_load_checks_exclude_failing_tag() -> None:
+    loaded_checks = doctor.load_checks(
+        Repository(os.path.join(os.path.dirname(__file__))), {"test"}, {"fail"}
+    )
+    loaded_check_names = [check.name for check in loaded_checks]
+    assert len(loaded_check_names) == 3
+    assert "passing check" in loaded_check_names
+    assert "failing check" not in loaded_check_names
+    assert "failing check with msg" not in loaded_check_names