motion-spec 0.3.0 → 0.4.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MDBmZjcyMjlmMTg2NDcyMWVjN2Q5NDJmZGE3MGZlYzAzMGY1MDhhMA==
+    ZGVhMjk5MWRlZGU0NTEwMDNkMmQ1YTQxYzA3NGU0NGI5MjMwMjk5Nw==
   data.tar.gz: !binary |-
-    M2ExODNlMzdhZWE2YmYwODY5OTBiY2ZhODVjNjg3ZGI1MDdmY2JjNg==
+    NTNiZTMwZTMyMDFjY2ZlNDcyYjNhOTJlZDM4OGI1NTdiNTgzOTYxZA==
 SHA512:
   metadata.gz: !binary |-
-    YWVjYjI1YTNhMDQxZmMwMDY3OGIzYTkyNjYzZjIzYWY5Nzk4ZjlhZDI2MjQ5
-    ZDhhOTZhNzkwMDMwNGQ0YjlkZTI2NWQ0YmI4NzI3ZjhjY2RmNWYzMjM1MGNk
-    MzU0OTA3YmNhOTM3MmE0OTI4MjJjNmVmNmQ5ZDQzODM4NmQzMWI=
+    ODdiODllYjUyODFkNzhkODM2MTY1MjQ1MDA4NmZlZWExZTA4YjU3MmYxZWRh
+    ZDJjMWQyOGYyYjNkNzk1NTNlMTJkYTY4YTY2YWExYzY0YjQwZDFiZGNiYjM1
+    YjE2OTE2ZjU3OTk2NmU5MzJhMmEzYjJkN2Y5MmFmMjdkMGIxNDQ=
   data.tar.gz: !binary |-
-    MTg0YzkzNGYwZDZlYWYwNTlmMGNlYmQyODYwNTI5ZmVmYmM2YmQ5NWVlZWI1
-    MGExNWNlNmQyNmM4ZGQ5MDgwMTNlMzNjYzZlOTViZGU1ZTQ5ZmExNWZjZGFk
-    M2ZjZmMxY2U3ZjY5OTJkN2ZhMWZiN2NjNjJmMDk0MWQ5OWEwNzI=
+    MWIzZDdjNjI5ODhlOWIwMmNlNzQyZTg3YWQ5MzNjYTM0MjEzNzMxNDhlY2Qy
+    YjQ3MjMyMjQ1NTg3OGM2YmFiOWNhZDdjMzdiOTY0MGJkYzNiMjZkZDIxZWRh
+    ODg0MmFhNzQ1ODU2M2YxYjE2ZjE3NmU3YTQ1OWNkNjMwMDdiYjQ=

data/.rubocop.yml

@@ -46,6 +46,12 @@ Metrics/ParameterLists:
   StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#too-many-params'
   Enabled: false
 
+Style/DotPosition:
+  EnforcedStyle: leading
+  SupportedStyles:
+    - leading
+    - trailing
+
 Style/EmptyLineBetweenDefs:
   AllowAdjacentOneLineDefs: true
 
@@ -57,6 +63,12 @@ Style/IndentationWidth:
 Style/MethodName:
   Enabled: false
 
+Style/MultilineOperationIndentation:
+  EnforcedStyle: indented
+  SupportedStyles:
+    - aligned
+    - indented
+
 Style/StringLiterals:
   Description: Checks if uses of quotes match the configured preference.
   StyleGuide: https://github.com/bbatsov/ruby-style-guide#consistent-string-literals

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    motion-spec (0.3.0)
+    motion-spec (0.4.0)
       motion-require (>= 0.0.6)
 
 GEM

data/lib/motion-spec/context.rb

@@ -4,6 +4,7 @@ module MotionSpec
     include ContextHelper::Matchers
     include ContextHelper::Should
     include ContextHelper::Expectation
+    include ContextHelper::MemoizedHelpers
 
     attr_reader :name, :block
 
