volt 0.9.1 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 089a3893d8b7611b25bad4b8fcbdedd84be6579b
-  data.tar.gz: 066dc262d293f911faafb6bafddedd065546484c
+  metadata.gz: c57d60611600de9f132243981453e12dd14d7d2b
+  data.tar.gz: e14a1bbead84d93c8cd03bcac1bdaf54d315d2ff
 SHA512:
-  metadata.gz: 5c7a7190b94631d3cc27baf85d11881dbe242cedf01b180ee187d1253b92fa3ea5567e32d32302016935be383c360fbdd69380dbec88aa00ed82d4d362b65a21
-  data.tar.gz: 0d099ac0fbf72d705c9c9e9be7cf3396c6737723da156cc5dc8f26d77b2bec36739c59b5f164fe537edf48a0a1823a760ae7028aa59e0a3af28721873deeb3c9
+  metadata.gz: 4fc5686a5121b45d4186acd8c59e0b46e45ad9796752a1bc0b915234fe864cb90bc8f6ad58954e4f757516a744e07cfe3da20d22e7283b49c12aab7bf59b12f6
+  data.tar.gz: 5b1b6ac243e17387a564b42be1341d378282883766a7e3002f21faa2396bd6c2006b53333ad39936194d5c84ba6ca29fb34c9dd63043f241251d5727481691b7

data/.rubocop.yml

@@ -1,3 +1,54 @@
+AllCops:
+  Include:
+  - "**/*.gemspec"
+  - "**/*.podspec"
+  - "**/*.jbuilder"
+  - "**/*.rake"
+  - "**/*.opal"
+  - "**/Gemfile"
+  - "**/Rakefile"
+  - "**/Capfile"
+  - "**/Guardfile"
+  - "**/Podfile"
+  - "**/Thorfile"
+  - "**/Vagrantfile"
+  - "**/Berksfile"
+  - "**/Cheffile"
+  - "**/Vagabondfile"
+  Exclude:
+  - "vendor/**/*"
+  - "db/schema.rb"
+  RunRailsCops: false
+  DisplayCopNames: false
+  StyleGuideCopsOnly: false
+
+
+Style/AccessModifierIndentation:
+  Description: Check indentation of private/protected visibility modifiers.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#indent-public-private-protected
+  Enabled: true
+  EnforcedStyle: indent
+  SupportedStyles:
+  - outdent
+  - indent
+
+# Metrics/LineLength:
+#   Description: Limit lines to 80 characters.
+#   StyleGuide: https://github.com/bbatsov/ruby-style-guide#80-character-limits
+#   Enabled: false
+#   Max: 80
+#   AllowURI: true
+#   URISchemes:
+#   - http
+#   - https
+# Style/GlobalVars:
+#   Description: Do not introduce global variables.
+#   StyleGuide: https://github.com/bbatsov/ruby-style-guide#instance-vars
+#   Enabled: false
+#   AllowedVariables: []
+
+# ^ From hound
+
 Style/Documentation:
   Enabled: false
 
@@ -12,3 +63,8 @@ UnderscorePrefixedVariableName:
 
 UnusedBlockArgument:
   Enabled: false
