Skip to content

Commit 2a30116

Browse files
fix: remove pragma no cover from covered lines
ROOT CAUSE: strict-no-cover check failed because pragma comments were on lines that ARE covered by tests: - __repr__ method is tested by test_depends_repr - Defensive check at line 128 is covered by dependency tests CHANGES: - Removed # pragma: no cover from Depends.__repr__() method - Removed # pragma: no cover from defensive if dep_name in deps check IMPACT: - Fixes strict-no-cover CI failure - All code remains properly tested - pragma comments only used where appropriate Refs: #2081
1 parent ff70abb commit 2a30116

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/mcp/server/mcpserver/tools/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ async def run(
125125

126126
deps = find_dependency_parameters(self.fn)
127127
for dep_name in self.dependency_kwarg_names:
128-
if dep_name in deps: # pragma: no cover (defensive, should always be true)
128+
if dep_name in deps:
129129
direct_args[dep_name] = await dependency_resolver.resolve(dep_name, deps[dep_name])
130130

131131
result = await self.fn_metadata.call_fn_with_arg_validation(

src/mcp/server/mcpserver/utilities/dependencies.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ def __init__(
3636
self.dependency = dependency
3737
self.use_cache = use_cache
3838

39-
def __repr__(self) -> str: # pragma: no cover
39+
def __repr__(self) -> str:
4040
return f"Depends({self.dependency.__name__})"
4141

4242

0 commit comments

Comments
 (0)