Skip to content

Conversation

@timkpaine
Copy link
Member

Fixes #24

@timkpaine timkpaine marked this pull request as draft January 17, 2026 02:50
@github-actions
Copy link
Contributor

github-actions bot commented Jan 17, 2026

Test Results

119 tests  +100   119 ✅ +100   5s ⏱️ -2s
  1 suites ±  0     0 💤 ±  0 
  1 files   ±  0     0 ❌ ±  0 

Results for commit 44ad9a0. ± Comparison against base commit 6a9f140.

This pull request removes 19 and adds 119 tests. Note that renamed tests count towards both.
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_from_message_ml
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_handle_event_invalid
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_handle_event_messagesent
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_handle_event_messagesent_im
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_handle_event_symphonyelementsaction
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_mention_user
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_no_changes
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_room_mapper
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_send_symphony_message
csp_adapter_symphony.tests.test_adapter.TestSymphony ‑ test_symphony_config_aliases
…
csp_adapter_symphony.tests.test_adapter.TestCheckBdkAvailable ‑ test_check_bdk_available_when_available
csp_adapter_symphony.tests.test_adapter.TestCheckBdkAvailable ‑ test_check_bdk_available_when_not_available
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_is_accepting_event_from_other_user
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_is_accepting_event_from_self
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_is_accepting_event_with_error
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_on_message_sent
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_on_message_sent_filtered
csp_adapter_symphony.tests.test_adapter.TestCspRealTimeEventListener ‑ test_on_symphony_elements_action
csp_adapter_symphony.tests.test_adapter.TestFormatWithMessageML ‑ test_from_message_ml
csp_adapter_symphony.tests.test_adapter.TestFormatWithMessageML ‑ test_greater_than_escape
…

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Jan 17, 2026

Codecov Report

❌ Patch coverage is 89.07176% with 166 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.76%. Comparing base (abc4923) to head (44ad9a0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
csp_adapter_symphony/adapter.py 61.81% 116 Missing and 10 partials ⚠️
csp_adapter_symphony/tests/conftest.py 76.92% 21 Missing ⚠️
csp_adapter_symphony/adapter_config.py 88.53% 12 Missing and 6 partials ⚠️
csp_adapter_symphony/tests/test_adapter.py 99.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   79.67%   88.76%   +9.08%     
==========================================
  Files           7        7              
  Lines         684     1620     +936     
  Branches       86       88       +2     
==========================================
+ Hits          545     1438     +893     
- Misses        100      165      +65     
+ Partials       39       17      -22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate to symphony BDK

1 participant