+
+Lint/UselessAssignment:
+  Description: Checks for useless assignment to a local variable.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#underscore-unused-vars
+  Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+## 0.9.2
+### Changed
+- We released 0.9.1 with a bug for destroy (doh!).  Specs added and bug fixed.
+
 ## 0.9.1
 ### Added
 - Mailer! - volt now includes the volt-mailer gem out of the box.  (you can remove it if you don't need/want it).  See https://github.com/voltrb/volt-mailer for more info.

data/Rakefile

@@ -19,7 +19,6 @@ Opal::RSpec::RakeTask.new('opal:rspec') do |s|
   s.append_path('app')
 end
 
-
 task default: [:test]
 
 require 'rspec/core/rake_task'

data/app/volt/models/user.rb

@@ -1,6 +1,4 @@
-unless RUBY_PLATFORM == 'opal'
-  require 'bcrypt'
-end
+require 'bcrypt' unless RUBY_PLATFORM == 'opal'
 
 module Volt
   class User < Model

data/app/volt/tasks/live_query/live_query.rb

@@ -129,6 +129,6 @@ class LiveQuery
   end
 
   def inspect
-    "<#{self.class.to_s} #{@collection}: #{@query.inspect}>"
+    "<#{self.class} #{@collection}: #{@query.inspect}>"
   end
 end

data/app/volt/tasks/store_tasks.rb

@@ -50,10 +50,9 @@ class StoreTasks < Volt::Task
     # who sent the update.
     #
     # return another promise
-    return promise.then do |model|
+    promise.then do |model|
       Thread.current['in_channel'] = @channel
       save_promise = model.save!.then do |result|
-
         next nil
       end.fail do |err|
         # An error object, convert to hash
@@ -77,14 +76,14 @@ class StoreTasks < Volt::Task
     query.fetch_first do |model|
       if model
         if model.can_delete?
-          db.delete('_id' => id)
+          db.delete(collection, '_id' => id)
         else
-          raise "Permissions did not allow #{collection} #{id} to be deleted."
+          fail "Permissions did not allow #{collection} #{id} to be deleted."
         end
 
         QueryTasks.live_query_pool.updated_collection(collection, @channel)
       else
-        raise "Could not find #{id} in #{collection}"
+        fail "Could not find #{id} in #{collection}"
       end
     end
   end

data/app/volt/tasks/user_tasks.rb

@@ -34,4 +34,4 @@ class UserTasks < Volt::Task
   def salty_user_id(user_id)
     "#{Volt.config.app_secret}::#{user_id}"
   end
-end
+end

data/lib/volt.rb

@@ -7,9 +7,7 @@ require 'volt/utils/volt_user_error'
 require 'volt/utils/boolean_patch'
 
 require 'volt/config'
-unless RUBY_PLATFORM == 'opal'
-  require 'volt/data_stores/data_store'
-end
+require 'volt/data_stores/data_store' unless RUBY_PLATFORM == 'opal'
 require 'volt/volt/users'
 
 module Volt
@@ -23,9 +21,7 @@ module Volt
 
   class << self
     def root
-      if self.client?
-        raise "Volt.root can not be called from the client."
-      end
+      fail 'Volt.root can not be called from the client.' if self.client?
       @root ||= File.expand_path(Dir.pwd)
     end
 

data/lib/volt/boot.rb

@@ -21,6 +21,4 @@ module Volt
     # Boot the app
     App.new(app_path)
   end
-
-
 end

data/lib/volt/cli.rb

@@ -72,7 +72,6 @@ module Volt
           server.timeout = 0
         end
       end
-
     end
 
     desc 'runner FILEPATH', 'Runs a ruby file at FILEPATH in the volt app'

data/lib/volt/cli/asset_compile.rb

@@ -19,7 +19,6 @@ module Volt
       require 'volt/volt/core'
       require 'volt/boot'
 
-
       @root_path ||= Dir.pwd
       Volt.root  = @root_path
 
@@ -31,7 +30,6 @@ module Volt
       require 'volt/server/rack/index_files'
       require 'volt/server/component_handler'
 
-
       @app_path = File.expand_path(File.join(@root_path, 'app'))
 
       @component_paths   = ComponentPaths.new(@root_path)

data/lib/volt/cli/console.rb

@@ -9,13 +9,13 @@ class Pry
       loop do
         case val = read
         when :control_c
-          output.puts ""
+          output.puts ''
           pry.reset_eval_string
         when :no_more_input
-          output.puts "" if output.tty?
+          output.puts '' if output.tty?
           break
         else
-          output.puts "" if val.nil? && output.tty?
+          output.puts '' if val.nil? && output.tty?
           return pry.exit_value unless pry.eval(val)
         end
 
@@ -39,7 +39,6 @@ module Volt
       end
     end
 
-
     def self.start
       require 'pry'
 

data/lib/volt/cli/generate.rb

@@ -22,7 +22,6 @@ class Generate < Thor
     directory('component_specs', component_spec_folder)
   end
 
-
   desc 'gem GEM', 'Creates a component gem where you can share a component'
   method_option :bin, type: :boolean, default: false, aliases: '-b', banner: 'Generate a binary for your library.'
   method_option :test, type: :string, lazy_default: 'rspec', aliases: '-t', banner: "Generate a test directory for your library: 'rspec' is the default, but 'minitest' is also supported."
@@ -35,7 +34,7 @@ class Generate < Thor
     require 'volt/cli/new_gem'
 
     if name =~ /[-]/
-      Volt.logger.error("Gem names should use underscores for their names.  Currently volt only supports a single namespace for a component.")
+      Volt.logger.error('Gem names should use underscores for their names.  Currently volt only supports a single namespace for a component.')
       return
     end
 
@@ -68,7 +67,7 @@ class Generate < Thor
     spec_file = Dir.pwd + "/spec/app/#{component.underscore}/integration/#{name.underscore.pluralize}_spec.rb"
 
     template('controller/model_controller.rb.tt', output_file, component_module: component.camelize, model_controller_name: controller_name.camelize)
-    template('controller/model_controller_spec.rb.tt', spec_file, describe: name.underscore.pluralize )
+    template('controller/model_controller_spec.rb.tt', spec_file, describe: name.underscore.pluralize)
   end
 
   desc 'task NAME COMPONENT', 'Creates a task named NAME in the app folder of the component named COMPONENT.'
@@ -96,6 +95,6 @@ class Generate < Thor
 
   def controller_exists?(name, component = 'main')
     dir = Dir.pwd + "/app/#{component}/controllers/"
-    File.exists?(dir + name.downcase.underscore.singularize + '.rb')
+    File.exist?(dir + name.downcase.underscore.singularize + '.rb')
   end
 end

data/lib/volt/cli/new_gem.rb

@@ -56,9 +56,7 @@ class NewGem
   end
 
   def copy_options
-    if @options[:bin]
-      copy('newgem/bin/newgem.tt', "bin/#{@name}")
-    end
+    copy('newgem/bin/newgem.tt', "bin/#{@name}") if @options[:bin]
     case @options[:test]
     when 'rspec'
       copy('newgem/rspec.tt', '.rspec')

data/lib/volt/controllers/actions.rb

@@ -12,7 +12,7 @@ module Volt
   module Actions
     # StopChainException inherits from Exception directly so it will not be handled by a
     # default rescue.
-    class StopChainException < Exception ; end
+    class StopChainException < Exception; end
 
     module ClassMethods
       # Takes a list of action groups (as symbols).  An action group is typically used for before/after, but
@@ -29,7 +29,7 @@ module Volt
             # Add the block in place of the symbol
             args.unshift(block) if block
 
-            raise "No callback symbol or block provided" unless args[0]
+            fail 'No callback symbol or block provided' unless args[0]
 
             callbacks = send(callbacks_var_name)
 
@@ -75,7 +75,7 @@ module Volt
     # raise an exception under the hood which will stop the chain and evaluation
     # from where stop_chain is called.
     def stop_chain
-      raise StopChainException
+      fail StopChainException
     end
 
     def self.included(base)
@@ -83,6 +83,7 @@ module Volt
     end
 
     private
+
     # TODO: currently we filter during the call, we could maybe improve performance
     # here by storing by action and having an all category as well.
     def filter_actions_by_only_exclude(callbacks, action)
@@ -94,8 +95,7 @@ module Volt
           # If no only, include it
           true
         end
-      end.map {|v| v[0] }
+      end.map { |v| v[0] }
     end
-
   end
-end
+end

data/lib/volt/controllers/model_controller.rb

@@ -50,7 +50,7 @@ module Volt
 
         val.then do |result|
           # Only assign if nothing else has been assigned since we started the resolve
-          self.model = result if self.last_promise == val
+          self.model = result if last_promise == val
         end.fail do |err|
           Volt.logger.error("Unable to resolve promise assigned to model on #{inspect}")
         end
@@ -80,9 +80,7 @@ module Volt
       model = self.current_model
 
       # If the model is a proc, call it now
-      if model && model.is_a?(Proc)
-        model = model.call
-      end
+      model = model.call if model && model.is_a?(Proc)
 
       model
     end
@@ -106,9 +104,7 @@ module Volt
         self.attrs = args[0]
 
         # If a model attribute is passed in, we assign it directly
-        if attrs.respond_to?(:model)
-          self.model = attrs.locals[:model]
-        end
+        self.model = attrs.locals[:model] if attrs.respond_to?(:model)
       end
     end
 
@@ -191,7 +187,7 @@ module Volt
       end
     end
 
-    def require_login(message="You must login to access this area.")
+    def require_login(message = 'You must login to access this area.')
       unless Volt.current_user_id
         flash._notices << message
         go '/login'

data/lib/volt/data_stores/base.rb

@@ -1,7 +1,6 @@
 module Volt
   class DataStore
     class Base
-
     end
   end
-end
+end

data/lib/volt/data_stores/data_store.rb

@@ -10,10 +10,10 @@ module Volt
       driver_name = database_name.camelize + 'Driver'
 
       begin
-        driver = self.const_get(driver_name)
+        driver = const_get(driver_name)
         @driver = MongoDriver.new
       rescue NameError => e
-        fail "#{database_name} is not a supported database"
+        raise "#{database_name} is not a supported database"
       end
     end
   end

data/lib/volt/data_stores/mongo_driver.rb

@@ -36,11 +36,11 @@ module Volt
           end
         end
 
-        return nil
+        nil
       end
 
       def query(collection, query)
-        allowed_methods = ['find', 'skip', 'limit']
+        allowed_methods = %w(find skip limit)
 
         cursor = @db[collection]
 
@@ -48,7 +48,7 @@ module Volt
           method_name, *args = query_part
 
           unless allowed_methods.include?(method_name.to_s)
-            raise "`#{method_name}` is not part of a valid query"
+            fail "`#{method_name}` is not part of a valid query"
           end
 
           cursor = cursor.send(method_name, *args)

data/lib/volt/extra_core/hash.rb

@@ -23,4 +23,4 @@
 #       key.is_a?(String) ? key : key.to_sym
 #     end
 #   end
-# end
+# end

data/lib/volt/extra_core/inflector/inflections.rb

@@ -28,7 +28,12 @@ module Volt
       attr_reader :plurals, :singulars, :uncountables, :humans, :acronyms, :acronym_regex
 
       def initialize
-        @plurals, @singulars, @uncountables, @humans, @acronyms, @acronym_regex = [], [], [], [], {}, /(?=a)b/
+        @plurals = []
+        @singulars = []
+        @uncountables = []
+        @humans = []
+        @acronyms = {}
+        @acronym_regex = /(?=a)b/
       end
 
       # Private, for the test suite.
@@ -179,7 +184,10 @@ module Volt
       def clear(scope = :all)
         case scope
           when :all
-            @plurals, @singulars, @uncountables, @humans = [], [], [], []
+            @plurals = []
+            @singulars = []
+            @uncountables = []
+            @humans = []
           else
             instance_variable_set "@#{scope}", []
         end