18812500213

Committed 26 Oct 2025 03:42AM UTC coverage: 80.284% (+0.005%) from 80.279%

Build # 18812500213

Build Type

Pull #22804

github

Committed by

web-flow

Commit Message

Merge 2a56fdb46 into 4834308dc

Pull Request Pull Request #22804: test_shell_command: use correct default cache scope for a test's environment

Run Details

29 of 31 new or added lines in 2 files covered. (93.55%)

1314 existing lines in 64 files now uncovered.

77900 of 97030 relevant lines covered (80.28%)

3.35 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

100.0

/src/python/pants/option/options_diff.py

# Copyright 2025 Pants project contributors (see CONTRIBUTORS.md).
# Licensed under the Apache License, Version 2.0 (see LICENSE).

from dataclasses import fields
from enum import Enum
from typing import Any

from pants.option.bootstrap_options import DynamicRemoteOptions


def summarize_options_map_diff(old: dict[str, Any], new: dict[str, Any]) -> str:
    """Compare two options map (as produced by `OptionsFingerprinter.options_map_for_scope`), and
    return a one-liner summarizing all differences, e.g.:

    `key1: 'old' -> "new'; list_key: ['x', 'y'] -> ['x']`
    """
    diffs = []
    for key in sorted({*old, *new}):
        o = old.get(key)
        n = new.get(key)

        if o == n:
            continue

        diffs.append(f"{key}: {o!r} -> {n!r}")

    return "; ".join(diffs)


def summarize_dynamic_options_diff(old: DynamicRemoteOptions, new: DynamicRemoteOptions) -> str:
    """Compare two `DynamicRemoteOptions` and return a one-liner summarizing all differences, e.g.:

    `provider: 'reapi' -> 'experimental-file'; cache_read: False -> True`
    """
    diffs = []
    for field in fields(DynamicRemoteOptions):
        key = field.name
        o = getattr(old, key)
        n = getattr(new, key)

        if o == n:
            continue

        o = o.value if isinstance(o, Enum) else o
        n = n.value if isinstance(n, Enum) else n

        diffs.append(f"{key}: {o!r} -> {n!r}")

    return "; ".join(diffs)

1	# Copyright 2025 Pants project contributors (see CONTRIBUTORS.md).
2	# Licensed under the Apache License, Version 2.0 (see LICENSE).
3
4	from dataclasses import fields	12✔
5	from enum import Enum	12✔
6	from typing import Any	12✔
7
8	from pants.option.bootstrap_options import DynamicRemoteOptions	12✔
9
10
11	def summarize_options_map_diff(old: dict[str, Any], new: dict[str, Any]) -> str:	12✔
12	"""Compare two options map (as produced by `OptionsFingerprinter.options_map_for_scope`), and
13	return a one-liner summarizing all differences, e.g.:
14
15	`key1: 'old' -> "new'; list_key: ['x', 'y'] -> ['x']`
16	"""
17	diffs = []	2✔
18	for key in sorted({old, new}):	2✔
19	o = old.get(key)	2✔
20	n = new.get(key)	2✔
21
22	if o == n:	2✔
23	continue	2✔
24
25	diffs.append(f"{key}: {o!r} -> {n!r}")	2✔
26
27	return "; ".join(diffs)	2✔
28
29
30	def summarize_dynamic_options_diff(old: DynamicRemoteOptions, new: DynamicRemoteOptions) -> str:	12✔
31	"""Compare two `DynamicRemoteOptions` and return a one-liner summarizing all differences, e.g.:
32
33	`provider: 'reapi' -> 'experimental-file'; cache_read: False -> True`
34	"""
UNCOV 35	diffs = []	1✔
UNCOV 36	for field in fields(DynamicRemoteOptions):	1✔
UNCOV 37	key = field.name	1✔
UNCOV 38	o = getattr(old, key)	1✔
UNCOV 39	n = getattr(new, key)	1✔
40
UNCOV 41	if o == n:	1✔
UNCOV 42	continue	1✔
43
UNCOV 44	o = o.value if isinstance(o, Enum) else o	1✔
UNCOV 45	n = n.value if isinstance(n, Enum) else n	1✔
46
UNCOV 47	diffs.append(f"{key}: {o!r} -> {n!r}")	1✔
48
UNCOV 49	return "; ".join(diffs)	1✔

pantsbuild / pants / 18812500213

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous