test-prof 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 29df0c008d08c25f6924b35fd76cf02e7747c116
-  data.tar.gz: 1216e9ee73060c8d20e2caa94b2d1fd5f9c70292
+  metadata.gz: 801b84df6d3f7ee580f2cdd0bb907a5d973d0f12
+  data.tar.gz: 7a1295fa606ba8fcbdb6d22867e0b6dfb5709519
 SHA512:
-  metadata.gz: d0b7e7e899c7a81512d686e4bfac89937f8c37df476ffc94daf7e351476b131c6301c8528731e263fc36a4c1397be5876fc053cf837cb51777a2dcb5ae1687f5
-  data.tar.gz: 143c0eb3c07ecbad124299395e06bc4174bda227a5ec36c4e71eb5c8d67eb6079e7f62a138c24da98d6dbc1c887529bf8105dc3c2e46b4df1cd4bbd06a65144b
+  metadata.gz: '078e932d7605e8399820ecdd3d7b4ee1098093c73d4f0f647c94ba095fad4b64531848304f86a3e1debce528bf815d2f78085b983c20ab906d0ca388d1874522'
+  data.tar.gz: '09c1d29d373158828d5d44d0a96a2413960934a60170b81a1f3974bc9c10e59fcdf4bcb20df2f842cd2985d82510c9e46ba024eca4f080d6008656e72a86a649'

data/CHANGELOG.md

@@ -1,7 +1,27 @@
 # Change log
 
+## master
+
+## 0.2.0
+
+- Ensure output directory exists. ([@danielwestendorf][])
+
+**Change default output dir** to "tmp/test_prof".
+
+Rename `#artefact_path` to `#artifact_path` to be more US-like
+
+Ensure output dir exists in `#artifact_path` method.
+
+- FactoryDoctor: print success message when no bad examples found. ([@palkan][])
+
+## 0.1.1
+
+- AnyFixture: clean tables in reverse order to not fail when foreign keys exist. ([@marshall-lee][])
+
 ## 0.1.0
 
 - Initial version. ([@palkan][])
 
 [@palkan]: https://github.com/palkan
+[@marshall-lee]: https://github.com/marshall-lee
+[@danielwestendorf]: https://github.com/danielwestendorf

data/README.md

@@ -1,4 +1,4 @@
-[![Gem Version](https://badge.fury.io/rb/test-prof.svg)](https://rubygems.org/gems/test-prof) [![Build Status](https://travis-ci.org/palkan/test-prof.svg?branch=master)](https://travis-ci.org/palkan/test-prof) [![CircleCI](https://circleci.com/gh/palkan/test-prof.svg?style=svg)](https://circleci.com/gh/palkan/test-prof)
+[![Gem Version](https://badge.fury.io/rb/test-prof.svg)](https://rubygems.org/gems/test-prof) [![Build Status](https://travis-ci.org/palkan/test-prof.svg?branch=master)](https://travis-ci.org/palkan/test-prof) [![CircleCI](https://circleci.com/gh/palkan/test-prof.svg?style=svg)](https://circleci.com/gh/palkan/test-prof) [![Code Triagers Badge](https://www.codetriage.com/palkan/test-prof/badges/users.svg)](https://www.codetriage.com/palkan/test-prof)
 
 # Ruby Tests Profiling Toolbox
 
@@ -29,6 +29,8 @@ See [Table of Contents](#table-of-contents) for more.
 
 ## Resources
 
+- [TestProf: a good doctor for slow Ruby tests](https://evilmartians.com/chronicles/testprof-a-good-doctor-for-slow-ruby-tests)
+
 - RubyConfBy, 2017, "Run Test Run" talk [[video](https://www.youtube.com/watch?v=q52n4p0wkIs), [slides](https://speakerdeck.com/palkan/rubyconfby-minsk-2017-run-test-run)]
 
 - [Tips to improve speed of your test suite](https://medium.com/appaloosa-store-engineering/tips-to-improve-speed-of-your-test-suite-8418b485205c) by [Benoit Tigeot](https://github.com/benoittgt)
@@ -83,7 +85,7 @@ TestProf global configuration is used by most of the profilers:
 
 ```ruby
 TestProf.configure do |config|
-  # the directory to put artifacts (reports) in ("tmp" by default)
+  # the directory to put artifacts (reports) in ("tmp/test_prof" by default)
   config.output_dir = "tmp/test_prof"
 
   # use unique filenames for reports (by simply appending current timestamp)

data/guides/factory_default.md

@@ -30,7 +30,7 @@ And we want to test the `Task` model:
 
 ```ruby
 describe "PATCH #update" do
-  let(:task( { create(:task) }
+  let(:task) { create(:task) }
 
   it "works" do
     patch :update, id: task.id, task: { completed: 't' }

data/lib/test_prof/any_fixture.rb

@@ -34,7 +34,7 @@ module TestProf
 
       # Clean all affected tables (but do not reset cache)
       def clean
-        tables_cache.keys.each do |table|
+        tables_cache.keys.reverse_each do |table|
           ActiveRecord::Base.connection.execute %(
             DELETE FROM #{table}
           )

data/lib/test_prof/factory_doctor/rspec.rb

@@ -8,6 +8,8 @@ module TestProf
       include Logging
       using FloatDuration
 
+      SUCCESS_MESSAGE = 'FactoryDoctor says: "Looks good to me!"'
+
       NOTIFICATIONS = %i[
         example_started
         example_finished
@@ -42,7 +44,7 @@ module TestProf
       end
 
       def print
-        return if @example_groups.empty?
+        return log(:info, SUCCESS_MESSAGE) if @example_groups.empty?
 
         msgs = []
 

data/lib/test_prof/factory_prof/printers/flamegraph.rb

@@ -61,7 +61,7 @@ module TestProf::FactoryProf
           template = File.read(TestProf.asset_path("flamegraph.template.html"))
           template.sub! '/**REPORT-DATA**/', data.to_json
 
-          outpath = TestProf.artefact_path("factory-flame.html")
+          outpath = TestProf.artifact_path("factory-flame.html")
           File.write(outpath, template)
           outpath
         end

data/lib/test_prof/ruby_prof.rb

@@ -107,7 +107,7 @@ module TestProf
       private
 
       def build_path(name, printer)
-        TestProf.artefact_path(
+        TestProf.artifact_path(
           "ruby-prof-report-#{printer}-#{config.mode}-#{name}.html"
         )
       end

data/lib/test_prof/stack_prof.rb

@@ -98,7 +98,7 @@ module TestProf
       private
 
       def build_path(name)
-        TestProf.artefact_path(
+        TestProf.artifact_path(
           "stack-prof-report-#{config.mode}#{config.raw ? '-raw' : ''}-#{name}.dump"
         )
       end

data/lib/test_prof/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TestProf
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/test_prof.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "fileutils"
 require "test_prof/version"
 require "test_prof/logging"
 
@@ -58,8 +59,10 @@ module TestProf
       ::File.expand_path(filename, ::File.join(::File.dirname(__FILE__), "..", "assets"))
     end
 
-    # Return a path to store artefact
-    def artefact_path(filename)
+    # Return a path to store artifact
+    def artifact_path(filename)
+      FileUtils.mkdir_p(config.output_dir)
+
       with_timestamps(
         ::File.join(
           config.output_dir,
@@ -91,7 +94,7 @@ module TestProf
     def initialize
       @output = $stdout
       @color = true
-      @output_dir = "tmp"
+      @output_dir = "tmp/test_prof"
       @timestamps = false
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-15 00:00:00.000000000 Z
+date: 2017-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler