[whimsy] branch master updated: Logger needs to be nil for all tests

[sebb]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 5ee4d4ee Logger needs to be nil for all tests
5ee4d4ee is described below

commit 5ee4d4ee102598eeee60af0e331bc966e916846b
Author: Sebb <s...@apache.org>
AuthorDate: Wed Aug 3 10:44:24 2022 +0100

    Logger needs to be nil for all tests
---
 lib/spec/lib/committee_spec.rb | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/lib/spec/lib/committee_spec.rb b/lib/spec/lib/committee_spec.rb
index 1a5b4f4f..b31a567c 100644
--- a/lib/spec/lib/committee_spec.rb
+++ b/lib/spec/lib/committee_spec.rb
@@ -20,6 +20,9 @@ require 'spec_helper'
 require 'whimsy/asf'
 
 describe ASF::Committee do
+  before {
+    Wunderbar.logger = nil # ensure we see warnings
+  }
   describe "ASF::Committee::site" do
     it "should return string for 'httpd'" do
       res = ASF::Committee.find('HTTP Server').site
@@ -78,9 +81,7 @@ describe ASF::Committee do
     input = File.read(file)
     it "should fail for 'httpd'" do
       res = nil
-      # Wunderbar.logger = nil; is needed to ensure logging output works as 
expected
       expect {
-        Wunderbar.logger = nil
         res = ASF::Committee.appendtlpmetadata(input, 'httpd', 'description', 
date_established)
       }.to output("_WARN Entry for 'httpd' already exists under 
:tlps\n").to_stderr
       expect(res).to eql(input)
@@ -88,9 +89,7 @@ describe ASF::Committee do
 
     it "should fail for 'comdev'" do
       res = nil
-      # Wunderbar.logger = nil; is needed to ensure logging output works as 
expected
       expect {
-        Wunderbar.logger = nil
         res = ASF::Committee.appendtlpmetadata(input, 'comdev', 'description', 
date_established)
       }.to output("_WARN Entry for 'comdev' already exists under 
:cttees\n").to_stderr
       expect(res).to eql(input)