Move git daemon setup/teardown into context block
authorJohn Duarte <john.duarte@puppetlabs.com>
Tue, 6 May 2014 15:52:20 +0000 (08:52 -0700)
committerJohn Duarte <john.duarte@puppetlabs.com>
Sun, 18 May 2014 18:38:18 +0000 (11:38 -0700)
In order to ensure that the git daemon is running for the proper test, the setup
and teardown calls for git daemon hav been moved into the context block for
testing the git protocol.

spec/acceptance/git_clone_protocols_spec.rb

index ec99629..77a1adb 100644 (file)
@@ -29,8 +29,6 @@ hosts.each do |host|
       # copy ssl keys
       scp_to(host, "#{my_root}/acceptance/files/server.crt", tmpdir)
       scp_to(host, "#{my_root}/acceptance/files/server.key", tmpdir)
-
-      host.execute("nohup git daemon  --detach --base-path=/#{tmpdir}")
       # }}}
     end
 
@@ -96,6 +94,7 @@ hosts.each do |host|
     context 'using git protocol' do
       before(:all) do
         on(host,apply_manifest("file {'#{tmpdir}/testrepo': ensure => directory, purge => true, recurse => true, recurselimit => 1, force => true; }"))
+        host.execute("nohup git daemon  --detach --base-path=/#{tmpdir}")
       end
 
       it 'should have HEAD pointing to master' do
@@ -115,6 +114,9 @@ hosts.each do |host|
         it { should contain 'ref: refs/heads/master' }
       end
 
+      after(:all) do
+        host.execute('pkill -9 git')
+      end
     end
 
   end