Merge pull request #10 from jobready/update-ruby-for-ci

Fix CI build config

Gemfile

@@ -1,4 +1,4 @@
 source "https://rubygems.org"
 gemspec
 
-gem "codeclimate-test-reporter", group: :test, require: nil
+gem "simplecov", group: :test, require: nil

boolean_dsl.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency 'bundler', '~> 1.0'
   s.add_development_dependency 'rspec', '~> 2.14'
-  s.add_development_dependency 'factory_girl', '~> 4.4'
+  s.add_development_dependency 'factory_bot', '~> 4.4'
   s.add_development_dependency 'cane', '~> 2.6'
   s.add_development_dependency 'byebug', '~> 2.7'
   s.add_development_dependency 'rake', '~> 10.1'

scripts/buildbox.sh

@@ -2,7 +2,7 @@
 set -e
 
 echo '--- setting ruby version'
-rbenv local 2.1.2
+rbenv local 2.3.6
 
 echo '--- bundling'
 bundle install -j $(nproc) --without production --quiet

spec/evaluator/context_spec.rb

@@ -15,8 +15,8 @@
   context 'attribute' do
     specify { expect(evaluator.evaluate(attribute: 'alpha')).to eq('omega') }
     specify { expect(evaluator.evaluate(attribute: 'gamma')).to eq(7) }
-    specify { expect(evaluator.evaluate(attribute: 'delta')).to be_true }
-    specify { expect(evaluator.evaluate(attribute: 'yotta')).to be_false }
+    specify { expect(evaluator.evaluate(attribute: 'delta')).to be_truthy }
+    specify { expect(evaluator.evaluate(attribute: 'yotta')).to be_falsey }
     specify do
       expect { evaluator.evaluate(attribute: 'beta') }.to(
         raise_error(BooleanDsl::EvaluationFailed, 'Context does not have key beta'))
@@ -55,4 +55,4 @@
       )
     end
   end
-end
+end

spec/evaluator/evaluate_spec.rb

@@ -11,15 +11,15 @@
       left: { integer: '1' },
       comparison_operator: '==',
       right: { integer: '1' },
-    )).to be_true
+    )).to be_truthy
   end
-  
+
   specify do
     expect(evaluator.evaluate(
       left: { integer: '1' },
       comparison_operator: '<',
       right: { integer: '1' },
-    )).to be_false
+    )).to be_falsey
   end
 
   specify do
@@ -35,7 +35,7 @@
         comparison_operator: '==',
         right: { integer: '1' },
       }
-    )).to be_true
+    )).to be_truthy
   end
 
   specify do
@@ -51,7 +51,7 @@
         comparison_operator: '==',
         right: { integer: '1' },
       }
-    )).to be_false
+    )).to be_falsey
   end
 
   specify do
@@ -67,7 +67,7 @@
         comparison_operator: '==',
         right: { integer: '1' },
       }
-    )).to be_true
+    )).to be_truthy
   end
 
   specify do
@@ -83,7 +83,7 @@
         comparison_operator: '==',
         right: { integer: '2' },
       }
-    )).to be_false
+    )).to be_falsey
   end
 
   specify do
@@ -93,6 +93,6 @@
         comparison_operator: '!=',
         right: { integer: '1' }
       }
-    )).to be_false
+    )).to be_falsey
   end
-end
+end

spec/evaluator_spec.rb

@@ -4,22 +4,22 @@
   context '#evaluate_comparison' do
     let(:evaluator) { described_class.new(nil, nil) }
 
-    specify { expect(evaluator.evaluate_comparison(1, '==', 1)).to be_true }
-    specify { expect(evaluator.evaluate_comparison(1, '==', 0)).to be_false }
-    specify { expect(evaluator.evaluate_comparison(1, '!=', 1)).to be_false }
-    specify { expect(evaluator.evaluate_comparison(1, '!=', 0)).to be_true }
-
-    specify { expect(evaluator.evaluate_comparison('alpha', '==', 'alpha')).to be_true }
-    specify { expect(evaluator.evaluate_comparison('beta', '==', 'omega')).to be_false }
-    specify { expect(evaluator.evaluate_comparison('alpha', '!=', 'alpha')).to be_false }
-    specify { expect(evaluator.evaluate_comparison('beta', '!=', 'omega')).to be_true }
-
-    specify { expect(evaluator.evaluate_comparison(10, '<', 20)).to be_true }
-    specify { expect(evaluator.evaluate_comparison(20, '<', 10)).to be_false }
-    specify { expect(evaluator.evaluate_comparison(20, '>', 10)).to be_true }
-    specify { expect(evaluator.evaluate_comparison(10, '>', 20)).to be_false }
-    specify { expect(evaluator.evaluate_comparison(20, '<=', 20)).to be_true }
-    specify { expect(evaluator.evaluate_comparison(20, '>=', 20)).to be_true }
+    specify { expect(evaluator.evaluate_comparison(1, '==', 1)).to be_truthy }
+    specify { expect(evaluator.evaluate_comparison(1, '==', 0)).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison(1, '!=', 1)).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison(1, '!=', 0)).to be_truthy }
+
+    specify { expect(evaluator.evaluate_comparison('alpha', '==', 'alpha')).to be_truthy }
+    specify { expect(evaluator.evaluate_comparison('beta', '==', 'omega')).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison('alpha', '!=', 'alpha')).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison('beta', '!=', 'omega')).to be_truthy }
+
+    specify { expect(evaluator.evaluate_comparison(10, '<', 20)).to be_truthy }
+    specify { expect(evaluator.evaluate_comparison(20, '<', 10)).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison(20, '>', 10)).to be_truthy }
+    specify { expect(evaluator.evaluate_comparison(10, '>', 20)).to be_falsey }
+    specify { expect(evaluator.evaluate_comparison(20, '<=', 20)).to be_truthy }
+    specify { expect(evaluator.evaluate_comparison(20, '>=', 20)).to be_truthy }
   end
 
   context '#evaluate_boolean' do
