puppet 6.16.0-x86-mingw32 → 7.0.0-x86-mingw32
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of puppet might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/Gemfile +5 -3
- data/Gemfile.lock +31 -33
- data/README.md +4 -5
- 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 +1 -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.rb +2 -2
- data/lib/puppet/agent/locker.rb +0 -7
- data/lib/puppet/application.rb +172 -98
- data/lib/puppet/application/agent.rb +22 -6
- 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/configurer.rb +66 -31
- data/lib/puppet/configurer/downloader.rb +31 -10
- data/lib/puppet/configurer/plugin_handler.rb +21 -19
- data/lib/puppet/confine.rb +2 -2
- data/lib/puppet/confine/any.rb +1 -1
- data/lib/puppet/defaults.rb +166 -169
- data/lib/puppet/environments.rb +41 -15
- 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 +2 -2
- data/lib/puppet/face/plugin.rb +5 -8
- data/lib/puppet/feature/base.rb +1 -1
- 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 +13 -1
- data/lib/puppet/file_serving/metadata.rb +4 -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_serving/terminus_selector.rb +7 -8
- data/lib/puppet/file_system/file_impl.rb +4 -4
- data/lib/puppet/file_system/uniquefile.rb +8 -16
- data/lib/puppet/forge.rb +1 -1
- data/lib/puppet/forge/cache.rb +1 -1
- data/lib/puppet/forge/repository.rb +3 -8
- 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 +170 -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 +13 -19
- 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 +49 -48
- 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.rb +1 -1
- 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 +3 -7
- data/lib/puppet/indirector/file_metadata/http.rb +25 -5
- data/lib/puppet/indirector/file_metadata/rest.rb +5 -11
- 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 +2 -103
- 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 +3 -21
- 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/leaf.rb +5 -5
- data/lib/puppet/parser/ast/pops_bridge.rb +0 -42
- data/lib/puppet/parser/compiler.rb +1 -199
- 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 +3 -71
- data/lib/puppet/parser/resource/param.rb +6 -0
- 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 +27 -13
- 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/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 -22
- 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 -42
- data/lib/puppet/pops/validation/tasks_checker.rb +0 -12
- data/lib/puppet/pops/validation/validator_factory_4_0.rb +1 -1
- 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/aptitude.rb +1 -1
- 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 +2 -1
- data/lib/puppet/provider/package/zypper.rb +3 -0
- data/lib/puppet/provider/service/smf.rb +191 -73
- data/lib/puppet/provider/service/windows.rb +23 -7
- data/lib/puppet/provider/user/aix.rb +1 -1
- 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/useradd.rb +11 -4
- 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/reports/http.rb +2 -0
- data/lib/puppet/resource.rb +3 -90
- data/lib/puppet/resource/catalog.rb +1 -14
- data/lib/puppet/resource/type.rb +5 -112
- 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_context.rb +2 -2
- data/lib/puppet/ssl/ssl_provider.rb +37 -1
- data/lib/puppet/ssl/state_machine.rb +3 -1
- data/lib/puppet/ssl/verifier.rb +2 -0
- data/lib/puppet/test/test_helper.rb +19 -16
- 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 +29 -1
- data/lib/puppet/type.rb +9 -77
- data/lib/puppet/type/file.rb +45 -22
- data/lib/puppet/type/file/checksum.rb +5 -5
- data/lib/puppet/type/file/source.rb +33 -13
- 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 +53 -0
- data/lib/puppet/type/user.rb +18 -3
- data/lib/puppet/util.rb +41 -3
- data/lib/puppet/util/autoload.rb +9 -7
- data/lib/puppet/util/character_encoding.rb +9 -5
- data/lib/puppet/util/checksums.rb +19 -4
- data/lib/puppet/util/execution.rb +2 -13
- data/lib/puppet/util/fileparsing.rb +2 -2
- data/lib/puppet/util/http_proxy.rb +2 -215
- data/lib/puppet/util/monkey_patches.rb +0 -46
- data/lib/puppet/util/provider_features.rb +1 -1
- data/lib/puppet/util/rdoc.rb +0 -7
- data/lib/puppet/util/reference.rb +1 -1
- 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 +5 -15
- 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/principal.rb +8 -6
- data/lib/puppet/util/windows/process.rb +4 -226
- data/lib/puppet/util/windows/registry.rb +11 -11
- data/lib/puppet/util/windows/security.rb +4 -4
- data/lib/puppet/util/windows/service.rb +52 -486
- data/lib/puppet/util/windows/string.rb +12 -13
- data/lib/puppet/util/windows/user.rb +242 -8
- 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 +713 -1380
- data/man/man5/puppet.conf.5 +84 -98
- data/man/man8/puppet-agent.8 +7 -4
- 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/integration/application/agent_spec.rb +157 -59
- 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 +33 -5
- 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 +29 -30
- data/spec/integration/node/environment_spec.rb +1 -1
- data/spec/integration/parser/catalog_spec.rb +0 -38
- data/spec/integration/parser/compiler_spec.rb +11 -0
- 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 +6 -5
- 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/registry_spec.rb +7 -7
- data/spec/integration/util/windows/security_spec.rb +1 -1
- data/spec/integration/util/windows/user_spec.rb +47 -5
- 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 +9 -4
- data/spec/unit/certificate_factory_spec.rb +1 -1
- data/spec/unit/configurer/downloader_spec.rb +14 -0
- data/spec/unit/configurer/fact_handler_spec.rb +4 -4
- 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 +12 -10
- data/spec/unit/defaults_spec.rb +77 -28
- data/spec/unit/environments_spec.rb +96 -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 +2 -2
- 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/http_metadata_spec.rb +37 -14
- 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_serving/terminus_selector_spec.rb +45 -26
- 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 +71 -17
- 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/response_spec.rb +6 -0
- 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/http_spec.rb +27 -0
- 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 +3 -267
- data/spec/unit/indirector/rest_spec.rb +98 -752
- data/spec/unit/indirector/yaml_spec.rb +7 -7
- data/spec/unit/interface_spec.rb +3 -3
- 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 +3 -98
- 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 +61 -73
- 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/ast/block_expression_spec.rb +1 -1
- data/spec/unit/parser/functions/create_resources_spec.rb +2 -20
- data/spec/unit/parser/scope_spec.rb +1 -1
- data/spec/unit/pops/evaluator/evaluating_parser_spec.rb +19 -8
- data/spec/unit/pops/loaders/loaders_spec.rb +77 -22
- 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 +7 -17
- 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 +50 -15
- data/spec/unit/provider/user/openbsd_spec.rb +1 -0
- data/spec/unit/provider/user/useradd_spec.rb +22 -16
- 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 +3 -59
- 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 +78 -49
- data/spec/unit/ssl/state_machine_spec.rb +0 -1
- data/spec/unit/ssl/verifier_spec.rb +0 -21
- data/spec/unit/test/test_helper_spec.rb +17 -0
- data/spec/unit/transaction/persistence_spec.rb +15 -0
- data/spec/unit/transaction/report_spec.rb +3 -3
- 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 +4 -5
- data/spec/unit/type/file_spec.rb +134 -102
- 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 +209 -0
- data/spec/unit/type/user_spec.rb +31 -2
- data/spec/unit/type_spec.rb +70 -0
- data/spec/unit/util/backups_spec.rb +0 -2
- data/spec/unit/util/character_encoding_spec.rb +4 -4
- data/spec/unit/util/checksums_spec.rb +16 -0
- 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/rubygems_spec.rb +2 -2
- data/spec/unit/util/run_mode_spec.rb +27 -127
- data/spec/unit/util/windows/api_types_spec.rb +104 -40
- data/spec/unit/util/windows/service_spec.rb +4 -4
- 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 +3 -21
- data/spec/unit/x509/cert_provider_spec.rb +1 -1
- metadata +76 -270
- 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 -85
- 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 -182
- 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 -64
- data/lib/puppet/parser/compiler/catalog_validator/site_validator.rb +0 -20
- data/lib/puppet/parser/environment_compiler.rb +0 -199
- 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 -282
- 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/integration/test/test_helper_spec.rb +0 -31
- 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 -723
- 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,231 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
require 'puppet/face'
|
3
|
-
require 'puppet/application/module'
|
4
|
-
require 'puppet/module_tool'
|
5
|
-
|
6
|
-
describe "puppet module search" do
|
7
|
-
subject { Puppet::Face[:module, :current] }
|
8
|
-
|
9
|
-
let(:options) do
|
10
|
-
{}
|
11
|
-
end
|
12
|
-
|
13
|
-
describe Puppet::Application::Module do
|
14
|
-
subject do
|
15
|
-
app = Puppet::Application::Module.new
|
16
|
-
allow(app).to receive(:action).and_return(Puppet::Face.find_action(:module, :search))
|
17
|
-
app
|
18
|
-
end
|
19
|
-
|
20
|
-
before do
|
21
|
-
subject.render_as = :console
|
22
|
-
allow(Puppet::Util::Terminal).to receive(:width).and_return(100)
|
23
|
-
end
|
24
|
-
|
25
|
-
it 'should output nothing when receiving an empty dataset' do
|
26
|
-
expect(subject.render({:answers => [], :result => :success}, ['apache', {}])).to eq("No results found for 'apache'.")
|
27
|
-
end
|
28
|
-
|
29
|
-
it 'should return error and exit when error returned' do
|
30
|
-
results = {
|
31
|
-
:result => :failure,
|
32
|
-
:error => {
|
33
|
-
:oneline => 'Something failed',
|
34
|
-
:multiline => 'Something failed',
|
35
|
-
}
|
36
|
-
}
|
37
|
-
expect { subject.render(results, ['apache', {}]) }.to raise_error 'Something failed'
|
38
|
-
end
|
39
|
-
|
40
|
-
it 'should output a header when receiving a non-empty dataset' do
|
41
|
-
results = {
|
42
|
-
:result => :success,
|
43
|
-
:answers => [
|
44
|
-
{'full_name' => '', 'author' => '', 'desc' => '', 'tag_list' => [] },
|
45
|
-
],
|
46
|
-
}
|
47
|
-
|
48
|
-
expect(subject.render(results, ['apache', {}])).to match(/NAME/)
|
49
|
-
expect(subject.render(results, ['apache', {}])).to match(/DESCRIPTION/)
|
50
|
-
expect(subject.render(results, ['apache', {}])).to match(/AUTHOR/)
|
51
|
-
expect(subject.render(results, ['apache', {}])).to match(/KEYWORDS/)
|
52
|
-
end
|
53
|
-
|
54
|
-
it 'should output the relevant fields when receiving a non-empty dataset' do
|
55
|
-
results = {
|
56
|
-
:result => :success,
|
57
|
-
:answers => [
|
58
|
-
{'full_name' => 'Name', 'author' => 'Author', 'desc' => 'Summary', 'tag_list' => ['tag1', 'tag2'] },
|
59
|
-
]
|
60
|
-
}
|
61
|
-
|
62
|
-
expect(subject.render(results, ['apache', {}])).to match(/Name/)
|
63
|
-
expect(subject.render(results, ['apache', {}])).to match(/Author/)
|
64
|
-
expect(subject.render(results, ['apache', {}])).to match(/Summary/)
|
65
|
-
expect(subject.render(results, ['apache', {}])).to match(/tag1/)
|
66
|
-
expect(subject.render(results, ['apache', {}])).to match(/tag2/)
|
67
|
-
end
|
68
|
-
|
69
|
-
it 'should mark deprecated modules in search results' do
|
70
|
-
results = {
|
71
|
-
:result => :success,
|
72
|
-
:answers => [
|
73
|
-
{'full_name' => 'puppetlabs-corosync', 'deprecated_at' => Time.new, 'author' => 'Author', 'desc' => 'Summary', 'tag_list' => ['tag1', 'tag2'] },
|
74
|
-
]
|
75
|
-
}
|
76
|
-
|
77
|
-
expect(subject.render(results, ['apache', {}])).to match(/puppetlabs-corosync.*DEPRECATED/i)
|
78
|
-
end
|
79
|
-
|
80
|
-
it 'should elide really long descriptions' do
|
81
|
-
results = {
|
82
|
-
:result => :success,
|
83
|
-
:answers => [
|
84
|
-
{
|
85
|
-
'full_name' => 'Name',
|
86
|
-
'author' => 'Author',
|
87
|
-
'desc' => 'This description is really too long to fit in a single data table, guys -- we should probably set about truncating it',
|
88
|
-
'tag_list' => ['tag1', 'tag2'],
|
89
|
-
},
|
90
|
-
]
|
91
|
-
}
|
92
|
-
|
93
|
-
expect(subject.render(results, ['apache', {}])).to match(/\.{3} @Author/)
|
94
|
-
end
|
95
|
-
|
96
|
-
it 'should never truncate the module name' do
|
97
|
-
results = {
|
98
|
-
:result => :success,
|
99
|
-
:answers => [
|
100
|
-
{
|
101
|
-
'full_name' => 'This-module-has-a-really-really-long-name',
|
102
|
-
'author' => 'Author',
|
103
|
-
'desc' => 'Description',
|
104
|
-
'tag_list' => ['tag1', 'tag2'],
|
105
|
-
},
|
106
|
-
]
|
107
|
-
}
|
108
|
-
|
109
|
-
expect(subject.render(results, ['apache', {}])).to match(/This-module-has-a-really-really-long-name/)
|
110
|
-
end
|
111
|
-
|
112
|
-
it 'should never truncate the author name' do
|
113
|
-
results = {
|
114
|
-
:result => :success,
|
115
|
-
:answers => [
|
116
|
-
{
|
117
|
-
'full_name' => 'Name',
|
118
|
-
'author' => 'This-author-has-a-really-really-long-name',
|
119
|
-
'desc' => 'Description',
|
120
|
-
'tag_list' => ['tag1', 'tag2'],
|
121
|
-
},
|
122
|
-
]
|
123
|
-
}
|
124
|
-
|
125
|
-
expect(subject.render(results, ['apache', {}])).to match(/@This-author-has-a-really-really-long-name/)
|
126
|
-
end
|
127
|
-
|
128
|
-
it 'should never remove tags that match the search term' do
|
129
|
-
results = {
|
130
|
-
:results => :success,
|
131
|
-
:answers => [
|
132
|
-
{
|
133
|
-
'full_name' => 'Name',
|
134
|
-
'author' => 'Author',
|
135
|
-
'desc' => 'Description',
|
136
|
-
'tag_list' => ['Supercalifragilisticexpialidocious'] + (1..100).map { |i| "tag#{i}" },
|
137
|
-
},
|
138
|
-
]
|
139
|
-
}
|
140
|
-
|
141
|
-
expect(subject.render(results, ['Supercalifragilisticexpialidocious', {}])).to match(/Supercalifragilisticexpialidocious/)
|
142
|
-
expect(subject.render(results, ['Supercalifragilisticexpialidocious', {}])).not_to match(/tag/)
|
143
|
-
end
|
144
|
-
|
145
|
-
{
|
146
|
-
100 => "NAME DESCRIPTION AUTHOR KEYWORDS#{' '*15}\n"\
|
147
|
-
"Name This description is really too long to fit ... @JohnnyApples tag1 tag2 taggitty3#{' '*4}\n",
|
148
|
-
|
149
|
-
70 => "NAME DESCRIPTION AUTHOR KEYWORDS#{' '*5}\n"\
|
150
|
-
"Name This description is rea... @JohnnyApples tag1 tag2#{' '*4}\n",
|
151
|
-
|
152
|
-
80 => "NAME DESCRIPTION AUTHOR KEYWORDS#{' '*8}\n"\
|
153
|
-
"Name This description is really too... @JohnnyApples tag1 tag2#{' '*7}\n",
|
154
|
-
|
155
|
-
200 => "NAME DESCRIPTION AUTHOR KEYWORDS#{' '*48}\n"\
|
156
|
-
"Name This description is really too long to fit in a single data table, guys -- we should probably set about trunca... @JohnnyApples tag1 tag2 taggitty3#{' '*37}\n"
|
157
|
-
}.each do |width, expectation|
|
158
|
-
it "should resize the table to fit the screen, when #{width} columns" do
|
159
|
-
results = {
|
160
|
-
:result => :success,
|
161
|
-
:answers => [
|
162
|
-
{
|
163
|
-
'full_name' => 'Name',
|
164
|
-
'author' => 'JohnnyApples',
|
165
|
-
'desc' => 'This description is really too long to fit in a single data table, guys -- we should probably set about truncating it',
|
166
|
-
'tag_list' => ['tag1', 'tag2', 'taggitty3'],
|
167
|
-
},
|
168
|
-
]
|
169
|
-
}
|
170
|
-
|
171
|
-
expect(Puppet::Util::Terminal).to receive(:width).and_return(width)
|
172
|
-
result = subject.render(results, ['apache', {}])
|
173
|
-
expect(result.lines.sort_by(&:length).last.chomp.length).to be <= width
|
174
|
-
expect(result).to eq(expectation)
|
175
|
-
end
|
176
|
-
end
|
177
|
-
end
|
178
|
-
|
179
|
-
describe "option validation" do
|
180
|
-
context "without any options" do
|
181
|
-
it "should require a search term" do
|
182
|
-
pattern = /wrong number of arguments/
|
183
|
-
expect { subject.search }.to raise_error ArgumentError, pattern
|
184
|
-
end
|
185
|
-
end
|
186
|
-
|
187
|
-
it "should accept the --module_repository option" do
|
188
|
-
forge = double("Puppet::Forge")
|
189
|
-
searcher = double("Searcher")
|
190
|
-
options[:module_repository] = "http://forge.example.com"
|
191
|
-
|
192
|
-
expect(Puppet::Forge).to receive(:new).with("http://forge.example.com").and_return(forge)
|
193
|
-
expect(Puppet::ModuleTool::Applications::Searcher).to receive(:new).with("puppetlabs-apache", forge, hash_including(options)).and_return(searcher)
|
194
|
-
expect(searcher).to receive(:run)
|
195
|
-
|
196
|
-
subject.search("puppetlabs-apache", options)
|
197
|
-
end
|
198
|
-
end
|
199
|
-
|
200
|
-
describe "inline documentation" do
|
201
|
-
subject { Puppet::Face[:module, :current].get_action :search }
|
202
|
-
|
203
|
-
its(:summary) { should =~ /search.*module/im }
|
204
|
-
its(:description) { should =~ /search.*module/im }
|
205
|
-
its(:returns) { should =~ /array/i }
|
206
|
-
its(:examples) { should_not be_empty }
|
207
|
-
|
208
|
-
%w{ license copyright summary description returns examples }.each do |doc|
|
209
|
-
context "of the" do
|
210
|
-
its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
|
211
|
-
end
|
212
|
-
end
|
213
|
-
end
|
214
|
-
|
215
|
-
it "should include a deprecation warning" do
|
216
|
-
stub_request(:get, "https://forgeapi.puppet.com/v3/modules?query=puppetlabs-apache").to_return(status: 200, body: [answers: [], result: :success])
|
217
|
-
|
218
|
-
subject.search("puppetlabs-apache")
|
219
|
-
|
220
|
-
expect(@logs).to include(an_object_having_attributes(level: :warning, message: /This action has been deprecated. Please use the Puppet Forge to search for modules./))
|
221
|
-
end
|
222
|
-
|
223
|
-
it "omits the warning when deprecations are disabled" do
|
224
|
-
stub_request(:get, "https://forgeapi.puppet.com/v3/modules?query=puppetlabs-apache").to_return(status: 200, body: [answers: [], result: :success])
|
225
|
-
|
226
|
-
Puppet[:disable_warnings] = 'deprecations'
|
227
|
-
subject.search("puppetlabs-apache")
|
228
|
-
|
229
|
-
expect(@logs).not_to include(an_object_having_attributes(level: :warning))
|
230
|
-
end
|
231
|
-
end
|
@@ -1,14 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/indirector/certificate/file'
|
4
|
-
|
5
|
-
describe Puppet::SSL::Certificate::File do
|
6
|
-
it "should have documentation" do
|
7
|
-
expect(Puppet::SSL::Certificate::File.doc).to be_instance_of(String)
|
8
|
-
end
|
9
|
-
|
10
|
-
it "should use the :certdir as the collection directory" do
|
11
|
-
Puppet[:certdir] = File.expand_path("/cert/dir")
|
12
|
-
expect(Puppet::SSL::Certificate::File.collection_directory).to eq(Puppet[:certdir])
|
13
|
-
end
|
14
|
-
end
|
@@ -1,61 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
require 'puppet/indirector/certificate/rest'
|
3
|
-
|
4
|
-
describe Puppet::SSL::Certificate::Rest do
|
5
|
-
before do
|
6
|
-
@searcher = Puppet::SSL::Certificate::Rest.new
|
7
|
-
end
|
8
|
-
|
9
|
-
it "should be a sublcass of Puppet::Indirector::REST" do
|
10
|
-
expect(Puppet::SSL::Certificate::Rest.superclass).to equal(Puppet::Indirector::REST)
|
11
|
-
end
|
12
|
-
|
13
|
-
it "should set server_setting to :ca_server" do
|
14
|
-
expect(Puppet::SSL::Certificate::Rest.server_setting).to eq(:ca_server)
|
15
|
-
end
|
16
|
-
|
17
|
-
it "should set port_setting to :ca_port" do
|
18
|
-
expect(Puppet::SSL::Certificate::Rest.port_setting).to eq(:ca_port)
|
19
|
-
end
|
20
|
-
|
21
|
-
it "should use the :ca SRV service" do
|
22
|
-
expect(Puppet::SSL::Certificate::Rest.srv_service).to eq(:ca)
|
23
|
-
end
|
24
|
-
|
25
|
-
it "should make sure found certificates have their names set to the search string" do
|
26
|
-
terminus = Puppet::SSL::Certificate::Rest.new
|
27
|
-
|
28
|
-
# This has 'boo.com' in the CN
|
29
|
-
cert_string = "-----BEGIN CERTIFICATE-----
|
30
|
-
MIICPzCCAaigAwIBAgIBBDANBgkqhkiG9w0BAQUFADAWMRQwEgYDVQQDDAtidWNr
|
31
|
-
eS5sb2NhbDAeFw0wOTA5MTcxNzI1MzJaFw0xNDA5MTYxNzI1MzJaMBIxEDAOBgNV
|
32
|
-
BAMMB2Jvby5jb20wgZ8wDQYJKoZIhvcNAQEBBQADgY0AMIGJAoGBAKG9B+DkTCNh
|
33
|
-
F5xHchNDfnbC9NzWKM600oxrr84pgUVAG6B2wAZcdfoEtXszhsY9Jzpwqkvxk4Mx
|
34
|
-
AbYqo9+TCi4UoiH6e+vAKOOJD3DHrlf+/RW4hGtyaI41DBhf4+B4/oFz5PH9mvKe
|
35
|
-
NSfHFI/yPW+1IXYjxKLQNwF9E7q3JbnzAgMBAAGjgaAwgZ0wOAYJYIZIAYb4QgEN
|
36
|
-
BCsWKVB1cHBldCBSdWJ5L09wZW5TU0wgR2VuZXJhdGVkIENlcnRpZmljYXRlMAwG
|
37
|
-
A1UdEwEB/wQCMAAwHQYDVR0OBBYEFJOxEUeyf4cNOBmf9zIaE1JTuNdLMAsGA1Ud
|
38
|
-
DwQEAwIFoDAnBgNVHSUEIDAeBggrBgEFBQcDAQYIKwYBBQUHAwIGCCsGAQUFBwME
|
39
|
-
MA0GCSqGSIb3DQEBBQUAA4GBAFTJxKprMg6tfhGnvEvURPmlJrINn9c2b5Y4AGYp
|
40
|
-
tO86PFFkWw/EIJvvJzbj3s+Butr+eUo//+f1xxX7UCwwGqGxKqjtVS219oU/wkx8
|
41
|
-
h7rW4Xk7MrLl0auSS1p4wLcAMm+ZImf94+j8Cj+tkr8eGozZceRV13b8+EkdaE3S
|
42
|
-
rn/G
|
43
|
-
-----END CERTIFICATE-----
|
44
|
-
"
|
45
|
-
|
46
|
-
network = double('network')
|
47
|
-
allow(terminus).to receive(:network).and_return(network)
|
48
|
-
|
49
|
-
response = double('response', :code => "200", :body => cert_string)
|
50
|
-
allow(response).to receive(:[]).with('content-type').and_return("text/plain")
|
51
|
-
allow(response).to receive(:[]).with('content-encoding')
|
52
|
-
allow(response).to receive(:[]).with(Puppet::Network::HTTP::HEADER_PUPPET_VERSION).and_return(Puppet.version)
|
53
|
-
allow(network).to receive(:verify_callback=)
|
54
|
-
expect(network).to receive(:get).and_return(response)
|
55
|
-
|
56
|
-
request = Puppet::Indirector::Request.new(:certificate, :find, "foo.com", nil)
|
57
|
-
result = terminus.find(request)
|
58
|
-
expect(result).not_to be_nil
|
59
|
-
expect(result.name).to eq("foo.com")
|
60
|
-
end
|
61
|
-
end
|
@@ -1,14 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/indirector/certificate_request/file'
|
4
|
-
|
5
|
-
describe Puppet::SSL::CertificateRequest::File do
|
6
|
-
it "should have documentation" do
|
7
|
-
expect(Puppet::SSL::CertificateRequest::File.doc).to be_instance_of(String)
|
8
|
-
end
|
9
|
-
|
10
|
-
it "should use the :requestdir as the collection directory" do
|
11
|
-
Puppet[:requestdir] = File.expand_path("/request/dir")
|
12
|
-
expect(Puppet::SSL::CertificateRequest::File.collection_directory).to eq(Puppet[:requestdir])
|
13
|
-
end
|
14
|
-
end
|
@@ -1,25 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/indirector/certificate_request/rest'
|
4
|
-
|
5
|
-
describe Puppet::SSL::CertificateRequest::Rest do
|
6
|
-
before do
|
7
|
-
@searcher = Puppet::SSL::CertificateRequest::Rest.new
|
8
|
-
end
|
9
|
-
|
10
|
-
it "should be a sublcass of Puppet::Indirector::REST" do
|
11
|
-
expect(Puppet::SSL::CertificateRequest::Rest.superclass).to equal(Puppet::Indirector::REST)
|
12
|
-
end
|
13
|
-
|
14
|
-
it "should set server_setting to :ca_server" do
|
15
|
-
expect(Puppet::SSL::CertificateRequest::Rest.server_setting).to eq(:ca_server)
|
16
|
-
end
|
17
|
-
|
18
|
-
it "should set port_setting to :ca_port" do
|
19
|
-
expect(Puppet::SSL::CertificateRequest::Rest.port_setting).to eq(:ca_port)
|
20
|
-
end
|
21
|
-
|
22
|
-
it "should use the :ca SRV service" do
|
23
|
-
expect(Puppet::SSL::CertificateRequest::Rest.srv_service).to eq(:ca)
|
24
|
-
end
|
25
|
-
end
|
@@ -1,79 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/indirector/key/file'
|
4
|
-
|
5
|
-
describe Puppet::SSL::Key::File do
|
6
|
-
it "should have documentation" do
|
7
|
-
expect(Puppet::SSL::Key::File.doc).to be_instance_of(String)
|
8
|
-
end
|
9
|
-
|
10
|
-
it "should use the :privatekeydir as the collection directory" do
|
11
|
-
Puppet[:privatekeydir] = File.expand_path("/key/dir")
|
12
|
-
expect(Puppet::SSL::Key::File.collection_directory).to eq(Puppet[:privatekeydir])
|
13
|
-
end
|
14
|
-
|
15
|
-
describe "when choosing the path for the public key" do
|
16
|
-
it "should use the host name plus '.pem' in :publickeydir for normal hosts" do
|
17
|
-
Puppet[:privatekeydir] = File.expand_path("/private/key/dir")
|
18
|
-
Puppet[:publickeydir] = File.expand_path("/public/key/dir")
|
19
|
-
allow(Puppet.settings).to receive(:use)
|
20
|
-
|
21
|
-
@searcher = Puppet::SSL::Key::File.new
|
22
|
-
allow(@searcher).to receive(:ca?).and_return(false)
|
23
|
-
expect(@searcher.public_key_path("whatever")).to eq(File.expand_path("/public/key/dir/whatever.pem"))
|
24
|
-
end
|
25
|
-
end
|
26
|
-
|
27
|
-
describe "when managing private keys" do
|
28
|
-
before do
|
29
|
-
@searcher = Puppet::SSL::Key::File.new
|
30
|
-
|
31
|
-
@private_key_path = File.join("/fake/key/path")
|
32
|
-
@public_key_path = File.join("/other/fake/key/path")
|
33
|
-
|
34
|
-
allow(@searcher).to receive(:public_key_path).and_return(@public_key_path)
|
35
|
-
allow(@searcher).to receive(:path).and_return(@private_key_path)
|
36
|
-
|
37
|
-
allow(FileTest).to receive(:directory?).and_return(true)
|
38
|
-
allow(FileTest).to receive(:writable?).and_return(true)
|
39
|
-
|
40
|
-
@public_key = double('public_key')
|
41
|
-
@real_key = double('sslkey', :public_key => @public_key)
|
42
|
-
|
43
|
-
@key = double('key', :name => "myname", :content => @real_key)
|
44
|
-
|
45
|
-
@request = double('request', :key => "myname", :instance => @key)
|
46
|
-
end
|
47
|
-
|
48
|
-
it "should save the public key when saving the private key" do
|
49
|
-
fh = StringIO.new
|
50
|
-
|
51
|
-
expect(Puppet.settings.setting(:publickeydir)).to receive(:open_file).with(@public_key_path, 'w:ASCII').and_yield(fh)
|
52
|
-
allow(Puppet.settings.setting(:privatekeydir)).to receive(:open_file)
|
53
|
-
expect(@public_key).to receive(:to_pem).and_return("my pem")
|
54
|
-
|
55
|
-
@searcher.save(@request)
|
56
|
-
|
57
|
-
expect(fh.string).to eq("my pem")
|
58
|
-
end
|
59
|
-
|
60
|
-
it "should destroy the public key when destroying the private key" do
|
61
|
-
expect(Puppet::FileSystem).to receive(:unlink).with(Puppet::FileSystem.pathname(@private_key_path))
|
62
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(Puppet::FileSystem.pathname(@private_key_path)).and_return(true)
|
63
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(Puppet::FileSystem.pathname(@public_key_path)).and_return(true)
|
64
|
-
expect(Puppet::FileSystem).to receive(:unlink).with(Puppet::FileSystem.pathname(@public_key_path))
|
65
|
-
|
66
|
-
@searcher.destroy(@request)
|
67
|
-
end
|
68
|
-
|
69
|
-
it "should not fail if the public key does not exist when deleting the private key" do
|
70
|
-
allow(Puppet::FileSystem).to receive(:unlink).with(Puppet::FileSystem.pathname(@private_key_path))
|
71
|
-
|
72
|
-
allow(Puppet::FileSystem).to receive(:exist?).with(Puppet::FileSystem.pathname(@private_key_path)).and_return(true)
|
73
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(Puppet::FileSystem.pathname(@public_key_path)).and_return(false)
|
74
|
-
expect(Puppet::FileSystem).not_to receive(:unlink).with(Puppet::FileSystem.pathname(@public_key_path))
|
75
|
-
|
76
|
-
@searcher.destroy(@request)
|
77
|
-
end
|
78
|
-
end
|
79
|
-
end
|
@@ -1,305 +0,0 @@
|
|
1
|
-
require 'spec_helper'
|
2
|
-
|
3
|
-
require 'puppet/indirector/ssl_file'
|
4
|
-
|
5
|
-
describe Puppet::Indirector::SslFile do
|
6
|
-
include PuppetSpec::Files
|
7
|
-
|
8
|
-
before(:all) do
|
9
|
-
class Puppet::SslTestModel
|
10
|
-
extend Puppet::Indirector
|
11
|
-
indirects :ssl_test_model
|
12
|
-
end
|
13
|
-
|
14
|
-
class Puppet::SslTestModel::SslFile < Puppet::Indirector::SslFile
|
15
|
-
end
|
16
|
-
|
17
|
-
Puppet::SslTestModel.indirection.terminus_class = :ssl_file
|
18
|
-
end
|
19
|
-
|
20
|
-
after(:all) do
|
21
|
-
Puppet::SslTestModel.indirection.delete
|
22
|
-
Puppet.send(:remove_const, :SslTestModel)
|
23
|
-
end
|
24
|
-
|
25
|
-
let(:terminus_class) { Puppet::SslTestModel::SslFile }
|
26
|
-
let(:model) { Puppet::SslTestModel }
|
27
|
-
|
28
|
-
before :each do
|
29
|
-
@setting = :certdir
|
30
|
-
terminus_class.store_in @setting
|
31
|
-
terminus_class.store_at nil
|
32
|
-
@path = make_absolute("/thisdoesntexist/my_directory")
|
33
|
-
Puppet[:noop] = false
|
34
|
-
Puppet[@setting] = @path
|
35
|
-
Puppet[:trace] = false
|
36
|
-
end
|
37
|
-
|
38
|
-
after :each do
|
39
|
-
terminus_class.store_in nil
|
40
|
-
terminus_class.store_at nil
|
41
|
-
end
|
42
|
-
|
43
|
-
it "should use :main and :ssl upon initialization" do
|
44
|
-
expect(Puppet.settings).to receive(:use).with(:main, :ssl)
|
45
|
-
terminus_class.new
|
46
|
-
end
|
47
|
-
|
48
|
-
it "should return a nil collection directory if no directory setting has been provided" do
|
49
|
-
terminus_class.store_in nil
|
50
|
-
expect(terminus_class.collection_directory).to be_nil
|
51
|
-
end
|
52
|
-
|
53
|
-
it "should return a nil file location if no location has been provided" do
|
54
|
-
terminus_class.store_at nil
|
55
|
-
expect(terminus_class.file_location).to be_nil
|
56
|
-
end
|
57
|
-
|
58
|
-
it "should fail if no store directory or file location has been set" do
|
59
|
-
expect(Puppet.settings).to receive(:use).with(:main, :ssl)
|
60
|
-
terminus_class.store_in nil
|
61
|
-
terminus_class.store_at nil
|
62
|
-
expect {
|
63
|
-
terminus_class.new
|
64
|
-
}.to raise_error(Puppet::DevError, /No file or directory setting provided/)
|
65
|
-
end
|
66
|
-
|
67
|
-
describe "when managing ssl files" do
|
68
|
-
before do
|
69
|
-
allow(Puppet.settings).to receive(:use)
|
70
|
-
@searcher = terminus_class.new
|
71
|
-
|
72
|
-
@cert = double('certificate', :name => "myname")
|
73
|
-
@certpath = File.join(@path, "myname.pem")
|
74
|
-
|
75
|
-
@request = double('request', :key => @cert.name, :instance => @cert)
|
76
|
-
end
|
77
|
-
|
78
|
-
describe "when choosing the location for certificates" do
|
79
|
-
it "should set them at the file location if a file setting is available" do
|
80
|
-
terminus_class.store_in nil
|
81
|
-
terminus_class.store_at :hostcrl
|
82
|
-
|
83
|
-
Puppet[:hostcrl] = File.expand_path("/some/file")
|
84
|
-
|
85
|
-
expect(@searcher.path(@cert.name)).to eq(Puppet[:hostcrl])
|
86
|
-
end
|
87
|
-
|
88
|
-
it "should set them in the setting directory, with the certificate name plus '.pem', if a directory setting is available" do
|
89
|
-
expect(@searcher.path(@cert.name)).to eq(@certpath)
|
90
|
-
end
|
91
|
-
|
92
|
-
['../foo', '..\\foo', './../foo', '.\\..\\foo',
|
93
|
-
'/foo', '//foo', '\\foo', '\\\\goo',
|
94
|
-
"test\0/../bar", "test\0\\..\\bar",
|
95
|
-
"..\\/bar", "/tmp/bar", "/tmp\\bar", "tmp\\bar",
|
96
|
-
" / bar", " /../ bar", " \\..\\ bar",
|
97
|
-
"c:\\foo", "c:/foo", "\\\\?\\UNC\\bar", "\\\\foo\\bar",
|
98
|
-
"\\\\?\\c:\\foo", "//?/UNC/bar", "//foo/bar",
|
99
|
-
"//?/c:/foo",
|
100
|
-
].each do |input|
|
101
|
-
it "should resist directory traversal attacks (#{input.inspect})" do
|
102
|
-
expect { @searcher.path(input) }.to raise_error(ArgumentError, /invalid key/)
|
103
|
-
end
|
104
|
-
end
|
105
|
-
|
106
|
-
# REVISIT: Should probably test MS-DOS reserved names here, too, since
|
107
|
-
# they would represent a vulnerability on a Win32 system, should we ever
|
108
|
-
# support that path. Don't forget that 'CON.foo' == 'CON'
|
109
|
-
# --daniel 2011-09-24
|
110
|
-
end
|
111
|
-
|
112
|
-
describe "when finding certificates on disk" do
|
113
|
-
describe "and no certificate is present" do
|
114
|
-
it "should return nil" do
|
115
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(@path).and_return(true)
|
116
|
-
expect(Dir).to receive(:entries).with(@path).and_return([])
|
117
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(@certpath).and_return(false)
|
118
|
-
|
119
|
-
expect(@searcher.find(@request)).to be_nil
|
120
|
-
end
|
121
|
-
end
|
122
|
-
|
123
|
-
describe "and a certificate is present" do
|
124
|
-
let(:cert) { double('cert') }
|
125
|
-
let(:model) { double('model') }
|
126
|
-
|
127
|
-
before(:each) do
|
128
|
-
allow(terminus_class).to receive(:model).and_return(model)
|
129
|
-
end
|
130
|
-
|
131
|
-
context "is readable" do
|
132
|
-
it "should return an instance of the model, which it should use to read the certificate" do
|
133
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(@certpath).and_return(true)
|
134
|
-
|
135
|
-
expect(model).to receive(:new).with("myname").and_return(cert)
|
136
|
-
expect(cert).to receive(:read).with(@certpath)
|
137
|
-
|
138
|
-
expect(@searcher.find(@request)).to equal(cert)
|
139
|
-
end
|
140
|
-
end
|
141
|
-
|
142
|
-
context "is unreadable" do
|
143
|
-
it "should raise an exception" do
|
144
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(@certpath).and_return(true)
|
145
|
-
|
146
|
-
expect(model).to receive(:new).with("myname").and_return(cert)
|
147
|
-
expect(cert).to receive(:read).with(@certpath).and_raise(Errno::EACCES)
|
148
|
-
|
149
|
-
expect {
|
150
|
-
@searcher.find(@request)
|
151
|
-
}.to raise_error(Errno::EACCES)
|
152
|
-
end
|
153
|
-
end
|
154
|
-
end
|
155
|
-
|
156
|
-
describe "and a certificate is present but has uppercase letters" do
|
157
|
-
before do
|
158
|
-
@request = double('request', :key => "myhost")
|
159
|
-
end
|
160
|
-
|
161
|
-
# This is kind of more an integration test; it's for #1382, until
|
162
|
-
# the support for upper-case certs can be removed around mid-2009.
|
163
|
-
it "should rename the existing file to the lower-case path" do
|
164
|
-
@path = @searcher.path("myhost")
|
165
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(@path).and_return(false)
|
166
|
-
dir, file = File.split(@path)
|
167
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(dir).and_return(true)
|
168
|
-
expect(Dir).to receive(:entries).with(dir).and_return([".", "..", "something.pem", file.upcase])
|
169
|
-
|
170
|
-
expect(File).to receive(:rename).with(File.join(dir, file.upcase), @path)
|
171
|
-
|
172
|
-
cert = double('cert')
|
173
|
-
model = double('model')
|
174
|
-
allow(@searcher).to receive(:model).and_return(model)
|
175
|
-
expect(@searcher.model).to receive(:new).with("myhost").and_return(cert)
|
176
|
-
expect(cert).to receive(:read).with(@path)
|
177
|
-
|
178
|
-
@searcher.find(@request)
|
179
|
-
end
|
180
|
-
end
|
181
|
-
end
|
182
|
-
|
183
|
-
describe "when saving certificates to disk" do
|
184
|
-
before do
|
185
|
-
allow(FileTest).to receive(:directory?).and_return(true)
|
186
|
-
allow(FileTest).to receive(:writable?).and_return(true)
|
187
|
-
end
|
188
|
-
|
189
|
-
it "should fail if the directory is absent" do
|
190
|
-
expect(FileTest).to receive(:directory?).with(File.dirname(@certpath)).and_return(false)
|
191
|
-
expect { @searcher.save(@request) }.to raise_error(Puppet::Error)
|
192
|
-
end
|
193
|
-
|
194
|
-
it "should fail if the directory is not writeable" do
|
195
|
-
allow(FileTest).to receive(:directory?).and_return(true)
|
196
|
-
expect(FileTest).to receive(:writable?).with(File.dirname(@certpath)).and_return(false)
|
197
|
-
expect { @searcher.save(@request) }.to raise_error(Puppet::Error)
|
198
|
-
end
|
199
|
-
|
200
|
-
it "should save to the path the output of converting the certificate to a string" do
|
201
|
-
fh = double('filehandle')
|
202
|
-
expect(fh).to receive(:print).with("mycert")
|
203
|
-
|
204
|
-
allow(@searcher).to receive(:write).and_yield(fh)
|
205
|
-
expect(@cert).to receive(:to_s).and_return("mycert")
|
206
|
-
|
207
|
-
@searcher.save(@request)
|
208
|
-
end
|
209
|
-
|
210
|
-
describe "and a directory setting is set" do
|
211
|
-
it "should use the Settings class to write the file" do
|
212
|
-
@searcher.class.store_in @setting
|
213
|
-
fh = double('filehandle')
|
214
|
-
allow(fh).to receive(:print)
|
215
|
-
expect(Puppet.settings.setting(@setting)).to receive(:open_file).with(@certpath, 'w:ASCII').and_yield(fh)
|
216
|
-
|
217
|
-
@searcher.save(@request)
|
218
|
-
end
|
219
|
-
end
|
220
|
-
|
221
|
-
describe "and a file location is set" do
|
222
|
-
it "should use the filehandle provided by the Settings" do
|
223
|
-
@searcher.class.store_at @setting
|
224
|
-
|
225
|
-
fh = double('filehandle')
|
226
|
-
allow(fh).to receive(:print)
|
227
|
-
expect(Puppet.settings.setting(@setting)).to receive(:open).with('w:ASCII').and_yield(fh)
|
228
|
-
@searcher.save(@request)
|
229
|
-
end
|
230
|
-
end
|
231
|
-
end
|
232
|
-
|
233
|
-
describe "when destroying certificates" do
|
234
|
-
describe "that do not exist" do
|
235
|
-
before do
|
236
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(Puppet::FileSystem.pathname(@certpath)).and_return(false)
|
237
|
-
end
|
238
|
-
|
239
|
-
it "should return false" do
|
240
|
-
expect(@searcher.destroy(@request)).to be_falsey
|
241
|
-
end
|
242
|
-
end
|
243
|
-
|
244
|
-
describe "that exist" do
|
245
|
-
it "should unlink the certificate file" do
|
246
|
-
path = Puppet::FileSystem.pathname(@certpath)
|
247
|
-
expect(Puppet::FileSystem).to receive(:exist?).with(path).and_return(true)
|
248
|
-
expect(Puppet::FileSystem).to receive(:unlink).with(path)
|
249
|
-
@searcher.destroy(@request)
|
250
|
-
end
|
251
|
-
|
252
|
-
it "should log that is removing the file" do
|
253
|
-
allow(Puppet::FileSystem).to receive(:exist?).and_return(true)
|
254
|
-
allow(Puppet::FileSystem).to receive(:unlink)
|
255
|
-
expect(Puppet).to receive(:notice)
|
256
|
-
@searcher.destroy(@request)
|
257
|
-
end
|
258
|
-
end
|
259
|
-
end
|
260
|
-
|
261
|
-
describe "when searching for certificates" do
|
262
|
-
let(:one) { double('one') }
|
263
|
-
let(:two) { double('two') }
|
264
|
-
let(:one_path) { File.join(@path, 'one.pem') }
|
265
|
-
let(:two_path) { File.join(@path, 'two.pem') }
|
266
|
-
let(:model) { double('model') }
|
267
|
-
|
268
|
-
before :each do
|
269
|
-
allow(terminus_class).to receive(:model).and_return(model)
|
270
|
-
end
|
271
|
-
|
272
|
-
it "should return a certificate instance for all files that exist" do
|
273
|
-
expect(Dir).to receive(:entries).with(@path).and_return(%w{. .. one.pem two.pem})
|
274
|
-
|
275
|
-
expect(model).to receive(:new).with("one").and_return(one)
|
276
|
-
expect(one).to receive(:read).with(one_path)
|
277
|
-
expect(model).to receive(:new).with("two").and_return(two)
|
278
|
-
expect(two).to receive(:read).with(two_path)
|
279
|
-
|
280
|
-
expect(@searcher.search(@request)).to eq([one, two])
|
281
|
-
end
|
282
|
-
|
283
|
-
it "should raise an exception if any file is unreadable" do
|
284
|
-
expect(Dir).to receive(:entries).with(@path).and_return(%w{. .. one.pem two.pem})
|
285
|
-
|
286
|
-
expect(model).to receive(:new).with("one").and_return(one)
|
287
|
-
expect(one).to receive(:read).with(one_path)
|
288
|
-
expect(model).to receive(:new).with("two").and_return(two)
|
289
|
-
expect(two).to receive(:read).and_raise(Errno::EACCES)
|
290
|
-
|
291
|
-
expect {
|
292
|
-
@searcher.search(@request)
|
293
|
-
}.to raise_error(Errno::EACCES)
|
294
|
-
end
|
295
|
-
|
296
|
-
it "should skip any files that do not match /\.pem$/" do
|
297
|
-
expect(Dir).to receive(:entries).with(@path).and_return(%w{. .. one two.notpem})
|
298
|
-
|
299
|
-
expect(model).not_to receive(:new)
|
300
|
-
|
301
|
-
expect(@searcher.search(@request)).to eq([])
|
302
|
-
end
|
303
|
-
end
|
304
|
-
end
|
305
|
-
end
|