coverband 4.2.6 → 5.0.0.rc.5

data/test/coverband/utils/html_formatter_test.rb

@@ -1,41 +1,27 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 class HTMLFormatterTest < Minitest::Test
   def setup
     super
-    @store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: 'coverband_test')
+    @store = Coverband::Adapters::RedisStore.new(Coverband::Test.redis, redis_namespace: "coverband_test")
   end
 
-  test 'generate dynamic content hosted html report' do
+  test "generate dynamic content hosted html report" do
     Coverband.configure do |config|
-      config.store             = @store
-      config.ignore            = ['notsomething.rb']
+      config.store = @store
+      config.ignore = ["notsomething.rb"]
     end
     mock_file_hash
     @store.send(:save_report, basic_coverage_full_path)
 
     notice = nil
-    base_path = '/coverage'
+    base_path = "/coverage"
     filtered_report_files = Coverband::Reporters::Base.report(@store, {})
     html = Coverband::Utils::HTMLFormatter.new(filtered_report_files,
-                                               base_path: base_path,
-                                               notice: notice).format_dynamic_html!
-    assert_match 'loading source data', html
-  end
-
-  test 'generate static HTML report file' do
-    Coverband.configure do |config|
-      config.store             = @store
-      config.ignore            = ['notsomething.rb']
-    end
-    mock_file_hash
-    @store.send(:save_report, basic_coverage_full_path)
-
-    filtered_report_files = Coverband::Reporters::Base.report(@store, {})
-    Coverband::Utils::HTMLFormatter.new(filtered_report_files).format_static_html!
-    html = File.read("#{Coverband.configuration.root}/coverage/index.html")
-    assert_match 'Coverage first seen', html
+      base_path: base_path,
+      notice: notice).format_dynamic_html!
+    assert_match "loading source data", html
   end
 end

data/test/coverband/utils/lines_classifier_test.rb

@@ -1,34 +1,34 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 ####
 # Thanks for all the help SimpleCov https://github.com/colszowka/simplecov-html
 # initial version of test pulled into Coverband from Simplecov 12/17/2018
 ####
 describe Coverband::Utils::LinesClassifier do
-  describe '#classify' do
+  describe "#classify" do
     def subject
       Coverband::Utils::LinesClassifier.new
     end
 
-    describe 'relevant lines' do
-      it 'determines code as relevant' do
+    describe "relevant lines" do
+      it "determines code as relevant" do
         classified_lines = subject.classify [
-          'module Foo',
-          '  class Baz',
-          '    def Bar',
+          "module Foo",
+          "  class Baz",
+          "    def Bar",
           "      puts 'hi'",
-          '    end',
-          '  end',
-          'end'
+          "    end",
+          "  end",
+          "end"
         ]
 
         assert_equal 7, classified_lines.length
         assert(classified_lines.all? { |line| line == Coverband::Utils::LinesClassifier::RELEVANT })
       end
 
-      it 'determines invalid UTF-8 byte sequences as relevant' do
+      it "determines invalid UTF-8 byte sequences as relevant" do
         classified_lines = subject.classify [
           "bytes = \"\xF1t\xEBrn\xE2ti\xF4n\xE0liz\xE6ti\xF8n\""
         ]
@@ -38,11 +38,11 @@ describe Coverband::Utils::LinesClassifier do
       end
     end
 
-    describe 'not-relevant lines' do
-      it 'determines whitespace is not-relevant' do
+    describe "not-relevant lines" do
+      it "determines whitespace is not-relevant" do
         classified_lines = subject.classify [
-          '',
-          '   ',
+          "",
+          "   ",
           "\t\t"
         ]
 
@@ -50,11 +50,11 @@ describe Coverband::Utils::LinesClassifier do
         assert(classified_lines.all? { |line| line == Coverband::Utils::LinesClassifier::NOT_RELEVANT })
       end
 
