pry-theme 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 915606c125b12d2db78eb2fbcc80a733abb5d97e
-  data.tar.gz: c74ffb434932c0e71a84d9892e677b88398ac169
+  metadata.gz: 188f6b205f68db1fd9447846a4cc3a57a341db8a
+  data.tar.gz: 0c5dbc3a6ba7b80985b7e2681d6ac51f78eec12c
 SHA512:
-  metadata.gz: 8b85ec4855ee1e104b5c0df4ab64d470e8678112ca6b5ceef3087befed492010bb7b6f117c20cf93b9ccfcb8e35e4eeef8b7e55d2301893cdf7499a405d67360
-  data.tar.gz: b7b7f6c96befaeb1ffc5ad4d642b45fc204bd0cf76033075b2976e2c168465ddc31dcf0030fd7f3cbfe76324729d8e71573ddef57a46d27923eaf1368b417383
+  metadata.gz: 94225f995a114b537297163a0a5830c2ee88b9ecb4cfd5925e573c3b954647993373f0a9676a204a9c745779e9fc7199f96e0a8ae74284a90cf4d2829fff5cc3
+  data.tar.gz: 9b47e162fb0583da740e50534131fdae076eb9452161b879f78ac0d72d8f1ab154f1b7c7ef26613d79fde36402f883b6e7f235405418703a1bc1aec320e0d75c

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,303 @@
+# This configuration was generated by `rubocop --auto-gen-config`
+# on 2014-06-14 18:01:49 +0300 using RuboCop version 0.23.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 3
+Lint/AmbiguousOperator:
+  Enabled: false
+
+# Offense count: 18
+Lint/AmbiguousRegexpLiteral:
+  Enabled: false
+
+# Offense count: 44
+Lint/BlockAlignment:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: AlignWith, SupportedStyles.
+Lint/EndAlignment:
+  Enabled: false
+
+# Offense count: 1
+Lint/InvalidCharacterLiteral:
+  Enabled: false
+
+# Offense count: 1
+Lint/ShadowingOuterLocalVariable:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+Lint/StringConversionInInterpolation:
+  Enabled: false
+
+# Offense count: 7
+# Cop supports --auto-correct.
+Lint/UnusedBlockArgument:
+  Enabled: false
+
+# Offense count: 4
+# Cop supports --auto-correct.
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+# Offense count: 215
+Lint/Void:
+  Enabled: false
+
+# Offense count: 1
+Style/AccessorMethodName:
+  Enabled: false
+
+# Offense count: 16
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle, SupportedLastArgumentHashStyles.
+Style/AlignHash:
+  Enabled: false
+
+# Offense count: 8
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/AlignParameters:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/AndOr:
+  Enabled: false
+
+# Offense count: 78
+# Cop supports --auto-correct.
+Style/Blocks:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+# Offense count: 3
+# Configuration parameters: CountComments.
+Style/ClassLength:
+  Max: 400
+
+# Offense count: 1
+Style/ClassVars:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredMethods.
+Style/CollectionMethods:
+  Enabled: false
+
+# Offense count: 14
+# Cop supports --auto-correct.
+Style/CommentIndentation:
+  Enabled: false
+
+# Offense count: 2
+Style/CyclomaticComplexity:
+  Max: 10
+
+# Offense count: 2
+# Cop supports --auto-correct.
+Style/DeprecatedHashMethods:
+  Enabled: false
+
+# Offense count: 15
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 37
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/DotPosition:
+  Enabled: false
+
+# Offense count: 3
+Style/DoubleNegation:
+  Enabled: false
+
+# Offense count: 1
+Style/EachWithObject:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: AllowAdjacentOneLineDefs.
+Style/EmptyLineBetweenDefs:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/EmptyLines:
+  Enabled: false
+
+# Offense count: 32
+# Cop supports --auto-correct.
+Style/EmptyLinesAroundBody:
+  Enabled: false
+
+# Offense count: 19
+# Configuration parameters: Exclude.
+Style/FileName:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/For:
+  Enabled: false
+
+# Offense count: 3
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/FormatString:
+  Enabled: false
+
+# Offense count: 5
+# Configuration parameters: MinBodyLength.
+Style/GuardClause:
+  Enabled: false
+
+# Offense count: 416
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/HashSyntax:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: MaxLineLength.
+Style/IfUnlessModifier:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+Style/IndentationWidth:
+  Enabled: false
+
+# Offense count: 36
+Style/Lambda:
+  Enabled: false
+
+# Offense count: 26
+# Cop supports --auto-correct.
+Style/LeadingCommentSpace:
+  Enabled: false
+
+# Offense count: 2
+# Cop supports --auto-correct.
+Style/LineEndConcatenation:
+  Enabled: false
+
+# Offense count: 25
+Style/LineLength:
+  Max: 80
+
+# Offense count: 20
+# Configuration parameters: CountComments.
+Style/MethodLength:
+  Max: 45
+
+# Offense count: 1
+Style/ModuleFunction:
+  Enabled: false
+
+# Offense count: 1
+Style/MultilineTernaryOperator:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/Next:
+  Enabled: false
+
+# Offense count: 11
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredDelimiters.
+Style/PercentLiteralDelimiters:
+  Enabled: false
+
+# Offense count: 2
+# Cop supports --auto-correct.
+# Configuration parameters: AllowAsExpressionSeparator.
+Style/Semicolon:
+  Enabled: false
+
+# Offense count: 17
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/SignalException:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: Methods.
+Style/SingleLineBlockParams:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: AllowIfMethodIsEmpty.
+Style/SingleLineMethods:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/SpaceAfterColon:
+  Enabled: false
+
+# Offense count: 2
+# Cop supports --auto-correct.
+Style/SpaceAroundOperators:
+  Enabled: false
+
+# Offense count: 47
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/SpaceBeforeBlockBraces:
+  Enabled: false
+
+# Offense count: 4
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
+Style/SpaceInsideBlockBraces:
+  Enabled: false
+
+# Offense count: 79
+# Cop supports --auto-correct.
+Style/SpaceInsideBrackets:
+  Enabled: false
+
+# Offense count: 6
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SupportedStyles.
+Style/SpaceInsideHashLiteralBraces:
+  Enabled: false
+
+# Offense count: 235
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/StringLiterals:
+  Enabled: false
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/TrailingBlankLines:
+  Enabled: false
+
+# Offense count: 11
+# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
+Style/TrailingComma:
+  Enabled: false
+
+# Offense count: 1
+Style/UnlessElse:
+  Enabled: false
+
+# Offense count: 3
+# Cop supports --auto-correct.
+Style/VariableInterpolation:
+  Enabled: false

