puppet 4.6.2-universal-darwin → 4.7.0-universal-darwin

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.

data/Rakefile CHANGED
@@ -77,10 +77,15 @@ end
77
77
 
78
78
  desc "verify that commit messages match CONTRIBUTING.md requirements"
79
79
  task(:commits) do
80
- # This git command looks at the summary from every commit from this branch not in master.
81
- # Ideally this would compare against the branch that a PR is submitted against, but I don't
82
- # know how to get that information. Absent that, comparing with master should work in most cases.
83
- %x{git log --no-merges --pretty=%s master..$HEAD}.each_line do |commit_summary|
80
+ # This rake task looks at the summary from every commit from this branch not
81
+ # in the branch targeted for a PR. This is accomplished by using the
82
+ # TRAVIS_COMMIT_RANGE environment variable, which is present in travis CI and
83
+ # populated with the range of commits the PR contains. If not available, this
84
+ # falls back to `master..HEAD` as a next best bet as `master` is unlikely to
85
+ # ever be absent.
86
+ commit_range = ENV['TRAVIS_COMMIT_RANGE'].nil? ? 'master..HEAD' : ENV['TRAVIS_COMMIT_RANGE']
87
+ puts "Checking commits..."
88
+ %x{git log --no-merges --pretty=%s #{commit_range}}.each_line do |commit_summary|
84
89
  # This regex tests for the currently supported commit summary tokens: maint, doc, packaging, or pup-<number>.
85
90
  # The exception tries to explain it in more full.
86
91
  if /^\((maint|doc|docs|packaging|pup-\d+)\)|revert/i.match(commit_summary).nil?
@@ -92,6 +97,9 @@ task(:commits) do
92
97
  "\t\t(maint)\n" \
93
98
  "\t\t(packaging)\n" \
94
99
  "\n\tThis test for the commit summary is case-insensitive.\n\n\n"
100
+ else
101
+ puts "#{commit_summary}"
95
102
  end
103
+ puts "...passed"
96
104
  end
97
105
  end
@@ -2,7 +2,7 @@
2
2
  packaging_url: 'git://github.com/puppetlabs/packaging.git --branch=master'
3
3
  packaging_repo: 'packaging'
4
4
  packager: 'puppetlabs'
5
- gpg_key: '4BD6EC30'
5
+ gpg_key: '7F438280EF8D349F'
6
6
 
7
7
  # These are the build targets used by the packaging repo. Uncomment to allow use.
8
8
  #final_mocks: 'pl-el-5-i386 pl-el-6-i386 pl-el-7-x86_64'
@@ -16,7 +16,7 @@ class Puppet::Network::HTTP::API::Master::V3
16
16
  end)
17
17
 
18
18
  ENVIRONMENT = Puppet::Network::HTTP::Route.
19
- path(%r{/environment/[^/]+$}).get(AUTHZ.wrap do
19
+ path(%r{^/environment/[^/]+$}).get(AUTHZ.wrap do
20
20
  Environment.new
21
21
  end)
22
22
 
@@ -7,7 +7,7 @@
7
7
 
8
8
 
9
9
  module Puppet
10
- PUPPETVERSION = '4.6.2'
10
+ PUPPETVERSION = '4.7.0'
11
11
 
12
12
  ##
13
13
  # version is a public API method intended to always provide a fast and
@@ -26,11 +26,12 @@ describe Puppet::Type.type(:service).provider(:windows), '(integration)',
26
26
 
27
27
  context 'should return valid values when querying a service that does exist' do
28
28
  let(:service) do
29
+ # This service should be ubiquitous across all supported Windows platforms
29
30
  Puppet::Type.type(:service).new(:name => 'lmhosts')
30
31
  end
31
32
 
32
- it "with a valid boolean when asked if enabled" do
33
- expect([:true, :false]).to include(service.provider.enabled?)
33
+ it "with a valid enabled? value when asked if enabled" do
34
+ expect([:true, :false, :manual]).to include(service.provider.enabled?)
34
35
  end
35
36
 
36
37
  it "with a valid status when asked about status" do
@@ -22,6 +22,21 @@ describe Puppet::Network::HTTP::API::Master::V3 do
22
22
  expect(response.code).to eq(200)
23
23
  end
24
24
 
25
+ it "mounts the environment endpoint" do
26
+ request = Puppet::Network::HTTP::Request.from_hash(:path => "#{master_url_prefix}/environment/production")
27
+ master_routes.process(request, response)
28
+
29
+ expect(response.code).to eq(200)
30
+ end
31
+
32
+ it "matches only complete routes" do
33
+ request = Puppet::Network::HTTP::Request.from_hash(:path => "#{master_url_prefix}/foo/environments")
34
+ expect { master_routes.process(request, response) }.to raise_error(Puppet::Network::HTTP::Error::HTTPNotFoundError)
35
+
36
+ request = Puppet::Network::HTTP::Request.from_hash(:path => "#{master_url_prefix}/foo/environment/production")
37
+ expect { master_routes.process(request, response) }.to raise_error(Puppet::Network::HTTP::Error::HTTPNotFoundError)
38
+ end
39
+
25
40
  it "mounts indirected routes" do
26
41
  request = Puppet::Network::HTTP::Request.
27
42
  from_hash(:path => "#{master_url_prefix}/node/foo",
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: puppet
3
3
  version: !ruby/object:Gem::Version
4
- version: 4.6.2
4
+ version: 4.7.0
5
5
  prerelease:
6
6
  platform: universal-darwin
7
7
  authors:
@@ -9,7 +9,7 @@ authors:
9
9
  autorequire:
10
10
  bindir: bin
11
11
  cert_chain: []
12
- date: 2016-09-02 00:00:00.000000000 Z
12
+ date: 2016-09-22 00:00:00.000000000 Z
13
13
  dependencies:
14
14
  - !ruby/object:Gem::Dependency
15
15
  name: facter