fix specs

Author: DannyBen

lib/bashly/settings.rb

@@ -56,9 +56,9 @@ def strict
       end
 
       def strict_string
-        if Settings.strict.is_a? String
-          Settings.strict
-        elsif Settings.strict
+        if strict.is_a? String
+          strict
+        elsif strict
           'set -euo pipefail'
         else
           'set -e'

spec/bashly/settings_spec.rb

@@ -1,24 +1,20 @@
 require 'spec_helper'
 
 describe Settings do
-  subject { described_class }
+  subject { BaselineSettings.clone }
 
   describe 'standard value' do
     it 'returns a predefined default value' do
       expect(subject.tab_indent).to be false
     end
 
     context 'when settings.yml exists' do
-      original_value = described_class.source_dir
-
       before do
         reset_tmp_dir
         File.write 'spec/tmp/settings.yml', 'source_dir: somedir'
         subject.source_dir = nil
       end
 
-      after { described_class.source_dir = original_value }
-
       it 'returns the value from the settings file' do
         Dir.chdir 'spec/tmp' do
           expect(subject.source_dir).to eq 'somedir'
@@ -27,16 +23,12 @@
     end
 
     context 'when bashly-settings.yml exists' do
-      original_value = described_class.source_dir
-
       before do
         reset_tmp_dir
         File.write 'spec/tmp/bashly-settings.yml', 'source_dir: somedir'
         subject.source_dir = nil
       end
 
-      after { described_class.source_dir = original_value }
-
       it 'returns the value from the settings file' do
         Dir.chdir 'spec/tmp' do
           expect(subject.source_dir).to eq 'somedir'
@@ -93,10 +85,6 @@
   end
 
   describe 'strict_string' do
-    original_value = described_class.strict
-
-    after { subject.strict = original_value }
-
     context 'when strict is true' do
       before { subject.strict = true }
 

spec/spec_helper.rb

@@ -34,6 +34,10 @@
 Settings.env = :development
 reset_tmp_dir
 
+# This is used in the Settings spec, to ensure we are working with an
+# eigenclass that has a known state
+BaselineSettings = Settings.clone
+
 # Consistent Colsole output (for rspec_fixtures)
 ENV['TTY'] = 'off'
 ENV['COLUMNS'] = '80'