Skip to content

Commit fb2b43c

Browse files
test(teams): consolidate duplicate certificate-auth validation test
Addresses bot review feedback on PR #65 — the same startup-throw check existed in both test_teams_coverage.py and test_teams_extended.py. Kept the detailed version, removed the duplicate. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 9b3a04e commit fb2b43c

1 file changed

Lines changed: 0 additions & 21 deletions

File tree

tests/test_teams_coverage.py

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
)
3030
from chat_sdk.adapters.teams.types import (
3131
TeamsAdapterConfig,
32-
TeamsAuthCertificate,
3332
TeamsThreadId,
3433
)
3534
from chat_sdk.shared.errors import (
@@ -1482,26 +1481,6 @@ def routed_post(url, **kwargs):
14821481
assert len(result.messages) >= 0
14831482

14841483

1485-
# ---------------------------------------------------------------------------
1486-
# Certificate config raises
1487-
# ---------------------------------------------------------------------------
1488-
1489-
1490-
class TestCertificateConfig:
1491-
def test_certificate_raises_validation_error(self):
1492-
with pytest.raises(ValidationError, match="Certificate"):
1493-
TeamsAdapter(
1494-
TeamsAdapterConfig(
1495-
app_id="test",
1496-
app_password="test",
1497-
certificate=TeamsAuthCertificate(
1498-
certificate_private_key="key",
1499-
certificate_thumbprint="abc",
1500-
),
1501-
)
1502-
)
1503-
1504-
15051484
# ---------------------------------------------------------------------------
15061485
# _extract_attachments_from_graph_message
15071486
# ---------------------------------------------------------------------------

0 commit comments

Comments
 (0)