data/CHANGELOG.md

@@ -1,6 +1,11 @@
 Pry Theme changelog
 ===================
 
+### v1.1.1 (June 14, 2014)
+
+* Fixed error message when `theme_options` is not set at
+  allff. [→](https://github.com/kyrylo/pry-theme/issues/40)
+
 ### v1.1.0 (June 13, 2014)
 
 * Added `theme_options` config hash. It is a general config for Pry Theme

data/Gemfile

@@ -2,7 +2,7 @@ source 'https://rubygems.org'
 gemspec
 
 group :development, :test do
-  gem 'pry', '0.10.0rc1'
+  gem 'pry', '~> 0.10'
 end
 
 group :development do

data/VERSION

@@ -1 +1 @@
-1.1.0
+1.1.1

data/lib/pry-theme/config.rb

@@ -1,18 +1,24 @@
 module PryTheme
+  # @since 1.1.0
+  # @api private
+  #
+  # Does what the +Pry.config.theme_options+ hash orders.  Currently, it
+  # supports only boolean values.
   class Config
+    # Responsible for actions that are defined in +theme_options+.
     class Executor
       # @note The method amends the default behaviour of CodeRay.
       #
-      # Sets the colour of the key token to the colour of the symbol token (akin to
-      # Pygments).
+      # Sets the colour of the key token to the colour of the symbol token (akin
+      # to Pygments).
       # @example
       #   {foo: 1, :bar => 2}
       #   #  ^       ^
       #   #  |       |
       #   # key    symbol
-      # Without this patch keys and symbols have generally different colours. It's
-      # impossible to set the colour of the key token, but with help of this method
-      # you can make it look like a symbol.
+      # Without this patch keys and symbols have generally different colours.
+      # It's impossible to set the colour of the key token, but with help of
+      # this method you can make it look like a symbol.
       # @return [void]
       # @see https://github.com/kyrylo/pry-theme/issues/30
       def paint_key_as_symbol

data/lib/pry-theme/hex.rb

@@ -67,7 +67,7 @@ module PryTheme
         raise TypeError, "can't convert #{ value.class } into PryTheme::HEX"
       end
       if value !~ PryTheme::HEX::PATTERN
-        raise ArgumentError, %|invalid value for PryTheme::HEX#new(): "#{ value }"|
+        fail ArgumentError, %|invalid value for PryTheme::HEX#new(): "#{value}"|
       end
       true
     end

data/lib/pry-theme/when_started_hook.rb

@@ -1,6 +1,8 @@
 module PryTheme
+  # This is a hook to Pry. It executes upon Pry's launch. The hook is
+  # responsible for bootstrapping Pry Theme.
   class WhenStartedHook
-    def call(target, options, _pry_)
+    def call(_, _, _)
       recreate_user_themes_from_default_ones
       load_themes
 
@@ -17,20 +19,20 @@ module PryTheme
     private
 
     def apply_config
+      Pry.config.theme_options ||= {}
       PryTheme::Config.new(Pry.config.theme_options).apply
     end
 
     # Copy a default theme to theme directory, but only if it isn't there yet.
     def recreate_user_themes_from_default_ones
       FileUtils.mkdir_p(USER_THEMES_DIR) unless File.exist?(USER_THEMES_DIR)
-      default_themes = Dir.entries(DEF_THEMES_DIR) - %w{. ..}
+      default_themes = Dir.entries(DEF_THEMES_DIR) - %w(. ..)
 
       default_themes.each do |theme|
         user_theme_path = File.join(USER_THEMES_DIR, theme)
-        unless File.exist?(user_theme_path)
-          def_theme_path = File.join(DEF_THEMES_DIR, theme)
-          FileUtils.cp(def_theme_path, USER_THEMES_DIR)
-        end
+        next if File.exist?(user_theme_path)
+        def_theme_path = File.join(DEF_THEMES_DIR, theme)
+        FileUtils.cp(def_theme_path, USER_THEMES_DIR)
       end
     end
 
@@ -46,9 +48,9 @@ module PryTheme
     end
 
     def display_warning(pry)
-      pry.output.puts 'Pry Theme Warning: Pry.config.theme is set to ' +
-        "\"#{ Pry.config.theme }\". There's no such a theme in your system. " +
-        "All installed themes live inside #{ USER_THEMES_DIR }. Falling back " +
+      pry.output.puts 'Pry Theme Warning: Pry.config.theme is set to ' \
+        "\"#{ Pry.config.theme }\". There's no such a theme in your system. " \
+        "All installed themes live inside #{ USER_THEMES_DIR }. Falling back " \
         'to the default theme for now.'
     end
   end

data/spec/color_table.rb

@@ -30,7 +30,7 @@ describe PryTheme::ColorTable do
       table = PryTheme::ColorTable.t256
       table.should =~ /Color model 256/
       table.should =~ /\e\[7;38;5;147m147\e\[0m:\e\[38;5;147mwisteria02\e\[0m/
-      table.should =~ /\n  \e\[7;38;5;24m24\e\[0m:\e\[38;5;24mcerulean_grey01\e\[0m/
+      table.should =~ /\e\[7;38;5;24m24\e\[0m:\e\[38;5;24mcerulean_grey01\e\[0m/
       table.should =~ /\e\[7;38;5;0m0\e\[0m:\e\[38;5;0mblack\e\[0m/
     end
 

data/spec/colors/color256_spec.rb

@@ -85,7 +85,11 @@ describe PryTheme::Color256 do
         end
 
         it "can be used with the default colour" do
-          color = Color256.new(:bold => true, :italic => true, :underline => true)
+          color = Color256.new(
+            :bold => true,
+            :italic => true,
+            :underline => true
+          )
           color.bold?.should == true
           color.underline?.should == true
           color.italic?.should == true
@@ -295,7 +299,11 @@ describe PryTheme::Color256 do
       end
 
       it "sets background and foreground properly" do
-        color = Color256.new(:from => :term, :foreground => 4, :background => 84)
+        color = Color256.new(
+          :from => :term,
+          :foreground => 4,
+          :background => 84
+        )
         color.foreground.should == 4
         color.foreground(true).should == 'navy_blue'
         color.background.should == 84

data/spec/commands_spec.rb

@@ -56,14 +56,15 @@ describe PryTheme::Command::PryTheme do
       pry_eval('pry-theme try sick').should == "Using \"sick\" theme\n"
       PryTheme::ThemeList.current_theme.name.should == 'sick'
 
-      pry_eval('pry-theme try wholesome').should == "Using \"wholesome\" theme\n"
+      pry_eval('pry-theme try wholesome')
+        .should == "Using \"wholesome\" theme\n"
       PryTheme::ThemeList.current_theme.name.should == 'wholesome'
     end
 
     it "displays error message if can't find the given theme" do
       cur = PryTheme::ThemeList.current_theme.name
-      pry_eval('pry-theme try g-system').
-        should == %|Cannot find "g-system" amongst themes in #{PryTheme::USER_THEMES_DIR}\n|
+      pry_eval('pry-theme try g-system')
+        .should =~ /Cannot find "g-system" amongst themes in/
       PryTheme::ThemeList.current_theme.name.should == cur
     end
   end
@@ -193,13 +194,14 @@ describe PryTheme::Command::PryTheme do
       end
     end
 
-    describe "error handling" do
-      it "outputs the error message if colour model was specified without a colour" do
+    describe "displaying of error messages" do
+      it "if colour model was specified without a colour" do
         pry_eval('pry-theme convert -m 8').should =~ /Provide a color value/
       end
 
-      it "outputs the error message if colour model is invalid" do
-        pry_eval('pry-theme convert -m 23 -t 32').should =~ /Unknown color model/
+      it "if colour model is invalid" do
+        pry_eval('pry-theme convert -m 23 -t 32')
+          .should =~ /Unknown color model/
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pry-theme
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Kyrylo Silin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-13 00:00:00.000000000 Z
+date: 2014-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -59,6 +59,8 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile