Skip to content

Commit

Permalink
Switch to Delivery local for testing
Browse files Browse the repository at this point in the history
Signed-off-by: Tim Smith <[email protected]>
  • Loading branch information
tas50 committed Mar 24, 2017
1 parent 7c69695 commit 9b765e6
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 83 deletions.
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ vendor/
.coverage/
.zero-knife.rb
Policyfile.lock.json
Cheffile.lock
.librarian/

# vagrant stuff
.vagrant/
Expand Down
6 changes: 3 additions & 3 deletions .kitchen.dokken.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ platforms:
platform: rhel
pid_one_command: /sbin/init
intermediate_instructions:
- RUN yum -y install which initscripts net-tools wget
- RUN yum -y install lsof which initscripts net-tools wget net-tools

- name: centos-7
driver:
Expand All @@ -50,7 +50,7 @@ platforms:
image: fedora:latest
pid_one_command: /usr/lib/systemd/systemd
intermediate_instructions:
- RUN dnf -y install yum which systemd-sysv initscripts wget net-tools
- RUN dnf -y install which systemd-sysv initscripts wget net-tools

- name: ubuntu-14.04
driver:
Expand All @@ -73,7 +73,7 @@ platforms:
image: opensuse:leap
pid_one_command: /bin/systemd
intermediate_instructions:
- RUN zypper --non-interactive install aaa_base perl-Getopt-Long-Descriptive which hostname
- RUN zypper --non-interactive install aaa_base perl-Getopt-Long-Descriptive which net-tools

suites:
- name: multi_instance
Expand Down
2 changes: 1 addition & 1 deletion .kitchen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ platforms:
- name: debian-7.11
- name: debian-8.6
- name: fedora-25
- name: opensuse-leap-42.1
- name: opensuse-leap-42.2
- name: ubuntu-14.04
- name: ubuntu-16.04

Expand Down
6 changes: 1 addition & 5 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
sudo: required
dist: trusty

# install the pre-release chef-dk. Use chef-stable-trusty to install the stable release
addons:
apt:
sources:
Expand Down Expand Up @@ -40,9 +39,6 @@ script: KITCHEN_LOCAL_YAML=.kitchen.dokken.yml /opt/chefdk/embedded/bin/kitchen

matrix:
include:
- before_script:
- eval "$(/opt/chefdk/bin/chef shell-init bash)"

- script:
- /opt/chefdk/bin/chef exec rake
- /opt/chefdk/bin/chef exec delivery local all
env: UNIT_AND_LINT=1
1 change: 0 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
source 'https://rubygems.org'

gem 'tomlrb'
gem 'rake'
gem 'stove'
gem 'community_cookbook_releaser'
69 changes: 0 additions & 69 deletions Rakefile

This file was deleted.

5 changes: 2 additions & 3 deletions libraries/tomcat.rb
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,8 @@ def derived_install_path

# make sure catalina base is in the env_var has no matter what
def ensure_catalina_base
unless new_resource.env_vars.any? { |env_hash| env_hash.key?('CATALINA_BASE') }
new_resource.env_vars.unshift('CATALINA_BASE' => derived_install_path)
end
return if new_resource.env_vars.any? { |env_hash| env_hash.key?('CATALINA_BASE') }
new_resource.env_vars.unshift('CATALINA_BASE' => derived_install_path)
end

# choose the right platform init class
Expand Down
2 changes: 1 addition & 1 deletion spec/unit/recipes/default_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

describe 'default recipe on ubuntu 16.04' do
let(:runner) { ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04') }
let(:chef_run) { runner.converge('tomcat::default') }
let(:chef_run) { runner.converge('test::default') }

it 'converges successfully' do
expect { :chef_run }.to_not raise_error
Expand Down

0 comments on commit 9b765e6

Please sign in to comment.