Skip to content

Commit

Permalink
fix(rubocop): fix remaining errors manually
Browse files Browse the repository at this point in the history
```bash
letsencrypt-formula$ rubocop -d
...
Inspecting 5 files
Scanning /home/imran/Salt/formulas/letsencrypt-formula/Gemfile
.Scanning /home/imran/Salt/formulas/letsencrypt-formula/bin/kitchen
.Scanning /home/imran/Salt/formulas/letsencrypt-formula/test/integration/deb/controls/letsencrypt_spec.rb
CScanning /home/imran/Salt/formulas/letsencrypt-formula/test/integration/git/controls/letsencrypt_spec.rb
CScanning /home/imran/Salt/formulas/letsencrypt-formula/test/integration/rpm/controls/letsencrypt_spec.rb
C

Offenses:

test/integration/deb/controls/letsencrypt_spec.rb:13:94: C: Metrics/LineLength: Line is too long. [95/88]
  its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
                                                                                             ^^
test/integration/deb/controls/letsencrypt_spec.rb:45:89: C: Metrics/LineLength: Line is too long. [91/88]
  its('content') { should match 'deb http://deb.debian.org/debian stretch-backports main' }
                                                                                        ^^^
test/integration/git/controls/letsencrypt_spec.rb:17:94: C: Metrics/LineLength: Line is too long. [95/88]
  its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
                                                                                             ^^
test/integration/rpm/controls/letsencrypt_spec.rb:14:94: C: Metrics/LineLength: Line is too long. [95/88]
  its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
                                                                                             ^^

5 files inspected, 4 offenses detected
```
  • Loading branch information
myii committed Oct 23, 2019
1 parent 94e8abe commit f313483
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 4 deletions.
8 changes: 6 additions & 2 deletions test/integration/deb/controls/letsencrypt_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@
it { should be_grouped_into 'root' }
it { should be_readable }
its('size') { should be > 1 }
its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
its('content') do
should match 'server = https://acme-staging.api.letsencrypt.org/directory'
end
its('content') { should match 'authenticator = webroot' }
end

Expand Down Expand Up @@ -42,5 +44,7 @@
it { should be_grouped_into 'root' }
it { should be_readable }
its('size') { should be > 1 }
its('content') { should match 'deb http://deb.debian.org/debian stretch-backports main' }
its('content') do
should match 'deb http://deb.debian.org/debian stretch-backports main'
end
end
4 changes: 3 additions & 1 deletion test/integration/git/controls/letsencrypt_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
it { should be_grouped_into 'root' }
it { should be_readable }
its('size') { should be > 1 }
its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
its('content') do
should match 'server = https://acme-staging.api.letsencrypt.org/directory'
end
its('content') { should match 'authenticator = standalone' }
end
4 changes: 3 additions & 1 deletion test/integration/rpm/controls/letsencrypt_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,9 @@
it { should be_readable }
its('size') { should be > 1 }

its('content') { should match 'server = https://acme-staging.api.letsencrypt.org/directory' }
its('content') do
should match 'server = https://acme-staging.api.letsencrypt.org/directory'
end
its('content') { should match 'authenticator = webroot' }
end

Expand Down

0 comments on commit f313483

Please sign in to comment.