Skip to content

Commit 6ab5763

Browse files
committed
fix: correct broken test assertions and exclude .agents templates from lint
Three test assertions had 'MINIMAL' string instead of 0, causing comparison failures. Also exclude .agents/skills/ template files from ruff to fix invalid-syntax errors caused by <placeholder> syntax in templates.
1 parent 5bfa9c9 commit 6ab5763

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

ruff.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ exclude = [
2121
"allure-results",
2222
"local_models",
2323
".agent",
24+
".agents",
2425
]
2526

2627
[lint]

tests/common/test_attachment_handler.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ def test_skip_files_with_postfix(self, mock_fetch_bytes):
164164

165165
result = attachment_handler.fetch_all_attachments(["path/to/image_SKIP.png"])
166166

167-
assert len(result) == "MINIMAL"
167+
assert len(result) == 0
168168
mock_fetch_bytes.assert_not_called()
169169

170170
@patch.object(attachment_handler, "_fetch_file_bytes")
@@ -175,7 +175,7 @@ def test_skip_unsupported_mime_types(self, mock_fetch_bytes):
175175

176176
result = attachment_handler.fetch_all_attachments(["path/to/archive.zip"])
177177

178-
assert len(result) == "MINIMAL"
178+
assert len(result) == 0
179179

180180
@patch.object(attachment_handler, "_fetch_file_bytes")
181181
@patch.object(attachment_handler.config, "JIRA_ATTACHMENT_SKIP_POSTFIX", "_SKIP")
@@ -204,7 +204,7 @@ def test_handle_fetch_errors_gracefully(self, mock_fetch_bytes):
204204

205205
result = attachment_handler.fetch_all_attachments(["path/to/missing.png"])
206206

207-
assert len(result) == "MINIMAL"
207+
assert len(result) == 0
208208

209209
def test_empty_attachment_list(self):
210210
"""Test empty attachment list returns empty result."""

0 commit comments

Comments
 (0)