-      describe 'comments' do
-        it 'determines comments are not-relevant' do
+      describe "comments" do
+        it "determines comments are not-relevant" do
           classified_lines = subject.classify [
-            '#Comment',
-            ' # Leading space comment',
+            "#Comment",
+            " # Leading space comment",
             "\t# Leading tab comment"
           ]
 
@@ -72,27 +72,27 @@ describe Coverband::Utils::LinesClassifier do
         end
       end
 
-      describe ':nocov: blocks' do
-        it 'determines :nocov: blocks are not-relevant' do
+      describe ":nocov: blocks" do
+        it "determines :nocov: blocks are not-relevant" do
           classified_lines = subject.classify [
-            '# :nocov:',
-            'def hi',
-            'end',
-            '# :nocov:'
+            "# :nocov:",
+            "def hi",
+            "end",
+            "# :nocov:"
           ]
 
           assert_equal 4, classified_lines.length
           assert(classified_lines.all? { |line| line == Coverband::Utils::LinesClassifier::NOT_RELEVANT })
         end
 
-        it 'determines all lines after a non-closing :nocov: as not-relevant' do
+        it "determines all lines after a non-closing :nocov: as not-relevant" do
           classified_lines = subject.classify [
-            '# :nocov:',
+            "# :nocov:",
             "puts 'Not relevant'",
-            '# :nocov:',
+            "# :nocov:",
             "puts 'Relevant again'",
             "puts 'Still relevant'",
-            '# :nocov:',
+            "# :nocov:",
             "puts 'Not relevant till the end'",
             "puts 'Ditto'"
           ]

data/test/coverband/utils/relative_file_converter_test.rb

@@ -1,38 +1,38 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 module Coverband
   module Utils
     class RelativeFileConverterTest < ::Minitest::Test
       def test_convert
-        converter = RelativeFileConverter.new(['/bar/tmp/'])
-        assert_equal('./gracie.rb', converter.convert('/bar/tmp/gracie.rb'))
+        converter = RelativeFileConverter.new(["/bar/tmp/"])
+        assert_equal("./gracie.rb", converter.convert("/bar/tmp/gracie.rb"))
       end
 
       def test_convert_without_leading_forward_slash
-        converter = RelativeFileConverter.new(['/foo/bar'])
-        assert_equal('./file.rb', converter.convert('/foo/bar/file.rb'))
+        converter = RelativeFileConverter.new(["/foo/bar"])
+        assert_equal("./file.rb", converter.convert("/foo/bar/file.rb"))
       end
 
       def test_multiple_roots
-        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
-        assert_equal('./josie.rb', converter.convert('/foo/bar/josie.rb'))
+        converter = RelativeFileConverter.new(["/bar/tmp/", "/foo/bar/"])
+        assert_equal("./josie.rb", converter.convert("/foo/bar/josie.rb"))
       end
 
       def test_no_match
-        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
-        assert_equal('/foo/josie.rb', converter.convert('/foo/josie.rb'))
+        converter = RelativeFileConverter.new(["/bar/tmp/", "/foo/bar/"])
+        assert_equal("/foo/josie.rb", converter.convert("/foo/josie.rb"))
       end
 
       def test_middle_path_match
-        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
-        assert_equal('/tmp/foo/bar/josie.rb', converter.convert('/tmp/foo/bar/josie.rb'))
+        converter = RelativeFileConverter.new(["/bar/tmp/", "/foo/bar/"])
+        assert_equal("/tmp/foo/bar/josie.rb", converter.convert("/tmp/foo/bar/josie.rb"))
       end
 
       def test_already_relative_file
-        converter = RelativeFileConverter.new(['/bar/tmp/', '/foo/bar/'])
-        assert_equal('./foo/bar/josie.rb', converter.convert('./foo/bar/josie.rb'))
+        converter = RelativeFileConverter.new(["/bar/tmp/", "/foo/bar/"])
+        assert_equal("./foo/bar/josie.rb", converter.convert("./foo/bar/josie.rb"))
       end
     end
   end

