rspec_profiling 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: fc2c3039e171c0935a2c938945e538e5ca46f1c2
-  data.tar.gz: bf4e57b9289fe5a396084b0e80a836586259ad2c
+SHA256:
+  metadata.gz: 24110d7ae61d04956ff76ff9e13c088c238a5fe66218d9e1aa4ef8a62bdbd9c6
+  data.tar.gz: 44d1a03208cba2c4249cbc81b27657dc535157d8fbe7ae302e618645732aebb5
 SHA512:
-  metadata.gz: 526049fc2f4db27873427c16cde7da2ab1a1abc87f513fac462fdef5c96aad5f214cb2a59658a4d04ee9fe0faf1c9ecaf577b76829cbefab2f914d54fd3ecea1
-  data.tar.gz: ba80e7353e2b5f1b2187a08172a2b4bc280b16aaee9a79de1c99ac1db7f3f306e2b8de10a09b0c2ba9910e5e0f5f2d1b0380df16bbb66725f994e964a9884d1b
+  metadata.gz: 3ad4f846af0ec602a4f8211f7f53f4358764e8aeed1e558dd46a3e8b8f70db66e02d53aa1242fb0715eb596c08c302d0d9cf23c499cf45246ae55f9f37e38c63
+  data.tar.gz: f9b030d41c7a68b511254bcdeb153a0b2dfd047928e9b38f1ffba123423dbd908d10badf0d461934c5f8e5bfc9944f0260e80558fbd8b57f51d800ef9e986506

data/.ruby-version

@@ -1 +1 @@
-ruby-2.2.1
+ruby-2.7.1

data/CHANGELOG.md

