Skip to content

Commit 7f5c40a

Browse files
committed
Safely include quoted examples.
1 parent 401a2b4 commit 7f5c40a

26 files changed

+117
-117
lines changed

lib/slack/web/api/templates/method_spec.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ RSpec.describe Slack::Web::Api::Endpoints::<%= group.gsub(".", "_").camelize %>
1212
context '<%= group %>_<%= name %>' do
1313
<% required_params.each do |arg_name, arg_v| %>
1414
it 'requires <%= arg_name %>' do
15-
<% params_except_required = required_params.reject{ |name, _| name == arg_name }.map{|var, opts| "#{var}: '#{opts['example'].gsub("'", '"')}'"}.join(', ') %>
15+
<% params_except_required = required_params.reject{ |name, _| name == arg_name }.map{ |var, opts| "#{var}: %q[#{opts['example']}]"}.join(', ') %>
1616
expect { client.<%= group.gsub(".", "_") %>_<%= name %><%= params_except_required.empty? ? '' : "(#{params_except_required})" %> }.to raise_error ArgumentError, /Required arguments :<%= arg_name %> missing/
1717
end
1818
<% end %>

spec/slack/web/api/endpoints/admin_conversations_restrictAccess_spec.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.conversations.restrictAccess_addGroup' do
99
it 'requires channel_id' do
10-
expect { client.admin_conversations_restrictAccess_addGroup(group_id: '') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
10+
expect { client.admin_conversations_restrictAccess_addGroup(group_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
1111
end
1212
it 'requires group_id' do
13-
expect { client.admin_conversations_restrictAccess_addGroup(channel_id: '') }.to raise_error ArgumentError, /Required arguments :group_id missing/
13+
expect { client.admin_conversations_restrictAccess_addGroup(channel_id: %q[]) }.to raise_error ArgumentError, /Required arguments :group_id missing/
1414
end
1515
end
1616
context 'admin.conversations.restrictAccess_listGroups' do
@@ -20,13 +20,13 @@
2020
end
2121
context 'admin.conversations.restrictAccess_removeGroup' do
2222
it 'requires channel_id' do
23-
expect { client.admin_conversations_restrictAccess_removeGroup(group_id: '', team_id: '') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
23+
expect { client.admin_conversations_restrictAccess_removeGroup(group_id: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
2424
end
2525
it 'requires group_id' do
26-
expect { client.admin_conversations_restrictAccess_removeGroup(channel_id: '', team_id: '') }.to raise_error ArgumentError, /Required arguments :group_id missing/
26+
expect { client.admin_conversations_restrictAccess_removeGroup(channel_id: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :group_id missing/
2727
end
2828
it 'requires team_id' do
29-
expect { client.admin_conversations_restrictAccess_removeGroup(channel_id: '', group_id: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
29+
expect { client.admin_conversations_restrictAccess_removeGroup(channel_id: %q[], group_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
3030
end
3131
end
3232
end

spec/slack/web/api/endpoints/admin_conversations_spec.rb

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@
1717
end
1818
context 'admin.conversations_create' do
1919
it 'requires is_private' do
20-
expect { client.admin_conversations_create(name: 'mychannel') }.to raise_error ArgumentError, /Required arguments :is_private missing/
20+
expect { client.admin_conversations_create(name: %q[mychannel]) }.to raise_error ArgumentError, /Required arguments :is_private missing/
2121
end
2222
it 'requires name' do
23-
expect { client.admin_conversations_create(is_private: 'true') }.to raise_error ArgumentError, /Required arguments :name missing/
23+
expect { client.admin_conversations_create(is_private: %q[true]) }.to raise_error ArgumentError, /Required arguments :name missing/
2424
end
2525
end
2626
context 'admin.conversations_delete' do
@@ -45,26 +45,26 @@
4545
end
4646
context 'admin.conversations_invite' do
4747
it 'requires channel_id' do
48-
expect { client.admin_conversations_invite(user_ids: 'U1234,U2345,U3456') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
48+
expect { client.admin_conversations_invite(user_ids: %q[U1234,U2345,U3456]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
4949
end
5050
it 'requires user_ids' do
51-
expect { client.admin_conversations_invite(channel_id: 'C12345') }.to raise_error ArgumentError, /Required arguments :user_ids missing/
51+
expect { client.admin_conversations_invite(channel_id: %q[C12345]) }.to raise_error ArgumentError, /Required arguments :user_ids missing/
5252
end
5353
end
5454
context 'admin.conversations_rename' do
5555
it 'requires channel_id' do
56-
expect { client.admin_conversations_rename(name: '') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
56+
expect { client.admin_conversations_rename(name: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
5757
end
5858
it 'requires name' do
59-
expect { client.admin_conversations_rename(channel_id: 'C12345') }.to raise_error ArgumentError, /Required arguments :name missing/
59+
expect { client.admin_conversations_rename(channel_id: %q[C12345]) }.to raise_error ArgumentError, /Required arguments :name missing/
6060
end
6161
end
6262
context 'admin.conversations_setConversationPrefs' do
6363
it 'requires channel_id' do
64-
expect { client.admin_conversations_setConversationPrefs(prefs: '{"who_can_post":"type:admin,user:U1234,subteam:S1234"}') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
64+
expect { client.admin_conversations_setConversationPrefs(prefs: %q[{'who_can_post':'type:admin,user:U1234,subteam:S1234'}]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
6565
end
6666
it 'requires prefs' do
67-
expect { client.admin_conversations_setConversationPrefs(channel_id: 'C1234') }.to raise_error ArgumentError, /Required arguments :prefs missing/
67+
expect { client.admin_conversations_setConversationPrefs(channel_id: %q[C1234]) }.to raise_error ArgumentError, /Required arguments :prefs missing/
6868
end
6969
end
7070
context 'admin.conversations_setTeams' do

spec/slack/web/api/endpoints/admin_conversations_whitelist_spec.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.conversations.whitelist_add' do
99
it 'requires channel_id' do
10-
expect { client.admin_conversations_whitelist_add(group_id: '') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
10+
expect { client.admin_conversations_whitelist_add(group_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
1111
end
1212
it 'requires group_id' do
13-
expect { client.admin_conversations_whitelist_add(channel_id: '') }.to raise_error ArgumentError, /Required arguments :group_id missing/
13+
expect { client.admin_conversations_whitelist_add(channel_id: %q[]) }.to raise_error ArgumentError, /Required arguments :group_id missing/
1414
end
1515
end
1616
context 'admin.conversations.whitelist_listGroupsLinkedToChannel' do
@@ -20,13 +20,13 @@
2020
end
2121
context 'admin.conversations.whitelist_remove' do
2222
it 'requires channel_id' do
23-
expect { client.admin_conversations_whitelist_remove(group_id: '', team_id: '') }.to raise_error ArgumentError, /Required arguments :channel_id missing/
23+
expect { client.admin_conversations_whitelist_remove(group_id: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_id missing/
2424
end
2525
it 'requires group_id' do
26-
expect { client.admin_conversations_whitelist_remove(channel_id: '', team_id: '') }.to raise_error ArgumentError, /Required arguments :group_id missing/
26+
expect { client.admin_conversations_whitelist_remove(channel_id: %q[], team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :group_id missing/
2727
end
2828
it 'requires team_id' do
29-
expect { client.admin_conversations_whitelist_remove(channel_id: '', group_id: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
29+
expect { client.admin_conversations_whitelist_remove(channel_id: %q[], group_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
3030
end
3131
end
3232
end

spec/slack/web/api/endpoints/admin_emoji_spec.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,18 +7,18 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.emoji_add' do
99
it 'requires name' do
10-
expect { client.admin_emoji_add(url: '') }.to raise_error ArgumentError, /Required arguments :name missing/
10+
expect { client.admin_emoji_add(url: %q[]) }.to raise_error ArgumentError, /Required arguments :name missing/
1111
end
1212
it 'requires url' do
13-
expect { client.admin_emoji_add(name: '') }.to raise_error ArgumentError, /Required arguments :url missing/
13+
expect { client.admin_emoji_add(name: %q[]) }.to raise_error ArgumentError, /Required arguments :url missing/
1414
end
1515
end
1616
context 'admin.emoji_addAlias' do
1717
it 'requires alias_for' do
18-
expect { client.admin_emoji_addAlias(name: '') }.to raise_error ArgumentError, /Required arguments :alias_for missing/
18+
expect { client.admin_emoji_addAlias(name: %q[]) }.to raise_error ArgumentError, /Required arguments :alias_for missing/
1919
end
2020
it 'requires name' do
21-
expect { client.admin_emoji_addAlias(alias_for: '') }.to raise_error ArgumentError, /Required arguments :name missing/
21+
expect { client.admin_emoji_addAlias(alias_for: %q[]) }.to raise_error ArgumentError, /Required arguments :name missing/
2222
end
2323
end
2424
context 'admin.emoji_remove' do
@@ -28,10 +28,10 @@
2828
end
2929
context 'admin.emoji_rename' do
3030
it 'requires name' do
31-
expect { client.admin_emoji_rename(new_name: '') }.to raise_error ArgumentError, /Required arguments :name missing/
31+
expect { client.admin_emoji_rename(new_name: %q[]) }.to raise_error ArgumentError, /Required arguments :name missing/
3232
end
3333
it 'requires new_name' do
34-
expect { client.admin_emoji_rename(name: '') }.to raise_error ArgumentError, /Required arguments :new_name missing/
34+
expect { client.admin_emoji_rename(name: %q[]) }.to raise_error ArgumentError, /Required arguments :new_name missing/
3535
end
3636
end
3737
end

spec/slack/web/api/endpoints/admin_teams_settings_spec.rb

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -12,42 +12,42 @@
1212
end
1313
context 'admin.teams.settings_setDefaultChannels' do
1414
it 'requires channel_ids' do
15-
expect { client.admin_teams_settings_setDefaultChannels(team_id: '') }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
15+
expect { client.admin_teams_settings_setDefaultChannels(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
1616
end
1717
it 'requires team_id' do
18-
expect { client.admin_teams_settings_setDefaultChannels(channel_ids: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
18+
expect { client.admin_teams_settings_setDefaultChannels(channel_ids: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
1919
end
2020
end
2121
context 'admin.teams.settings_setDescription' do
2222
it 'requires description' do
23-
expect { client.admin_teams_settings_setDescription(team_id: '') }.to raise_error ArgumentError, /Required arguments :description missing/
23+
expect { client.admin_teams_settings_setDescription(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :description missing/
2424
end
2525
it 'requires team_id' do
26-
expect { client.admin_teams_settings_setDescription(description: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
26+
expect { client.admin_teams_settings_setDescription(description: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
2727
end
2828
end
2929
context 'admin.teams.settings_setDiscoverability' do
3030
it 'requires discoverability' do
31-
expect { client.admin_teams_settings_setDiscoverability(team_id: '') }.to raise_error ArgumentError, /Required arguments :discoverability missing/
31+
expect { client.admin_teams_settings_setDiscoverability(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :discoverability missing/
3232
end
3333
it 'requires team_id' do
34-
expect { client.admin_teams_settings_setDiscoverability(discoverability: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
34+
expect { client.admin_teams_settings_setDiscoverability(discoverability: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
3535
end
3636
end
3737
context 'admin.teams.settings_setIcon' do
3838
it 'requires image_url' do
39-
expect { client.admin_teams_settings_setIcon(team_id: '') }.to raise_error ArgumentError, /Required arguments :image_url missing/
39+
expect { client.admin_teams_settings_setIcon(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :image_url missing/
4040
end
4141
it 'requires team_id' do
42-
expect { client.admin_teams_settings_setIcon(image_url: 'http://mysite.com/icon.jpeg') }.to raise_error ArgumentError, /Required arguments :team_id missing/
42+
expect { client.admin_teams_settings_setIcon(image_url: %q[http://mysite.com/icon.jpeg]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
4343
end
4444
end
4545
context 'admin.teams.settings_setName' do
4646
it 'requires name' do
47-
expect { client.admin_teams_settings_setName(team_id: '') }.to raise_error ArgumentError, /Required arguments :name missing/
47+
expect { client.admin_teams_settings_setName(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :name missing/
4848
end
4949
it 'requires team_id' do
50-
expect { client.admin_teams_settings_setName(name: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
50+
expect { client.admin_teams_settings_setName(name: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
5151
end
5252
end
5353
end

spec/slack/web/api/endpoints/admin_teams_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.teams_create' do
99
it 'requires team_domain' do
10-
expect { client.admin_teams_create(team_name: '') }.to raise_error ArgumentError, /Required arguments :team_domain missing/
10+
expect { client.admin_teams_create(team_name: %q[]) }.to raise_error ArgumentError, /Required arguments :team_domain missing/
1111
end
1212
it 'requires team_name' do
13-
expect { client.admin_teams_create(team_domain: '') }.to raise_error ArgumentError, /Required arguments :team_name missing/
13+
expect { client.admin_teams_create(team_domain: %q[]) }.to raise_error ArgumentError, /Required arguments :team_name missing/
1414
end
1515
end
1616
end

spec/slack/web/api/endpoints/admin_usergroups_spec.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,18 +7,18 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.usergroups_addChannels' do
99
it 'requires channel_ids' do
10-
expect { client.admin_usergroups_addChannels(usergroup_id: 'S00000000') }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
10+
expect { client.admin_usergroups_addChannels(usergroup_id: %q[S00000000]) }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
1111
end
1212
it 'requires usergroup_id' do
13-
expect { client.admin_usergroups_addChannels(channel_ids: 'C00000000,C00000001') }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
13+
expect { client.admin_usergroups_addChannels(channel_ids: %q[C00000000,C00000001]) }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
1414
end
1515
end
1616
context 'admin.usergroups_addTeams' do
1717
it 'requires team_ids' do
18-
expect { client.admin_usergroups_addTeams(usergroup_id: 'S12345678') }.to raise_error ArgumentError, /Required arguments :team_ids missing/
18+
expect { client.admin_usergroups_addTeams(usergroup_id: %q[S12345678]) }.to raise_error ArgumentError, /Required arguments :team_ids missing/
1919
end
2020
it 'requires usergroup_id' do
21-
expect { client.admin_usergroups_addTeams(team_ids: 'T12345678,T98765432') }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
21+
expect { client.admin_usergroups_addTeams(team_ids: %q[T12345678,T98765432]) }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
2222
end
2323
end
2424
context 'admin.usergroups_listChannels' do
@@ -28,10 +28,10 @@
2828
end
2929
context 'admin.usergroups_removeChannels' do
3030
it 'requires channel_ids' do
31-
expect { client.admin_usergroups_removeChannels(usergroup_id: 'S00000000') }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
31+
expect { client.admin_usergroups_removeChannels(usergroup_id: %q[S00000000]) }.to raise_error ArgumentError, /Required arguments :channel_ids missing/
3232
end
3333
it 'requires usergroup_id' do
34-
expect { client.admin_usergroups_removeChannels(channel_ids: 'C00000000,C00000001') }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
34+
expect { client.admin_usergroups_removeChannels(channel_ids: %q[C00000000,C00000001]) }.to raise_error ArgumentError, /Required arguments :usergroup_id missing/
3535
end
3636
end
3737
end

spec/slack/web/api/endpoints/admin_users_session_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77
let(:client) { Slack::Web::Client.new }
88
context 'admin.users.session_invalidate' do
99
it 'requires session_id' do
10-
expect { client.admin_users_session_invalidate(team_id: '') }.to raise_error ArgumentError, /Required arguments :session_id missing/
10+
expect { client.admin_users_session_invalidate(team_id: %q[]) }.to raise_error ArgumentError, /Required arguments :session_id missing/
1111
end
1212
it 'requires team_id' do
13-
expect { client.admin_users_session_invalidate(session_id: '') }.to raise_error ArgumentError, /Required arguments :team_id missing/
13+
expect { client.admin_users_session_invalidate(session_id: %q[]) }.to raise_error ArgumentError, /Required arguments :team_id missing/
1414
end
1515
end
1616
context 'admin.users.session_reset' do

0 commit comments

Comments
 (0)