puppet 6.17.0-x86-mingw32 → 7.1.0-x86-mingw32
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Potentially problematic release.
This version of puppet might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/CODEOWNERS +2 -16
- data/Gemfile +3 -4
- data/Gemfile.lock +32 -30
- data/README.md +2 -3
- data/Rakefile +4 -12
- data/conf/fileserver.conf +5 -10
- data/ext/build_defaults.yaml +1 -1
- data/ext/osx/file_mapping.yaml +0 -5
- data/ext/project_data.yaml +2 -14
- data/ext/redhat/puppet.spec.erb +0 -1
- data/ext/windows/service/daemon.rb +6 -5
- data/install.rb +21 -17
- data/lib/puppet.rb +11 -20
- data/lib/puppet/agent/locker.rb +0 -7
- data/lib/puppet/application.rb +172 -98
- data/lib/puppet/application/agent.rb +8 -3
- data/lib/puppet/application/apply.rb +18 -20
- data/lib/puppet/application/device.rb +100 -104
- data/lib/puppet/application/doc.rb +1 -1
- data/lib/puppet/application/filebucket.rb +15 -11
- data/lib/puppet/application/lookup.rb +16 -4
- data/lib/puppet/application/ssl.rb +1 -1
- data/lib/puppet/application_support.rb +7 -0
- data/lib/puppet/configurer.rb +46 -19
- data/lib/puppet/configurer/downloader.rb +31 -10
- data/lib/puppet/configurer/plugin_handler.rb +21 -19
- data/lib/puppet/confine.rb +1 -1
- data/lib/puppet/confine/any.rb +1 -1
- data/lib/puppet/defaults.rb +116 -162
- data/lib/puppet/environments.rb +72 -62
- data/lib/puppet/face/catalog.rb +1 -1
- data/lib/puppet/face/config.rb +56 -16
- data/lib/puppet/face/epp.rb +12 -2
- data/lib/puppet/face/facts.rb +66 -6
- data/lib/puppet/face/help.rb +1 -1
- data/lib/puppet/face/node.rb +3 -3
- data/lib/puppet/face/node/clean.rb +10 -2
- data/lib/puppet/face/plugin.rb +5 -8
- data/lib/puppet/feature/base.rb +1 -1
- data/lib/puppet/ffi/posix.rb +10 -0
- data/lib/puppet/ffi/posix/constants.rb +14 -0
- data/lib/puppet/ffi/posix/functions.rb +24 -0
- data/lib/puppet/ffi/windows.rb +12 -0
- data/lib/puppet/ffi/windows/api_types.rb +311 -0
- data/lib/puppet/ffi/windows/constants.rb +404 -0
- data/lib/puppet/ffi/windows/functions.rb +628 -0
- data/lib/puppet/ffi/windows/structs.rb +338 -0
- data/lib/puppet/file_bucket/dipper.rb +1 -1
- data/lib/puppet/file_serving/configuration.rb +0 -5
- data/lib/puppet/file_serving/configuration/parser.rb +3 -32
- data/lib/puppet/file_serving/http_metadata.rb +1 -1
- data/lib/puppet/file_serving/mount.rb +1 -2
- data/lib/puppet/file_serving/mount/locales.rb +1 -2
- data/lib/puppet/file_serving/mount/pluginfacts.rb +1 -2
- data/lib/puppet/file_serving/mount/plugins.rb +1 -2
- data/lib/puppet/file_system/file_impl.rb +3 -3
- data/lib/puppet/forge/repository.rb +0 -1
- data/lib/puppet/functions/epp.rb +1 -0
- data/lib/puppet/functions/inline_epp.rb +1 -0
- data/lib/puppet/functions/lstrip.rb +4 -4
- data/lib/puppet/functions/new.rb +8 -3
- data/lib/puppet/functions/reverse_each.rb +1 -1
- data/lib/puppet/functions/rstrip.rb +4 -4
- data/lib/puppet/functions/step.rb +1 -1
- data/lib/puppet/functions/strip.rb +4 -4
- data/lib/puppet/generate/models/type/type.rb +4 -1
- data/lib/puppet/gettext/config.rb +5 -5
- data/lib/puppet/gettext/module_translations.rb +4 -4
- data/lib/puppet/http.rb +23 -13
- data/lib/puppet/http/client.rb +165 -115
- data/lib/puppet/{network/resolver.rb → http/dns.rb} +2 -2
- data/lib/puppet/http/errors.rb +16 -0
- data/lib/puppet/http/external_client.rb +5 -7
- data/lib/puppet/{network/http → http}/factory.rb +8 -11
- data/lib/puppet/{network/http → http}/pool.rb +61 -26
- data/lib/puppet/{network/http/session.rb → http/pool_entry.rb} +2 -3
- data/lib/puppet/http/proxy.rb +137 -0
- data/lib/puppet/http/redirector.rb +4 -12
- data/lib/puppet/http/resolver.rb +10 -23
- data/lib/puppet/http/resolver/server_list.rb +23 -45
- data/lib/puppet/http/resolver/settings.rb +7 -10
- data/lib/puppet/http/resolver/srv.rb +11 -15
- data/lib/puppet/http/response.rb +36 -54
- data/lib/puppet/http/response_converter.rb +24 -0
- data/lib/puppet/http/response_net_http.rb +42 -0
- data/lib/puppet/http/retry_after_handler.rb +4 -13
- data/lib/puppet/http/service.rb +15 -27
- data/lib/puppet/http/service/ca.rb +11 -22
- data/lib/puppet/http/service/compiler.rb +23 -70
- data/lib/puppet/http/service/file_server.rb +19 -28
- data/lib/puppet/http/service/puppetserver.rb +53 -0
- data/lib/puppet/http/service/report.rb +8 -10
- data/lib/puppet/http/session.rb +16 -24
- data/lib/puppet/{network/http → http}/site.rb +1 -2
- data/lib/puppet/indirector/catalog/compiler.rb +1 -1
- data/lib/puppet/indirector/catalog/rest.rb +2 -4
- data/lib/puppet/indirector/exec.rb +1 -1
- data/lib/puppet/indirector/fact_search.rb +60 -0
- data/lib/puppet/indirector/facts/facter.rb +27 -6
- data/lib/puppet/indirector/facts/json.rb +27 -0
- data/lib/puppet/indirector/facts/rest.rb +3 -22
- data/lib/puppet/indirector/facts/yaml.rb +4 -59
- data/lib/puppet/indirector/file_bucket_file/rest.rb +3 -9
- data/lib/puppet/indirector/file_content/rest.rb +2 -6
- data/lib/puppet/indirector/file_metadata/http.rb +1 -0
- data/lib/puppet/indirector/file_metadata/rest.rb +3 -9
- data/lib/puppet/indirector/file_server.rb +1 -8
- data/lib/puppet/indirector/generic_http.rb +0 -11
- data/lib/puppet/indirector/hiera.rb +4 -0
- data/lib/puppet/indirector/indirection.rb +1 -1
- data/lib/puppet/indirector/json.rb +5 -1
- data/lib/puppet/indirector/msgpack.rb +1 -1
- data/lib/puppet/indirector/node/json.rb +8 -0
- data/lib/puppet/indirector/node/rest.rb +2 -4
- data/lib/puppet/indirector/report/json.rb +34 -0
- data/lib/puppet/indirector/report/processor.rb +2 -2
- data/lib/puppet/indirector/report/rest.rb +3 -8
- data/lib/puppet/indirector/request.rb +1 -102
- data/lib/puppet/indirector/rest.rb +12 -263
- data/lib/puppet/indirector/yaml.rb +1 -1
- data/lib/puppet/module.rb +1 -2
- data/lib/puppet/module_tool/applications.rb +0 -1
- data/lib/puppet/network/authconfig.rb +2 -96
- data/lib/puppet/network/authorization.rb +13 -35
- data/lib/puppet/network/format_support.rb +2 -2
- data/lib/puppet/network/formats.rb +2 -1
- data/lib/puppet/network/http.rb +3 -3
- data/lib/puppet/network/http/api/indirected_routes.rb +2 -20
- data/lib/puppet/network/http/api/master/v3.rb +11 -13
- data/lib/puppet/network/http/api/master/v3/environments.rb +0 -1
- data/lib/puppet/network/http/connection.rb +247 -316
- data/lib/puppet/network/http/handler.rb +0 -1
- data/lib/puppet/network/http/route.rb +2 -2
- data/lib/puppet/network/http_pool.rb +16 -34
- data/lib/puppet/node.rb +1 -30
- data/lib/puppet/node/environment.rb +12 -5
- data/lib/puppet/node/facts.rb +17 -0
- data/lib/puppet/pal/json_catalog_encoder.rb +4 -0
- data/lib/puppet/pal/pal_impl.rb +93 -14
- data/lib/puppet/parameter.rb +1 -1
- data/lib/puppet/parser/ast/pops_bridge.rb +0 -38
- data/lib/puppet/parser/compiler.rb +0 -198
- data/lib/puppet/parser/compiler/catalog_validator/relationship_validator.rb +14 -39
- data/lib/puppet/parser/functions.rb +21 -17
- data/lib/puppet/parser/functions/create_resources.rb +11 -7
- data/lib/puppet/parser/resource.rb +0 -69
- data/lib/puppet/parser/templatewrapper.rb +1 -1
- data/lib/puppet/parser/type_loader.rb +2 -2
- data/lib/puppet/pops/adaptable.rb +7 -13
- data/lib/puppet/pops/adapters.rb +8 -4
- data/lib/puppet/pops/evaluator/collectors/abstract_collector.rb +1 -3
- data/lib/puppet/pops/evaluator/evaluator_impl.rb +22 -8
- data/lib/puppet/pops/evaluator/runtime3_converter.rb +2 -2
- data/lib/puppet/pops/evaluator/runtime3_resource_support.rb +3 -3
- data/lib/puppet/pops/evaluator/runtime3_support.rb +1 -1
- data/lib/puppet/pops/issues.rb +0 -5
- data/lib/puppet/pops/loader/ruby_legacy_function_instantiator.rb +6 -8
- data/lib/puppet/pops/loader/runtime3_type_loader.rb +4 -2
- data/lib/puppet/pops/loaders.rb +18 -11
- data/lib/puppet/pops/lookup/context.rb +1 -1
- data/lib/puppet/pops/lookup/hiera_config.rb +14 -1
- data/lib/puppet/pops/model/ast.pp +0 -42
- data/lib/puppet/pops/model/ast.rb +0 -290
- data/lib/puppet/pops/model/factory.rb +0 -45
- data/lib/puppet/pops/model/model_label_provider.rb +0 -5
- data/lib/puppet/pops/model/model_tree_dumper.rb +0 -22
- data/lib/puppet/pops/model/pn_transformer.rb +0 -16
- data/lib/puppet/pops/parser/egrammar.ra +0 -56
- data/lib/puppet/pops/parser/eparser.rb +1520 -1712
- data/lib/puppet/pops/parser/lexer2.rb +4 -4
- data/lib/puppet/pops/parser/parser_support.rb +0 -5
- data/lib/puppet/pops/resource/resource_type_impl.rb +2 -24
- data/lib/puppet/pops/types/iterable.rb +34 -8
- data/lib/puppet/pops/types/p_meta_type.rb +1 -1
- data/lib/puppet/pops/types/p_type_set_type.rb +4 -0
- data/lib/puppet/pops/types/type_calculator.rb +0 -7
- data/lib/puppet/pops/types/type_parser.rb +0 -4
- data/lib/puppet/pops/types/types.rb +0 -1
- data/lib/puppet/pops/validation/checker4_0.rb +28 -52
- data/lib/puppet/pops/validation/tasks_checker.rb +0 -12
- data/lib/puppet/pops/validation/validator_factory_4_0.rb +1 -2
- data/lib/puppet/provider.rb +0 -13
- data/lib/puppet/provider/file/windows.rb +1 -1
- data/lib/puppet/provider/nameservice.rb +0 -18
- data/lib/puppet/provider/package/apt.rb +34 -0
- data/lib/puppet/provider/package/dpkg.rb +1 -11
- data/lib/puppet/provider/package/gem.rb +27 -5
- data/lib/puppet/provider/package/pip.rb +0 -1
- data/lib/puppet/provider/package/pip2.rb +17 -0
- data/lib/puppet/provider/package/pkg.rb +0 -4
- data/lib/puppet/provider/package/portage.rb +1 -1
- data/lib/puppet/provider/package/puppet_gem.rb +6 -4
- data/lib/puppet/provider/package/puppetserver_gem.rb +180 -0
- data/lib/puppet/provider/package/yum.rb +1 -0
- data/lib/puppet/provider/package/zypper.rb +3 -0
- data/lib/puppet/provider/service/smf.rb +191 -73
- data/lib/puppet/provider/user/aix.rb +3 -3
- data/lib/puppet/provider/user/directoryservice.rb +0 -10
- data/lib/puppet/provider/user/user_role_add.rb +1 -1
- data/lib/puppet/provider/user/windows_adsi.rb +18 -1
- data/lib/puppet/reference/configuration.rb +2 -0
- data/lib/puppet/reference/indirection.rb +1 -1
- data/lib/puppet/resource.rb +1 -89
- data/lib/puppet/resource/catalog.rb +1 -14
- data/lib/puppet/resource/type.rb +5 -120
- data/lib/puppet/resource/type_collection.rb +3 -48
- data/lib/puppet/runtime.rb +1 -2
- data/lib/puppet/settings.rb +84 -35
- data/lib/puppet/settings/base_setting.rb +26 -2
- data/lib/puppet/settings/integer_setting.rb +17 -0
- data/lib/puppet/settings/port_setting.rb +15 -0
- data/lib/puppet/settings/priority_setting.rb +5 -4
- data/lib/puppet/ssl.rb +10 -6
- data/lib/puppet/ssl/base.rb +3 -5
- data/lib/puppet/ssl/certificate.rb +0 -6
- data/lib/puppet/ssl/certificate_request.rb +1 -12
- data/lib/puppet/ssl/certificate_signer.rb +6 -0
- data/lib/puppet/ssl/oids.rb +3 -1
- data/lib/puppet/ssl/ssl_provider.rb +17 -0
- data/lib/puppet/ssl/state_machine.rb +3 -1
- data/lib/puppet/ssl/verifier.rb +2 -0
- data/lib/puppet/test/test_helper.rb +11 -6
- data/lib/puppet/transaction.rb +3 -9
- data/lib/puppet/transaction/persistence.rb +1 -1
- data/lib/puppet/transaction/report.rb +10 -8
- data/lib/puppet/trusted_external.rb +2 -2
- data/lib/puppet/type.rb +4 -79
- data/lib/puppet/type/file.rb +7 -9
- data/lib/puppet/type/file/checksum.rb +1 -1
- data/lib/puppet/type/file/source.rb +29 -9
- data/lib/puppet/type/filebucket.rb +4 -4
- data/lib/puppet/type/notify.rb +2 -2
- data/lib/puppet/type/package.rb +5 -13
- data/lib/puppet/type/service.rb +4 -0
- data/lib/puppet/type/user.rb +19 -4
- data/lib/puppet/util.rb +26 -12
- data/lib/puppet/util/autoload.rb +9 -7
- data/lib/puppet/util/character_encoding.rb +9 -5
- data/lib/puppet/util/execution.rb +2 -13
- data/lib/puppet/util/http_proxy.rb +2 -215
- data/lib/puppet/util/monkey_patches.rb +0 -46
- data/lib/puppet/util/posix.rb +53 -4
- data/lib/puppet/util/rdoc.rb +0 -7
- data/lib/puppet/util/retry_action.rb +1 -1
- data/lib/puppet/util/rubygems.rb +5 -1
- data/lib/puppet/util/run_mode.rb +14 -2
- data/lib/puppet/util/windows.rb +3 -7
- data/lib/puppet/util/windows/daemon.rb +360 -0
- data/lib/puppet/util/windows/error.rb +1 -0
- data/lib/puppet/util/windows/eventlog.rb +4 -9
- data/lib/puppet/util/windows/file.rb +8 -242
- data/lib/puppet/util/windows/monkey_patches/process.rb +414 -0
- data/lib/puppet/util/windows/process.rb +4 -226
- data/lib/puppet/util/windows/security.rb +4 -4
- data/lib/puppet/util/windows/service.rb +9 -460
- data/lib/puppet/util/windows/string.rb +12 -13
- data/lib/puppet/util/windows/user.rb +219 -0
- data/lib/puppet/util/yaml.rb +0 -22
- data/lib/puppet/vendor/require_vendored.rb +0 -1
- data/lib/puppet/version.rb +1 -1
- data/lib/puppet/x509.rb +5 -1
- data/lib/puppet/x509/cert_provider.rb +29 -1
- data/locales/puppet.pot +656 -1351
- data/man/man5/puppet.conf.5 +72 -97
- data/man/man8/puppet-agent.8 +6 -3
- data/man/man8/puppet-apply.8 +1 -1
- data/man/man8/puppet-catalog.8 +1 -1
- data/man/man8/puppet-config.8 +6 -6
- data/man/man8/puppet-describe.8 +1 -1
- data/man/man8/puppet-device.8 +1 -1
- data/man/man8/puppet-doc.8 +1 -1
- data/man/man8/puppet-epp.8 +1 -1
- data/man/man8/puppet-facts.8 +55 -9
- data/man/man8/puppet-filebucket.8 +6 -6
- data/man/man8/puppet-generate.8 +1 -1
- data/man/man8/puppet-help.8 +1 -1
- data/man/man8/puppet-lookup.8 +2 -2
- data/man/man8/puppet-module.8 +1 -58
- data/man/man8/puppet-node.8 +7 -4
- data/man/man8/puppet-parser.8 +1 -1
- data/man/man8/puppet-plugin.8 +1 -1
- data/man/man8/puppet-report.8 +4 -1
- data/man/man8/puppet-resource.8 +1 -1
- data/man/man8/puppet-script.8 +1 -1
- data/man/man8/puppet-ssl.8 +1 -1
- data/man/man8/puppet.8 +2 -2
- data/spec/fixtures/integration/application/apply/environments/spec/modules/amod/lib/puppet/provider/applytest/applytest.rb +2 -0
- data/spec/fixtures/integration/application/apply/environments/spec/modules/amod/lib/puppet/type/applytest.rb +25 -0
- data/spec/fixtures/unit/forge/bacula-releases.json +128 -0
- data/spec/fixtures/unit/forge/bacula.tar.gz +0 -0
- data/spec/fixtures/unit/provider/package/puppetserver_gem/gem-list-local-packages +30 -0
- data/spec/fixtures/unit/provider/service/smf/{svcs.out → svcs_instances.out} +0 -0
- data/spec/fixtures/unit/provider/user/aix/aix_passwd_file.out +4 -0
- data/spec/integration/application/agent_spec.rb +70 -61
- data/spec/integration/application/apply_spec.rb +150 -150
- data/spec/integration/application/doc_spec.rb +16 -6
- data/spec/integration/application/filebucket_spec.rb +78 -29
- data/spec/integration/application/help_spec.rb +44 -0
- data/spec/integration/application/lookup_spec.rb +13 -0
- data/spec/integration/application/module_spec.rb +68 -0
- data/spec/integration/application/plugin_spec.rb +76 -4
- data/spec/integration/configurer_spec.rb +14 -0
- data/spec/integration/data_binding_spec.rb +82 -0
- data/spec/integration/defaults_spec.rb +32 -3
- data/spec/integration/directory_environments_spec.rb +17 -17
- data/spec/integration/environments/setting_hooks_spec.rb +1 -1
- data/spec/integration/indirector/facts/facter_spec.rb +8 -6
- data/spec/integration/network/http_pool_spec.rb +3 -21
- data/spec/integration/node/environment_spec.rb +1 -1
- data/spec/integration/parser/catalog_spec.rb +0 -38
- data/spec/integration/parser/node_spec.rb +0 -9
- data/spec/integration/parser/pcore_resource_spec.rb +0 -37
- data/spec/integration/type/file_spec.rb +5 -4
- data/spec/integration/util/execution_spec.rb +22 -0
- data/spec/integration/util/windows/adsi_spec.rb +2 -2
- data/spec/integration/util/windows/monkey_patches/process_spec.rb +231 -0
- data/spec/integration/util/windows/process_spec.rb +26 -32
- data/spec/integration/util/windows/security_spec.rb +1 -1
- data/spec/integration/util/windows/user_spec.rb +7 -0
- data/spec/integration/util_spec.rb +7 -33
- data/spec/lib/puppet_spec/matchers.rb +0 -80
- data/spec/lib/puppet_spec/puppetserver.rb +9 -1
- data/spec/lib/puppet_spec/settings.rb +7 -1
- data/spec/shared_contexts/types_setup.rb +2 -0
- data/spec/spec_helper.rb +2 -0
- data/spec/unit/agent_spec.rb +0 -2
- data/spec/unit/application/agent_spec.rb +3 -4
- data/spec/unit/application/config_spec.rb +224 -4
- data/spec/unit/application/doc_spec.rb +2 -2
- data/spec/unit/application/face_base_spec.rb +6 -4
- data/spec/unit/application/facts_spec.rb +74 -8
- data/spec/unit/application/filebucket_spec.rb +41 -39
- data/spec/unit/application/resource_spec.rb +3 -1
- data/spec/unit/application/ssl_spec.rb +17 -4
- data/spec/unit/application_spec.rb +43 -4
- data/spec/unit/certificate_factory_spec.rb +1 -1
- data/spec/unit/configurer/downloader_spec.rb +14 -0
- data/spec/unit/configurer/plugin_handler_spec.rb +56 -18
- data/spec/unit/configurer_spec.rb +96 -44
- data/spec/unit/confine_spec.rb +2 -1
- data/spec/unit/context/trusted_information_spec.rb +2 -6
- data/spec/unit/defaults_spec.rb +26 -32
- data/spec/unit/environments_spec.rb +173 -32
- data/spec/unit/face/config_spec.rb +65 -12
- data/spec/unit/face/facts_spec.rb +4 -0
- data/spec/unit/face/node_spec.rb +16 -4
- data/spec/unit/face/plugin_spec.rb +73 -33
- data/spec/unit/file_bucket/file_spec.rb +1 -1
- data/spec/unit/file_serving/configuration/parser_spec.rb +14 -18
- data/spec/unit/file_serving/configuration_spec.rb +6 -12
- data/spec/unit/file_serving/mount/locales_spec.rb +2 -2
- data/spec/unit/file_serving/mount/pluginfacts_spec.rb +2 -2
- data/spec/unit/file_serving/mount/plugins_spec.rb +2 -2
- data/spec/unit/file_system/uniquefile_spec.rb +18 -0
- data/spec/unit/file_system_spec.rb +1 -2
- data/spec/unit/functions/camelcase_spec.rb +1 -1
- data/spec/unit/functions/capitalize_spec.rb +1 -1
- data/spec/unit/functions/downcase_spec.rb +1 -1
- data/spec/unit/functions/inline_epp_spec.rb +26 -1
- data/spec/unit/functions/upcase_spec.rb +1 -1
- data/spec/unit/http/client_spec.rb +7 -9
- data/spec/unit/{network/resolver_spec.rb → http/dns_spec.rb} +3 -3
- data/spec/unit/http/external_client_spec.rb +4 -4
- data/spec/unit/{network/http → http}/factory_spec.rb +5 -11
- data/spec/unit/{network/http/session_spec.rb → http/pool_entry_spec.rb} +3 -3
- data/spec/unit/{network/http → http}/pool_spec.rb +12 -17
- data/spec/unit/{util/http_proxy_spec.rb → http/proxy_spec.rb} +2 -69
- data/spec/unit/http/resolver_spec.rb +34 -15
- data/spec/unit/http/service/ca_spec.rb +2 -3
- data/spec/unit/http/service/compiler_spec.rb +51 -65
- data/spec/unit/http/service/file_server_spec.rb +5 -6
- data/spec/unit/http/service/puppetserver_spec.rb +112 -0
- data/spec/unit/http/service/report_spec.rb +2 -3
- data/spec/unit/http/service_spec.rb +1 -3
- data/spec/unit/http/session_spec.rb +24 -35
- data/spec/unit/{network/http → http}/site_spec.rb +3 -3
- data/spec/unit/indirector/catalog/json_spec.rb +1 -1
- data/spec/unit/indirector/catalog/rest_spec.rb +1 -1
- data/spec/unit/indirector/facts/facter_spec.rb +97 -0
- data/spec/unit/indirector/facts/json_spec.rb +255 -0
- data/spec/unit/indirector/facts/rest_spec.rb +1 -1
- data/spec/unit/indirector/file_bucket_file/file_spec.rb +5 -3
- data/spec/unit/indirector/file_content/rest_spec.rb +0 -4
- data/spec/unit/indirector/file_metadata/rest_spec.rb +0 -4
- data/spec/unit/indirector/file_server_spec.rb +1 -15
- data/spec/unit/indirector/json_spec.rb +8 -8
- data/spec/unit/indirector/msgpack_spec.rb +8 -8
- data/spec/unit/indirector/node/json_spec.rb +33 -0
- data/spec/unit/indirector/node/rest_spec.rb +1 -1
- data/spec/{integration/indirector/report/yaml.rb → unit/indirector/report/json_spec.rb} +13 -24
- data/spec/unit/indirector/report/rest_spec.rb +2 -17
- data/spec/unit/indirector/report/yaml_spec.rb +72 -8
- data/spec/unit/indirector/request_spec.rb +2 -266
- data/spec/unit/indirector/rest_spec.rb +98 -752
- data/spec/unit/indirector/yaml_spec.rb +7 -7
- data/spec/unit/module_tool/tar/mini_spec.rb +20 -0
- data/spec/unit/network/authconfig_spec.rb +2 -132
- data/spec/unit/network/authorization_spec.rb +2 -55
- data/spec/unit/network/format_support_spec.rb +3 -2
- data/spec/unit/network/formats_spec.rb +4 -4
- data/spec/unit/network/http/api/indirected_routes_spec.rb +1 -97
- data/spec/unit/network/http/api/master/v3/environments_spec.rb +12 -23
- data/spec/unit/network/http/api/master/v3_spec.rb +28 -7
- data/spec/unit/network/http/api_spec.rb +10 -0
- data/spec/unit/network/http/connection_spec.rb +19 -41
- data/spec/unit/network/http/handler_spec.rb +0 -6
- data/spec/unit/network/http_pool_spec.rb +0 -4
- data/spec/unit/node/environment_spec.rb +51 -22
- data/spec/unit/node_spec.rb +2 -54
- data/spec/unit/parser/functions/create_resources_spec.rb +2 -20
- data/spec/unit/parser/templatewrapper_spec.rb +4 -3
- data/spec/unit/pops/evaluator/evaluating_parser_spec.rb +4 -7
- data/spec/unit/pops/loaders/loaders_spec.rb +76 -21
- data/spec/unit/pops/lookup/lookup_spec.rb +25 -0
- data/spec/unit/pops/parser/parse_application_spec.rb +4 -22
- data/spec/unit/pops/parser/parse_basic_expressions_spec.rb +0 -1
- data/spec/unit/pops/parser/parse_capabilities_spec.rb +8 -21
- data/spec/unit/pops/parser/parse_site_spec.rb +20 -24
- data/spec/unit/pops/resource/resource_type_impl_spec.rb +0 -71
- data/spec/unit/pops/serialization/to_from_hr_spec.rb +1 -1
- data/spec/unit/pops/types/type_calculator_spec.rb +6 -6
- data/spec/unit/pops/types/type_factory_spec.rb +1 -1
- data/spec/unit/pops/validator/validator_spec.rb +61 -46
- data/spec/unit/pops/visitor_spec.rb +1 -1
- data/spec/unit/provider/exec_spec.rb +4 -3
- data/spec/unit/provider/nameservice_spec.rb +0 -57
- data/spec/unit/provider/package/apt_spec.rb +77 -0
- data/spec/unit/provider/package/aptitude_spec.rb +1 -0
- data/spec/unit/provider/package/dpkg_spec.rb +22 -55
- data/spec/unit/provider/package/gem_spec.rb +32 -0
- data/spec/unit/provider/package/openbsd_spec.rb +2 -0
- data/spec/unit/provider/package/pip2_spec.rb +36 -0
- data/spec/unit/provider/package/puppet_gem_spec.rb +6 -2
- data/spec/unit/provider/package/puppetserver_gem_spec.rb +137 -0
- data/spec/unit/provider/package/yum_spec.rb +31 -0
- data/spec/unit/provider/package/zypper_spec.rb +14 -0
- data/spec/unit/provider/service/base_spec.rb +2 -4
- data/spec/unit/provider/service/bsd_spec.rb +5 -1
- data/spec/unit/provider/service/daemontools_spec.rb +1 -1
- data/spec/unit/provider/service/debian_spec.rb +3 -5
- data/spec/unit/provider/service/freebsd_spec.rb +1 -1
- data/spec/unit/provider/service/gentoo_spec.rb +4 -5
- data/spec/unit/provider/service/init_spec.rb +45 -5
- data/spec/unit/provider/service/launchd_spec.rb +5 -6
- data/spec/unit/provider/service/openrc_spec.rb +4 -5
- data/spec/unit/provider/service/openwrt_spec.rb +1 -1
- data/spec/unit/provider/service/redhat_spec.rb +1 -1
- data/spec/unit/provider/service/runit_spec.rb +2 -1
- data/spec/unit/provider/service/smf_spec.rb +402 -166
- data/spec/unit/provider/service/src_spec.rb +3 -5
- data/spec/unit/provider/service/systemd_spec.rb +3 -6
- data/spec/unit/provider/service/upstart_spec.rb +4 -5
- data/spec/unit/provider/service/windows_spec.rb +28 -1
- data/spec/unit/provider/user/aix_spec.rb +5 -0
- data/spec/unit/provider/user/pw_spec.rb +2 -0
- data/spec/unit/provider/user/useradd_spec.rb +1 -0
- data/spec/unit/provider/user/windows_adsi_spec.rb +82 -0
- data/spec/unit/provider_spec.rb +0 -12
- data/spec/unit/puppet_pal_2pec.rb +40 -0
- data/spec/unit/puppet_pal_catalog_spec.rb +45 -0
- data/spec/unit/reports/store_spec.rb +17 -13
- data/spec/unit/resource/type_collection_spec.rb +2 -22
- data/spec/unit/resource_spec.rb +0 -56
- data/spec/unit/settings/http_extra_headers_spec.rb +2 -4
- data/spec/unit/settings/integer_setting_spec.rb +42 -0
- data/spec/unit/settings/port_setting_spec.rb +31 -0
- data/spec/unit/settings/priority_setting_spec.rb +4 -4
- data/spec/unit/settings_spec.rb +586 -239
- data/spec/unit/ssl/base_spec.rb +36 -3
- data/spec/unit/ssl/certificate_request_spec.rb +15 -45
- data/spec/unit/ssl/certificate_spec.rb +2 -11
- data/spec/unit/ssl/ssl_provider_spec.rb +11 -8
- data/spec/unit/ssl/state_machine_spec.rb +0 -1
- data/spec/unit/ssl/verifier_spec.rb +0 -21
- data/spec/unit/transaction/persistence_spec.rb +15 -0
- data/spec/unit/transaction/report_spec.rb +2 -2
- data/spec/unit/transaction/resource_harness_spec.rb +2 -2
- data/spec/unit/transaction_spec.rb +45 -79
- data/spec/unit/type/file/checksum_spec.rb +6 -6
- data/spec/unit/type/file/content_spec.rb +1 -1
- data/spec/unit/type/file/ensure_spec.rb +1 -1
- data/spec/unit/type/file/mode_spec.rb +1 -1
- data/spec/unit/type/file/source_spec.rb +1 -2
- data/spec/unit/type/file_spec.rb +12 -6
- data/spec/unit/type/filebucket_spec.rb +1 -1
- data/spec/unit/type/package_spec.rb +1 -1
- data/spec/unit/type/service_spec.rb +35 -2
- data/spec/unit/type/user_spec.rb +31 -2
- data/spec/unit/type_spec.rb +20 -0
- data/spec/unit/util/backups_spec.rb +0 -2
- data/spec/unit/util/character_encoding_spec.rb +4 -4
- data/spec/unit/util/command_line_spec.rb +11 -6
- data/spec/unit/util/execution_spec.rb +0 -29
- data/spec/unit/util/monkey_patches_spec.rb +0 -6
- data/spec/unit/util/posix_spec.rb +357 -15
- data/spec/unit/util/rubygems_spec.rb +2 -2
- data/spec/unit/util/run_mode_spec.rb +27 -127
- data/spec/unit/util/storage_spec.rb +3 -1
- data/spec/unit/util/windows/string_spec.rb +1 -3
- data/spec/unit/util/yaml_spec.rb +0 -54
- data/spec/unit/util_spec.rb +0 -18
- metadata +84 -261
- data/conf/auth.conf +0 -150
- data/lib/puppet/application/cert.rb +0 -76
- data/lib/puppet/application/key.rb +0 -4
- data/lib/puppet/application/man.rb +0 -4
- data/lib/puppet/application/status.rb +0 -4
- data/lib/puppet/face/key.rb +0 -16
- data/lib/puppet/face/man.rb +0 -145
- data/lib/puppet/face/module/build.rb +0 -14
- data/lib/puppet/face/module/generate.rb +0 -14
- data/lib/puppet/face/module/search.rb +0 -103
- data/lib/puppet/face/status.rb +0 -51
- data/lib/puppet/indirector/certificate/file.rb +0 -9
- data/lib/puppet/indirector/certificate/rest.rb +0 -18
- data/lib/puppet/indirector/certificate_request/file.rb +0 -9
- data/lib/puppet/indirector/certificate_request/memory.rb +0 -7
- data/lib/puppet/indirector/certificate_request/rest.rb +0 -11
- data/lib/puppet/indirector/file_content/http.rb +0 -22
- data/lib/puppet/indirector/key/file.rb +0 -46
- data/lib/puppet/indirector/key/memory.rb +0 -7
- data/lib/puppet/indirector/ssl_file.rb +0 -162
- data/lib/puppet/indirector/status.rb +0 -3
- data/lib/puppet/indirector/status/local.rb +0 -12
- data/lib/puppet/indirector/status/rest.rb +0 -27
- data/lib/puppet/module_tool/applications/searcher.rb +0 -29
- data/lib/puppet/network/auth_config_parser.rb +0 -90
- data/lib/puppet/network/authstore.rb +0 -283
- data/lib/puppet/network/http/api/master/v3/authorization.rb +0 -18
- data/lib/puppet/network/http/api/master/v3/environment.rb +0 -88
- data/lib/puppet/network/http/base_pool.rb +0 -36
- data/lib/puppet/network/http/compression.rb +0 -127
- data/lib/puppet/network/http/connection_adapter.rb +0 -184
- data/lib/puppet/network/http/nocache_pool.rb +0 -28
- data/lib/puppet/network/rest_controller.rb +0 -2
- data/lib/puppet/network/rights.rb +0 -210
- data/lib/puppet/parser/compiler/catalog_validator/env_relationship_validator.rb +0 -66
- data/lib/puppet/parser/compiler/catalog_validator/site_validator.rb +0 -22
- data/lib/puppet/parser/environment_compiler.rb +0 -202
- data/lib/puppet/pops/types/enumeration.rb +0 -16
- data/lib/puppet/resource/capability_finder.rb +0 -154
- data/lib/puppet/rest/errors.rb +0 -15
- data/lib/puppet/rest/response.rb +0 -35
- data/lib/puppet/rest/route.rb +0 -85
- data/lib/puppet/rest/routes.rb +0 -135
- data/lib/puppet/ssl/host.rb +0 -505
- data/lib/puppet/ssl/key.rb +0 -61
- data/lib/puppet/ssl/validator.rb +0 -61
- data/lib/puppet/ssl/validator/default_validator.rb +0 -209
- data/lib/puppet/ssl/validator/no_validator.rb +0 -22
- data/lib/puppet/ssl/verifier_adapter.rb +0 -58
- data/lib/puppet/status.rb +0 -40
- data/lib/puppet/util/connection.rb +0 -88
- data/lib/puppet/util/ssl.rb +0 -83
- data/lib/puppet/util/windows/api_types.rb +0 -295
- data/lib/puppet/vendor/load_pathspec.rb +0 -1
- data/lib/puppet/vendor/pathspec/CHANGELOG.md +0 -2
- data/lib/puppet/vendor/pathspec/LICENSE +0 -201
- data/lib/puppet/vendor/pathspec/PUPPET_README.md +0 -6
- data/lib/puppet/vendor/pathspec/README.md +0 -53
- data/lib/puppet/vendor/pathspec/lib/pathspec.rb +0 -122
- data/lib/puppet/vendor/pathspec/lib/pathspec/gitignorespec.rb +0 -275
- data/lib/puppet/vendor/pathspec/lib/pathspec/regexspec.rb +0 -17
- data/lib/puppet/vendor/pathspec/lib/pathspec/spec.rb +0 -14
- data/man/man8/puppet-key.8 +0 -126
- data/man/man8/puppet-man.8 +0 -76
- data/man/man8/puppet-status.8 +0 -108
- data/spec/integration/faces/config_spec.rb +0 -91
- data/spec/integration/faces/documentation_spec.rb +0 -57
- data/spec/integration/file_bucket/file_spec.rb +0 -50
- data/spec/integration/file_serving/content_spec.rb +0 -7
- data/spec/integration/file_serving/fileset_spec.rb +0 -12
- data/spec/integration/file_serving/metadata_spec.rb +0 -8
- data/spec/integration/file_serving/terminus_helper_spec.rb +0 -20
- data/spec/integration/file_system/uniquefile_spec.rb +0 -26
- data/spec/integration/module_tool/forge_spec.rb +0 -51
- data/spec/integration/module_tool/tar/mini_spec.rb +0 -28
- data/spec/integration/network/authconfig_spec.rb +0 -256
- data/spec/integration/provider/service/init_spec.rb +0 -48
- data/spec/integration/provider/service/systemd_spec.rb +0 -25
- data/spec/integration/provider/service/windows_spec.rb +0 -50
- data/spec/integration/reference/providers_spec.rb +0 -21
- data/spec/integration/reports_spec.rb +0 -13
- data/spec/integration/ssl/certificate_request_spec.rb +0 -44
- data/spec/integration/ssl/host_spec.rb +0 -72
- data/spec/integration/ssl/key_spec.rb +0 -99
- data/spec/shared_behaviours/file_serving_model.rb +0 -51
- data/spec/unit/capability_spec.rb +0 -414
- data/spec/unit/face/catalog_spec.rb +0 -6
- data/spec/unit/face/key_spec.rb +0 -9
- data/spec/unit/face/man_spec.rb +0 -25
- data/spec/unit/face/module/search_spec.rb +0 -231
- data/spec/unit/face/module_spec.rb +0 -3
- data/spec/unit/face/status_spec.rb +0 -9
- data/spec/unit/indirector/certificate/file_spec.rb +0 -14
- data/spec/unit/indirector/certificate/rest_spec.rb +0 -61
- data/spec/unit/indirector/certificate_request/file_spec.rb +0 -14
- data/spec/unit/indirector/certificate_request/rest_spec.rb +0 -25
- data/spec/unit/indirector/key/file_spec.rb +0 -79
- data/spec/unit/indirector/ssl_file_spec.rb +0 -305
- data/spec/unit/indirector/status/local_spec.rb +0 -10
- data/spec/unit/indirector/status/rest_spec.rb +0 -50
- data/spec/unit/man_spec.rb +0 -31
- data/spec/unit/module_tool/applications/searcher_spec.rb +0 -38
- data/spec/unit/network/auth_config_parser_spec.rb +0 -115
- data/spec/unit/network/authstore_spec.rb +0 -422
- data/spec/unit/network/http/api/master/v3/authorization_spec.rb +0 -57
- data/spec/unit/network/http/api/master/v3/environment_spec.rb +0 -185
- data/spec/unit/network/http/compression_spec.rb +0 -240
- data/spec/unit/network/http/nocache_pool_spec.rb +0 -64
- data/spec/unit/network/http_spec.rb +0 -9
- data/spec/unit/network/rights_spec.rb +0 -439
- data/spec/unit/parser/environment_compiler_spec.rb +0 -730
- data/spec/unit/pops/types/enumeration_spec.rb +0 -51
- data/spec/unit/resource/capability_finder_spec.rb +0 -143
- data/spec/unit/rest/route_spec.rb +0 -132
- data/spec/unit/ssl/host_spec.rb +0 -650
- data/spec/unit/ssl/key_spec.rb +0 -173
- data/spec/unit/ssl/validator_spec.rb +0 -278
- data/spec/unit/status_spec.rb +0 -45
- data/spec/unit/util/ssl_spec.rb +0 -91
@@ -1,48 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
test_title = 'Integration Tests for Puppet::Type::Service::Provider::Init'
|
4
|
-
|
5
|
-
describe test_title, unless: Puppet::Util::Platform.jruby? do
|
6
|
-
let(:provider) { Puppet::Type.type(:service).provider(:init) }
|
7
|
-
|
8
|
-
describe "when running on FreeBSD" do
|
9
|
-
before :each do
|
10
|
-
allow(Facter).to receive(:value).with(:operatingsystem).and_return('FreeBSD')
|
11
|
-
allow(Facter).to receive(:value).with(:osfamily).and_return('FreeBSD')
|
12
|
-
end
|
13
|
-
|
14
|
-
it "should set its default path to include /etc/rc.d and /usr/local/etc/rc.d" do
|
15
|
-
expect(provider.defpath).to eq(["/etc/rc.d", "/usr/local/etc/rc.d"])
|
16
|
-
end
|
17
|
-
end
|
18
|
-
|
19
|
-
describe "when running on HP-UX" do
|
20
|
-
before :each do
|
21
|
-
allow(Facter).to receive(:value).with(:operatingsystem).and_return('HP-UX')
|
22
|
-
end
|
23
|
-
|
24
|
-
it "should set its default path to include /sbin/init.d" do
|
25
|
-
expect(provider.defpath).to eq("/sbin/init.d")
|
26
|
-
end
|
27
|
-
end
|
28
|
-
|
29
|
-
describe "when running on Archlinux" do
|
30
|
-
before :each do
|
31
|
-
allow(Facter).to receive(:value).with(:operatingsystem).and_return('Archlinux')
|
32
|
-
end
|
33
|
-
|
34
|
-
it "should set its default path to include /etc/rc.d" do
|
35
|
-
expect(provider.defpath).to eq("/etc/rc.d")
|
36
|
-
end
|
37
|
-
end
|
38
|
-
|
39
|
-
describe "when not running on FreeBSD, HP-UX or Archlinux" do
|
40
|
-
before :each do
|
41
|
-
allow(Facter).to receive(:value).with(:operatingsystem).and_return('RedHat')
|
42
|
-
end
|
43
|
-
|
44
|
-
it "should set its default path to include /etc/init.d" do
|
45
|
-
expect(provider.defpath).to eq("/etc/init.d")
|
46
|
-
end
|
47
|
-
end
|
48
|
-
end
|
@@ -1,25 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
test_title = 'Integration Tests for Puppet::Type::Service::Provider::Systemd'
|
4
|
-
|
5
|
-
describe test_title, unless: Puppet::Util::Platform.jruby? do
|
6
|
-
let(:provider_class) { Puppet::Type.type(:service).provider(:systemd) }
|
7
|
-
|
8
|
-
# TODO: Unfortunately there does not seem a way to stub the executable
|
9
|
-
# checks in the systemd provider because they happen at load time.
|
10
|
-
|
11
|
-
it "should be considered suitable if /proc/1/comm is present and contains 'systemd'",
|
12
|
-
:if => File.exist?('/proc/1/comm') && Puppet::FileSystem.read('/proc/1/comm').include?('systemd') do
|
13
|
-
expect(provider_class).to be_suitable
|
14
|
-
end
|
15
|
-
|
16
|
-
it "should not be considered suitable if /proc/1/comm is present it does not contain 'systemd'",
|
17
|
-
:if => File.exist?('/proc/1/comm') && !Puppet::FileSystem.read('/proc/1/comm').include?('systemd') do
|
18
|
-
expect(provider_class).not_to be_suitable
|
19
|
-
end
|
20
|
-
|
21
|
-
it "should not be considered suitable if /proc/1/comm is absent",
|
22
|
-
:if => !File.exist?('/proc/1/comm') do
|
23
|
-
expect(provider_class).not_to be_suitable
|
24
|
-
end
|
25
|
-
end
|
@@ -1,50 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
test_title = 'Integration Tests for Puppet::Type::Service::Provider::Windows'
|
4
|
-
|
5
|
-
describe test_title, '(integration)', :if => Puppet::Util::Platform.windows? do
|
6
|
-
let(:provider_class) { Puppet::Type.type(:service).provider(:windows) }
|
7
|
-
|
8
|
-
require 'puppet/util/windows'
|
9
|
-
|
10
|
-
before :each do
|
11
|
-
allow(Puppet::Type.type(:service)).to receive(:defaultprovider).and_return(provider_class)
|
12
|
-
end
|
13
|
-
|
14
|
-
context 'should return valid values when querying a service that does not exist' do
|
15
|
-
let(:service) do
|
16
|
-
Puppet::Type.type(:service).new(:name => 'foobarservice1234')
|
17
|
-
end
|
18
|
-
|
19
|
-
it "with :false when asked if enabled" do
|
20
|
-
expect(service.provider.enabled?).to eql(:false)
|
21
|
-
end
|
22
|
-
|
23
|
-
it "with :stopped when asked about status" do
|
24
|
-
expect(service.provider.status).to eql(:stopped)
|
25
|
-
end
|
26
|
-
end
|
27
|
-
|
28
|
-
context 'should return valid values when querying a service that does exist' do
|
29
|
-
let(:service) do
|
30
|
-
# This service should be ubiquitous across all supported Windows platforms
|
31
|
-
Puppet::Type.type(:service).new(:name => 'lmhosts')
|
32
|
-
end
|
33
|
-
|
34
|
-
it "with a valid enabled? value when asked if enabled" do
|
35
|
-
expect([:true, :false, :manual]).to include(service.provider.enabled?)
|
36
|
-
end
|
37
|
-
|
38
|
-
it "with a valid status when asked about status" do
|
39
|
-
expect([
|
40
|
-
:running,
|
41
|
-
:'continue pending',
|
42
|
-
:'pause pending',
|
43
|
-
:paused,
|
44
|
-
:running,
|
45
|
-
:'start pending',
|
46
|
-
:'stop pending',
|
47
|
-
:stopped]).to include(service.provider.status)
|
48
|
-
end
|
49
|
-
end
|
50
|
-
end
|
@@ -1,21 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/util/reference'
|
4
|
-
|
5
|
-
reference = Puppet::Util::Reference.reference(:providers)
|
6
|
-
|
7
|
-
describe reference do
|
8
|
-
it "should exist" do
|
9
|
-
expect(reference).not_to be_nil
|
10
|
-
end
|
11
|
-
|
12
|
-
it "should be able to be rendered as markdown" do
|
13
|
-
# We have a :confine block that calls execute in our upstart provider, which fails
|
14
|
-
# on jruby. Thus, we stub it out here since we don't care to do any assertions on it.
|
15
|
-
# This is only an issue if you're running these unit tests on a platform where upstart
|
16
|
-
# is a default provider, like Ubuntu trusty.
|
17
|
-
allow(Puppet::Util::Execution).to receive(:execute)
|
18
|
-
|
19
|
-
reference.to_markdown
|
20
|
-
end
|
21
|
-
end
|
@@ -1,13 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/reports'
|
4
|
-
|
5
|
-
describe Puppet::Reports, " when using report types" do
|
6
|
-
before do
|
7
|
-
allow(Puppet.settings).to receive(:use)
|
8
|
-
end
|
9
|
-
|
10
|
-
it "should load report types as modules" do
|
11
|
-
expect(Puppet::Reports.report(:store)).to be_instance_of(Module)
|
12
|
-
end
|
13
|
-
end
|
@@ -1,44 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/ssl/certificate_request'
|
4
|
-
|
5
|
-
describe Puppet::SSL::CertificateRequest do
|
6
|
-
include PuppetSpec::Files
|
7
|
-
|
8
|
-
before do
|
9
|
-
# Get a safe temporary file
|
10
|
-
dir = tmpdir("csr_integration_testing")
|
11
|
-
|
12
|
-
Puppet.settings[:confdir] = dir
|
13
|
-
Puppet.settings[:vardir] = dir
|
14
|
-
|
15
|
-
@csr = Puppet::SSL::CertificateRequest.new("luke.madstop.com")
|
16
|
-
|
17
|
-
@key = OpenSSL::PKey::RSA.new(512)
|
18
|
-
|
19
|
-
# This is necessary so the terminus instances don't lie around.
|
20
|
-
Puppet::SSL::CertificateRequest.indirection.termini.clear
|
21
|
-
end
|
22
|
-
|
23
|
-
it "should be able to generate CSRs" do
|
24
|
-
@csr.generate(@key)
|
25
|
-
end
|
26
|
-
|
27
|
-
it "should be able to save CSRs" do
|
28
|
-
Puppet::SSL::CertificateRequest.indirection.save(@csr)
|
29
|
-
end
|
30
|
-
|
31
|
-
it "should be able to find saved certificate requests via the Indirector" do
|
32
|
-
@csr.generate(@key)
|
33
|
-
Puppet::SSL::CertificateRequest.indirection.save(@csr)
|
34
|
-
|
35
|
-
expect(Puppet::SSL::CertificateRequest.indirection.find("luke.madstop.com")).to be_instance_of(Puppet::SSL::CertificateRequest)
|
36
|
-
end
|
37
|
-
|
38
|
-
it "should save the completely CSR when saving" do
|
39
|
-
@csr.generate(@key)
|
40
|
-
Puppet::SSL::CertificateRequest.indirection.save(@csr)
|
41
|
-
|
42
|
-
expect(Puppet::SSL::CertificateRequest.indirection.find("luke.madstop.com").content.to_s).to eq(@csr.content.to_s)
|
43
|
-
end
|
44
|
-
end
|
@@ -1,72 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
require 'puppet/test_ca'
|
3
|
-
|
4
|
-
require 'puppet/ssl/host'
|
5
|
-
|
6
|
-
describe Puppet::SSL::Host, if: !Puppet::Util::Platform.jruby? do
|
7
|
-
include PuppetSpec::Files
|
8
|
-
|
9
|
-
before do
|
10
|
-
# Get a safe temporary file
|
11
|
-
dir = tmpdir("host_integration_testing")
|
12
|
-
|
13
|
-
Puppet.settings[:confdir] = dir
|
14
|
-
Puppet.settings[:vardir] = dir
|
15
|
-
Puppet.settings.use :main, :ssl
|
16
|
-
|
17
|
-
@host = Puppet::SSL::Host.new("luke.madstop.com")
|
18
|
-
allow(@host).to receive(:submit_certificate_request)
|
19
|
-
|
20
|
-
@ca = Puppet::TestCa.new
|
21
|
-
Puppet::Util.replace_file(Puppet[:localcacert], 0644) do |f|
|
22
|
-
f.write(@ca.ca_cert.to_s)
|
23
|
-
end
|
24
|
-
Puppet::Util.replace_file(Puppet[:hostcrl], 0644) do |f|
|
25
|
-
f.write(@ca.ca_crl.to_s)
|
26
|
-
end
|
27
|
-
end
|
28
|
-
|
29
|
-
describe "when managing its key" do
|
30
|
-
it "should be able to generate and save a key" do
|
31
|
-
@host.generate_key
|
32
|
-
end
|
33
|
-
|
34
|
-
it "should save the key such that the Indirector can find it" do
|
35
|
-
@host.generate_key
|
36
|
-
|
37
|
-
expect(Puppet::SSL::Key.indirection.find(@host.name).content.to_s).to eq(@host.key.to_s)
|
38
|
-
end
|
39
|
-
|
40
|
-
it "should save the private key into the :privatekeydir" do
|
41
|
-
@host.generate_key
|
42
|
-
expect(File.read(File.join(Puppet.settings[:privatekeydir], "luke.madstop.com.pem"))).to eq(@host.key.to_s)
|
43
|
-
end
|
44
|
-
end
|
45
|
-
|
46
|
-
describe "when managing its certificate request" do
|
47
|
-
it "should be able to generate and save a certificate request" do
|
48
|
-
@host.generate_certificate_request
|
49
|
-
end
|
50
|
-
|
51
|
-
it "should save the certificate request such that the Indirector can find it" do
|
52
|
-
@host.generate_certificate_request
|
53
|
-
|
54
|
-
expect(Puppet::SSL::CertificateRequest.indirection.find(@host.name).content.to_s).to eq(@host.certificate_request.to_s)
|
55
|
-
end
|
56
|
-
|
57
|
-
it "should save the private certificate request into the :privatekeydir" do
|
58
|
-
@host.generate_certificate_request
|
59
|
-
expect(File.read(File.join(Puppet.settings[:requestdir], "luke.madstop.com.pem"))).to eq(@host.certificate_request.to_s)
|
60
|
-
end
|
61
|
-
end
|
62
|
-
|
63
|
-
it "should pass the verification of its own SSL store", :unless => Puppet.features.microsoft_windows? do
|
64
|
-
@host.generate_certificate_request
|
65
|
-
cert = @ca.sign(@host.certificate_request.content)
|
66
|
-
Puppet::Util.replace_file(File.join(Puppet[:certdir], "#{@host.name}.pem"), 0644) do |f|
|
67
|
-
f.write(cert)
|
68
|
-
end
|
69
|
-
|
70
|
-
expect(@host.ssl_store.verify(@host.certificate.content)).to be_truthy
|
71
|
-
end
|
72
|
-
end
|
@@ -1,99 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/ssl/key'
|
4
|
-
|
5
|
-
describe Puppet::SSL::Key, unless: Puppet::Util::Platform.jruby? do
|
6
|
-
include PuppetSpec::Files
|
7
|
-
|
8
|
-
# different UTF-8 widths
|
9
|
-
# 1-byte A
|
10
|
-
# 2-byte ۿ - http://www.fileformat.info/info/unicode/char/06ff/index.htm - 0xDB 0xBF / 219 191
|
11
|
-
# 3-byte ᚠ - http://www.fileformat.info/info/unicode/char/16A0/index.htm - 0xE1 0x9A 0xA0 / 225 154 160
|
12
|
-
# 4-byte - http://www.fileformat.info/info/unicode/char/2070E/index.htm - 0xF0 0xA0 0x9C 0x8E / 240 160 156 142
|
13
|
-
let (:mixed_utf8) { "A\u06FF\u16A0\u{2070E}" } # Aۿᚠ
|
14
|
-
|
15
|
-
before do
|
16
|
-
# Get a safe temporary file
|
17
|
-
dir = tmpdir('key_integration_testing')
|
18
|
-
|
19
|
-
Puppet.settings[:confdir] = dir
|
20
|
-
Puppet.settings[:vardir] = dir
|
21
|
-
|
22
|
-
# This is necessary so the terminus instances don't lie around.
|
23
|
-
# and so that Puppet::SSL::Key.indirection.save may be used
|
24
|
-
Puppet::SSL::Key.indirection.termini.clear
|
25
|
-
end
|
26
|
-
|
27
|
-
describe 'with a custom user-specified passfile' do
|
28
|
-
|
29
|
-
before do
|
30
|
-
# write custom password file to where Puppet expects
|
31
|
-
password_file = tmpfile('passfile')
|
32
|
-
Puppet[:passfile] = password_file
|
33
|
-
Puppet::FileSystem.open(password_file, nil, 'w:UTF-8') { |f| f.print(mixed_utf8) }
|
34
|
-
end
|
35
|
-
|
36
|
-
it 'should use the configured password file if it is not the CA key' do
|
37
|
-
key = Puppet::SSL::Key.new('test')
|
38
|
-
expect(key.password_file).to eq(Puppet[:passfile])
|
39
|
-
expect(key.password).to eq(mixed_utf8.force_encoding(Encoding::BINARY))
|
40
|
-
end
|
41
|
-
|
42
|
-
it "should be able to read an existing private key given the correct password" do
|
43
|
-
key_name = 'test'
|
44
|
-
# use OpenSSL APIs to generate a private key
|
45
|
-
private_key = OpenSSL::PKey::RSA.generate(512)
|
46
|
-
|
47
|
-
# stash it in Puppets private key directory
|
48
|
-
FileUtils.mkdir_p(Puppet[:privatekeydir])
|
49
|
-
pem_path = File.join(Puppet[:privatekeydir], "#{key_name}.pem")
|
50
|
-
Puppet::FileSystem.open(pem_path, nil, 'w:UTF-8') do |f|
|
51
|
-
# with password protection enabled
|
52
|
-
pem = private_key.to_pem(OpenSSL::Cipher::DES.new(:EDE3, :CBC), mixed_utf8)
|
53
|
-
f.print(pem)
|
54
|
-
end
|
55
|
-
|
56
|
-
# indirector loads existing .pem off disk instead of replacing it
|
57
|
-
host = Puppet::SSL::Host.new(key_name)
|
58
|
-
host.key
|
59
|
-
|
60
|
-
# newly loaded host private key matches the manually created key
|
61
|
-
# Private-Key: (512 bit) style data
|
62
|
-
expect(host.key.content.to_text).to eq(private_key.to_text)
|
63
|
-
# -----BEGIN RSA PRIVATE KEY-----
|
64
|
-
expect(host.key.content.to_s).to eq(private_key.to_s)
|
65
|
-
expect(host.key.password).to eq(mixed_utf8.force_encoding(Encoding::BINARY))
|
66
|
-
end
|
67
|
-
|
68
|
-
it 'should export the private key to PEM using the password' do
|
69
|
-
key_name = 'test'
|
70
|
-
|
71
|
-
# uses specified :passfile when writing the private key
|
72
|
-
key = Puppet::SSL::Key.new(key_name)
|
73
|
-
key.generate
|
74
|
-
Puppet::SSL::Key.indirection.save(key)
|
75
|
-
|
76
|
-
# indirector writes file here
|
77
|
-
pem_path = File.join(Puppet[:privatekeydir], "#{key_name}.pem")
|
78
|
-
|
79
|
-
# note incorrect password is an error
|
80
|
-
expect do
|
81
|
-
Puppet::FileSystem.open(pem_path, nil, 'r:ASCII') do |f|
|
82
|
-
OpenSSL::PKey::RSA.new(f.read, 'invalid_password')
|
83
|
-
end
|
84
|
-
end.to raise_error(OpenSSL::PKey::RSAError)
|
85
|
-
|
86
|
-
# but when specifying the correct password
|
87
|
-
reloaded_key = nil
|
88
|
-
Puppet::FileSystem.open(pem_path, nil, 'r:ASCII') do |f|
|
89
|
-
reloaded_key = OpenSSL::PKey::RSA.new(f.read, mixed_utf8)
|
90
|
-
end
|
91
|
-
|
92
|
-
# the original key matches the manually reloaded key
|
93
|
-
# Private-Key: (512 bit) style data
|
94
|
-
expect(key.content.to_text).to eq(reloaded_key.to_text)
|
95
|
-
# -----BEGIN RSA PRIVATE KEY-----
|
96
|
-
expect(key.content.to_s).to eq(reloaded_key.to_s)
|
97
|
-
end
|
98
|
-
end
|
99
|
-
end
|
@@ -1,51 +0,0 @@
|
|
1
|
-
shared_examples_for "a file_serving model" do
|
2
|
-
include PuppetSpec::Files
|
3
|
-
|
4
|
-
describe "#indirection" do
|
5
|
-
localpath = PuppetSpec::Files.make_absolute("/etc/sudoers")
|
6
|
-
localurl = "file://" + localpath
|
7
|
-
|
8
|
-
before :each do
|
9
|
-
# Never connect to the network, no matter what
|
10
|
-
allow_any_instance_of(described_class.indirection.terminus(:rest).class).to receive(:find)
|
11
|
-
end
|
12
|
-
|
13
|
-
describe "when running the apply application" do
|
14
|
-
before :each do
|
15
|
-
Puppet[:default_file_terminus] = 'file_server'
|
16
|
-
end
|
17
|
-
|
18
|
-
{
|
19
|
-
localpath => :file,
|
20
|
-
localurl => :file,
|
21
|
-
"puppet:///modules/foo/bar" => :file_server,
|
22
|
-
"puppet://server/modules/foo/bar" => :rest,
|
23
|
-
}.each do |key, terminus|
|
24
|
-
it "should use the #{terminus} terminus when requesting #{key.inspect}" do
|
25
|
-
expect_any_instance_of(described_class.indirection.terminus(terminus).class).to receive(:find)
|
26
|
-
|
27
|
-
described_class.indirection.find(key)
|
28
|
-
end
|
29
|
-
end
|
30
|
-
end
|
31
|
-
|
32
|
-
describe "when running another application" do
|
33
|
-
before :each do
|
34
|
-
Puppet[:default_file_terminus] = 'rest'
|
35
|
-
end
|
36
|
-
|
37
|
-
{
|
38
|
-
localpath => :file,
|
39
|
-
localurl => :file,
|
40
|
-
"puppet:///modules/foo/bar" => :rest,
|
41
|
-
"puppet://server/modules/foo/bar" => :rest,
|
42
|
-
}.each do |key, terminus|
|
43
|
-
it "should use the #{terminus} terminus when requesting #{key.inspect}" do
|
44
|
-
expect_any_instance_of(described_class.indirection.terminus(terminus).class).to receive(:find)
|
45
|
-
|
46
|
-
described_class.indirection.find(key)
|
47
|
-
end
|
48
|
-
end
|
49
|
-
end
|
50
|
-
end
|
51
|
-
end
|
@@ -1,414 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
require 'puppet_spec/compiler'
|
3
|
-
|
4
|
-
describe 'Capability types' do
|
5
|
-
include PuppetSpec::Compiler
|
6
|
-
let(:env) { Puppet::Node::Environment.create(:testing, []) }
|
7
|
-
let(:node) { Puppet::Node.new('test', :environment => env) }
|
8
|
-
let(:loaders) { Puppet::Pops::Loaders.new(env) }
|
9
|
-
|
10
|
-
before(:each) do
|
11
|
-
allow_any_instance_of(Puppet::Parser::Compiler).to receive(:loaders).and_return(loaders)
|
12
|
-
Puppet.push_context({:loaders => loaders, :current_environment => env})
|
13
|
-
Puppet::Type.newtype :cap, :is_capability => true do
|
14
|
-
newparam :name
|
15
|
-
newparam :host
|
16
|
-
end
|
17
|
-
end
|
18
|
-
|
19
|
-
after(:each) do
|
20
|
-
Puppet::Type.rmtype(:cap)
|
21
|
-
Puppet.pop_context()
|
22
|
-
end
|
23
|
-
|
24
|
-
context 'annotations' do
|
25
|
-
it "adds a blueprint for a produced resource" do
|
26
|
-
catalog = compile_to_catalog(<<-MANIFEST, node)
|
27
|
-
define test($hostname) {
|
28
|
-
notify { "hostname ${hostname}":}
|
29
|
-
}
|
30
|
-
|
31
|
-
Test produces Cap {
|
32
|
-
host => $hostname
|
33
|
-
}
|
34
|
-
MANIFEST
|
35
|
-
|
36
|
-
krt = catalog.environment_instance.known_resource_types
|
37
|
-
type = krt.definition(:test)
|
38
|
-
expect(type.produces).to be_instance_of(Array)
|
39
|
-
prd = type.produces.first
|
40
|
-
|
41
|
-
expect(prd).to be_instance_of(Hash)
|
42
|
-
expect(prd[:capability]).to eq("Cap")
|
43
|
-
expect(prd[:mappings]).to be_instance_of(Hash)
|
44
|
-
expect(prd[:mappings]["host"]).to be_instance_of(Puppet::Parser::AST::PopsBridge::Expression)
|
45
|
-
end
|
46
|
-
|
47
|
-
it "adds a blueprint for a consumed resource" do
|
48
|
-
catalog = compile_to_catalog(<<-MANIFEST, node)
|
49
|
-
define test($hostname) {
|
50
|
-
notify { "hostname ${hostname}":}
|
51
|
-
}
|
52
|
-
|
53
|
-
Test consumes Cap {
|
54
|
-
host => $hostname
|
55
|
-
}
|
56
|
-
MANIFEST
|
57
|
-
|
58
|
-
krt = catalog.environment_instance.known_resource_types
|
59
|
-
type = krt.definition(:test)
|
60
|
-
expect(type.produces).to be_instance_of(Array)
|
61
|
-
cns = type.consumes.first
|
62
|
-
|
63
|
-
expect(cns).to be_instance_of(Hash)
|
64
|
-
expect(cns[:capability]).to eq("Cap")
|
65
|
-
expect(cns[:mappings]).to be_instance_of(Hash)
|
66
|
-
expect(cns[:mappings]["host"]).to be_instance_of(Puppet::Parser::AST::PopsBridge::Expression)
|
67
|
-
end
|
68
|
-
|
69
|
-
it 'can place define and consumes/produces in separate manifests' do
|
70
|
-
parse_results = []
|
71
|
-
parser = Puppet::Parser::ParserFactory.parser
|
72
|
-
|
73
|
-
parser.string = <<-MANIFEST
|
74
|
-
define test($hostname) {
|
75
|
-
notify { "hostname ${hostname}":}
|
76
|
-
}
|
77
|
-
MANIFEST
|
78
|
-
parse_results << parser.parse
|
79
|
-
|
80
|
-
parser.string = <<-MANIFEST
|
81
|
-
Test consumes Cap {
|
82
|
-
host => $hostname
|
83
|
-
}
|
84
|
-
MANIFEST
|
85
|
-
parse_results << parser.parse
|
86
|
-
|
87
|
-
main = Puppet::Parser::AST::Hostclass.new('', :code => Puppet::Parser::ParserFactory.code_merger.concatenate(parse_results))
|
88
|
-
allow_any_instance_of(Puppet::Node::Environment).to receive(:perform_initial_import).and_return(main)
|
89
|
-
|
90
|
-
type = compile_to_catalog(nil).environment_instance.known_resource_types.definition(:test)
|
91
|
-
expect(type.produces).to be_instance_of(Array)
|
92
|
-
cns = type.consumes.first
|
93
|
-
|
94
|
-
expect(cns).to be_instance_of(Hash)
|
95
|
-
expect(cns[:capability]).to eq('Cap')
|
96
|
-
expect(cns[:mappings]).to be_instance_of(Hash)
|
97
|
-
expect(cns[:mappings]['host']).to be_instance_of(Puppet::Parser::AST::PopsBridge::Expression)
|
98
|
-
end
|
99
|
-
|
100
|
-
it 'can place use a qualified name for defines that produces capabilities' do
|
101
|
-
parse_results = []
|
102
|
-
parser = Puppet::Parser::ParserFactory.parser
|
103
|
-
|
104
|
-
parser.string = <<-MANIFEST
|
105
|
-
class mod {
|
106
|
-
define test($hostname) {
|
107
|
-
notify { "hostname ${hostname}":}
|
108
|
-
}
|
109
|
-
}
|
110
|
-
include mod
|
111
|
-
MANIFEST
|
112
|
-
parse_results << parser.parse
|
113
|
-
|
114
|
-
parser.string = <<-MANIFEST
|
115
|
-
Mod::Test consumes Cap {
|
116
|
-
host => $hostname
|
117
|
-
}
|
118
|
-
MANIFEST
|
119
|
-
parse_results << parser.parse
|
120
|
-
|
121
|
-
main = Puppet::Parser::AST::Hostclass.new('', :code => Puppet::Parser::ParserFactory.code_merger.concatenate(parse_results))
|
122
|
-
allow_any_instance_of(Puppet::Node::Environment).to receive(:perform_initial_import).and_return(main)
|
123
|
-
|
124
|
-
type = compile_to_catalog(nil).environment_instance.known_resource_types.definition('Mod::Test')
|
125
|
-
expect(type.produces).to be_instance_of(Array)
|
126
|
-
cns = type.consumes.first
|
127
|
-
|
128
|
-
expect(cns).to be_instance_of(Hash)
|
129
|
-
expect(cns[:capability]).to eq('Cap')
|
130
|
-
expect(cns[:mappings]).to be_instance_of(Hash)
|
131
|
-
expect(cns[:mappings]['host']).to be_instance_of(Puppet::Parser::AST::PopsBridge::Expression)
|
132
|
-
end
|
133
|
-
|
134
|
-
it "does not allow operator '+>' in a mapping" do
|
135
|
-
expect do
|
136
|
-
compile_to_catalog(<<-MANIFEST, node)
|
137
|
-
define test($hostname) {
|
138
|
-
notify { "hostname ${hostname}":}
|
139
|
-
}
|
140
|
-
|
141
|
-
Test consumes Cap {
|
142
|
-
host +> $hostname
|
143
|
-
}
|
144
|
-
MANIFEST
|
145
|
-
end.to raise_error(Puppet::ParseErrorWithIssue, /Illegal \+> operation.*This operator can not be used in a Capability Mapping/)
|
146
|
-
end
|
147
|
-
|
148
|
-
it "does not allow operator '*=>' in a mapping" do
|
149
|
-
expect do
|
150
|
-
compile_to_catalog(<<-MANIFEST, node)
|
151
|
-
define test($hostname) {
|
152
|
-
notify { "hostname ${hostname}":}
|
153
|
-
}
|
154
|
-
|
155
|
-
Test consumes Cap {
|
156
|
-
*=> { host => $hostname }
|
157
|
-
}
|
158
|
-
MANIFEST
|
159
|
-
end.to raise_error(Puppet::ParseError, /The operator '\* =>' in a Capability Mapping is not supported/)
|
160
|
-
end
|
161
|
-
|
162
|
-
it "does not allow 'before' relationship to capability mapping" do
|
163
|
-
expect do
|
164
|
-
compile_to_catalog(<<-MANIFEST, node)
|
165
|
-
define test() {
|
166
|
-
notify { "hello":}
|
167
|
-
}
|
168
|
-
|
169
|
-
Test consumes Cap {}
|
170
|
-
|
171
|
-
test { one: before => Cap[cap] }
|
172
|
-
MANIFEST
|
173
|
-
end.to raise_error(Puppet::Error, /'before' is not a valid relationship to a capability/)
|
174
|
-
end
|
175
|
-
|
176
|
-
["produces", "consumes"].each do |kw|
|
177
|
-
it "creates an error when #{kw} references nonexistent type" do
|
178
|
-
manifest = <<-MANIFEST
|
179
|
-
Test #{kw} Cap {
|
180
|
-
host => $hostname
|
181
|
-
}
|
182
|
-
MANIFEST
|
183
|
-
|
184
|
-
expect {
|
185
|
-
compile_to_catalog(manifest, node)
|
186
|
-
}.to raise_error(Puppet::Error,
|
187
|
-
/#{kw} clause references nonexistent type Test/)
|
188
|
-
end
|
189
|
-
end
|
190
|
-
end
|
191
|
-
|
192
|
-
context 'exporting a capability' do
|
193
|
-
it "does not add produced resources that are not exported" do
|
194
|
-
manifest = <<-MANIFEST
|
195
|
-
define test($hostname) {
|
196
|
-
notify { "hostname ${hostname}":}
|
197
|
-
}
|
198
|
-
|
199
|
-
Test produces Cap {
|
200
|
-
host => $hostname
|
201
|
-
}
|
202
|
-
|
203
|
-
test { one: hostname => "ahost" }
|
204
|
-
MANIFEST
|
205
|
-
catalog = compile_to_catalog(manifest, node)
|
206
|
-
expect(catalog.resource("Test[one]")).to be_instance_of(Puppet::Resource)
|
207
|
-
expect(catalog.resource_keys.find { |type, _| type == "Cap" }).to be_nil
|
208
|
-
end
|
209
|
-
|
210
|
-
it "adds produced resources that are exported" do
|
211
|
-
manifest = <<-MANIFEST
|
212
|
-
define test($hostname) {
|
213
|
-
notify { "hostname ${hostname}":}
|
214
|
-
}
|
215
|
-
|
216
|
-
# The $hostname in the produces clause does not refer to this variable,
|
217
|
-
# instead, it referes to the hostname property of the Test resource
|
218
|
-
# that is producing the Cap
|
219
|
-
$hostname = "other_host"
|
220
|
-
|
221
|
-
Test produces Cap {
|
222
|
-
host => $hostname
|
223
|
-
}
|
224
|
-
|
225
|
-
test { one: hostname => "ahost", export => Cap[two] }
|
226
|
-
MANIFEST
|
227
|
-
catalog = compile_to_catalog(manifest, node)
|
228
|
-
expect(catalog.resource("Test[one]")).to be_instance_of(Puppet::Resource)
|
229
|
-
|
230
|
-
caps = catalog.resource_keys.select { |type, _| type == "Cap" }
|
231
|
-
expect(caps.size).to eq(1)
|
232
|
-
|
233
|
-
cap = catalog.resource("Cap[two]")
|
234
|
-
expect(cap).to be_instance_of(Puppet::Resource)
|
235
|
-
expect(cap["require"]).to eq("Test[one]")
|
236
|
-
expect(cap["host"]).to eq("ahost")
|
237
|
-
expect(cap.resource_type).to eq(Puppet::Type::Cap)
|
238
|
-
expect(cap.tags.any? { |t| t == 'producer:testing' }).to eq(true)
|
239
|
-
end
|
240
|
-
end
|
241
|
-
|
242
|
-
context 'consuming a capability' do
|
243
|
-
def make_catalog(instance)
|
244
|
-
manifest = <<-MANIFEST
|
245
|
-
define test($hostname = nohost) {
|
246
|
-
notify { "hostname ${hostname}":}
|
247
|
-
}
|
248
|
-
|
249
|
-
Test consumes Cap {
|
250
|
-
hostname => $host
|
251
|
-
}
|
252
|
-
MANIFEST
|
253
|
-
compile_to_catalog(manifest + instance, node)
|
254
|
-
end
|
255
|
-
|
256
|
-
def mock_cap_finding
|
257
|
-
cap = Puppet::Resource.new("Cap", "two")
|
258
|
-
cap["host"] = "ahost"
|
259
|
-
expect(Puppet::Resource::CapabilityFinder).to receive(:find).and_return(cap)
|
260
|
-
cap
|
261
|
-
end
|
262
|
-
|
263
|
-
it "does not fetch a consumed resource when consume metaparam not set" do
|
264
|
-
expect(Puppet::Resource::CapabilityFinder).not_to receive(:find)
|
265
|
-
catalog = make_catalog("test { one: }")
|
266
|
-
expect(catalog.resource_keys.find { |type, _| type == "Cap" }).to be_nil
|
267
|
-
expect(catalog.resource("Test", "one")["hostname"]).to eq("nohost")
|
268
|
-
end
|
269
|
-
|
270
|
-
it "sets hostname from consumed capability" do
|
271
|
-
cap = mock_cap_finding
|
272
|
-
catalog = make_catalog("test { one: consume => Cap[two] }")
|
273
|
-
expect(catalog.resource("Cap[two]")).to eq(cap)
|
274
|
-
expect(catalog.resource("Cap[two]")["host"]).to eq("ahost")
|
275
|
-
expect(catalog.resource("Test", "one")["hostname"]).to eq("ahost")
|
276
|
-
end
|
277
|
-
|
278
|
-
it "does not override explicit hostname property when consuming" do
|
279
|
-
cap = mock_cap_finding
|
280
|
-
catalog = make_catalog("test { one: hostname => other_host, consume => Cap[two] }")
|
281
|
-
expect(catalog.resource("Cap[two]")).to eq(cap)
|
282
|
-
expect(catalog.resource("Cap[two]")["host"]).to eq("ahost")
|
283
|
-
expect(catalog.resource("Test", "one")["hostname"]).to eq("other_host")
|
284
|
-
end
|
285
|
-
|
286
|
-
it "fetches required capability" do
|
287
|
-
cap = mock_cap_finding
|
288
|
-
catalog = make_catalog("test { one: require => Cap[two] }")
|
289
|
-
expect(catalog.resource("Cap[two]")).to eq(cap)
|
290
|
-
expect(catalog.resource("Cap[two]")["host"]).to eq("ahost")
|
291
|
-
expect(catalog.resource("Test", "one")["hostname"]).to eq("nohost")
|
292
|
-
end
|
293
|
-
|
294
|
-
['export', 'consume'].each do |metaparam|
|
295
|
-
|
296
|
-
it "validates that #{metaparam} metaparameter rejects values that are not resources" do
|
297
|
-
expect { make_catalog("test { one: #{metaparam} => 'hello' }") }.to raise_error(Puppet::Error, /not a resource/)
|
298
|
-
end
|
299
|
-
|
300
|
-
it "validates that #{metaparam} metaparameter rejects resources that are not capability resources" do
|
301
|
-
expect { make_catalog("notify{hello:} test { one: #{metaparam} => Notify[hello] }") }.to raise_error(Puppet::Error, /not a capability resource/)
|
302
|
-
end
|
303
|
-
end
|
304
|
-
|
305
|
-
context 'producing/consuming resources' do
|
306
|
-
|
307
|
-
let(:ral) do
|
308
|
-
compile_to_ral(<<-MANIFEST, node)
|
309
|
-
define producer() {
|
310
|
-
notify { "producer":}
|
311
|
-
}
|
312
|
-
|
313
|
-
define consumer() {
|
314
|
-
notify { $title:}
|
315
|
-
}
|
316
|
-
|
317
|
-
Producer produces Cap {}
|
318
|
-
|
319
|
-
Consumer consumes Cap {}
|
320
|
-
|
321
|
-
producer {x: export => Cap[cap]}
|
322
|
-
consumer {x: consume => Cap[cap]}
|
323
|
-
consumer {y: require => Cap[cap]}
|
324
|
-
MANIFEST
|
325
|
-
end
|
326
|
-
|
327
|
-
let(:graph) do
|
328
|
-
graph = Puppet::Graph::RelationshipGraph.new(Puppet::Graph::SequentialPrioritizer.new)
|
329
|
-
graph.populate_from(ral)
|
330
|
-
graph
|
331
|
-
end
|
332
|
-
|
333
|
-
let(:capability) { ral.resource('Cap[cap]') }
|
334
|
-
|
335
|
-
it 'the produced resource depends on the producer' do
|
336
|
-
expect(graph.dependencies(capability).map {|d| d.to_s }).to include('Producer[x]')
|
337
|
-
end
|
338
|
-
|
339
|
-
it 'the consumer depends on the consumed resource' do
|
340
|
-
expect(graph.dependents(capability).map {|d| d.to_s }).to include('Consumer[x]')
|
341
|
-
end
|
342
|
-
|
343
|
-
it 'the consumer depends on the required resource' do
|
344
|
-
expect(graph.dependents(capability).map {|d| d.to_s }).to include('Consumer[y]')
|
345
|
-
end
|
346
|
-
end
|
347
|
-
|
348
|
-
context 'producing/consuming resources to/from classes' do
|
349
|
-
|
350
|
-
let(:ral) do
|
351
|
-
compile_to_ral(<<-MANIFEST, node)
|
352
|
-
define test($hostname) {
|
353
|
-
notify { $hostname:}
|
354
|
-
}
|
355
|
-
|
356
|
-
class producer($host) {
|
357
|
-
notify { p: }
|
358
|
-
}
|
359
|
-
|
360
|
-
class consumer($host) {
|
361
|
-
test { c: hostname => $host }
|
362
|
-
}
|
363
|
-
|
364
|
-
Class[producer] produces Cap {}
|
365
|
-
|
366
|
-
Class[consumer] consumes Cap {}
|
367
|
-
|
368
|
-
class { producer: host => 'produced.host', export => Cap[one]}
|
369
|
-
class { consumer: consume => Cap[one]}
|
370
|
-
MANIFEST
|
371
|
-
end
|
372
|
-
|
373
|
-
let(:graph) do
|
374
|
-
graph = Puppet::Graph::RelationshipGraph.new(Puppet::Graph::SequentialPrioritizer.new)
|
375
|
-
graph.populate_from(ral)
|
376
|
-
graph
|
377
|
-
end
|
378
|
-
|
379
|
-
let(:capability) { ral.resource('Cap[one]') }
|
380
|
-
|
381
|
-
it 'the produced resource depends on the producer' do
|
382
|
-
expect(graph.dependencies(capability).map {|d| d.to_s }).to include('Class[Producer]')
|
383
|
-
end
|
384
|
-
|
385
|
-
it 'the consumer depends on the consumed resource' do
|
386
|
-
expect(graph.dependents(capability).map {|d| d.to_s }).to include('Class[Consumer]')
|
387
|
-
end
|
388
|
-
|
389
|
-
it 'resource in the consumer class gets values from producer via the capability resource' do
|
390
|
-
expect(graph.dependents(capability).map {|d| d.to_s }).to include('Notify[produced.host]')
|
391
|
-
end
|
392
|
-
end
|
393
|
-
end
|
394
|
-
|
395
|
-
context 'and aliased resources' do
|
396
|
-
let(:drive) { Puppet::Util::Platform.windows? ? 'C:' : '' }
|
397
|
-
let(:code) { <<-PUPPET }
|
398
|
-
$dir='#{drive}/tmp/test'
|
399
|
-
$same_dir='#{drive}/tmp/test/'
|
400
|
-
|
401
|
-
file {$dir:
|
402
|
-
ensure => directory
|
403
|
-
}
|
404
|
-
|
405
|
-
file { $same_dir:
|
406
|
-
ensure => directory
|
407
|
-
}
|
408
|
-
PUPPET
|
409
|
-
|
410
|
-
it 'fails if a resource is defined and then redefined using name that results in the same alias' do
|
411
|
-
expect { compile_to_ral(code) }.to raise_error(/resource \["File", "#{drive}\/tmp\/test"\] already declared/)
|
412
|
-
end
|
413
|
-
end
|
414
|
-
end
|