Merge pull request #7 from saveriomiroddi/coveralls_integration_fixes

Coveralls integration fixes

Author: 64kramsystem

Gemfile

@@ -4,6 +4,10 @@ source "https://rubygems.org"
 
 gemspec
 
+group :test do
+  gem "coveralls", "~> 0.8.21", require: false
+end
+
 group :tools do
   gem "byebug"
   gem "rubocop", "~> 0.52"

json_on_rails.gemspec

@@ -27,7 +27,6 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency "activerecord", "~> 4.0"
   s.add_runtime_dependency "mysql2", "~> 0.3"
 
-  s.add_development_dependency "coveralls", "~> 0.8.21"
   s.add_development_dependency "rake", "~> 12.3"
   s.add_development_dependency "rspec", "~> 3.7"
   s.add_development_dependency "rspec-rails", "~> 3.7"

spec/setup/spec_environment_helper.rb

@@ -6,16 +6,22 @@ class SpecEnvironmentHelper
   TEST_MODELS_PATH = File.expand_path("models", __dir__)
 
   def setup_spec_environment
+    setup_coverage_tool
     boot_mini_app
     require_components
     connection_configuration = load_connection_configuration
     prepare_database(connection_configuration)
     autoload_models
-    setup_coverage_tool
   end
 
   private
 
+  def setup_coverage_tool
+    require "coveralls"
+
+    Coveralls.wear!
+  end
+
   def boot_mini_app
     # Requiring rails (/all) is the minimum for using rspec-rails, which gives transactional UTs.
     require "rails/all"
@@ -58,10 +64,4 @@ def autoload_models
     # paths directly. See http://guides.rubyonrails.org/autoloading_and_reloading_constants.html#autoload-paths
     ActiveSupport::Dependencies.autoload_paths << TEST_MODELS_PATH
   end
-
-  def setup_coverage_tool
-    require "coveralls"
-
-    Coveralls.wear!
-  end
 end