data/test/coverband/utils/result_test.rb

@@ -1,59 +1,59 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 ####
 # Thanks for all the help SimpleCov https://github.com/colszowka/simplecov-html
 # initial version of test pulled into Coverband from Simplecov 12/19/2018
 ####
-describe 'result' do
-  describe 'with a (mocked) Coverage.result' do
+describe "result" do
+  describe "with a (mocked) Coverage.result" do
     let(:original_result) do
       {
-        source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
-        source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
-        source_fixture('app/controllers/sample_controller.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil]
+        source_fixture("sample.rb") => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
+        source_fixture("app/models/user.rb") => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
+        source_fixture("app/controllers/sample_controller.rb") => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil]
       }
     end
 
-    describe 'a simple cov result initialized from that' do
+    describe "a simple cov result initialized from that" do
       subject { Coverband::Utils::Result.new(original_result) }
 
-      it 'has 3 filenames' do
+      it "has 3 filenames" do
         assert_equal 3, subject.filenames.count
       end
 
-      it 'has 3 source files' do
+      it "has 3 source files" do
         assert_equal 3, subject.source_files.count
         subject.source_files.each do |source_file|
           assert source_file.is_a?(Coverband::Utils::SourceFile)
         end
       end
 
-      it 'returns an instance of Coverband::Utils::FileList for source_files and files' do
+      it "returns an instance of Coverband::Utils::FileList for source_files and files" do
         assert subject.files.is_a?(Coverband::Utils::FileList)
         assert subject.source_files.is_a?(Coverband::Utils::FileList)
       end
 
-      it 'has files equal to source_files' do
+      it "has files equal to source_files" do
         assert_equal subject.source_files, subject.files
       end
 
-      it 'has accurate covered percent' do
+      it "has accurate covered percent" do
         # in our fixture, there are 13 covered line (result in 1) in all 15 relevant line (result in non-nil)
         assert_equal 86.66666666666667, subject.covered_percent
       end
 
-      it 'has accurate covered percentages' do
+      it "has accurate covered percentages" do
         assert_equal [80.0, 80.0, 100.0], subject.covered_percentages
       end
 
       %i[covered_percent
-         covered_percentages
-         covered_strength
-         covered_lines
-         missed_lines
-         total_lines].each do |msg|
+        covered_percentages
+        covered_strength
+        covered_lines
+        missed_lines
+        total_lines].each do |msg|
         it "responds to #{msg}" do
           assert(subject.respond_to?(msg))
         end

data/test/coverband/utils/results_test.rb

@@ -1,52 +1,52 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
-describe 'results' do
-  describe 'with a (mocked) Coverage.result' do
-    let(:source_file) { Coverband::Utils::SourceFile.new(source_fixture('app/models/user.rb'), run_lines) }
+describe "results" do
+  describe "with a (mocked) Coverage.result" do
+    let(:source_file) { Coverband::Utils::SourceFile.new(source_fixture("app/models/user.rb"), run_lines) }
     let(:eager_lines) { [nil, 1, 1, 0, nil, nil, 1, 0, nil, nil] }
-    let(:run_lines) { [nil, nil, nil, 1, nil, nil, nil,nil, nil, nil] }
+    let(:run_lines) { [nil, nil, nil, 1, nil, nil, nil, nil, nil, nil] }
     let(:original_result) do
       orig = {
-        Coverband::MERGED_TYPE => {source_fixture('app/models/user.rb') => eager_lines},
+        Coverband::MERGED_TYPE => {source_fixture("app/models/user.rb") => eager_lines}
       }