@@ -21,8 +22,7 @@ module MotionSpec
     end
 
     def run
-      # TODO
-      # return unless name =~ RestrictContext
+      # TODO: return unless name =~ RestrictContext
 
       if Platform.android?
         @specifications.each(&:run)
@@ -69,11 +69,16 @@ module MotionSpec
       instance_eval(&Shared[name])
     end
 
-    def it(description, &block)
+    def it(description = nil, &block)
       return unless description =~ RestrictName
+
       block ||= proc { should.flunk 'not implemented' }
+
       Counter[:specifications] += 1
-      @specifications << Specification.new(self, description, block, @before, @after)
+
+      @specifications << Specification.new(
+        self, description, block, @before, @after
+      )
     end
 
     def should(*args, &block)
@@ -135,11 +140,25 @@ module MotionSpec
     private
 
     def build_ios_parent_context(context)
-      (parent_context = self).methods(false).each do |e|
-        class << context
-          self
-        end.send(:define_method, e) do |*args|
-          parent_context.send(e, *args)
+      parent_context = self
+
+      # object.methods(false) returns duplicate method names where one ends in
+      # a ':' (e.g. ['foo:', 'foo']). This was causing a low-level Ruby error:
+      # Assertion failed: (b != NULL), function rb_vm_block_method_imp, file vm.cpp, line 3386.
+      # To fix the issue we removed the 'foo:' version of the method names.
+      methods = parent_context
+        .methods(false)
+        .map { |name| name.to_s.chomp(':') }
+        .uniq
+
+      context_eigenclass = (class << context; self; end)
+      context_eigenclass.send(:define_method, :parent_context) { parent_context }
+
+      methods.each do |method_name|
+        next if context.respond_to?(method_name)
+
+        context_eigenclass.send(:define_method, method_name) do |*args|
+          parent_context.send(method_name, *args)
         end
       end
     end

data/lib/motion-spec/context_helper/memoized_helpers.rb

@@ -0,0 +1,51 @@
+# -*- encoding : utf-8 -*-
+module MotionSpec
+  module ContextHelper
+    module MemoizedHelpers
+      attr_accessor :__memoized
+
+      def let(name, &block)
+        raise '#let or #subject called without a block' unless block_given?
+
+        (class << self; self; end).class_eval do
+          define_method(name) do
+            self.__memoized ||= {}
+            __memoized.fetch(name) { __memoized[name] = block.call }
+          end
+        end
+
+        # The way that nested contexts are implemented requires us to manually
+        # reset any memoized values after each spec via an 'after'.
+        after { reset_memoized }
+      end
+
+      def let!(name, &block)
+        let(name, &block)
+        before { __send__(name) }
+      end
+
+      def subject(name = nil, &block)
+        if name
+          let(name, &block)
+          alias_method :subject, name
+        else
+          let(:subject, &block)
+        end
+      end
+
+      def subject!(name = nil, &block)
+        subject(name, &block)
+        before { subject }
+      end
+
+      def is_expected # rubocop:disable Style/PredicateName
+        expect(subject)
+      end
+
+      def reset_memoized
+        @__memoized = nil
+        parent_context.reset_memoized if respond_to?(:parent_context)
+      end
+    end
+  end
+end

data/lib/motion-spec/version.rb

@@ -1,5 +1,5 @@
 # -*- encoding : utf-8 -*-
 
 module MotionSpec
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: motion-spec
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Jonathan Bender
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-05 00:00:00.000000000 Z
+date: 2015-11-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: motion-require
@@ -60,6 +60,7 @@ files:
 - lib/motion-spec/context.rb
 - lib/motion-spec/context_helper/expectation.rb
 - lib/motion-spec/context_helper/matchers.rb
+- lib/motion-spec/context_helper/memoized_helpers.rb
 - lib/motion-spec/context_helper/should.rb
 - lib/motion-spec/core.rb
 - lib/motion-spec/error.rb