Closed GMayank0310 closed 4 days ago
The changes involve modifications to the remove_bot_access
function in kairon/shared/account/processor.py
, specifically altering how active bot access records are retrieved. Additionally, several test functions in tests/integration_test/services_test.py
have been removed and replaced with new tests that utilize different user accounts and bot identifiers, ensuring consistent functionality while adapting to a more flexible testing approach.
File | Change Summary |
---|---|
kairon/shared/account/processor.py |
Modified the retrieval method for active bot access records in the remove_bot_access function to call get() directly on the query for active_bot_access . |
tests/integration_test/services_test.py |
Removed several bot membership management test functions and added new tests with different user accounts and bot identifiers, maintaining the same logical assertions. |
kairon/shared/account/processor.py
, specifically adding the process_leave_bot
method, which relates to the remove_bot_access
function modified in the main PR, as both deal with user access management for bots.🐰 In the code, we hop and play,
New tests and changes lead the way.
With bots and users, roles anew,
The access flows, fresh as the dew.
Let's celebrate with a joyful cheer,
For every change brings us near! 🥕
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
fix audit log issues when user leaves bot, deletes a member and when a bot is deleted
Summary by CodeRabbit
Bug Fixes
Tests