Skip to content

Increase test coverage for ChatAuthorizer and remove `chat_authorized?`

What does this MR do and why?

This MR increases test coverage in the chat_authorizer_spec.rb to 100% by fixing a typo in a test

Also, we noticed that the chat_authorized? method is not needed, as callers can use the context method directly.

MR acceptance checklist

Screenshots or screen recordings

Before image

After image

How to set up and validate locally

This is mainly an RSpec change, but it is good to test Duo Chat locally to make sure authorization still functions properly.

Related to #443876 (closed)

Edited by Lesley Razzaghian

Merge request reports

Loading