Skip to content

Fix licensed feature stubbing in Duo Chat QA specs

Alejandro Rodríguez requested to merge fix-duo-chat-qa into master

What does this MR do and why?

Whenever stub_licensed_features is called, it removes previous stubs by invoking and_call_original. This means that to correctly stub two features, you must do so in a single stub_licensed_features, or the second will override the first. Us doing that in the Chat QA specs led to responses being "This feature is only allowed in groups that enable this feature" (see !136268 (comment 1652428808)).

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports