stackprof-webnav 0.1.0 → 1.0.3

data/lib/stackprof-webnav/views/layout.haml

@@ -7,6 +7,4 @@
     %link(rel="stylesheet" href="/css/application.css")
 
   %body
-    .row
-      .large-12.columns
-        = yield
+    = yield

data/lib/stackprof-webnav/views/method.haml

@@ -1,12 +1,15 @@
 %h3 StackProf Navigator - #{@action} (#{@frames.count} frames)
 %hr
 
-%a{:href => '/'} ← Back to overview
-%br
+%p
+  %a{:href => url_for("/overview")} ← Back to overview
 
 - @frames.each do |frame|
-  %h4
-    %a{:href => file_url(frame[:location])}= frame[:location]
+  %a{:href => url_for("/file", path: frame[:location])}
+    %button.btn
+      Browse
+      = frame[:location]
+
   - if frame[:callers].any?
     %table
       %thead
@@ -21,8 +24,8 @@
             %td= caller[:weight]
             %td= caller[:pct]
             %td
-              %a{:href => method_url(caller[:method])}
-                = caller[:method]
+              %a{:href => url_for("/method", name: caller[:method])}
+                &= caller[:method]
 
   - if frame[:callees].any?
     %table
@@ -38,8 +41,8 @@
             %td= caller[:weight]
             %td= caller[:pct]
             %td
-              %a{:href => method_url(caller[:method])}
-                = caller[:method]
+              %a{:href => url_for("/method", name: caller[:method])}
+                &= caller[:method]
 
   %h4 Code
   != frame[:source]

data/lib/stackprof-webnav/views/overview.haml

@@ -3,7 +3,21 @@
 
 %p
   Viewing dump
-  %b= @file
+  %b= current_dump.path
+
+%a{href: "/"}
+  %button.btn Browse directory
+
+
+%a{href: url_for("/graph")}
+  %button.btn View call graph
+
+- if current_report.data[:raw]
+  %a{href: url_for("/flamegraph")}
+    %button View flamegraph
+- else
+  %a{href: "https://github.com/tmm1/stackprof#all-options"}
+    %button.secondary Flamegraph is not available
 
 %table.centered
   %thead
@@ -11,15 +25,21 @@
     %th %
     %th Samples
     %th %
-    %th Method
+    %th
+      Method
+      %input.filter{:placeholder => "filter methods", :type => "text"}/
 
   %tbody
     - @frames.each do |frame|
-      %tr
+      - # Filter keys are preprocessed to better match acronyms.
+      %tr{"data-filter-key" => frame[:method].gsub(/[A-Z]/, " \\0").gsub(/[\W_]+/, " ").strip}
         %td= frame[:total]
         %td= frame[:total_pct]
         %td= frame[:samples]
         %td= frame[:samples_pct]
         %td
-          %a{:href => method_url(frame[:method])}
-            = frame[:method]
+          %a{:href => url_for("/method", name: frame[:method])}
+            &= frame[:method]
+
+%script{src: "lib/string_score.js"}
+%script{src: "overview.js"}

data/spec/helpers.rb

@@ -0,0 +1,17 @@
+require 'rack/test'
+
+module Helpers
+  def build_app(options={})
+    StackProf::Webnav::Server.cmd_options = options
+    Rack::Test::Session.new(Rack::MockSession.new(StackProf::Webnav::Server))
+  end
+
+  def fixture_path(name)
+    File.join(File.dirname(__FILE__), "fixtures", name)
+  end
+
+  def build_presenter(path)
+    report = StackProf::Report.new(Marshal.load(File.read(path)))
+    StackProf::Webnav::Presenter.new(report)
+  end
+end

data/spec/integration_spec.rb

@@ -0,0 +1,79 @@
+require "spec_helper"
+
+RSpec.describe "integration tests" do
+  let(:app) { build_app }
+  let(:presenter) { build_presenter(fixture_path("test.dump")) }
+  let(:response) { app.last_response }
+
+  after(:each) do
+    Dir.glob("spec/fixtures/*.flames.json").each {|file| File.delete(file) }
+    Dir.glob("spec/fixtures/*.digraph.dot").each {|file| File.delete(file) }
+    Dir.glob("spec/fixtures/*.graph.svg").each {|file| File.delete(file) }
+  end
+
+  describe "index" do
+    it "lists the files in a folder" do
+      app = build_app(directory: "spec/fixtures")
+      app.get "/"
+      expect(app.last_response.body).to include("test.dump")
+    end
+
+    it "redirects to overview if path is specified" do
+      app = build_app(filepath: fixture_path("test.dump"))
+      app.get "/"
+      app.follow_redirect!
+      expect(app.last_response.body).to include("Dummy")
+    end
+  end
+
+  it "does not crash on unknown requests" do
+    app.get "/favicon.ico"
+    expect(response.status).to eq(404)
+  end
+
+  describe "overview" do
+    it "works with a valid dump" do
+      app.get "/overview", dump: fixture_path("test.dump")
+
+      frame = presenter.overview_frames.first
+      expect(response.body).to include(frame[:method])
+    end
+
+    it "communicates that flamegraph is not available for dump" do
+      app.get "/overview", dump: fixture_path("test.dump")
+    end
+
+    it "does not crash if selected file is not a dump" do
+      app.get "/overview", dump: "Rakefile"
+      expect(response.body).to include("Unable to open")
+    end
+  end
+
+  it "is able to generate flames.json file" do
+    app.get "/flames.json", dump: fixture_path("test-raw.dump")
+    expect(response.body).to include("flamegraph")
+  end
+
+  it "is able to render graph" do
+    app.get "/graph.svg", dump: fixture_path("test.dump")
+    expect(response.get_header("Content-Type")).to eq("image/svg+xml")
+    expect(response.body.size).to_not eq(0)
+  end
+
+  it "method page renders information about a method" do
+    frame = presenter.overview_frames.first
+    app.get "/method", dump: fixture_path("test.dump"), name: frame[:method]
+    expect(response.body).to include("Callers")
+  end
+
+  it "does not crash on a file page" do
+    frame = presenter.overview_frames.first
+    method_info = presenter.method_info frame[:method]
+
+    expect {
+      app.get "/file",
+        dump: fixture_path("test.dump"),
+        path: method_info.first[:location]
+    }.to_not raise_error
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,15 @@
+ENV['APP_ENV'] = 'test'
+$LOAD_PATH.unshift File.join(Dir.pwd, "lib")
+
+require "stackprof-webnav"
+require "rspec"
+require_relative 'helpers'
+
+RSpec.configure do |config|
+  config.include Helpers
+  config.backtrace_exclusion_patterns << /.gem/
+
+  config.after(:each) do
+    StackProf::Webnav::Server.cmd_options = {}
+  end
+end

data/stackprof-webnav.gemspec

@@ -21,10 +21,15 @@ Gem::Specification.new do |spec|
   spec.bindir = 'bin'
   spec.executables << 'stackprof-webnav'
 
-  spec.add_dependency "sinatra", "~> 2.0.1"
-  spec.add_dependency "haml", "~> 4.0"
-  spec.add_dependency "stackprof", "~> 0.2"
+  spec.add_dependency "sinatra", "~> 2.1.0"
+  spec.add_dependency "haml", "~> 5.1.2"
+  spec.add_dependency "stackprof", ">= 0.2.13"
   spec.add_dependency "better_errors", "~> 1.1.0"
-  spec.add_development_dependency "bundler", "~> 1.5"
+  spec.add_dependency "ruby-graphviz", "~> 1.2.4"
+  spec.add_dependency "sinatra-contrib", "~> 2.1.0"
+  spec.add_dependency "webrick", "~> 1.7.0"
+  spec.add_development_dependency "bundler", "~> 2.2"
   spec.add_development_dependency "rake", "~> 10.1"
+  spec.add_development_dependency "rspec", "~> 3.9.0"
+  spec.add_development_dependency "rack-test", "~> 1.1.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stackprof-webnav
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 1.0.3
 platform: ruby
 authors:
 - Andrei Lisnic
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-03-01 00:00:00.000000000 Z
+date: 2021-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra
@@ -16,42 +16,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.1
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: haml
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: 5.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: 5.1.2
 - !ruby/object:Gem::Dependency
   name: stackprof
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: 0.2.13
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: 0.2.13
 - !ruby/object:Gem::Dependency
   name: better_errors
   requirement: !ruby/object:Gem::Requirement
@@ -66,20 +66,62 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.1.0
+- !ruby/object:Gem::Dependency
+  name: ruby-graphviz
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.4
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.2.4
+- !ruby/object:Gem::Dependency
+  name: sinatra-contrib
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+- !ruby/object:Gem::Dependency
+  name: webrick
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.7.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.7.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '2.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +136,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10.1'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.9.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.9.0
+- !ruby/object:Gem::Dependency
+  name: rack-test
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.0
 description: Provides the ability to analyze StackProf dumps
 email:
 - andrei.lisnic@gmail.com
@@ -102,6 +172,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - Gemfile
 - Gemfile.lock
@@ -111,27 +182,43 @@ files:
 - Rakefile
 - bin/stackprof-webnav
 - lib/stackprof-webnav.rb
+- lib/stackprof-webnav/dump.rb
 - lib/stackprof-webnav/presenter.rb
 - lib/stackprof-webnav/public/css/application.css
 - lib/stackprof-webnav/public/css/code.css
 - lib/stackprof-webnav/public/css/foundation.min.css
 - lib/stackprof-webnav/public/css/normalize.css
+- lib/stackprof-webnav/public/flamegraph.js
+- lib/stackprof-webnav/public/lib/string_score.js
+- lib/stackprof-webnav/public/overview.js
 - lib/stackprof-webnav/server.rb
 - lib/stackprof-webnav/version.rb
+- lib/stackprof-webnav/views/error.haml
 - lib/stackprof-webnav/views/file.haml
+- lib/stackprof-webnav/views/flamegraph.haml
+- lib/stackprof-webnav/views/graph.haml
+- lib/stackprof-webnav/views/index.haml
+- lib/stackprof-webnav/views/invalid_dump.haml
 - lib/stackprof-webnav/views/layout.haml
-- lib/stackprof-webnav/views/listing.haml
 - lib/stackprof-webnav/views/method.haml
 - lib/stackprof-webnav/views/overview.haml
+- screenshots/callgraph.png
+- screenshots/directory.png
 - screenshots/file.png
-- screenshots/main.png
+- screenshots/flamegraph.png
 - screenshots/method.png
+- screenshots/overview.png
+- spec/fixtures/test-raw.dump
+- spec/fixtures/test.dump
+- spec/helpers.rb
+- spec/integration_spec.rb
+- spec/spec_helper.rb
 - stackprof-webnav.gemspec
 homepage: https://github.com/alisnic/stackprof-webnav
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -146,9 +233,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.5
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: View stackprof dumps in a web UI
-test_files: []
+test_files:
+- spec/fixtures/test-raw.dump
+- spec/fixtures/test.dump
+- spec/helpers.rb
+- spec/integration_spec.rb
+- spec/spec_helper.rb