Skip to content

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Oct 31, 2025

Summary

This PR outputs verbose logs for the "apps.activities.list" command if the verbose flag is used for better insight into unexpected errors.

Notes

Initial changes are being used to test in CI but I'm not sure if this is something we want to release? Perhaps this method was skipped to avoid cluttering debug logs during development.

Requirements

@zimeg zimeg self-assigned this Oct 31, 2025
@zimeg zimeg added the code health M-T: Test improvements and anything that improves code health label Oct 31, 2025
@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.60%. Comparing base (37106f5) to head (b87ee20).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #256   +/-   ##
=======================================
  Coverage   64.60%   64.60%           
=======================================
  Files         212      212           
  Lines       17505    17504    -1     
=======================================
  Hits        11309    11309           
- Misses       5130     5131    +1     
+ Partials     1066     1064    -2     

☔ 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.

@zimeg
Copy link
Member Author

zimeg commented Oct 31, 2025

🗣️ Closing this PR since the build was useful enough in testing!

🐛 For the curious, the unexpected errors occurred when expecting certain logs had arrived when stopping a test CLI process after certain events, persay a Slack message, happened in the workspace. The polling activity command retries every three seconds and therefore might not have returned all logs at that time.

@zimeg zimeg closed this Oct 31, 2025
@zimeg zimeg deleted the zimeg-feat-verbose branch October 31, 2025 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code health M-T: Test improvements and anything that improves code health

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants