aws-flow 1.0.8 → 1.0.9

data/{test/aws → spec/aws/unit}/async_backtrace_spec.rb

@@ -12,6 +12,7 @@
 # express or implied. See the License for the specific language governing
 # permissions and limitations under the License.
 ##
+require 'spec_helper'
 
 def validate_stacktrace_content(method_to_call_on_async_backtrace, thing_to_look_for, should_it_be_there)
     it "makes sure that any of #{thing_to_look_for.join", "} #{should_it_be_there} be printed when we call #{method_to_call_on_async_backtrace} on AsyncBacktrace" do

data/{test/aws → spec/aws/unit}/begin_rescue_ensure_spec.rb

@@ -62,6 +62,7 @@ describe BeginRescueEnsure do
   let(:catching_BRE) do
     make_catching_BRE
   end
+
   context "BeginRescueEnsure in the :trying state" do
     subject { trying_BRE }
     its(:heirs) { should_not be_empty }
@@ -361,7 +362,7 @@ describe BeginRescueEnsure do
     end
     # TODO Make rescue give a specific error in this case
 
-    expect { scope.eventLoop }.to raise_error /You have already registered RescueTestingError/
+    expect { scope.eventLoop }.to raise_error /You have already registered \[RescueTestingError\]/
   end
 
   it "ensures that stack traces work properly" do
@@ -499,7 +500,94 @@ describe BeginRescueEnsure do
     trace.should == []
   end
 
-  describe "BRE's  closed behavior" do
+  # Tests in the following context are used to test the feature
+  context "when handling multiple exceptions in one block" do
+    it "ensures that rescue blocks can take in multiple exceptions" do
+      scope = AsyncScope.new do
+        error_handler do |t|
+          t.begin do
+            trace << :in_the_begin
+            raise RescueTestingError
+          end
+          t.rescue(RescueTestingError, ArgumentError) { trace << :in_the_rescue }
+        end
+      end
+      scope.eventLoop
+      trace.should == [:in_the_begin, :in_the_rescue]
+    end
+
+    it "ensures that blocks go to the first applicable block" do
+      scope = AsyncScope.new do
+        error_handler do |t|
+          t.begin do
+            trace << :in_the_begin
+            raise RescueTestingError
+          end
+          t.rescue(IOError) { trace << :bad_rescue }
+          t.rescue(RescueTestingError, ArgumentError) { trace << :in_the_rescue }
+          t.rescue(RescueTestingError) { trace << :bad_rescue_2 }
+        end
+      end
+      scope.eventLoop
+      trace.should == [:in_the_begin, :in_the_rescue]
+    end
+
+    it "makes sure an error is raised if none of the rescue blocks apply to it" do
+      scope = AsyncScope.new  do
+        error_handler do |t|
+          t.begin { raise RescueTestingError }
+          t.rescue(IOError, ArgumentError) { p "nothing doing here" }
+        end
+      end
+      expect { scope.eventLoop }.to raise_error RescueTestingError
+    end
+
+    it "ensures that the ensure block is called with a rescue path" do
+      scope = AsyncScope.new do
+        error_handler do |t|
+          t.begin do
+            trace << :in_the_begin
+            raise RescueTestingError
+          end
+          t.rescue(IOError, RescueTestingError) { trace << :in_the_rescue }
+          t.ensure { trace << :in_the_ensure }
+          end
+      end
+      scope.eventLoop
+      trace.should == [:in_the_begin, :in_the_rescue, :in_the_ensure]
+    end
+
+    it "ensures that the ensure block is called and the error is passed up
+    without an applicable rescue case" do
+      scope = AsyncScope.new do
+        error_handler do |t|
+          t.begin do
+            trace << :in_the_begin
+            raise RescueTestingError
+          end
+          t.rescue(IOError, ArgumentError) { trace << :in_the_rescue }
+          t.ensure { trace << :in_the_ensure }
+          expect { scope.eventLoop }.to raise_error RescueTestingError
+          trace.should == [:in_the_begin, :in_the_ensure]
+          end
+      end
+    end
+
+    it "ensures that the same rescue exception twice causes an exception" do
+      scope = AsyncScope.new do
+        error_handler do |t|
+          t.begin {}
+          t.rescue(RescueTestingError, ArgumentError) {}
+          t.rescue(RescueTestingError, ArgumentError) {}
+        end
+      end
+      # TODO Make rescue give a specific error in this case
+
+      expect { scope.eventLoop }.to raise_error /You have already registered \[RescueTestingError, ArgumentError\]/
+    end
+  end
+
+  describe "BRE's closed behavior" do
     before (:each) do
       @scope = AsyncScope.new do
         @error_handler = error_handler do |t|

data/{test/aws → spec/aws/unit}/decider_spec.rb

@@ -22,6 +22,30 @@ class FakeConfig
 
   end
 end
+class FakeServiceClient
+  attr_accessor :trace
+  def respond_decision_task_completed(task_completed_request)
+    @trace ||= []
+    @trace << task_completed_request
+  end
+  def start_workflow_execution(options)
+    @trace ||= []
+    @trace << options
+    {"runId" => "blah"}
+  end
+  def register_activity_type(options)
+  end
+  def register_workflow_type(options)
+  end
+  def respond_activity_task_completed(task_token, result)
+  end
+  def start_workflow_execution(options)
+    {"runId" => "blah"}
+  end
+  def config
+    FakeConfig.new
+  end
+end
 
 $RUBYFLOW_DECIDER_DOMAIN = "rubyflow_decider_domain_06-12-2012"
 $RUBYFLOW_DECIDER_TASK_LIST = 'test_task_list'
