spoom 1.1.11 → 1.1.12

data/lib/spoom/coverage/d3/pie.rb

@@ -18,34 +18,38 @@ module Spoom
           @title = title
         end
 
-        sig { returns(String) }
-        def self.header_style
-          <<~CSS
-            .pie .title {
-              font: 18px Arial, sans-serif;
-              font-weight: bold;
-              fill: #212529;
-              text-anchor: middle;
-              pointer-events: none;
-            }
+        class << self
+          extend T::Sig
 
-            .pie .arc {
-              stroke: #fff;
-              stroke-width: 2px;
-            }
-          CSS
-        end
+          sig { returns(String) }
+          def header_style
+            <<~CSS
+              .pie .title {
+                font: 18px Arial, sans-serif;
+                font-weight: bold;
+                fill: #212529;
+                text-anchor: middle;
+                pointer-events: none;
+              }
 
-        sig { returns(String) }
-        def self.header_script
-          <<~JS
-            function tooltipPie(d, title, kind, sum) {
-              moveTooltip(d)
-                .html("<b>" + title + "</b><br><br>"
-                  + "<b>" + d.data.value + "</b> " + kind + "<br>"
-                  + "<b>" + toPercent(d.data.value, sum) + "</b>%")
-            }
-          JS
+              .pie .arc {
+                stroke: #fff;
+                stroke-width: 2px;
+              }
+            CSS
+          end
+
+          sig { returns(String) }
+          def header_script
+            <<~JS
+              function tooltipPie(d, title, kind, sum) {
+                moveTooltip(d)
+                  .html("<b>" + title + "</b><br><br>"
+                    + "<b>" + d.data.value + "</b> " + kind + "<br>"
+                    + "<b>" + toPercent(d.data.value, sum) + "</b>%")
+              }
+            JS
+          end
         end
 
         sig { override.returns(String) }

data/lib/spoom/coverage/d3/timeline.rb

@@ -18,79 +18,83 @@ module Spoom
           @keys = keys
         end
 
-        sig { returns(String) }
-        def self.header_style
-          <<~CSS
-            .domain {
-              stroke: transparent;
-            }
+        class << self
+          extend T::Sig
 
-            .grid line {
-              stroke: #ccc;
-            }
+          sig { returns(String) }
+          def header_style
+            <<~CSS
+              .domain {
+                stroke: transparent;
+              }
 
-            .axis text {
-              font: 12px Arial, sans-serif;
-              fill: #333;
-              text-anchor: right;
-              pointer-events: none;
-            }
+              .grid line {
+                stroke: #ccc;
+              }
 
-            .area {
-              fill-opacity: 0.5;
-            }
+              .axis text {
+                font: 12px Arial, sans-serif;
+                fill: #333;
+                text-anchor: right;
+                pointer-events: none;
+              }
 
-            .line {
-              stroke-width: 2;
-              fill: transparent;
-            }
+              .area {
+                fill-opacity: 0.5;
+              }
 
-            .dot {
-              r: 2;
-              fill: #888;
-            }
+              .line {
+                stroke-width: 2;
+                fill: transparent;
+              }
 
-            .inverted .grid line {
-              stroke: #777;
-            }
+              .dot {
+                r: 2;
+                fill: #888;
+              }
 
-            .inverted .area {
-              fill-opacity: 0.9;
-            }
+              .inverted .grid line {
+                stroke: #777;
+              }
 
-            .inverted .axis text {
-              fill: #fff;
-            }
+              .inverted .area {
+                fill-opacity: 0.9;
+              }
 
-            .inverted .axis line {
-              stroke: #fff;
-            }
+              .inverted .axis text {
+                fill: #fff;
+              }
 
-            .inverted .dot {
-              fill: #fff;
-            }
-          CSS
-        end
+              .inverted .axis line {
+                stroke: #fff;
+              }
 
-        sig { returns(String) }
-        def self.header_script
-          <<~JS
-            var parseVersion = function(version) {
-              if (!version) {
-                return null;
+              .inverted .dot {
+                fill: #fff;
               }
-              return parseFloat(version.replaceAll("0.", ""));
-            }
+            CSS
+          end
 
-            function tooltipTimeline(d, kind) {
-              moveTooltip(d)
-                .html("commit <b>" + d.data.commit + "</b><br>"
-                  + d3.timeFormat("%y/%m/%d")(parseDate(d.data.timestamp)) + "<br><br>"
-                  + "<b>typed: " + d.key + "</b><br><br>"
-                  + "<b>" + (d.data.values[d.key] ? d.data.values[d.key] : 0) + "</b> " + kind +"<br>"
-                  + "<b>" + toPercent(d.data.values[d.key] ? d.data.values[d.key] : 0, d.data.total) + "%")
-            }
-          JS
+          sig { returns(String) }
+          def header_script
+            <<~JS
+              var parseVersion = function(version) {
+                if (!version) {
+                  return null;
+                }
+                return parseFloat(version.replaceAll("0.", ""));
+              }
+
+              function tooltipTimeline(d, kind) {
+                moveTooltip(d)
+                  .html("commit <b>" + d.data.commit + "</b><br>"
+                    + d3.timeFormat("%y/%m/%d")(parseDate(d.data.timestamp)) + "<br><br>"
+                    + "<b>typed: " + d.key + "</b><br><br>"
+                    + "<b>" + (d.data.values[d.key] ? d.data.values[d.key] : 0) + "</b> " + kind +"<br>"
+                    + "<b>" + toPercent(d.data.values[d.key] ? d.data.values[d.key] : 0, d.data.total) + "%")
+              }
+            JS
+          end
         end
 
         sig { override.returns(String) }
@@ -264,13 +268,13 @@ module Spoom
                               d3.min(data_#{id}, (d) => parseVersion(d.runtime))]) - 0.01",
                 max: "d3.max([d3.max(data_#{id}, (d) => parseVersion(d.static)),
                               d3.max(data_#{id}, (d) => parseVersion(d.runtime))]) + 0.01",
-                ticks: 'ticks(8)'
+                ticks: "ticks(8)"
               )}
-              #{line(y: 'parseVersion(d.runtime)', color: '#e83e8c', curve: 'curveStepAfter')}
-              #{line(y: 'parseVersion(d.static)', color: '#007bff', curve: 'curveStepAfter')}
-              #{points(y: 'parseVersion(d.static)')}
+              #{line(y: "parseVersion(d.runtime)", color: "#e83e8c", curve: "curveStepAfter")}
+              #{line(y: "parseVersion(d.static)", color: "#007bff", curve: "curveStepAfter")}
+              #{points(y: "parseVersion(d.static)")}
               #{x_ticks}
-              #{y_ticks(ticks: 'ticks(4)', format: "'v0.' + d.toFixed(2)", padding: 50)}
+              #{y_ticks(ticks: "ticks(4)", format: "'v0.' + d.toFixed(2)", padding: 50)}
             JS
           end
         end
@@ -308,15 +312,15 @@ module Spoom
             <<~JS
               #{x_scale}
               #{y_scale(
-                min: '0',
+                min: "0",
                 max: "d3.max(data_#{id}, (d) => d.runtime)",
-                ticks: 'ticks(10)'
+                ticks: "ticks(10)"
               )}
-              #{area(y: 'd.runtime')}
-              #{line(y: 'd.runtime')}
-              #{points(y: 'd.runtime')}
+              #{area(y: "d.runtime")}
+              #{line(y: "d.runtime")}
+              #{points(y: "d.runtime")}
               #{x_ticks}
-              #{y_ticks(ticks: 'ticks(5)', format: 'd.toFixed(2) + "s"', padding: 40)}
+              #{y_ticks(ticks: "ticks(5)", format: 'd.toFixed(2) + "s"', padding: 40)}
                 .call(g => g.selectAll(".tick:first-of-type text").remove())
             JS
           end
@@ -373,15 +377,15 @@ module Spoom
           def plot
             <<~JS
               #{x_scale}
-              #{y_scale(min: '0', max: '100', ticks: 'tickValues([0, 25, 50, 75, 100])')}
-              #{line(y: 'd.data.timestamp')}
+              #{y_scale(min: "0", max: "100", ticks: "tickValues([0, 25, 50, 75, 100])")}
+              #{line(y: "d.data.timestamp")}
               #{x_ticks}
-              #{y_ticks(ticks: 'tickValues([25, 50, 75])', format: "d + '%'", padding: 30)}
+              #{y_ticks(ticks: "tickValues([25, 50, 75])", format: "d + '%'", padding: 30)}
             JS
           end
 
           sig { override.params(y: String, color: String, curve: String).returns(String) }
-          def line(y:, color: 'strictnessColor(d.key)', curve: 'curveCatmullRom.alpha(1)')
+          def line(y:, color: "strictnessColor(d.key)", curve: "curveCatmullRom.alpha(1)")
             <<~JS
               var area_#{id} = d3.area()
                 .x((d) => xScale_#{id}(parseDate(#{y})))
@@ -446,7 +450,7 @@ module Spoom
 
           sig { params(id: String, snapshots: T::Array[Snapshot]).void }
           def initialize(id, snapshots)
-            keys = ['false', 'true']
+            keys = ["false", "true"]
             data = snapshots.map do |snapshot|
               {
                 timestamp: snapshot.commit_timestamp,
@@ -473,7 +477,7 @@ module Spoom
 
           sig { params(id: String, snapshots: T::Array[Snapshot]).void }
           def initialize(id, snapshots)
-            keys = ['false', 'true']
+            keys = ["false", "true"]
             data = snapshots.map do |snapshot|
               {
                 timestamp: snapshot.commit_timestamp,
@@ -500,7 +504,7 @@ module Spoom
 
           sig { params(id: String, snapshots: T::Array[Snapshot]).void }
           def initialize(id, snapshots)
-            keys = ['rbis', 'files']
+            keys = ["rbis", "files"]
             data = snapshots.map do |snapshot|
               {
                 timestamp: snapshot.commit_timestamp,
@@ -567,7 +571,7 @@ module Spoom
           end
 
           sig { override.params(y: String, color: String, curve: String).returns(String) }
-          def line(y:, color: 'strictnessColor(d.key)', curve: 'curveCatmullRom.alpha(1)')
+          def line(y:, color: "strictnessColor(d.key)", curve: "curveCatmullRom.alpha(1)")
             <<~JS
               var area_#{id} = d3.area()
                 .x((d) => xScale_#{id}(parseDate(#{y})))
@@ -610,10 +614,10 @@ module Spoom
           def plot
             <<~JS
               #{x_scale}
-              #{y_scale(min: '0', max: "d3.max(data_#{id}, (d) => d.total + 10)", ticks: 'tickValues([0, 25, 50, 75, 100])')}
-              #{line(y: 'd.data.timestamp', color: "d.key == 'rbis' ? '#8673ff' : '#007bff'")}
+              #{y_scale(min: "0", max: "d3.max(data_#{id}, (d) => d.total + 10)", ticks: "tickValues([0, 25, 50, 75, 100])")}
+              #{line(y: "d.data.timestamp", color: "d.key == 'rbis' ? '#8673ff' : '#007bff'")}
               #{x_ticks}
-              #{y_ticks(ticks: 'tickValues([25, 50, 75])', format: 'd', padding: 20)}
+              #{y_ticks(ticks: "tickValues([25, 50, 75])", format: "d", padding: 20)}
             JS
           end
         end

data/lib/spoom/coverage/d3.rb

@@ -8,94 +8,96 @@ require_relative "d3/timeline"
 module Spoom
   module Coverage
     module D3
-      extend T::Sig
-
       COLOR_IGNORE = "#999"
       COLOR_FALSE = "#db4437"
       COLOR_TRUE = "#0f9d58"
       COLOR_STRICT = "#0a7340"
       COLOR_STRONG = "#064828"
 
-      sig { returns(String) }
-      def self.header_style
-        <<~CSS
-          svg {
-            width: 100%;
-            height: 100%;
-          }
+      class << self
+        extend T::Sig
 
-          .tooltip {
-            font: 12px Arial, sans-serif;
-            color: #fff;
-            text-align: center;
-            background: rgba(0, 0, 0, 0.6);
-            padding: 5px;
-            border: 0px;
-            border-radius: 4px;
-            position: absolute;
-            top: 0;
-            left: 0;
-            opacity: 0;
-          }
+        sig { returns(String) }
+        def header_style
+          <<~CSS
+            svg {
+              width: 100%;
+              height: 100%;
+            }
 
-          .label {
-            font: 14px Arial, sans-serif;
-            font-weight: bold;
-            fill: #fff;
-            text-anchor: middle;
-            pointer-events: none;
-          }
+            .tooltip {
+              font: 12px Arial, sans-serif;
+              color: #fff;
+              text-align: center;
+              background: rgba(0, 0, 0, 0.6);
+              padding: 5px;
+              border: 0px;
+              border-radius: 4px;
+              position: absolute;
+              top: 0;
+              left: 0;
+              opacity: 0;
+            }
 
-          .label .small {
-            font-size: 10px;
-          }
+            .label {
+              font: 14px Arial, sans-serif;
+              font-weight: bold;
+              fill: #fff;
+              text-anchor: middle;
+              pointer-events: none;
+            }
 
-          #{Pie.header_style}
-          #{CircleMap.header_style}
-          #{Timeline.header_style}
-        CSS
-      end
+            .label .small {
+              font-size: 10px;
+            }
+
+            #{Pie.header_style}
+            #{CircleMap.header_style}
+            #{Timeline.header_style}
+          CSS
+        end
 
-      sig { params(palette: ColorPalette).returns(String) }
-      def self.header_script(palette)
-        <<~JS
-          var parseDate = d3.timeParse("%s");
+        sig { params(palette: ColorPalette).returns(String) }
+        def header_script(palette)
+          <<~JS
+            var parseDate = d3.timeParse("%s");
 
-          function strictnessColor(strictness) {
-            switch(strictness) {
-              case "ignore":
-                return "#{palette.ignore}";
-              case "false":
-                return "#{palette.false}";
-              case "true":
-                return "#{palette.true}";
-              case "strict":
-                return "#{palette.strict}";
-              case "strong":
-                return "#{palette.strong}";
+            function strictnessColor(strictness) {
+              switch(strictness) {
+                case "ignore":
+                  return "#{palette.ignore}";
+                case "false":
+                  return "#{palette.false}";
+                case "true":
+                  return "#{palette.true}";
+                case "strict":
+                  return "#{palette.strict}";
+                case "strong":
+                  return "#{palette.strong}";
+              }
+              return "#{palette.false}";
             }
-            return "#{palette.false}";
-          }
 
-          function toPercent(value, sum) {
-            return value ? Math.round(value * 100 / sum) : 0;
-          }
+            function toPercent(value, sum) {
+              return value ? Math.round(value * 100 / sum) : 0;
+            }
 
-          var tooltip = d3.select("body")
-            .append("div")
+            var tooltip = d3.select("body")
               .append("div")
-                .attr("class", "tooltip");
+                .append("div")
+                  .attr("class", "tooltip");
 
-          function moveTooltip(d) {
-            return tooltip
-              .style("left", (d3.event.pageX + 20) + "px")
-              .style("top", (d3.event.pageY) + "px")
-          }
+            function moveTooltip(d) {
+              return tooltip
+                .style("left", (d3.event.pageX + 20) + "px")
+                .style("top", (d3.event.pageY) + "px")
+            }
 
-          #{Pie.header_script}
-          #{CircleMap.header_script}
-          #{Timeline.header_script}
-        JS
+            #{Pie.header_script}
+            #{CircleMap.header_script}
+            #{Timeline.header_script}
+          JS
+        end
       end
 
       class ColorPalette < T::Struct

data/lib/spoom/coverage/report.rb

@@ -109,7 +109,7 @@ module Spoom
         abstract!
 
         sig { void }
-        def initialize; end
+        def initialize; end # rubocop:disable Lint/MissingSuper
 
         sig { override.returns(String) }
         def html
@@ -136,17 +136,17 @@ module Spoom
 
         sig { returns(D3::Pie::Sigils) }
         def pie_sigils
-          D3::Pie::Sigils.new('pie_sigils', 'Sigils', snapshot)
+          D3::Pie::Sigils.new("pie_sigils", "Sigils", snapshot)
         end
 
         sig { returns(D3::Pie::Calls) }
         def pie_calls
-          D3::Pie::Calls.new('pie_calls', 'Calls', snapshot)
+          D3::Pie::Calls.new("pie_calls", "Calls", snapshot)
         end
 
         sig { returns(D3::Pie::Sigs) }
         def pie_sigs
-          D3::Pie::Sigs.new('pie_sigs', 'Sigs', snapshot)
+          D3::Pie::Sigs.new("pie_sigs", "Sigs", snapshot)
         end
       end
 
@@ -226,7 +226,7 @@ module Spoom
         extend T::Sig
 
         sig { params(sorbet_intro_commit: T.nilable(String), sorbet_intro_date: T.nilable(Time)).void }
-        def initialize(sorbet_intro_commit: nil, sorbet_intro_date: nil)
+        def initialize(sorbet_intro_commit: nil, sorbet_intro_date: nil) # rubocop:disable Lint/MissingSuper
           @sorbet_intro_commit = sorbet_intro_commit
           @sorbet_intro_date = sorbet_intro_date
         end
@@ -235,7 +235,7 @@ module Spoom
         def erb
           <<~ERB
             <div class="text-center" style="margin-top: 30px">
-              Typchecked by Sorbet since <b>#{@sorbet_intro_date&.strftime('%F')}</b>
+              Typchecked by Sorbet since <b>#{@sorbet_intro_date&.strftime("%F")}</b>
               (commit <b>#{@sorbet_intro_commit}</b>).
             </div>
           ERB

data/lib/spoom/coverage/snapshot.rb

@@ -32,44 +32,49 @@ module Spoom
         printer.print_snapshot(self)
       end
 
-      sig { params(json: String).returns(Snapshot) }
-      def self.from_json(json)
-        from_obj(JSON.parse(json))
+      sig { params(arg: T.untyped).returns(String) }
+      def to_json(*arg)
+        serialize.to_json(*arg)
       end
 
-      sig { params(obj: T::Hash[String, T.untyped]).returns(Snapshot) }
-      def self.from_obj(obj)
-        snapshot = Snapshot.new
-        snapshot.timestamp = obj.fetch("timestamp", 0)
-        snapshot.version_static = obj.fetch("version_static", nil)
-        snapshot.version_runtime = obj.fetch("version_runtime", nil)
-        snapshot.duration = obj.fetch("duration", 0)
-        snapshot.commit_sha = obj.fetch("commit_sha", nil)
-        snapshot.commit_timestamp = obj.fetch("commit_timestamp", nil)
-        snapshot.files = obj.fetch("files", 0)
-        snapshot.rbi_files = obj.fetch("rbi_files", 0)
-        snapshot.modules = obj.fetch("modules", 0)
-        snapshot.classes = obj.fetch("classes", 0)
-        snapshot.singleton_classes = obj.fetch("singleton_classes", 0)
-        snapshot.methods_with_sig = obj.fetch("methods_with_sig", 0)
-        snapshot.methods_without_sig = obj.fetch("methods_without_sig", 0)
-        snapshot.calls_typed = obj.fetch("calls_typed", 0)
-        snapshot.calls_untyped = obj.fetch("calls_untyped", 0)
-
-        sigils = obj.fetch("sigils", {})
-        if sigils
-          Snapshot::STRICTNESSES.each do |strictness|
-            next unless sigils.key?(strictness)
-            snapshot.sigils[strictness] = sigils[strictness]
-          end
+      class << self
+        extend T::Sig
+
+        sig { params(json: String).returns(Snapshot) }
+        def from_json(json)
+          from_obj(JSON.parse(json))
         end
 
-        snapshot
-      end
+        sig { params(obj: T::Hash[String, T.untyped]).returns(Snapshot) }
+        def from_obj(obj)
+          snapshot = Snapshot.new
+          snapshot.timestamp = obj.fetch("timestamp", 0)
+          snapshot.version_static = obj.fetch("version_static", nil)
+          snapshot.version_runtime = obj.fetch("version_runtime", nil)
+          snapshot.duration = obj.fetch("duration", 0)
+          snapshot.commit_sha = obj.fetch("commit_sha", nil)
+          snapshot.commit_timestamp = obj.fetch("commit_timestamp", nil)
+          snapshot.files = obj.fetch("files", 0)
+          snapshot.rbi_files = obj.fetch("rbi_files", 0)
+          snapshot.modules = obj.fetch("modules", 0)
+          snapshot.classes = obj.fetch("classes", 0)
+          snapshot.singleton_classes = obj.fetch("singleton_classes", 0)
+          snapshot.methods_with_sig = obj.fetch("methods_with_sig", 0)
+          snapshot.methods_without_sig = obj.fetch("methods_without_sig", 0)
+          snapshot.calls_typed = obj.fetch("calls_typed", 0)
+          snapshot.calls_untyped = obj.fetch("calls_untyped", 0)
 
-      sig { params(arg: T.untyped).returns(String) }
-      def to_json(*arg)
-        serialize.to_json(*arg)
+          sigils = obj.fetch("sigils", {})
+          if sigils
+            Snapshot::STRICTNESSES.each do |strictness|
+              next unless sigils.key?(strictness)
+
+              snapshot.sigils[strictness] = sigils[strictness]
+            end
+          end
+
+          snapshot
+        end
       end
     end
 
@@ -119,6 +124,7 @@ module Spoom
         indent
         hash.each do |key, value|
           next unless value > 0
+
           printl("#{key}: #{value}#{percent(value, total)}")
         end
         dedent
@@ -127,6 +133,7 @@ module Spoom
       sig { params(value: T.nilable(Integer), total: T.nilable(Integer)).returns(String) }
       def percent(value, total)
         return "" if value.nil? || total.nil? || total == 0
+
         " (#{(value.to_f * 100.0 / total.to_f).round}%)"
       end
     end