2017-07-05 18:00:58 -04:00
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
RSpec.describe DiscourseChat::Manager do
|
|
|
|
|
|
|
|
let(:category) {Fabricate(:category)}
|
|
|
|
let(:tag1){Fabricate(:tag)}
|
2017-07-05 20:11:26 -04:00
|
|
|
let(:tag2){Fabricate(:tag)}
|
|
|
|
let(:tag3){Fabricate(:tag)}
|
2017-07-05 18:00:58 -04:00
|
|
|
|
|
|
|
describe '.status_for_channel' do
|
|
|
|
|
|
|
|
context 'with no rules' do
|
|
|
|
it 'includes the heading' do
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string).to include('Rules for this channel')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'includes the no_rules string' do
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string).to include('no rules')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with some rules' do
|
|
|
|
before do
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#general', filter:'watch', category_id:category.id, tags:nil}).save!
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#general', filter:'mute', category_id:nil, tags:nil}).save!
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#general', filter:'follow', category_id:nil, tags:[tag1.name]}).save!
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#otherchannel', filter:'watch', category_id:1, tags:nil}).save!
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'displays the correct rules' do
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string.scan('watch').size).to eq(1)
|
|
|
|
expect(string.scan('mute').size).to eq(1)
|
|
|
|
expect(string.scan('follow').size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'enumerates the rules correctly' do
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string.scan('1)').size).to eq(1)
|
|
|
|
expect(string.scan('2)').size).to eq(1)
|
|
|
|
expect(string.scan('3)').size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only displays tags for rules with tags' do
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string.scan('with tags').size).to eq(0)
|
|
|
|
|
|
|
|
SiteSetting.tagging_enabled = true
|
|
|
|
string = DiscourseChat::Helper.status_for_channel('slack','#general')
|
|
|
|
expect(string.scan('with tags').size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2017-07-06 16:36:50 -04:00
|
|
|
describe '.delete_by_index' do
|
|
|
|
let(:category2) {Fabricate(:category)}
|
|
|
|
let(:category3) {Fabricate(:category)}
|
|
|
|
|
|
|
|
it 'deletes the correct rule' do
|
|
|
|
# Three identical rules, with different categories
|
|
|
|
# Status will be sorted by category id, so they should
|
|
|
|
# be in this order
|
|
|
|
rule1 = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
}).save!
|
|
|
|
rule2 = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category2.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
}).save!
|
|
|
|
rule3 = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category3.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
}).save!
|
|
|
|
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(3)
|
|
|
|
|
|
|
|
expect(DiscourseChat::Helper.delete_by_index('slack','#general',2)).to eq(:deleted)
|
|
|
|
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(2)
|
|
|
|
expect(DiscourseChat::Rule.all.map(&:category_id)).to contain_exactly(category.id, category3.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'fails gracefully for out of range indexes' do
|
|
|
|
rule1 = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
}).save!
|
|
|
|
|
|
|
|
expect(DiscourseChat::Helper.delete_by_index('slack','#general',-1)).to eq(false)
|
|
|
|
expect(DiscourseChat::Helper.delete_by_index('slack','#general',0)).to eq(false)
|
|
|
|
expect(DiscourseChat::Helper.delete_by_index('slack','#general',2)).to eq(false)
|
|
|
|
|
|
|
|
expect(DiscourseChat::Helper.delete_by_index('slack','#general',1)).to eq(:deleted)
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2017-07-05 20:11:26 -04:00
|
|
|
describe '.smart_create_rule' do
|
|
|
|
|
|
|
|
it 'creates a rule when there are none' do
|
|
|
|
val = DiscourseChat::Helper.smart_create_rule(provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name]
|
|
|
|
)
|
|
|
|
expect(val).to eq(:created)
|
|
|
|
|
|
|
|
record = DiscourseChat::Rule.all.first
|
|
|
|
expect(record.provider).to eq('slack')
|
|
|
|
expect(record.channel).to eq('#general')
|
|
|
|
expect(record.filter).to eq('watch')
|
|
|
|
expect(record.category_id).to eq(category.id)
|
|
|
|
expect(record.tags).to eq([tag1.name])
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'updates a rule when it has the same category and tags' do
|
|
|
|
existing = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag2.name, tag1.name]
|
|
|
|
}).save!
|
|
|
|
|
|
|
|
val = DiscourseChat::Helper.smart_create_rule(provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'mute',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
)
|
|
|
|
|
|
|
|
expect(val).to eq(:updated)
|
|
|
|
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(1)
|
|
|
|
expect(DiscourseChat::Rule.all.first.filter).to eq('mute')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'updates a rule when it has the same category and filter' do
|
|
|
|
existing = DiscourseChat::Rule.new({provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name, tag2.name]
|
|
|
|
}).save!
|
|
|
|
|
|
|
|
val = DiscourseChat::Helper.smart_create_rule(provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: category.id,
|
|
|
|
tags: [tag1.name, tag3.name]
|
|
|
|
)
|
|
|
|
|
|
|
|
expect(val).to eq(:updated)
|
|
|
|
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(1)
|
|
|
|
expect(DiscourseChat::Rule.all.first.tags).to contain_exactly(tag1.name, tag2.name, tag3.name)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'destroys duplicate rules on save' do
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#general', filter: 'watch'}).save!
|
|
|
|
DiscourseChat::Rule.new({provider: 'slack', channel: '#general', filter: 'watch'}).save!
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(2)
|
|
|
|
val = DiscourseChat::Helper.smart_create_rule(provider: 'slack',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: nil,
|
|
|
|
tags: nil
|
|
|
|
)
|
|
|
|
expect(val).to eq(:updated)
|
|
|
|
expect(DiscourseChat::Rule.all.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false on error' do
|
|
|
|
val = DiscourseChat::Helper.smart_create_rule(provider: 'nonexistantprovider',
|
|
|
|
channel: '#general',
|
|
|
|
filter: 'watch',
|
|
|
|
category_id: nil,
|
|
|
|
tags: nil
|
|
|
|
)
|
|
|
|
|
|
|
|
expect(val).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-07-05 18:00:58 -04:00
|
|
|
end
|