test-prof 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ec87793d9a80b7f23974e847523944bdfe148b67
-  data.tar.gz: 2baef7b881fc96b8c48393c6ddc3dc0f0d967748
+  metadata.gz: 801b84df6d3f7ee580f2cdd0bb907a5d973d0f12
+  data.tar.gz: 7a1295fa606ba8fcbdb6d22867e0b6dfb5709519
 SHA512:
-  metadata.gz: 5cf770c4cc7d47cdbab88e35edfd94eca9e87d81f1d1a2d9c12eeade108fee8a7e86b136cc446a3c11eb6cdc7499c68cc0455313c4156622bae1344f0cef9925
-  data.tar.gz: 2e06a2110a04017da1ae8779c41a203a997b4a0644a6e5f26e14ae8a41e8cff0c7efb7b2d53fc3c031c75b28e6cde9b27dd74cef3f6263385725d6063ec1d9ce
+  metadata.gz: '078e932d7605e8399820ecdd3d7b4ee1098093c73d4f0f647c94ba095fad4b64531848304f86a3e1debce528bf815d2f78085b983c20ab906d0ca388d1874522'
+  data.tar.gz: '09c1d29d373158828d5d44d0a96a2413960934a60170b81a1f3974bc9c10e59fcdf4bcb20df2f842cd2985d82510c9e46ba024eca4f080d6008656e72a86a649'

data/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## 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][])
@@ -12,3 +24,4 @@
 
 [@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
 
@@ -85,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.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
 

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.1"
+  VERSION = "0.2.0"
 end

metadata

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