-      orig.merge!({Coverband::EAGER_TYPE => {source_fixture('app/models/user.rb') => eager_lines}}) if eager_lines
-      orig.merge!({Coverband::RUNTIME_TYPE => {source_fixture('app/models/user.rb') => run_lines}}) if run_lines
+      orig[Coverband::EAGER_TYPE] = {source_fixture("app/models/user.rb") => eager_lines} if eager_lines
+      orig[Coverband::RUNTIME_TYPE] = {source_fixture("app/models/user.rb") => run_lines} if run_lines
       orig
     end
     subject { Coverband::Utils::Results.new(original_result) }
 
-    describe 'runtime relevant lines is supported' do
-      it 'has correct runtime relevant coverage' do
+    describe "runtime relevant lines is supported" do
+      it "has correct runtime relevant coverage" do
         assert_equal 50.0, subject.runtime_relevant_coverage(source_file)
       end
 
-      it 'has correct runtime relevant lines' do
+      it "has correct runtime relevant lines" do
         assert_equal 2, subject.runtime_relavent_lines(source_file)
       end
     end
 
-    describe 'runtime relevant lines when no runtime coverage exists' do
+    describe "runtime relevant lines when no runtime coverage exists" do
       let(:run_lines) { nil }
 
-      it 'has correct runtime relevant lines' do
+      it "has correct runtime relevant lines" do
         assert_equal 0.0, subject.runtime_relevant_coverage(source_file)
       end
 
-      it 'has correct runtime relevant lines' do
+      it "has correct runtime relevant lines" do
         assert_equal 2, subject.runtime_relavent_lines(source_file)
       end
     end
 
-    describe 'runtime relevant lines when no eager coverage exists' do
+    describe "runtime relevant lines when no eager coverage exists" do
       let(:eager_lines) { nil }
 
-      it 'has correct runtime relevant lines' do
+      it "has correct runtime relevant lines" do
         assert_equal 100.0, subject.runtime_relevant_coverage(source_file)
       end
 
-      it 'has correct runtime relevant lines' do
+      it "has correct runtime relevant lines" do
         assert_equal 1, subject.runtime_relavent_lines(source_file)
       end
     end

data/test/coverband/utils/source_file_line_test.rb

@@ -1,165 +1,165 @@
 # frozen_string_literal: true
 
-require File.expand_path('../../test_helper', File.dirname(__FILE__))
+require File.expand_path("../../test_helper", File.dirname(__FILE__))
 
 ####
 # Thanks for all the help SimpleCov https://github.com/colszowka/simplecov-html
 # initial version of test pulled into Coverband from Simplecov 12/19/2018
 ####
 describe Coverband::Utils::SourceFile::Line do
-  describe 'a source line' do
+  describe "a source line" do
     subject do
-      Coverband::Utils::SourceFile::Line.new('# the ruby source', 5, 3)
+      Coverband::Utils::SourceFile::Line.new("# the ruby source", 5, 3)
     end
 
     it 'returns "# the ruby source" as src' do
-      assert_equal '# the ruby source', subject.src
+      assert_equal "# the ruby source", subject.src
     end
 
-    it 'returns the same for source as for src' do
+    it "returns the same for source as for src" do
       assert_equal subject.src, subject.source
     end
 
-    it 'has line number 5' do
+    it "has line number 5" do
       assert_equal 5, subject.line_number
     end
 
-    it 'has equal line_number, line and number' do
+    it "has equal line_number, line and number" do
       assert_equal subject.line, subject.line_number
       assert_equal subject.number, subject.line_number
     end
 
-    describe 'flagged as skipped!' do
+    describe "flagged as skipped!" do
       before do
         subject.skipped!
       end
-      it 'is not covered' do
+      it "is not covered" do
         refute subject.covered?
       end
 
-      it 'is skipped' do
+      it "is skipped" do
         assert subject.skipped?
       end
 
-      it 'is not missed' do
+      it "is not missed" do
         refute subject.missed?
       end
 
-      it 'is not never' do
+      it "is not never" do
         refute subject.never?
       end
 
