foodcritic 15.0.0 → 15.1.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/chef_dsl_metadata/{chef_14.7.17.json → chef_14.8.12.json} +61 -1
- data/foodcritic.gemspec +1 -3
- data/lib/foodcritic/linter.rb +1 -1
- data/lib/foodcritic/version.rb +1 -1
- metadata +5 -232
- data/Rakefile +0 -73
- data/features/002_check_string_interpolation.feature +0 -50
- data/features/005_check_for_resource_repetition.feature +0 -64
- data/features/006_check_file_mode.feature +0 -36
- data/features/009_check_for_unrecognised_resource_attributes.feature +0 -55
- data/features/014_check_for_long_ruby_blocks.feature +0 -30
- data/features/018_check_for_old_lwrp_notification_syntax.feature +0 -25
- data/features/019_check_for_consistent_node_access.feature +0 -108
- data/features/021_check_for_dodgy_lwrp_conditions.feature +0 -28
- data/features/022_check_for_dodgy_conditions_within_loop.feature +0 -43
- data/features/024_check_for_missing_platforms.feature +0 -43
- data/features/027_check_for_internal_attribute_use.feature +0 -22
- data/features/030_check_for_debugger_breakpoints.feature +0 -25
- data/features/032_check_for_invalid_notification_timing.feature +0 -24
- data/features/033_check_for_missing_template.feature +0 -39
- data/features/034_check_for_unused_template_variables.feature +0 -80
- data/features/037_check_for_invalid_notification_action.feature +0 -34
- data/features/038_check_for_invalid_action.feature +0 -39
- data/features/039_check_for_key_access_to_node_methods.feature +0 -33
- data/features/044_check_for_bare_attribute_keys.feature +0 -53
- data/features/046_check_for_assign_unless_nil_attributes.feature +0 -21
- data/features/047_check_for_attribute_assignment_without_precedence.feature +0 -49
- data/features/051_check_for_template_partial_loops.feature +0 -40
- data/features/build_framework_support.feature +0 -109
- data/features/checking_all_types_of_file.feature +0 -40
- data/features/choose_rules_to_apply.feature +0 -47
- data/features/command_line_help.feature +0 -58
- data/features/continuous_integration_support.feature +0 -33
- data/features/exclude_paths_to_lint.feature +0 -12
- data/features/ignore_via_line_comments.feature +0 -69
- data/features/include_custom_rules.feature +0 -29
- data/features/individual_file.feature +0 -28
- data/features/limit_rules_to_specific_versions.feature +0 -65
- data/features/multiple_paths.feature +0 -35
- data/features/show_lines_matched.feature +0 -20
- data/features/sort_warnings.feature +0 -10
- data/features/specify_search_grammar.feature +0 -25
- data/features/step_definitions/cookbook_steps.rb +0 -1826
- data/features/support/command_helpers.rb +0 -418
- data/features/support/cookbook_helpers.rb +0 -599
- data/features/support/env.rb +0 -19
- data/man/foodcritic.1.ronn +0 -76
- data/spec/functional/fc001_spec.rb +0 -72
- data/spec/functional/fc004_spec.rb +0 -84
- data/spec/functional/fc007_spec.rb +0 -127
- data/spec/functional/fc008_spec.rb +0 -33
- data/spec/functional/fc009_spec.rb +0 -37
- data/spec/functional/fc010_spec.rb +0 -32
- data/spec/functional/fc011_spec.rb +0 -20
- data/spec/functional/fc013_spec.rb +0 -39
- data/spec/functional/fc015_spec.rb +0 -19
- data/spec/functional/fc016_spec.rb +0 -57
- data/spec/functional/fc025_spec.rb +0 -60
- data/spec/functional/fc026_spec.rb +0 -66
- data/spec/functional/fc028_spec.rb +0 -93
- data/spec/functional/fc029_spec.rb +0 -33
- data/spec/functional/fc031_spec.rb +0 -14
- data/spec/functional/fc037_spec.rb +0 -75
- data/spec/functional/fc038_spec.rb +0 -28
- data/spec/functional/fc040_spec.rb +0 -156
- data/spec/functional/fc041_spec.rb +0 -121
- data/spec/functional/fc042_spec.rb +0 -13
- data/spec/functional/fc043_spec.rb +0 -28
- data/spec/functional/fc045_spec.rb +0 -18
- data/spec/functional/fc048_spec.rb +0 -67
- data/spec/functional/fc049_spec.rb +0 -56
- data/spec/functional/fc050_spec.rb +0 -85
- data/spec/functional/fc052_spec.rb +0 -13
- data/spec/functional/fc053_spec.rb +0 -13
- data/spec/functional/fc055_spec.rb +0 -18
- data/spec/functional/fc056_spec.rb +0 -18
- data/spec/functional/fc058_spec.rb +0 -102
- data/spec/functional/fc060_spec.rb +0 -45
- data/spec/functional/fc061_spec.rb +0 -48
- data/spec/functional/fc062_spec.rb +0 -13
- data/spec/functional/fc063_spec.rb +0 -18
- data/spec/functional/fc064_spec.rb +0 -13
- data/spec/functional/fc065_spec.rb +0 -13
- data/spec/functional/fc066_spec.rb +0 -33
- data/spec/functional/fc067_spec.rb +0 -13
- data/spec/functional/fc068_spec.rb +0 -13
- data/spec/functional/fc069_spec.rb +0 -38
- data/spec/functional/fc070_spec.rb +0 -65
- data/spec/functional/fc071_spec.rb +0 -36
- data/spec/functional/fc072_spec.rb +0 -13
- data/spec/functional/fc073_spec.rb +0 -56
- data/spec/functional/fc074_spec.rb +0 -39
- data/spec/functional/fc075_spec.rb +0 -41
- data/spec/functional/fc076_spec.rb +0 -13
- data/spec/functional/fc077_spec.rb +0 -13
- data/spec/functional/fc078_spec.rb +0 -35
- data/spec/functional/fc079_spec.rb +0 -21
- data/spec/functional/fc080_spec.rb +0 -68
- data/spec/functional/fc081_spec.rb +0 -25
- data/spec/functional/fc082_spec.rb +0 -23
- data/spec/functional/fc083_spec.rb +0 -34
- data/spec/functional/fc084_spec.rb +0 -49
- data/spec/functional/fc085_spec.rb +0 -77
- data/spec/functional/fc086_spec.rb +0 -45
- data/spec/functional/fc087_spec.rb +0 -43
- data/spec/functional/fc088_spec.rb +0 -18
- data/spec/functional/fc089_spec.rb +0 -23
- data/spec/functional/fc091_spec.rb +0 -38
- data/spec/functional/fc092_spec.rb +0 -31
- data/spec/functional/fc093_spec.rb +0 -27
- data/spec/functional/fc094_spec.rb +0 -18
- data/spec/functional/fc095_spec.rb +0 -18
- data/spec/functional/fc096_spec.rb +0 -18
- data/spec/functional/fc097_spec.rb +0 -18
- data/spec/functional/fc098_spec.rb +0 -18
- data/spec/functional/fc099_spec.rb +0 -18
- data/spec/functional/fc100_spec.rb +0 -18
- data/spec/functional/fc101_spec.rb +0 -23
- data/spec/functional/fc102_spec.rb +0 -18
- data/spec/functional/fc103_spec.rb +0 -30
- data/spec/functional/fc104_spec.rb +0 -54
- data/spec/functional/fc105_spec.rb +0 -36
- data/spec/functional/fc106_spec.rb +0 -21
- data/spec/functional/fc107_spec.rb +0 -22
- data/spec/functional/fc108_spec.rb +0 -43
- data/spec/functional/fc109_spec.rb +0 -22
- data/spec/functional/fc110_spec.rb +0 -21
- data/spec/functional/fc111_spec.rb +0 -17
- data/spec/functional/fc112_spec.rb +0 -17
- data/spec/functional/fc113_spec.rb +0 -98
- data/spec/functional/fc114_spec.rb +0 -31
- data/spec/functional/fc115_spec.rb +0 -24
- data/spec/functional/fc116_spec.rb +0 -18
- data/spec/functional/fc117_spec.rb +0 -23
- data/spec/functional/fc118_spec.rb +0 -17
- data/spec/functional/fc119_spec.rb +0 -27
- data/spec/functional/fc120_spec.rb +0 -21
- data/spec/functional/fc121_spec.rb +0 -45
- data/spec/functional/fc122_spec.rb +0 -18
- data/spec/functional/root_aliases_spec.rb +0 -13
- data/spec/regression/cookbooks.txt +0 -83
- data/spec/regression/expected/activemq.txt +0 -6
- data/spec/regression/expected/aix.txt +0 -27
- data/spec/regression/expected/apparmor.txt +0 -9
- data/spec/regression/expected/apt.txt +0 -14
- data/spec/regression/expected/ark.txt +0 -3
- data/spec/regression/expected/auditd.txt +0 -2
- data/spec/regression/expected/aws.txt +0 -16
- data/spec/regression/expected/bluepill.txt +0 -12
- data/spec/regression/expected/boost.txt +0 -8
- data/spec/regression/expected/build-essential.txt +0 -8
- data/spec/regression/expected/chef-client.txt +0 -17
- data/spec/regression/expected/chef-ingredient.txt +0 -6
- data/spec/regression/expected/chef-server.txt +0 -13
- data/spec/regression/expected/chef-vault.txt +0 -7
- data/spec/regression/expected/chef.txt +0 -104
- data/spec/regression/expected/chef_client_updater.txt +0 -2
- data/spec/regression/expected/chef_handler.txt +0 -7
- data/spec/regression/expected/chef_hostname.txt +0 -3
- data/spec/regression/expected/chef_nginx.txt +0 -4
- data/spec/regression/expected/cron.txt +0 -7
- data/spec/regression/expected/database.txt +0 -42
- data/spec/regression/expected/dmg.txt +0 -11
- data/spec/regression/expected/docker.txt +0 -4
- data/spec/regression/expected/drbd.txt +0 -29
- data/spec/regression/expected/dynect.txt +0 -33
- data/spec/regression/expected/erlang.txt +0 -6
- data/spec/regression/expected/fail2ban.txt +0 -5
- data/spec/regression/expected/firewall.txt +0 -11
- data/spec/regression/expected/gecode.txt +0 -10
- data/spec/regression/expected/gems.txt +0 -32
- data/spec/regression/expected/gunicorn.txt +0 -11
- data/spec/regression/expected/heartbeat.txt +0 -8
- data/spec/regression/expected/homebrew.txt +0 -7
- data/spec/regression/expected/iis.txt +0 -22
- data/spec/regression/expected/iptables.txt +0 -9
- data/spec/regression/expected/jetty.txt +0 -13
- data/spec/regression/expected/jpackage.txt +0 -9
- data/spec/regression/expected/keepalived.txt +0 -3
- data/spec/regression/expected/kickstart.txt +0 -15
- data/spec/regression/expected/libarchive.txt +0 -6
- data/spec/regression/expected/logwatch.txt +0 -5
- data/spec/regression/expected/lvm.txt +0 -14
- data/spec/regression/expected/maradns.txt +0 -14
- data/spec/regression/expected/maven.txt +0 -8
- data/spec/regression/expected/memcached.txt +0 -31
- data/spec/regression/expected/motd-tail.txt +0 -5
- data/spec/regression/expected/mysql.txt +0 -19
- data/spec/regression/expected/ohai.txt +0 -9
- data/spec/regression/expected/openldap.txt +0 -12
- data/spec/regression/expected/openssh.txt +0 -6
- data/spec/regression/expected/openssl.txt +0 -6
- data/spec/regression/expected/partial_search.txt +0 -7
- data/spec/regression/expected/passenger_apache2.txt +0 -31
- data/spec/regression/expected/perl.txt +0 -14
- data/spec/regression/expected/php.txt +0 -6
- data/spec/regression/expected/postfix.txt +0 -8
- data/spec/regression/expected/powershell.txt +0 -8
- data/spec/regression/expected/resolver.txt +0 -7
- data/spec/regression/expected/rsync.txt +0 -9
- data/spec/regression/expected/rsyslog.txt +0 -11
- data/spec/regression/expected/runit.txt +0 -4
- data/spec/regression/expected/smokeping.txt +0 -6
- data/spec/regression/expected/sql_server.txt +0 -7
- data/spec/regression/expected/ssh_known_hosts.txt +0 -6
- data/spec/regression/expected/stompserver.txt +0 -6
- data/spec/regression/expected/subversion.txt +0 -6
- data/spec/regression/expected/sudo.txt +0 -10
- data/spec/regression/expected/tar.txt +0 -10
- data/spec/regression/expected/tftp.txt +0 -6
- data/spec/regression/expected/tomcat.txt +0 -18
- data/spec/regression/expected/transmission.txt +0 -13
- data/spec/regression/expected/ubuntu.txt +0 -5
- data/spec/regression/expected/ufw.txt +0 -12
- data/spec/regression/expected/unicorn.txt +0 -7
- data/spec/regression/expected/users.txt +0 -12
- data/spec/regression/expected/vcruntime.txt +0 -8
- data/spec/regression/expected/vim.txt +0 -7
- data/spec/regression/expected/webpi.txt +0 -7
- data/spec/regression/expected/whitelist-node-attrs.txt +0 -7
- data/spec/regression/expected/windows.txt +0 -35
- data/spec/regression/expected/wix.txt +0 -5
- data/spec/regression/expected/xml.txt +0 -7
- data/spec/regression/expected/yum.txt +0 -15
- data/spec/regression/regression_spec.rb +0 -23
- data/spec/spec_helper.rb +0 -132
- data/spec/unit/api_spec.rb +0 -1939
- data/spec/unit/chef_spec.rb +0 -66
- data/spec/unit/command_line_spec.rb +0 -136
- data/spec/unit/domain_spec.rb +0 -78
- data/spec/unit/linter_spec.rb +0 -107
- data/spec/unit/template_spec.rb +0 -53
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC094" do
|
4
|
-
context "with a cookbook with a custom resource that includes node['filesystem2']" do
|
5
|
-
resource_file "node['filesystem2']"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes node['filesystem2']" do
|
10
|
-
recipe_file "node['filesystem2']"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes node['filesystem2']" do
|
15
|
-
library_file "node['filesystem2']"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC095" do
|
4
|
-
context "with a cookbook with a custom resource that includes node['cloud_v2']" do
|
5
|
-
resource_file "node['cloud_v2']"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes node['cloud_v2']" do
|
10
|
-
recipe_file "node['cloud_v2']"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes node['cloud_v2']" do
|
15
|
-
library_file "node['cloud_v2']"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC096" do
|
4
|
-
context "with a cookbook with a custom resource that includes node['virtualization']['uri']" do
|
5
|
-
resource_file "node['virtualization']['uri']"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes node['virtualization']['uri']" do
|
10
|
-
recipe_file "node['virtualization']['uri']"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes node['virtualization']['uri']" do
|
15
|
-
library_file "node['virtualization']['uri']"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC097" do
|
4
|
-
context "with a cookbook with a custom resource that includes Chef::Mixin::LanguageIncludeAttribute" do
|
5
|
-
resource_file "include Chef::Mixin::LanguageIncludeAttribute"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes Chef::Mixin::LanguageIncludeAttribute" do
|
10
|
-
recipe_file "include Chef::Mixin::LanguageIncludeAttribute"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes Chef::Mixin::LanguageIncludeAttribute" do
|
15
|
-
library_file "include Chef::Mixin::LanguageIncludeAttribute"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC098" do
|
4
|
-
context "with a cookbook with a custom resource that includes Chef::Mixin::RecipeDefinitionDSLCore" do
|
5
|
-
resource_file "include Chef::Mixin::RecipeDefinitionDSLCore"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes Chef::Mixin::RecipeDefinitionDSLCore" do
|
10
|
-
recipe_file "include Chef::Mixin::RecipeDefinitionDSLCore"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes Chef::Mixin::RecipeDefinitionDSLCore" do
|
15
|
-
library_file "include Chef::Mixin::RecipeDefinitionDSLCore"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC099" do
|
4
|
-
context "with a cookbook with a custom resource that includes Chef::Mixin::LanguageIncludeRecipe" do
|
5
|
-
resource_file "include Chef::Mixin::LanguageIncludeRecipe"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes Chef::Mixin::LanguageIncludeRecipe" do
|
10
|
-
recipe_file "include Chef::Mixin::LanguageIncludeRecipe"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes Chef::Mixin::LanguageIncludeRecipe" do
|
15
|
-
library_file "include Chef::Mixin::LanguageIncludeRecipe"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC100" do
|
4
|
-
context "with a cookbook with a custom resource that includes Chef::Mixin::Language" do
|
5
|
-
resource_file "include Chef::Mixin::Language"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes Chef::Mixin::Language" do
|
10
|
-
recipe_file "include Chef::Mixin::Language"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes Chef::Mixin::Language" do
|
15
|
-
library_file "include Chef::Mixin::Language"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,23 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC101" do
|
4
|
-
context "with a cookbook with a recipe that includes an deploy resource" do
|
5
|
-
recipe_file <<-EOF
|
6
|
-
deploy 'private_repo' do
|
7
|
-
repo 'git@github.com:acctname/private-repo.git'
|
8
|
-
user 'ubuntu'
|
9
|
-
deploy_to '/tmp/private_code'
|
10
|
-
ssh_wrapper '/tmp/private_code/wrap-ssh4git.sh'
|
11
|
-
action :deploy
|
12
|
-
end
|
13
|
-
EOF
|
14
|
-
it { is_expected.to violate_rule }
|
15
|
-
end
|
16
|
-
|
17
|
-
context "with a cookbook with a recipe that doesn't include an deploy resource" do
|
18
|
-
recipe_file <<-EOF
|
19
|
-
package "foo"
|
20
|
-
EOF
|
21
|
-
it { is_expected.not_to violate_rule }
|
22
|
-
end
|
23
|
-
end
|
@@ -1,18 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC102" do
|
4
|
-
context "with a cookbook with a custom resource that includes Chef::DSL::Recipe::FullDSL" do
|
5
|
-
resource_file "include Chef::DSL::Recipe::FullDSL"
|
6
|
-
it { is_expected.to violate_rule }
|
7
|
-
end
|
8
|
-
|
9
|
-
context "with a cookbook with a recipe that includes Chef::DSL::Recipe::FullDSL" do
|
10
|
-
recipe_file "include Chef::DSL::Recipe::FullDSL"
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a library that includes Chef::DSL::Recipe::FullDSL" do
|
15
|
-
library_file "include Chef::DSL::Recipe::FullDSL"
|
16
|
-
it { is_expected.to violate_rule }
|
17
|
-
end
|
18
|
-
end
|
@@ -1,30 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC103" do
|
4
|
-
context "with a cookbook with a custom resource that uses the :uninstall resource in chocolatey_package" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
chocolatey_package 'name' do
|
7
|
-
action :uninstall
|
8
|
-
end
|
9
|
-
EOF
|
10
|
-
it { is_expected.to violate_rule }
|
11
|
-
end
|
12
|
-
|
13
|
-
context "with a cookbook with a recipe that uses the :uninstall resource in chocolatey_package" do
|
14
|
-
recipe_file <<-EOF
|
15
|
-
chocolatey_package 'name' do
|
16
|
-
action :uninstall
|
17
|
-
end
|
18
|
-
EOF
|
19
|
-
it { is_expected.to violate_rule }
|
20
|
-
end
|
21
|
-
|
22
|
-
context "with a cookbook with a library that uses the :uninstall resource in chocolatey_package" do
|
23
|
-
library_file <<-EOF
|
24
|
-
chocolatey_package 'name' do
|
25
|
-
action :uninstall
|
26
|
-
end
|
27
|
-
EOF
|
28
|
-
it { is_expected.to violate_rule }
|
29
|
-
end
|
30
|
-
end
|
@@ -1,54 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC104" do
|
4
|
-
context "with a cookbook with a recipe where ruby_block specifies a create action" do
|
5
|
-
recipe_file <<-EOF
|
6
|
-
ruby_block 'puts' do
|
7
|
-
block do
|
8
|
-
puts "test test test"
|
9
|
-
end
|
10
|
-
action :create
|
11
|
-
end
|
12
|
-
EOF
|
13
|
-
it { is_expected.to violate_rule }
|
14
|
-
end
|
15
|
-
|
16
|
-
context "with a cookbook with a recipe where ruby_block specifies a run action" do
|
17
|
-
recipe_file <<-EOF
|
18
|
-
ruby_block 'puts' do
|
19
|
-
block do
|
20
|
-
puts "test test test"
|
21
|
-
end
|
22
|
-
action :run
|
23
|
-
end
|
24
|
-
EOF
|
25
|
-
it { is_expected.to_not violate_rule }
|
26
|
-
end
|
27
|
-
|
28
|
-
context "with a cookbook with a recipe that notifies :create on a ruby_block" do
|
29
|
-
recipe_file <<-EOF
|
30
|
-
file 'foo' do
|
31
|
-
notifies :create, 'ruby_block[bar]', :delayed
|
32
|
-
end
|
33
|
-
EOF
|
34
|
-
it { is_expected.to violate_rule }
|
35
|
-
end
|
36
|
-
|
37
|
-
context "with a cookbook with a recipe that notifies :run on a ruby_block" do
|
38
|
-
recipe_file <<-EOF
|
39
|
-
file 'foo' do
|
40
|
-
notifies :run, 'ruby_block[bar]', :delayed
|
41
|
-
end
|
42
|
-
EOF
|
43
|
-
it { is_expected.to_not violate_rule }
|
44
|
-
end
|
45
|
-
|
46
|
-
context "with a cookbook with a recipe that has a non-ruby_block resource with :create action" do
|
47
|
-
recipe_file <<-EOF
|
48
|
-
directory 'foo' do
|
49
|
-
action :create
|
50
|
-
end
|
51
|
-
EOF
|
52
|
-
it { is_expected.to_not violate_rule }
|
53
|
-
end
|
54
|
-
end
|
@@ -1,36 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC105" do
|
4
|
-
context "with a cookbook with a custom resource that includes an erl_call resource" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
erl_call 'list names' do
|
7
|
-
code 'net_adm:names().'
|
8
|
-
distributed true
|
9
|
-
node_name 'chef@latte'
|
10
|
-
end
|
11
|
-
EOF
|
12
|
-
it { is_expected.to violate_rule }
|
13
|
-
end
|
14
|
-
|
15
|
-
context "with a cookbook with a recipe that includes an erl_call resource" do
|
16
|
-
recipe_file <<-EOF
|
17
|
-
erl_call 'list names' do
|
18
|
-
code 'net_adm:names().'
|
19
|
-
distributed true
|
20
|
-
node_name 'chef@latte'
|
21
|
-
end
|
22
|
-
EOF
|
23
|
-
it { is_expected.to violate_rule }
|
24
|
-
end
|
25
|
-
|
26
|
-
context "with a cookbook with a library that includes an erl_call resource" do
|
27
|
-
library_file <<-EOF
|
28
|
-
erl_call 'list names' do
|
29
|
-
code 'net_adm:names().'
|
30
|
-
distributed true
|
31
|
-
node_name 'chef@latte'
|
32
|
-
end
|
33
|
-
EOF
|
34
|
-
it { is_expected.to violate_rule }
|
35
|
-
end
|
36
|
-
end
|
@@ -1,21 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC106" do
|
4
|
-
context "with a cookbook with a recipe that uses hash in a launchd resource" do
|
5
|
-
recipe_file <<-EOF
|
6
|
-
launchd 'foo' do
|
7
|
-
hash {}
|
8
|
-
end
|
9
|
-
EOF
|
10
|
-
it { is_expected.to violate_rule }
|
11
|
-
end
|
12
|
-
|
13
|
-
context "with a cookbook with a recipe that uses plist_hash in a launchd resource" do
|
14
|
-
library_file <<-EOF
|
15
|
-
launchd 'foo' do
|
16
|
-
plist_hash {}
|
17
|
-
end
|
18
|
-
EOF
|
19
|
-
it { is_expected.to_not violate_rule }
|
20
|
-
end
|
21
|
-
end
|
@@ -1,22 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC107" do
|
4
|
-
context "with a cookbook with a recipe that uses uses epic_fail in a resource" do
|
5
|
-
recipe_file <<-EOF
|
6
|
-
chocolatey_package 'name' do
|
7
|
-
epic_fail true
|
8
|
-
end
|
9
|
-
EOF
|
10
|
-
it { is_expected.to violate_rule }
|
11
|
-
end
|
12
|
-
|
13
|
-
context "with a cookbook with a recipe that uses uses ignore_failure in a resource" do
|
14
|
-
library_file <<-EOF
|
15
|
-
chocolatey_package 'name' do
|
16
|
-
action :install
|
17
|
-
ignore_failure true
|
18
|
-
end
|
19
|
-
EOF
|
20
|
-
it { is_expected.to_not violate_rule }
|
21
|
-
end
|
22
|
-
end
|
@@ -1,43 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC108" do
|
4
|
-
context "with a cookbook with a custom resource that defines a name property" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
property :name, String, name_property: true
|
7
|
-
|
8
|
-
action :create do
|
9
|
-
end
|
10
|
-
EOF
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a custom resource that does not defines a name property" do
|
15
|
-
recipe_file <<-EOF
|
16
|
-
property :not_name, String, name_property: true
|
17
|
-
|
18
|
-
action :create do
|
19
|
-
end
|
20
|
-
EOF
|
21
|
-
it { is_expected.to_not violate_rule }
|
22
|
-
end
|
23
|
-
|
24
|
-
context "with a cookbook with a custom resource where a property contains a name variable" do
|
25
|
-
recipe_file <<-EOF
|
26
|
-
property :home, String, default: lazy { ::File.join('/home/', name) }
|
27
|
-
|
28
|
-
action :create do
|
29
|
-
end
|
30
|
-
EOF
|
31
|
-
it { is_expected.to_not violate_rule }
|
32
|
-
end
|
33
|
-
|
34
|
-
context "with a cookbook with a custom resource where a property contains a name symbol" do
|
35
|
-
recipe_file <<-EOF
|
36
|
-
property :foo, Symbol, default: :name
|
37
|
-
|
38
|
-
action :create do
|
39
|
-
end
|
40
|
-
EOF
|
41
|
-
it { is_expected.to_not violate_rule }
|
42
|
-
end
|
43
|
-
end
|
@@ -1,22 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC109" do
|
4
|
-
context "with a cookbook with a package resource that defines a provider" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
package 'foo' do
|
7
|
-
provider Chef::Provider::Package::Rpm
|
8
|
-
action :install
|
9
|
-
end
|
10
|
-
EOF
|
11
|
-
it { is_expected.to violate_rule }
|
12
|
-
end
|
13
|
-
|
14
|
-
context "with a cookbook with a package resource that doesn't define a provider" do
|
15
|
-
resource_file <<-EOF
|
16
|
-
package 'foo' do
|
17
|
-
action :install
|
18
|
-
end
|
19
|
-
EOF
|
20
|
-
it { is_expected.to_not violate_rule }
|
21
|
-
end
|
22
|
-
end
|
@@ -1,21 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC110" do
|
4
|
-
context "with a cookbook with a script resource that uses command" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
bash 'foo' do
|
7
|
-
command 'cat /etc/passwd'
|
8
|
-
end
|
9
|
-
EOF
|
10
|
-
it { is_expected.to violate_rule }
|
11
|
-
end
|
12
|
-
|
13
|
-
context "with a cookbook with a script resource that uses code" do
|
14
|
-
resource_file <<-EOF
|
15
|
-
bash 'foo' do
|
16
|
-
code 'cat /etc/passwd'
|
17
|
-
end
|
18
|
-
EOF
|
19
|
-
it { is_expected.not_to violate_rule }
|
20
|
-
end
|
21
|
-
end
|
@@ -1,17 +0,0 @@
|
|
1
|
-
require "spec_helper"
|
2
|
-
|
3
|
-
describe "FC111" do
|
4
|
-
context "with a cookbook with a search that uses the deprecated sort flag" do
|
5
|
-
resource_file <<-EOF
|
6
|
-
search(:node, 'role:web', :sort => true)
|
7
|
-
EOF
|
8
|
-
it { is_expected.to violate_rule }
|
9
|
-
end
|
10
|
-
|
11
|
-
context "with a cookbook with a search that does not use sort" do
|
12
|
-
resource_file <<-EOF
|
13
|
-
search(:node, 'role:web')
|
14
|
-
EOF
|
15
|
-
it { is_expected.not_to violate_rule }
|
16
|
-
end
|
17
|
-
end
|