@@ -178,21 +202,6 @@ end
 
 describe WorkflowClient do
 
-  class FakeServiceClient
-    attr_accessor :trace
-    def respond_decision_task_completed(task_completed_request)
-      @trace ||= []
-      @trace << task_completed_request
-    end
-    def start_workflow_execution(options)
-      @trace ||= []
-      @trace << options
-      {"runId" => "blah"}
-    end
-    def register_workflow_type(options)
-    end
-
-  end
   class TestWorkflow
     extend Decider
 
@@ -388,20 +397,6 @@ end
 describe WorkflowFactory do
   it "ensures that you can create a workflow_client without access to the Workflow definition" do
     workflow_type_object = double("workflow_type", :name => "NonExistantWorkflow.some_entry_method", :start_execution => "" )
-    class FakeServiceClient
-      attr_accessor :trace
-      def respond_decision_task_completed(task_completed_request)
-        @trace ||= []
-        @trace << task_completed_request
-      end
-      def start_workflow_execution(options)
-        @trace ||= []
-        @trace << options
-        {"runId" => "blah"}
-      end
-      def register_workflow_type(options)
-      end
-    end
     domain = FakeDomain.new(workflow_type_object)
     swf_client = FakeServiceClient.new
     my_workflow_factory = workflow_factory(swf_client, domain) do |options|
@@ -431,30 +426,6 @@ describe "FakeHistory" do
       end
     end
 
-    class FakeServiceClient
-      attr_accessor :trace
-      def respond_decision_task_completed(task_completed_request)
-        @trace ||= []
-        @trace << task_completed_request
-      end
-      def start_workflow_execution(options)
-        @trace ||= []
-        @trace << options
-        {"runId" => "blah"}
-      end
-      def register_activity_type(options)
-      end
-      def register_workflow_type(options)
-      end
-      def respond_activity_task_completed(task_token, result)
-      end
-      def start_workflow_execution(options)
-        {"runId" => "blah"}
-      end
-      def config
-        FakeConfig.new
-      end
-    end
 
     class Hash
       def to_h; self; end
@@ -1820,4 +1791,21 @@ describe "testing changing default values in RetryOptions and RetryPolicy" do
     result = FlowConstants.exponential_retry_function.call(first, time_of_failure, attempts, options)
     result.should == 5
   end
+
+  it "ensures that the next_retry_delay_seconds honors -1 returned by the retry function" do
+    my_retry_func = lambda do |first, time_of_failure, attempts|
+      -1
+    end
+    options = {
+      :should_jitter => true
+    }
+    retry_policy = RetryPolicy.new(my_retry_func, RetryOptions.new(options))
+    result = retry_policy.next_retry_delay_seconds(Time.now, 0, {Exception=>10}, Exception, 1)
+    result.should == -1
+  end
+
+  it "ensures that the jitter function checks arguments passed to it" do
+    expect { FlowConstants.jitter_function.call(1, -1) }.to raise_error(
+      ArgumentError, "max_value should be greater than 0")
+  end
 end

metadata

@@ -1,46 +1,47 @@
 --- !ruby/object:Gem::Specification
 name: aws-flow
 version: !ruby/object:Gem::Version
-  version: 1.0.8
-  prerelease: 
+  version: 1.0.9
 platform: ruby
 authors:
 - Michael Steger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-03 00:00:00.000000000 Z
+date: 2014-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1'
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 1.36.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1'
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 1.36.2
 - !ruby/object:Gem::Dependency
   name: aws-flow-core
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 1.0.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: 1.0.1
 description: Library to provide the AWS Flow Framework for Ruby
@@ -90,44 +91,43 @@ files:
 - lib/aws/flow/implementation.rb
 - lib/aws/flow/simple_dfa.rb
 - lib/aws/flow/tasks.rb
-- test/aws/async_backtrace_spec.rb
-- test/aws/async_scope_spec.rb
-- test/aws/begin_rescue_ensure_spec.rb
-- test/aws/decider_spec.rb
-- test/aws/external_task_spec.rb
-- test/aws/factories.rb
-- test/aws/fiber_condition_variable_spec.rb
-- test/aws/fiber_spec.rb
-- test/aws/flow_spec.rb
-- test/aws/future_spec.rb
-- test/aws/integration_spec.rb
-- test/aws/preinclude_tests.rb
-- test/aws/rubyflow.rb
-- test/aws/simple_dfa_spec.rb
-- test/aws/spec_helper.rb
+- spec/aws/integration/integration_spec.rb
+- spec/aws/unit/async_backtrace_spec.rb
+- spec/aws/unit/async_scope_spec.rb
+- spec/aws/unit/begin_rescue_ensure_spec.rb
+- spec/aws/unit/decider_spec.rb
+- spec/aws/unit/external_task_spec.rb
+- spec/aws/unit/factories.rb
+- spec/aws/unit/fiber_condition_variable_spec.rb
+- spec/aws/unit/fiber_spec.rb
+- spec/aws/unit/flow_spec.rb
+- spec/aws/unit/future_spec.rb
+- spec/aws/unit/preinclude_tests.rb
+- spec/aws/unit/rubyflow.rb
+- spec/aws/unit/simple_dfa_spec.rb
+- spec/spec_helper.rb
 homepage: 
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 - lib/aws/
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: AWS Flow Decider package decider
 test_files: []