@@ -30,28 +30,28 @@
       allow(evaluator).to receive(:evaluate).with(false).and_return(false)
     end
 
-    specify { expect(evaluator.evaluate_boolean(true, 'and', true)).to be_true }
-    specify { expect(evaluator.evaluate_boolean(true, 'and', false)).to be_false }
-    specify { expect(evaluator.evaluate_boolean(false, 'and', true)).to be_false }
-    specify { expect(evaluator.evaluate_boolean(false, 'and', false)).to be_false }
+    specify { expect(evaluator.evaluate_boolean(true, 'and', true)).to be_truthy }
+    specify { expect(evaluator.evaluate_boolean(true, 'and', false)).to be_falsey }
+    specify { expect(evaluator.evaluate_boolean(false, 'and', true)).to be_falsey }
+    specify { expect(evaluator.evaluate_boolean(false, 'and', false)).to be_falsey }
 
-    specify { expect(evaluator.evaluate_boolean(true, 'or', true)).to be_true }
-    specify { expect(evaluator.evaluate_boolean(true, 'or', false)).to be_true }
-    specify { expect(evaluator.evaluate_boolean(false, 'or', true)).to be_true }
-    specify { expect(evaluator.evaluate_boolean(false, 'or', false)).to be_false }
+    specify { expect(evaluator.evaluate_boolean(true, 'or', true)).to be_truthy }
+    specify { expect(evaluator.evaluate_boolean(true, 'or', false)).to be_truthy }
+    specify { expect(evaluator.evaluate_boolean(false, 'or', true)).to be_truthy }
+    specify { expect(evaluator.evaluate_boolean(false, 'or', false)).to be_falsey }
   end
 
   context 'full parse' do
     def outcome_for(expression, context_hash = {})
       described_class.new(expression, context_hash).outcome
     end
-    specify { expect(outcome_for('1 == 1')).to be_true }
-    specify { expect(outcome_for('1 == 0')).to be_false }
-    specify { expect(outcome_for("(1 < 4 or 5 < 4) and (1 == 1 and 'alpha' == 'alpha')")).to be_true }
-    specify { expect(outcome_for('gamma == 7', { 'gamma' => 7 })).to be_true }
-    specify { expect(outcome_for('gamma == 8', { 'gamma' => 7 })).to be_false }
-    specify { expect(outcome_for("alpha == 'beta'", { 'alpha' => 'beta' })).to be_true }
-    specify { expect(outcome_for("alpha == 'beta'", { 'alpha' => 'delta' })).to be_false }
+    specify { expect(outcome_for('1 == 1')).to be_truthy }
+    specify { expect(outcome_for('1 == 0')).to be_falsey }
+    specify { expect(outcome_for("(1 < 4 or 5 < 4) and (1 == 1 and 'alpha' == 'alpha')")).to be_truthy }
+    specify { expect(outcome_for('gamma == 7', { 'gamma' => 7 })).to be_truthy }
+    specify { expect(outcome_for('gamma == 8', { 'gamma' => 7 })).to be_falsey }
+    specify { expect(outcome_for("alpha == 'beta'", { 'alpha' => 'beta' })).to be_truthy }
+    specify { expect(outcome_for("alpha == 'beta'", { 'alpha' => 'delta' })).to be_falsey }
   end
 end
 

spec/spec_helper.rb

@@ -1,9 +1,6 @@
 require 'simplecov'
 SimpleCov.start
 
-require "codeclimate-test-reporter"
-CodeClimate::TestReporter.start
-
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
   SimpleCov::Formatter::HTMLFormatter
 ]
@@ -21,7 +18,7 @@
   config.filter_run :focus
   config.filter_run_excluding perf: true
   config.order = 'random'
-#  config.include FactoryGirl::Syntax::Methods
+#  config.include FactoryBot::Syntax::Methods
 end
 
 require 'parslet/convenience'