Be a bit louder when skipping an entire integration suite

test/integration/default/_debug_spec.rb

@@ -1 +1,8 @@
-p "You are currently running on OS: #{os[:name] || 'unknown'}, OS release: #{os[:release] || 'unknown'}, OS family: #{os[:family] || 'unknown'}"
+puts "-----------------------------------"
+puts "        TEST ENVIRONMENT           "
+puts "-----------------------------------"
+puts "    Docker: #{!ENV['DOCKER'].nil?}"
+puts "   OS name: #{os[:name] || 'unknown' }"
+puts "OS release: #{os[:release] || 'unknown'}"
+puts " OS family: #{os[:family] || 'unknown'}"
+puts "-----------------------------------"

test/integration/default/apache_conf_spec.rb

@@ -1,8 +1,13 @@
 # encoding: utf-8
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
-return unless os.linux?
+if !os.linux?
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__} because we are not on linux.\033[0m"
+  return
+end
 
 # direct access to params of apache conf
 describe apache_conf do

test/integration/default/etc_group_spec.rb

@@ -1,7 +1,9 @@
 # encoding: utf-8
 
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 # lets define our own group
 root_group = 'root'

test/integration/default/file_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 if os[:family] == 'freebsd'
   filedata = {

test/integration/default/iptables_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 case os[:family]
 when 'ubuntu', 'fedora'

test/integration/default/kernel_module_spec.rb

@@ -1,8 +1,13 @@
 # encoding: utf-8
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
-return unless os.linux?
+if !os.linux?
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__} because we are not on linux.\033[0m"
+  return
+end
 
 # Test kernel modules on all linux systems
 describe kernel_module('video') do

test/integration/default/kernel_parameter_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 # prepare values
 if ['ubuntu', 'centos', 'fedora', 'opensuse', 'debian'].include?(os[:family])

test/integration/default/mount_spec.rb

@@ -1,8 +1,13 @@
 # encoding: utf-8
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
-return unless os.linux?
+if !os.linux?
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__} because we are not on linux.\033[0m"
+  return
+end
 
 # instead of `.with` or `.only_with` we recommend to use the `mount` resource
 describe mount '/mnt/iso-disk' do

test/integration/default/port_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 # check that ssh runs
 if os.unix?

test/integration/default/postgres_session_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 # postgres-server is installed on these platforms
 if ['ubuntu', 'centos'].include? os['family']

test/integration/default/service_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
-
-# TODO: do not run those tests on docker yet
-return if ENV['DOCKER']
+if ENV['DOCKER']
+  STDERR.puts "\033[1;33mTODO: Not running #{__FILE__.split("/").last} because we are running in docker\033[0m"
+  return
+end
 
 # based on operating system we select the available service
 if ['centos', 'fedora', 'freebsd', 'opensuse'].include?(os[:family])