chef 12.3.0 → 12.4.0.rc.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/Rakefile +86 -7
- data/distro/common/markdown/man1/chef-shell.mkd +4 -4
- data/distro/common/markdown/man1/knife-bootstrap.mkd +1 -1
- data/distro/common/markdown/man1/knife-client.mkd +1 -1
- data/distro/common/markdown/man1/knife-configure.mkd +1 -1
- data/distro/common/markdown/man1/knife-cookbook-site.mkd +1 -1
- data/distro/common/markdown/man1/knife-cookbook.mkd +4 -4
- data/distro/common/markdown/man1/knife-data-bag.mkd +1 -1
- data/distro/common/markdown/man1/knife-environment.mkd +3 -3
- data/distro/common/markdown/man1/knife-exec.mkd +1 -1
- data/distro/common/markdown/man1/knife-index.mkd +1 -1
- data/distro/common/markdown/man1/knife-node.mkd +1 -1
- data/distro/common/markdown/man1/knife-role.mkd +3 -3
- data/distro/common/markdown/man1/knife-search.mkd +2 -2
- data/distro/common/markdown/man1/knife-ssh.mkd +1 -1
- data/distro/common/markdown/man1/knife-status.mkd +1 -1
- data/distro/common/markdown/man1/knife-tag.mkd +1 -1
- data/distro/common/markdown/man1/knife.mkd +2 -2
- data/distro/common/markdown/man8/chef-client.mkd +1 -2
- data/distro/common/markdown/man8/chef-expander.mkd +1 -2
- data/distro/common/markdown/man8/chef-expanderctl.mkd +1 -2
- data/distro/common/markdown/man8/chef-server-webui.mkd +1 -1
- data/distro/common/markdown/man8/chef-server.mkd +1 -2
- data/distro/common/markdown/man8/chef-solo.mkd +2 -2
- data/distro/common/markdown/man8/chef-solr.mkd +1 -1
- data/lib/chef/client.rb +2 -2
- data/lib/chef/config.rb +17 -709
- data/lib/chef/cookbook/metadata.rb +9 -5
- data/lib/chef/cookbook_loader.rb +1 -1
- data/lib/chef/cookbook_site_streaming_uploader.rb +2 -18
- data/lib/chef/dsl/definitions.rb +44 -0
- data/lib/chef/dsl/recipe.rb +50 -35
- data/lib/chef/dsl/resources.rb +28 -0
- data/lib/chef/event_dispatch/dispatcher.rb +2 -0
- data/lib/chef/event_loggers/windows_eventlog.rb +1 -11
- data/lib/chef/exceptions.rb +4 -0
- data/lib/chef/file_access_control/unix.rb +5 -0
- data/lib/chef/formatters/error_inspectors/api_error_formatting.rb +17 -0
- data/lib/chef/formatters/error_inspectors/compile_error_inspector.rb +21 -9
- data/lib/chef/formatters/error_inspectors/cookbook_resolve_error_inspector.rb +2 -0
- data/lib/chef/formatters/error_inspectors/cookbook_sync_error_inspector.rb +2 -0
- data/lib/chef/formatters/error_inspectors/node_load_error_inspector.rb +2 -0
- data/lib/chef/formatters/error_inspectors/registration_error_inspector.rb +4 -0
- data/lib/chef/formatters/error_inspectors/resource_failure_inspector.rb +11 -1
- data/lib/chef/formatters/error_inspectors/run_list_expansion_error_inspector.rb +2 -0
- data/lib/chef/http/authenticator.rb +3 -0
- data/lib/chef/http/basic_client.rb +10 -6
- data/lib/chef/http/json_input.rb +6 -1
- data/lib/chef/key.rb +271 -0
- data/lib/chef/knife.rb +11 -1
- data/lib/chef/knife/bootstrap.rb +6 -0
- data/lib/chef/knife/bootstrap/templates/chef-full.erb +166 -23
- data/lib/chef/knife/client_key_create.rb +67 -0
- data/lib/chef/knife/client_key_delete.rb +76 -0
- data/lib/chef/knife/client_key_edit.rb +80 -0
- data/lib/chef/knife/client_key_list.rb +69 -0
- data/lib/chef/knife/client_key_show.rb +76 -0
- data/lib/chef/knife/key_create.rb +108 -0
- data/lib/chef/knife/key_create_base.rb +50 -0
- data/lib/chef/knife/key_delete.rb +55 -0
- data/lib/chef/knife/key_edit.rb +114 -0
- data/lib/chef/knife/key_edit_base.rb +55 -0
- data/lib/chef/knife/key_list.rb +88 -0
- data/lib/chef/knife/key_list_base.rb +45 -0
- data/lib/chef/knife/key_show.rb +53 -0
- data/lib/chef/knife/ssh.rb +26 -28
- data/lib/chef/knife/user_key_create.rb +69 -0
- data/lib/chef/knife/user_key_delete.rb +76 -0
- data/lib/chef/knife/user_key_edit.rb +80 -0
- data/lib/chef/knife/user_key_list.rb +69 -0
- data/lib/chef/knife/user_key_show.rb +76 -0
- data/lib/chef/log.rb +2 -0
- data/lib/chef/log/syslog.rb +46 -0
- data/lib/chef/log/winevt.rb +99 -0
- data/lib/chef/mixin/provides.rb +8 -1
- data/lib/chef/mixin/unformatter.rb +32 -0
- data/lib/chef/mixin/uris.rb +33 -0
- data/lib/chef/mixin/wstring.rb +31 -0
- data/lib/chef/node.rb +21 -2
- data/lib/chef/platform/provider_mapping.rb +8 -4
- data/lib/chef/platform/query_helpers.rb +1 -5
- data/lib/chef/platform/service_helpers.rb +21 -21
- data/lib/chef/provider.rb +33 -0
- data/lib/chef/provider/cron/unix.rb +1 -0
- data/lib/chef/provider/file.rb +5 -3
- data/lib/chef/provider/lwrp_base.rb +76 -58
- data/lib/chef/provider/ohai.rb +1 -0
- data/lib/chef/provider/package.rb +7 -4
- data/lib/chef/provider/package/aix.rb +1 -0
- data/lib/chef/provider/package/smartos.rb +5 -5
- data/lib/chef/provider/package/windows.rb +90 -6
- data/lib/chef/provider/package/yum.rb +102 -32
- data/lib/chef/provider/reboot.rb +1 -0
- data/lib/chef/provider/registry_key.rb +2 -0
- data/lib/chef/provider/remote_file.rb +1 -0
- data/lib/chef/provider/remote_file/content.rb +5 -1
- data/lib/chef/provider/remote_file/fetcher.rb +22 -8
- data/lib/chef/provider/remote_file/network_file.rb +48 -0
- data/lib/chef/provider/service/aix.rb +13 -12
- data/lib/chef/provider_resolver.rb +87 -0
- data/lib/chef/providers.rb +1 -0
- data/lib/chef/resource.rb +67 -8
- data/lib/chef/resource/bash.rb +1 -0
- data/lib/chef/resource/bff_package.rb +1 -2
- data/lib/chef/resource/breakpoint.rb +1 -0
- data/lib/chef/resource/csh.rb +1 -0
- data/lib/chef/resource/deploy.rb +1 -0
- data/lib/chef/resource/erl_call.rb +1 -0
- data/lib/chef/resource/execute.rb +1 -0
- data/lib/chef/resource/file.rb +18 -0
- data/lib/chef/resource/http_request.rb +1 -0
- data/lib/chef/resource/ifconfig.rb +1 -2
- data/lib/chef/resource/log.rb +1 -2
- data/lib/chef/resource/lwrp_base.rb +106 -87
- data/lib/chef/resource/ohai.rb +1 -0
- data/lib/chef/resource/package.rb +1 -0
- data/lib/chef/resource/perl.rb +1 -0
- data/lib/chef/resource/portage_package.rb +1 -0
- data/lib/chef/resource/python.rb +1 -0
- data/lib/chef/resource/reboot.rb +2 -0
- data/lib/chef/resource/registry_key.rb +1 -0
- data/lib/chef/resource/remote_file.rb +1 -1
- data/lib/chef/resource/route.rb +1 -2
- data/lib/chef/resource/ruby.rb +1 -0
- data/lib/chef/resource/ruby_block.rb +1 -0
- data/lib/chef/resource/scm.rb +1 -0
- data/lib/chef/resource/script.rb +1 -0
- data/lib/chef/resource/service.rb +1 -0
- data/lib/chef/resource/subversion.rb +1 -0
- data/lib/chef/resource/whyrun_safe_ruby_block.rb +1 -0
- data/lib/chef/resource/windows_package.rb +24 -2
- data/lib/chef/resource/windows_script.rb +1 -0
- data/lib/chef/resource/yum_package.rb +1 -1
- data/lib/chef/resource_definition.rb +1 -0
- data/lib/chef/resource_reporter.rb +3 -10
- data/lib/chef/resource_resolver.rb +20 -10
- data/lib/chef/run_context.rb +1 -0
- data/lib/chef/run_list/versioned_recipe_list.rb +18 -0
- data/lib/chef/run_status.rb +2 -4
- data/lib/chef/shell.rb +1 -1
- data/lib/chef/util/path_helper.rb +3 -204
- data/lib/chef/util/windows/net_user.rb +73 -118
- data/lib/chef/version.rb +8 -4
- data/lib/chef/win32/api.rb +2 -1
- data/lib/chef/win32/api/installer.rb +1 -1
- data/lib/chef/win32/api/net.rb +115 -2
- data/lib/chef/win32/api/security.rb +24 -0
- data/lib/chef/win32/api/unicode.rb +1 -1
- data/lib/chef/win32/eventlog.rb +31 -0
- data/lib/chef/win32/net.rb +190 -0
- data/lib/chef/win32/security.rb +51 -2
- data/lib/chef/win32/security/sid.rb +17 -0
- data/spec/data/lwrp/providers/buck_passer.rb +18 -2
- data/spec/data/lwrp/providers/buck_passer_2.rb +18 -2
- data/spec/data/lwrp/providers/embedded_resource_accesses_providers_scope.rb +14 -2
- data/spec/data/lwrp_override/resources/foo.rb +5 -0
- data/spec/functional/knife/ssh_spec.rb +2 -2
- data/spec/functional/rebooter_spec.rb +1 -1
- data/spec/functional/resource/aixinit_service_spec.rb +1 -1
- data/spec/functional/resource/user/windows_spec.rb +125 -0
- data/spec/functional/shell_spec.rb +25 -10
- data/spec/functional/win32/sid_spec.rb +55 -0
- data/spec/integration/client/client_spec.rb +53 -29
- data/spec/integration/knife/deps_spec.rb +8 -14
- data/spec/integration/knife/upload_spec.rb +18 -0
- data/spec/integration/recipes/lwrp_inline_resources_spec.rb +1 -1
- data/spec/integration/recipes/provider_choice.rb +41 -0
- data/spec/integration/recipes/recipe_dsl_spec.rb +259 -0
- data/spec/spec_helper.rb +6 -1
- data/spec/support/key_helpers.rb +104 -0
- data/spec/support/lib/chef/resource/cat.rb +1 -0
- data/spec/support/lib/chef/resource/one_two_three_four.rb +2 -0
- data/spec/support/lib/chef/resource/zen_follower.rb +1 -0
- data/spec/support/lib/chef/resource/zen_master.rb +2 -0
- data/spec/support/mock/platform.rb +1 -1
- data/spec/support/pedant/Gemfile.lock +67 -0
- data/spec/support/shared/functional/securable_resource.rb +34 -12
- data/spec/support/shared/integration/integration_helper.rb +6 -5
- data/spec/support/shared/unit/provider/file.rb +31 -8
- data/spec/unit/application/client_spec.rb +2 -2
- data/spec/unit/client_spec.rb +21 -4
- data/spec/unit/cookbook/cookbook_version_loader_spec.rb +1 -1
- data/spec/unit/cookbook/metadata_spec.rb +15 -0
- data/spec/unit/cookbook/syntax_check_spec.rb +1 -1
- data/spec/unit/cookbook_loader_spec.rb +1 -1
- data/spec/unit/cookbook_site_streaming_uploader_spec.rb +0 -21
- data/spec/unit/data_bag_spec.rb +1 -1
- data/spec/unit/event_dispatch/dispatcher_spec.rb +61 -0
- data/spec/unit/formatters/error_inspectors/api_error_formatting_spec.rb +75 -0
- data/spec/unit/formatters/error_inspectors/compile_error_inspector_spec.rb +149 -112
- data/spec/unit/formatters/error_inspectors/resource_failure_inspector_spec.rb +7 -0
- data/spec/unit/guard_interpreter/resource_guard_interpreter_spec.rb +1 -1
- data/spec/unit/http/authenticator_spec.rb +69 -0
- data/spec/unit/http/basic_client_spec.rb +16 -0
- data/spec/unit/key_spec.rb +634 -0
- data/spec/unit/knife/bootstrap_spec.rb +14 -1
- data/spec/unit/knife/core/subcommand_loader_spec.rb +1 -1
- data/spec/unit/knife/core/ui_spec.rb +1 -1
- data/spec/unit/knife/data_bag_from_file_spec.rb +1 -1
- data/spec/unit/knife/environment_from_file_spec.rb +1 -1
- data/spec/unit/knife/key_create_spec.rb +224 -0
- data/spec/unit/knife/key_delete_spec.rb +135 -0
- data/spec/unit/knife/key_edit_spec.rb +267 -0
- data/spec/unit/knife/key_helper.rb +74 -0
- data/spec/unit/knife/key_list_spec.rb +216 -0
- data/spec/unit/knife/key_show_spec.rb +126 -0
- data/spec/unit/knife/ssh_spec.rb +23 -26
- data/spec/unit/knife_spec.rb +33 -1
- data/spec/unit/log/syslog_spec.rb +53 -0
- data/spec/unit/log/winevt_spec.rb +55 -0
- data/spec/unit/lwrp_spec.rb +105 -51
- data/spec/unit/mixin/path_sanity_spec.rb +2 -2
- data/spec/unit/mixin/template_spec.rb +2 -2
- data/spec/unit/mixin/unformatter_spec.rb +61 -0
- data/spec/unit/mixin/uris_spec.rb +45 -0
- data/spec/unit/platform/query_helpers_spec.rb +1 -1
- data/spec/unit/policy_builder/policyfile_spec.rb +7 -3
- data/spec/unit/provider/deploy/revision_spec.rb +1 -1
- data/spec/unit/provider/deploy_spec.rb +1 -1
- data/spec/unit/provider/directory_spec.rb +1 -1
- data/spec/unit/provider/execute_spec.rb +1 -1
- data/spec/unit/provider/package/aix_spec.rb +20 -8
- data/spec/unit/provider/package/smartos_spec.rb +50 -40
- data/spec/unit/provider/package/windows_spec.rb +104 -25
- data/spec/unit/provider/package/yum_spec.rb +111 -1
- data/spec/unit/provider/package_spec.rb +6 -0
- data/spec/unit/provider/remote_file/fetcher_spec.rb +20 -1
- data/spec/unit/provider/remote_file/network_file_spec.rb +45 -0
- data/spec/unit/provider/service/aix_service_spec.rb +26 -11
- data/spec/unit/provider/user/dscl_spec.rb +1 -1
- data/spec/unit/provider_spec.rb +20 -0
- data/spec/unit/recipe_spec.rb +1 -1
- data/spec/unit/resource/batch_spec.rb +1 -0
- data/spec/unit/resource/powershell_spec.rb +1 -0
- data/spec/unit/resource/remote_file_spec.rb +10 -0
- data/spec/unit/resource/windows_package_spec.rb +16 -2
- data/spec/unit/resource_spec.rb +40 -24
- data/spec/unit/rest_spec.rb +10 -20
- data/spec/unit/role_spec.rb +1 -1
- data/spec/unit/run_context_spec.rb +31 -0
- data/spec/unit/shell_spec.rb +4 -4
- data/tasks/external_tests.rb +29 -0
- data/tasks/rspec.rb +14 -1
- metadata +105 -28
- data/spec/unit/config_spec.rb +0 -544
- data/spec/unit/util/path_helper_spec.rb +0 -255
@@ -1,255 +0,0 @@
|
|
1
|
-
#
|
2
|
-
# Author:: Bryan McLellan <btm@loftninjas.org>
|
3
|
-
# Copyright:: Copyright (c) 2014 Chef Software, Inc.
|
4
|
-
# License:: Apache License, Version 2.0
|
5
|
-
#
|
6
|
-
# Licensed under the Apache License, Version 2.0 (the "License");
|
7
|
-
# you may not use this file except in compliance with the License.
|
8
|
-
# You may obtain a copy of the License at
|
9
|
-
#
|
10
|
-
# http://www.apache.org/licenses/LICENSE-2.0
|
11
|
-
#
|
12
|
-
# Unless required by applicable law or agreed to in writing, software
|
13
|
-
# distributed under the License is distributed on an "AS IS" BASIS,
|
14
|
-
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
15
|
-
# See the License for the specific language governing permissions and
|
16
|
-
# limitations under the License.
|
17
|
-
#
|
18
|
-
|
19
|
-
require 'chef/util/path_helper'
|
20
|
-
require 'spec_helper'
|
21
|
-
|
22
|
-
describe Chef::Util::PathHelper do
|
23
|
-
PathHelper = Chef::Util::PathHelper
|
24
|
-
|
25
|
-
[ false, true ].each do |is_windows|
|
26
|
-
context "on #{is_windows ? "windows" : "unix"}" do
|
27
|
-
before(:each) do
|
28
|
-
allow(Chef::Platform).to receive(:windows?).and_return(is_windows)
|
29
|
-
end
|
30
|
-
|
31
|
-
describe "join" do
|
32
|
-
it "joins components when some end with separators" do
|
33
|
-
expected = PathHelper.cleanpath("/foo/bar/baz")
|
34
|
-
expected = "C:#{expected}" if is_windows
|
35
|
-
expect(PathHelper.join(is_windows ? 'C:\\foo\\' : "/foo/", "bar", "baz")).to eq(expected)
|
36
|
-
end
|
37
|
-
|
38
|
-
it "joins components when some end and start with separators" do
|
39
|
-
expected = PathHelper.cleanpath("/foo/bar/baz")
|
40
|
-
expected = "C:#{expected}" if is_windows
|
41
|
-
expect(PathHelper.join(is_windows ? 'C:\\foo\\' : "/foo/", "bar/", "/baz")).to eq(expected)
|
42
|
-
end
|
43
|
-
|
44
|
-
it "joins components that don't end in separators" do
|
45
|
-
expected = PathHelper.cleanpath("/foo/bar/baz")
|
46
|
-
expected = "C:#{expected}" if is_windows
|
47
|
-
expect(PathHelper.join(is_windows ? 'C:\\foo' : "/foo", "bar", "baz")).to eq(expected)
|
48
|
-
end
|
49
|
-
|
50
|
-
it "joins starting with '' resolve to absolute paths" do
|
51
|
-
expect(PathHelper.join('', 'a', 'b')).to eq("#{PathHelper.path_separator}a#{PathHelper.path_separator}b")
|
52
|
-
end
|
53
|
-
|
54
|
-
it "joins ending with '' add a / to the end" do
|
55
|
-
expect(PathHelper.join('a', 'b', '')).to eq("a#{PathHelper.path_separator}b#{PathHelper.path_separator}")
|
56
|
-
end
|
57
|
-
|
58
|
-
if is_windows
|
59
|
-
it "joins components on Windows when some end with unix separators" do
|
60
|
-
expect(PathHelper.join('C:\\foo/', "bar", "baz")).to eq('C:\\foo\\bar\\baz')
|
61
|
-
end
|
62
|
-
end
|
63
|
-
end
|
64
|
-
|
65
|
-
if is_windows
|
66
|
-
it "path_separator is \\" do
|
67
|
-
expect(PathHelper.path_separator).to eq('\\')
|
68
|
-
end
|
69
|
-
else
|
70
|
-
it "path_separator is /" do
|
71
|
-
expect(PathHelper.path_separator).to eq('/')
|
72
|
-
end
|
73
|
-
end
|
74
|
-
|
75
|
-
if is_windows
|
76
|
-
it "cleanpath changes slashes into backslashes and leaves backslashes alone" do
|
77
|
-
expect(PathHelper.cleanpath('/a/b\\c/d/')).to eq('\\a\\b\\c\\d')
|
78
|
-
end
|
79
|
-
it "cleanpath does not remove leading double backslash" do
|
80
|
-
expect(PathHelper.cleanpath('\\\\a/b\\c/d/')).to eq('\\\\a\\b\\c\\d')
|
81
|
-
end
|
82
|
-
else
|
83
|
-
it "cleanpath removes extra slashes alone" do
|
84
|
-
expect(PathHelper.cleanpath('/a///b/c/d/')).to eq('/a/b/c/d')
|
85
|
-
end
|
86
|
-
end
|
87
|
-
|
88
|
-
describe "dirname" do
|
89
|
-
it "dirname('abc') is '.'" do
|
90
|
-
expect(PathHelper.dirname('abc')).to eq('.')
|
91
|
-
end
|
92
|
-
it "dirname('/') is '/'" do
|
93
|
-
expect(PathHelper.dirname(PathHelper.path_separator)).to eq(PathHelper.path_separator)
|
94
|
-
end
|
95
|
-
it "dirname('a/b/c') is 'a/b'" do
|
96
|
-
expect(PathHelper.dirname(PathHelper.join('a', 'b', 'c'))).to eq(PathHelper.join('a', 'b'))
|
97
|
-
end
|
98
|
-
it "dirname('a/b/c/') is 'a/b'" do
|
99
|
-
expect(PathHelper.dirname(PathHelper.join('a', 'b', 'c', ''))).to eq(PathHelper.join('a', 'b'))
|
100
|
-
end
|
101
|
-
it "dirname('/a/b/c') is '/a/b'" do
|
102
|
-
expect(PathHelper.dirname(PathHelper.join('', 'a', 'b', 'c'))).to eq(PathHelper.join('', 'a', 'b'))
|
103
|
-
end
|
104
|
-
end
|
105
|
-
end
|
106
|
-
end
|
107
|
-
|
108
|
-
describe "validate_path" do
|
109
|
-
context "on windows" do
|
110
|
-
before(:each) do
|
111
|
-
# pass by default
|
112
|
-
allow(Chef::Platform).to receive(:windows?).and_return(true)
|
113
|
-
allow(PathHelper).to receive(:printable?).and_return(true)
|
114
|
-
allow(PathHelper).to receive(:windows_max_length_exceeded?).and_return(false)
|
115
|
-
end
|
116
|
-
|
117
|
-
it "returns the path if the path passes the tests" do
|
118
|
-
expect(PathHelper.validate_path("C:\\ThisIsRigged")).to eql("C:\\ThisIsRigged")
|
119
|
-
end
|
120
|
-
|
121
|
-
it "does not raise an error if everything looks great" do
|
122
|
-
expect { PathHelper.validate_path("C:\\cool path\\dude.exe") }.not_to raise_error
|
123
|
-
end
|
124
|
-
|
125
|
-
it "raises an error if the path has invalid characters" do
|
126
|
-
allow(PathHelper).to receive(:printable?).and_return(false)
|
127
|
-
expect { PathHelper.validate_path("Newline!\n") }.to raise_error(Chef::Exceptions::ValidationFailed)
|
128
|
-
end
|
129
|
-
|
130
|
-
it "Adds the \\\\?\\ prefix if the path exceeds MAX_LENGTH and does not have it" do
|
131
|
-
long_path = "C:\\" + "a" * 250 + "\\" + "b" * 250
|
132
|
-
prefixed_long_path = "\\\\?\\" + long_path
|
133
|
-
allow(PathHelper).to receive(:windows_max_length_exceeded?).and_return(true)
|
134
|
-
expect(PathHelper.validate_path(long_path)).to eql(prefixed_long_path)
|
135
|
-
end
|
136
|
-
end
|
137
|
-
end
|
138
|
-
|
139
|
-
describe "windows_max_length_exceeded?" do
|
140
|
-
it "returns true if the path is too long (259 + NUL) for the API" do
|
141
|
-
expect(PathHelper.windows_max_length_exceeded?("C:\\" + "a" * 250 + "\\" + "b" * 6)).to be_truthy
|
142
|
-
end
|
143
|
-
|
144
|
-
it "returns false if the path is not too long (259 + NUL) for the standard API" do
|
145
|
-
expect(PathHelper.windows_max_length_exceeded?("C:\\" + "a" * 250 + "\\" + "b" * 5)).to be_falsey
|
146
|
-
end
|
147
|
-
|
148
|
-
it "returns false if the path is over 259 characters but uses the \\\\?\\ prefix" do
|
149
|
-
expect(PathHelper.windows_max_length_exceeded?("\\\\?\\C:\\" + "a" * 250 + "\\" + "b" * 250)).to be_falsey
|
150
|
-
end
|
151
|
-
end
|
152
|
-
|
153
|
-
describe "printable?" do
|
154
|
-
it "returns true if the string contains no non-printable characters" do
|
155
|
-
expect(PathHelper.printable?("C:\\Program Files (x86)\\Microsoft Office\\Files.lst")).to be_truthy
|
156
|
-
end
|
157
|
-
|
158
|
-
it "returns true when given 'abc' in unicode" do
|
159
|
-
expect(PathHelper.printable?("\u0061\u0062\u0063")).to be_truthy
|
160
|
-
end
|
161
|
-
|
162
|
-
it "returns true when given japanese unicode" do
|
163
|
-
expect(PathHelper.printable?("\uff86\uff87\uff88")).to be_truthy
|
164
|
-
end
|
165
|
-
|
166
|
-
it "returns false if the string contains a non-printable character" do
|
167
|
-
expect(PathHelper.printable?("\my files\work\notes.txt")).to be_falsey
|
168
|
-
end
|
169
|
-
|
170
|
-
# This isn't necessarily a requirement, but here to be explicit about functionality.
|
171
|
-
it "returns false if the string contains a newline or tab" do
|
172
|
-
expect(PathHelper.printable?("\tThere's no way,\n\t *no* way,\n\t that you came from my loins.\n")).to be_falsey
|
173
|
-
end
|
174
|
-
end
|
175
|
-
|
176
|
-
describe "canonical_path" do
|
177
|
-
context "on windows", :windows_only do
|
178
|
-
it "returns an absolute path with backslashes instead of slashes" do
|
179
|
-
expect(PathHelper.canonical_path("\\\\?\\C:/windows/win.ini")).to eq("\\\\?\\c:\\windows\\win.ini")
|
180
|
-
end
|
181
|
-
|
182
|
-
it "adds the \\\\?\\ prefix if it is missing" do
|
183
|
-
expect(PathHelper.canonical_path("C:/windows/win.ini")).to eq("\\\\?\\c:\\windows\\win.ini")
|
184
|
-
end
|
185
|
-
|
186
|
-
it "returns a lowercase path" do
|
187
|
-
expect(PathHelper.canonical_path("\\\\?\\C:\\CASE\\INSENSITIVE")).to eq("\\\\?\\c:\\case\\insensitive")
|
188
|
-
end
|
189
|
-
end
|
190
|
-
|
191
|
-
context "not on windows", :unix_only do
|
192
|
-
it "returns a canonical path" do
|
193
|
-
expect(PathHelper.canonical_path("/etc//apache.d/sites-enabled/../sites-available/default")).to eq("/etc/apache.d/sites-available/default")
|
194
|
-
end
|
195
|
-
end
|
196
|
-
end
|
197
|
-
|
198
|
-
describe "paths_eql?" do
|
199
|
-
it "returns true if the paths are the same" do
|
200
|
-
allow(PathHelper).to receive(:canonical_path).with("bandit").and_return("c:/bandit/bandit")
|
201
|
-
allow(PathHelper).to receive(:canonical_path).with("../bandit/bandit").and_return("c:/bandit/bandit")
|
202
|
-
expect(PathHelper.paths_eql?("bandit", "../bandit/bandit")).to be_truthy
|
203
|
-
end
|
204
|
-
|
205
|
-
it "returns false if the paths are different" do
|
206
|
-
allow(PathHelper).to receive(:canonical_path).with("bandit").and_return("c:/Bo/Bandit")
|
207
|
-
allow(PathHelper).to receive(:canonical_path).with("../bandit/bandit").and_return("c:/bandit/bandit")
|
208
|
-
expect(PathHelper.paths_eql?("bandit", "../bandit/bandit")).to be_falsey
|
209
|
-
end
|
210
|
-
end
|
211
|
-
|
212
|
-
describe "escape_glob" do
|
213
|
-
it "escapes characters reserved by glob" do
|
214
|
-
path = "C:\\this\\*path\\[needs]\\escaping?"
|
215
|
-
escaped_path = "C:\\\\this\\\\\\*path\\\\\\[needs\\]\\\\escaping\\?"
|
216
|
-
expect(PathHelper.escape_glob(path)).to eq(escaped_path)
|
217
|
-
end
|
218
|
-
|
219
|
-
context "when given more than one argument" do
|
220
|
-
it "joins, cleanpaths, and escapes characters reserved by glob" do
|
221
|
-
args = ["this/*path", "[needs]", "escaping?"]
|
222
|
-
escaped_path = if windows?
|
223
|
-
"this\\\\\\*path\\\\\\[needs\\]\\\\escaping\\?"
|
224
|
-
else
|
225
|
-
"this/\\*path/\\[needs\\]/escaping\\?"
|
226
|
-
end
|
227
|
-
expect(PathHelper).to receive(:join).with(*args).and_call_original
|
228
|
-
expect(PathHelper).to receive(:cleanpath).and_call_original
|
229
|
-
expect(PathHelper.escape_glob(*args)).to eq(escaped_path)
|
230
|
-
end
|
231
|
-
end
|
232
|
-
end
|
233
|
-
|
234
|
-
describe "all_homes" do
|
235
|
-
before do
|
236
|
-
stub_const('ENV', env)
|
237
|
-
allow(Chef::Platform).to receive(:windows?).and_return(is_windows)
|
238
|
-
end
|
239
|
-
|
240
|
-
context "on windows" do
|
241
|
-
let (:is_windows) { true }
|
242
|
-
end
|
243
|
-
|
244
|
-
context "on unix" do
|
245
|
-
let (:is_windows) { false }
|
246
|
-
|
247
|
-
context "when HOME is not set" do
|
248
|
-
let (:env) { {} }
|
249
|
-
it "returns an empty array" do
|
250
|
-
expect(PathHelper.all_homes).to eq([])
|
251
|
-
end
|
252
|
-
end
|
253
|
-
end
|
254
|
-
end
|
255
|
-
end
|