Replacing anchor functionality with contain

Author: Helen Campbell

manifests/init.pp

@@ -65,9 +65,12 @@
   # Anchor this as per #8040 - this ensures that classes won't float off and
   # mess everything up.  You can read about this at:
   # http://docs.puppetlabs.com/puppet/2.7/reference/lang_containment.html#known-issues
-  anchor { 'ntp::begin': } ->
-  class { '::ntp::install': } ->
-  class { '::ntp::config': } ~>
-  class { '::ntp::service': } ->
-  anchor { 'ntp::end': }
+
+  contain ntp::install
+  contain ntp::config
+  contain ntp::service
+
+  Class['::ntp::install'] ->
+  Class['::ntp::config'] ~>
+  Class['::ntp::service']
 }

spec/classes/contains_spec.rb

@@ -0,0 +1,23 @@
+# To check the correct dependancies are set up for NTP.
+
+require 'spec_helper'
+describe 'ntp' do
+  let(:facts) {{ :is_virtual => 'false' }}
+
+  on_supported_os.select { |_, f| f[:os]['family'] != 'Solaris' }.each do |os, f|
+    context "on #{os}" do
+      let(:facts) do
+        f.merge(super())
+      end
+
+      it { is_expected.to compile.with_all_deps }
+      describe "Testing the dependancies between the classes" do
+        it { should contain_class('ntp::install') }
+        it { should contain_class('ntp::config') }
+        it { should contain_class('ntp::service') }
+        it { is_expected.to contain_class('ntp::install').that_comes_before('Class[ntp::config]') }
+        it { is_expected.to contain_class('ntp::service').that_subscribes_to('Class[ntp::config]') }
+      end
+    end
+  end
+end