-      it 'status is skipped' do
-        assert_equal 'skipped', subject.status
+      it "status is skipped" do
+        assert_equal "skipped", subject.status
       end
     end
   end
 
-  describe 'A source line with coverage' do
+  describe "A source line with coverage" do
     subject do
-      Coverband::Utils::SourceFile::Line.new('# the ruby source', 5, 3)
+      Coverband::Utils::SourceFile::Line.new("# the ruby source", 5, 3)
     end
 
-    it 'has coverage of 3' do
+    it "has coverage of 3" do
       assert_equal 3, subject.coverage
     end
 
-    it 'is covered' do
+    it "is covered" do
       assert subject.covered?
     end
 
-    it 'is not skipped' do
+    it "is not skipped" do
       refute subject.skipped?
     end
 
-    it 'is not missed' do
+    it "is not missed" do
       refute subject.missed?
     end
 
-    it 'is not never' do
+    it "is not never" do
       refute subject.never?
     end
 
-    it 'status is covered' do
-      assert_equal 'covered', subject.status
+    it "status is covered" do
+      assert_equal "covered", subject.status
     end
   end
 
-  describe 'A source line without coverage' do
+  describe "A source line without coverage" do
     subject do
-      Coverband::Utils::SourceFile::Line.new('# the ruby source', 5, 0)
+      Coverband::Utils::SourceFile::Line.new("# the ruby source", 5, 0)
     end
 
-    it 'has coverage of 0' do
+    it "has coverage of 0" do
       assert_equal 0, subject.coverage
     end
 
-    it 'is not covered' do
+    it "is not covered" do
       refute subject.covered?
     end
 
-    it 'is not skipped' do
+    it "is not skipped" do
       refute subject.skipped?
     end
 
-    it 'is missed' do
+    it "is missed" do
       assert subject.missed?
     end
 
-    it 'is not never' do
+    it "is not never" do
       refute subject.never?
     end
 
-    it 'status is missed' do
-      assert_equal 'missed', subject.status
+    it "status is missed" do
+      assert_equal "missed", subject.status
     end
   end
 
-  describe 'A source line with no code' do
+  describe "A source line with no code" do
     subject do
-      Coverband::Utils::SourceFile::Line.new('# the ruby source', 5, nil)
+      Coverband::Utils::SourceFile::Line.new("# the ruby source", 5, nil)
     end
 
-    it 'has nil coverage' do
+    it "has nil coverage" do
       assert_nil subject.coverage
     end
 
-    it 'is not covered' do
+    it "is not covered" do
       refute subject.covered?
     end
 
-    it 'is not skipped' do
+    it "is not skipped" do
       refute subject.skipped?
     end
 
-    it 'is not missed' do
+    it "is not missed" do
       refute subject.missed?
     end
 
-    it 'is never' do
+    it "is never" do
       assert subject.never?
     end
 
-    it 'status is never' do
-      assert_equal 'never', subject.status
+    it "status is never" do
+      assert_equal "never", subject.status
     end
   end
 
-  it 'raises ArgumentError when initialized with invalid src' do
+  it "raises ArgumentError when initialized with invalid src" do
     assert_raises ArgumentError do
       Coverband::Utils::SourceFile::Line.new(:symbol, 5, 3)
     end
   end
 
-  it 'raises ArgumentError when initialized with invalid line_number' do
+  it "raises ArgumentError when initialized with invalid line_number" do
     assert_raises ArgumentError do
-      Coverband::Utils::SourceFile::Line.new('some source', 'five', 3)
+      Coverband::Utils::SourceFile::Line.new("some source", "five", 3)
     end
   end
 
-  it 'raises ArgumentError when initialized with invalid coverage' do
+  it "raises ArgumentError when initialized with invalid coverage" do
     assert_raises ArgumentError do
-      Coverband::Utils::SourceFile::Line.new('some source', 5, 'three')
+      Coverband::Utils::SourceFile::Line.new("some source", 5, "three")
     end
   end
 end