-
-
Notifications
You must be signed in to change notification settings - Fork 3k
/
string_chars_spec.rb
76 lines (63 loc) · 1.8 KB
/
string_chars_spec.rb
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
# frozen_string_literal: true
RSpec.describe RuboCop::Cop::Style::StringChars, :config do
it 'registers and corrects an offense when using `split(//)`' do
expect_offense(<<~RUBY)
string.split(//)
^^^^^^^^^ Use `chars` instead of `split(//)`.
RUBY
expect_correction(<<~RUBY)
string.chars
RUBY
end
it "registers and corrects an offense when using `split('')`" do
expect_offense(<<~RUBY)
string.split('')
^^^^^^^^^ Use `chars` instead of `split('')`.
RUBY
expect_correction(<<~RUBY)
string.chars
RUBY
end
it 'registers and corrects an offense when using `split("")`' do
expect_offense(<<~RUBY)
string.split("")
^^^^^^^^^ Use `chars` instead of `split("")`.
RUBY
expect_correction(<<~RUBY)
string.chars
RUBY
end
it 'registers and corrects an offense when using `split` without parentheses' do
expect_offense(<<~RUBY)
do_something { |foo| foo.split '' }
^^^^^^^^ Use `chars` instead of `split ''`.
RUBY
expect_correction(<<~RUBY)
do_something { |foo| foo.chars }
RUBY
end
it 'registers and corrects an offense when using safe navigation `split(//)` call' do
expect_offense(<<~RUBY)
string&.split(//)
^^^^^^^^^ Use `chars` instead of `split(//)`.
RUBY
expect_correction(<<~RUBY)
string&.chars
RUBY
end
it 'does not register an offense when using `chars`' do
expect_no_offenses(<<~RUBY)
string.chars
RUBY
end
it 'does not register an offense when using `split(/ /)`' do
expect_no_offenses(<<~RUBY)
string.split(/ /)
RUBY
end
it 'does not register an offense when using `split`' do
expect_no_offenses(<<~RUBY)
string.split
RUBY
end
end