Shivam MishraandGitHub
ddb0535a93
This improves the Captain overview by loading reporting metrics and FAQ stats from separate endpoints. Range changes now refresh only the metrics, while reopen-rate calculation reuses the resolved conversation count to avoid redundant database queries. ## What changed - Split Captain overview metrics and FAQ stats into separate APIs. - Fetch FAQ stats independently from range-based metrics. - Reuse resolved conversation totals when calculating reopen rate. - Skip the reopen query when there are no resolved conversations.
35 lines
1.2 KiB
Ruby
35 lines
1.2 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
RSpec.describe Captain::AssistantPolicy, type: :policy do
|
|
subject(:assistant_policy) { described_class }
|
|
|
|
let(:account) { create(:account) }
|
|
let(:administrator) { create(:user, :administrator, account: account) }
|
|
let(:agent) { create(:user, account: account) }
|
|
let(:assistant) { create(:captain_assistant, account: account) }
|
|
let(:administrator_context) { { user: administrator, account: account, account_user: account.account_users.first } }
|
|
let(:agent_context) { { user: agent, account: account, account_user: account.account_users.first } }
|
|
|
|
permissions :index?, :show?, :playground?, :metrics?, :faq_stats? do
|
|
context 'when administrator' do
|
|
it { expect(assistant_policy).to permit(administrator_context, assistant) }
|
|
end
|
|
|
|
context 'when agent' do
|
|
it { expect(assistant_policy).to permit(agent_context, assistant) }
|
|
end
|
|
end
|
|
|
|
permissions :tools?, :create?, :update?, :destroy?, :sync?, :drilldown? do
|
|
context 'when administrator' do
|
|
it { expect(assistant_policy).to permit(administrator_context, assistant) }
|
|
end
|
|
|
|
context 'when agent' do
|
|
it { expect(assistant_policy).not_to permit(agent_context, assistant) }
|
|
end
|
|
end
|
|
end
|