]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
drop mock dependency
authorEmilien Macchi <emilien@redhat.com>
Fri, 22 Jan 2016 14:31:14 +0000 (09:31 -0500)
committerEmilien Macchi <emilien@redhat.com>
Sun, 24 Jan 2016 17:52:02 +0000 (12:52 -0500)
mock is not used anywhere in the module, let's drop it.
This patch will help to hit this bug:
https://launchpad.net/bugs/1492636

Change-Id: I000953c7b215454baf9838813c04e94b7ebeca00
(cherry picked from commit aeb040c9354dd60c93ef3b3ebfd0ec31c20cae85)

Gemfile
spec/classes/ceilometer_collector_spec.rb
spec/spec_helper.rb

diff --git a/Gemfile b/Gemfile
index fc22143982eeb61f0b2552d71d47045134f3e296..984392186093afc9808acae811ace22eba50a975 100644 (file)
--- a/Gemfile
+++ b/Gemfile
@@ -13,7 +13,6 @@ group :development, :test do
   gem 'puppet-lint-variable_contains_upcase', :require => 'false'
   gem 'puppet-lint-numericvariable',          :require => 'false'
   gem 'json',                                 :require => 'false'
-  gem 'webmock',                              :require => 'false'
 end
 
 group :system_tests do
index b3bbfff802734c77e8f7c7ffd73e61d8751b6fb4..c3e54c8ce2ea776f60a5ebda810c8a4725cd9cbb 100644 (file)
@@ -1,6 +1,4 @@
 require 'spec_helper'
-# LP1492636 - Cohabitation of compile matcher and webmock
-WebMock.disable_net_connect!(:allow => "169.254.169.254")
 
 describe 'ceilometer::collector' do
 
index 3df4cede10e1f2e489b73e4d09dc4204eb9649d2..172614e99164c991fbb85ee8d547a9e48d23fe6a 100644 (file)
@@ -1,6 +1,5 @@
 require 'puppetlabs_spec_helper/module_spec_helper'
 require 'shared_examples'
-require 'webmock/rspec'
 
 RSpec.configure do |c|
   c.alias_it_should_behave_like_to :it_configures, 'configures'