@@ -0,0 +1,7 @@
+## 0.0.6
+
+* Create indexes for DB collectors [#26]
+* Use ActiveRecord v5 data_source_exists? if in use [#25]
+* Strip newlines and other whitespace in Git output [#24]
+
+## 0.0.5

data/README.md

@@ -169,7 +169,7 @@ require 'rspec_profiling'
 require 'rspec_profiling/console'
 ```
 
-Then `results` will be available as a variable to the script. 
+Then `results` will be available as a variable to the script.
 
 ## Uninstalling
 
@@ -183,10 +183,10 @@ To remove the results database, run `bundle exec rake rspec_profiling:uninstall`
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
 
-## About Foraker Labs
+## About Breakwater
 
-![Foraker Logo](http://assets.foraker.com/attribution_logo.png)
+![Breakwater Logo](https://images.squarespace-cdn.com/content/5d084fe43b0b620001239437/1565926359217-2LQ1BOFAO5846OAYAGZV/breakwater.png?content-type=image%2Fpng)
 
-Foraker Labs builds exciting web and mobile apps in Boulder, CO. Our work powers a wide variety of businesses with many different needs. We love open source software, and we're proud to contribute where we can. Interested to learn more? [Contact us today](https://www.foraker.com/contact-us).
+Breakwater builds exciting web and mobile apps in Louisville, CO. Our work powers a wide variety of businesses with many different needs. We love open source software, and we're proud to contribute where we can. Interested to learn more? [Contact us today](https://www.breakwaterltd.com/contact).
 
-This project is maintained by Foraker Labs. The names and logos of Foraker Labs are fully owned and copyright Foraker Design, LLC.
+This project is maintained by Breakwater. The names and logos of Breakwater are fully owned and copyright Breakwater Limited.

data/lib/rspec_profiling/collectors/csv.rb

@@ -5,7 +5,7 @@ module RspecProfiling
     class CSV
       HEADERS = %w{
         branch
-        commit
+        commit_hash
         date
         file
         line_number

data/lib/rspec_profiling/collectors/psql.rb

@@ -25,19 +25,19 @@ module RspecProfiling
         return if prepared?
 
         connection.create_table(table) do |t|
-          t.string    :branch
-          t.string    :commit
-          t.datetime  :date
-          t.text      :file
-          t.integer   :line_number
+          t.string    :branch, index: true
+          t.string    :commit_hash, index: true
+          t.datetime  :date, index: true
+          t.text      :file, index: true
+          t.integer   :line_number, index: true
           t.text      :description
-          t.decimal   :time
-          t.string    :status
+          t.decimal   :time, index: true
+          t.string    :status, index: true
           t.text      :exception
-          t.integer   :query_count
-          t.decimal   :query_time
-          t.integer   :request_count
-          t.decimal   :request_time
+          t.integer   :query_count, index: true
+          t.decimal   :query_time, index:true
+          t.integer   :request_count, index: true
+          t.decimal   :request_time, index: true
           t.timestamps null: true
         end
       end
@@ -63,7 +63,15 @@ module RspecProfiling
       private
 
       def prepared?
-        connection.table_exists?(table)
+        if active_record5_or_up?
+          connection.data_source_exists?(table)
+        else
+          connection.table_exists?(table)
+        end
+      end
+
+      def active_record5_or_up?
+        ActiveRecord::VERSION::STRING[0].to_i >= 5
       end
 
       def connection

data/lib/rspec_profiling/collectors/sql.rb

@@ -25,19 +25,19 @@ module RspecProfiling
         return if prepared?
 
         connection.create_table(table) do |t|
-          t.string    :branch
-          t.string    :commit
-          t.datetime  :date
-          t.text      :file
-          t.integer   :line_number
+          t.string    :branch, index: true
+          t.string    :commit_hash, index: true
+          t.datetime  :date, index: true
+          t.text      :file, index: true
+          t.integer   :line_number, index: true
           t.text      :description
-          t.decimal   :time
-          t.string    :status
+          t.decimal   :time, index: true
+          t.string    :status, index: true
           t.text      :exception
-          t.integer   :query_count
-          t.decimal   :query_time
-          t.integer   :request_count
-          t.decimal   :request_time
+          t.integer   :query_count, index: true
+          t.decimal   :query_time, index:true
+          t.integer   :request_count, index: true
+          t.decimal   :request_time, index: true
           t.timestamps null: true
         end
       end

data/lib/rspec_profiling/run.rb

@@ -28,7 +28,7 @@ module RspecProfiling
     def example_finished(*args)
       collector.insert({
         branch:        vcs.branch,
-        commit:        vcs.sha,
+        commit_hash:   vcs.sha,
         date:          vcs.time,
         file:          @current_example.file,
         line_number:   @current_example.line_number,

data/lib/rspec_profiling/vcs/git.rb

@@ -4,11 +4,11 @@ module RspecProfiling
   module VCS
     class Git
       def branch
-        `git rev-parse --abbrev-ref HEAD`
+        `git rev-parse --abbrev-ref HEAD`.chomp
       end
 
       def sha
-        `git rev-parse HEAD`
+        `git rev-parse HEAD`.chomp
       end
 
       def time

data/lib/rspec_profiling/version.rb

@@ -1,3 +1,3 @@
 module RspecProfiling
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/rspec_profiling.gemspec

@@ -23,7 +23,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "pg"
   spec.add_dependency "rails"
 
-  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "bundler", "> 1.3"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
 end

data/spec/collectors/psql_spec.rb

@@ -14,7 +14,7 @@ module RspecProfiling
         before do
           collector.insert({
             branch: "master",
-            commit: "ABC123",
+            commit_hash: "ABC123",
             date: "Thu Dec 18 12:00:00 2012",
             file: "/some/file.rb",
             line_number: 10,
@@ -37,11 +37,11 @@ module RspecProfiling
           expect(result.branch).to eq "master"
         end
 
-        it "records the commit SHA" do
-          expect(result.commit).to eq "ABC123"
+        it "records the commit_hash SHA" do
+          expect(result.commit_hash).to eq "ABC123"
         end
 
-        it "records the commit date" do
+        it "records the commit_hash date" do
           expect(result.date).to eq Time.utc(2012, 12, 18, 12)
         end
 

data/spec/collectors/sql_spec.rb

@@ -14,7 +14,7 @@ module RspecProfiling
         before do
           collector.insert({
             branch: "master",
-            commit: "ABC123",
+            commit_hash: "ABC123",
             date: "Thu Dec 18 12:00:00 2012",
             file: "/some/file.rb",
             line_number: 10,
@@ -37,11 +37,11 @@ module RspecProfiling
           expect(result.branch).to eq "master"
         end
 
-        it "records the commit SHA" do
-          expect(result.commit).to eq "ABC123"
+        it "records the commit_hash SHA" do
+          expect(result.commit_hash).to eq "ABC123"
         end
 
-        it "records the commit date" do
+        it "records the commit_hash date" do
           expect(result.date).to eq Time.utc(2012, 12, 18, 12)
         end
 

data/spec/run_spec.rb

@@ -52,8 +52,8 @@ module RspecProfiling
         expect(result.branch).to eq "master"
       end
 
-      it "records the commit SHA" do
-        expect(result.commit).to eq "abc123"
+      it "records the commit_hash SHA" do
+        expect(result.commit_hash).to eq "abc123"
       end
 
       it "counts two queries" do

data/spec/vcs/git_spec.rb

@@ -4,22 +4,22 @@ module RspecProfiling
   describe VCS::Git do
     describe "#branch" do
       it "calls Git to get the current branch" do
-        expect(subject).to receive(:`).with("git rev-parse --abbrev-ref HEAD").and_return("master")
+        expect(subject).to receive(:`).with("git rev-parse --abbrev-ref HEAD").and_return("master\n")
         expect(subject.branch).to eq "master"
       end
     end
 
     describe "#sha" do
       it "calls Git to get the current commit's SHA" do
-        expect(subject).to receive(:`).with("git rev-parse HEAD").and_return("abc123")
+        expect(subject).to receive(:`).with("git rev-parse HEAD").and_return("abc123\n")
         expect(subject.sha).to eq "abc123"
       end
     end
 
     describe "#time" do
       it "calls Git to get the current commit's datetime" do
-        expect(subject).to receive(:`).with("git rev-parse HEAD").and_return("abc123")
-        expect(subject).to receive(:`).with("git show -s --format=%ci abc123").and_return("2017-01-31")
+        expect(subject).to receive(:`).with("git rev-parse HEAD").and_return("abc123\n")
+        expect(subject).to receive(:`).with("git show -s --format=%ci abc123").and_return("2017-01-31\n")
         expect(subject.time).to eq Time.parse("2017-01-31")
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec_profiling
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Ben Eddy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-09 00:00:00.000000000 Z
+date: 2020-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sqlite3
@@ -70,14 +70,14 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
@@ -118,6 +118,7 @@ files:
 - ".gitignore"
 - ".ruby-gemset"
 - ".ruby-version"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -162,8 +163,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Profile RSpec test suites