fortitude 0.0.1

data/spec/system/rebuild_notifications_system_spec.rb

@@ -0,0 +1,208 @@
+describe "Fortitude rebuilding notifications", :type => :system do
+  before :each do
+    @notifications = [ ]
+    n = @notifications
+    ActiveSupport::Notifications.subscribe("fortitude.rebuilding") do |*args|
+      n << args
+    end
+
+    @wc = widget_class
+  end
+
+  def expect_notification(expected_payload)
+    expected_payload = { :class => @wc, :originating_class => @wc }.merge(expected_payload)
+    notification = @notifications.detect do |(name, start, finish, id, payload)|
+      payload == expected_payload
+    end
+    raise "Can't find notification with payload #{expected_payload.inspect}; have: #{@notifications.inspect}" unless notification
+    @notifications.delete(notification)
+  end
+
+  def expect_no_more_notifications!(what = nil)
+    remaining = @notifications
+    remaining = remaining.select { |n| n[4][:what] == what } if what
+
+    if remaining.length > 0
+      raise "Had more notifications that we didn't expect: #{remaining.inspect}"
+    end
+  end
+
+  describe "text methods" do
+    it "should fire a notification when rebuilding because format_output has changed" do
+      @wc.format_output true
+      expect_notification(:what => :text_methods, :why => :format_output_changed)
+      expect_no_more_notifications!(:text_methods)
+
+      @wc.format_output false
+      expect_notification(:what => :text_methods, :why => :format_output_changed)
+      expect_no_more_notifications!(:text_methods)
+    end
+  end
+
+  describe "needs" do
+    it "should fire a notification when rebuilding because a need was declared" do
+      @wc.needs :foo
+      expect_notification(:what => :needs, :why => :need_declared)
+      expect_no_more_notifications!(:needs)
+
+      @wc.needs :bar => :baz
+      expect_notification(:what => :needs, :why => :need_declared)
+      expect_no_more_notifications!(:needs)
+    end
+
+    it "should fire a notification when rebuilding because extra_assigns was changed" do
+      @wc.extra_assigns :use
+      expect_notification(:what => :needs, :why => :extra_assigns_changed)
+      expect_no_more_notifications!(:needs)
+
+      @wc.extra_assigns :error
+      expect_notification(:what => :needs, :why => :extra_assigns_changed)
+      expect_no_more_notifications!(:needs)
+    end
+
+    it "should fire a notification when rebuilding because use_instance_variables_for_assigns was changed" do
+      @wc.use_instance_variables_for_assigns true
+      expect_notification(:what => :needs, :why => :use_instance_variables_for_assigns_changed)
+      expect_no_more_notifications!(:needs)
+
+      @wc.use_instance_variables_for_assigns false
+      expect_notification(:what => :needs, :why => :use_instance_variables_for_assigns_changed)
+      expect_no_more_notifications!(:needs)
+    end
+  end
+
+  describe "run_content" do
+    it "should fire a notification when rebuilding because an around_content filter was added" do
+      @wc.around_content :around1
+      expect_notification(:what => :run_content, :why => :around_content_added)
+      expect_no_more_notifications!(:run_content)
+
+      @wc.around_content :around2
+      expect_notification(:what => :run_content, :why => :around_content_added)
+      expect_no_more_notifications!(:run_content)
+    end
+
+    it "should fire a notification when rebuilding because an around_content filter was removed" do
+      @wc.around_content :around1
+      expect_notification(:what => :run_content, :why => :around_content_added)
+      expect_no_more_notifications!(:run_content)
+
+      @wc.remove_around_content :around1
+      expect_notification(:what => :run_content, :why => :around_content_removed)
+      expect_no_more_notifications!(:run_content)
+    end
+
+    it "should fire a notification when rebuilding because a localized content method was added" do
+      @wc.class_eval do
+        def localized_content_en
+          text "foo"
+        end
+      end
+      expect_notification(:what => :run_content, :why => :localized_methods_presence_changed)
+      expect_no_more_notifications!(:run_content)
+
+      @wc.send(:remove_method, :localized_content_en)
+      expect_notification(:what => :run_content, :why => :localized_methods_presence_changed)
+      expect_no_more_notifications!(:run_content)
+    end
+  end
+
+  describe "tag_methods" do
+    it "should fire a notification when rebuilding because a tag was added" do
+      @wc.tag :foo
+      expect_notification(:what => :tag_methods, :why => :tags_declared)
+      expect_no_more_notifications!(:tag_methods)
+    end
+
+    it "should fire a notification when rebuilding because format_output has changed" do
+      @wc.format_output true
+      expect_notification(:what => :tag_methods, :why => :format_output_changed)
+      expect_no_more_notifications!(:tag_methods)
+
+      @wc.format_output false
+      expect_notification(:what => :tag_methods, :why => :format_output_changed)
+      expect_no_more_notifications!(:tag_methods)
+    end
+
+    it "should fire a notification when rebuilding because enforce_element_nesting_rules has changed" do
+      @wc.enforce_element_nesting_rules true
+      expect_notification(:what => :tag_methods, :why => :enforce_element_nesting_rules_changed)
+      expect_no_more_notifications!(:tag_methods)
+
+      @wc.enforce_element_nesting_rules false
+      expect_notification(:what => :tag_methods, :why => :enforce_element_nesting_rules_changed)
+      expect_no_more_notifications!(:tag_methods)
+    end
+
+    it "should fire a notification when rebuilding because enforce_attribute_rules has changed" do
+      @wc.enforce_attribute_rules true
+      expect_notification(:what => :tag_methods, :why => :enforce_attribute_rules_changed)
+      expect_no_more_notifications!(:tag_methods)
+
+      @wc.enforce_attribute_rules false
+      expect_notification(:what => :tag_methods, :why => :enforce_attribute_rules_changed)
+      expect_no_more_notifications!(:tag_methods)
+    end
+
+    it "should fire a notification when rebuilding because enforce_id_uniqueness has changed" do
+      @wc.enforce_id_uniqueness true
+      expect_notification(:what => :tag_methods, :why => :enforce_id_uniqueness_changed)
+      expect_no_more_notifications!(:tag_methods)
+
+      @wc.enforce_id_uniqueness false
+      expect_notification(:what => :tag_methods, :why => :enforce_id_uniqueness_changed)
+      expect_no_more_notifications!(:tag_methods)
+    end
+  end
+
+  describe "with a subclass" do
+    before :each do
+      @wc = widget_class
+      @wc_subclass = widget_class(:superclass => @wc)
+    end
+
+    it "should fire with the right class for text methods" do
+      @wc_subclass.format_output true
+      expect_notification(:what => :text_methods, :why => :format_output_changed, :class => @wc_subclass, :originating_class => @wc_subclass)
+      expect_no_more_notifications!(:text_methods)
+
+      @wc.format_output true
+      expect_notification(:what => :text_methods, :why => :format_output_changed, :class => @wc, :originating_class => @wc)
+      expect_notification(:what => :text_methods, :why => :format_output_changed, :class => @wc_subclass, :originating_class => @wc)
+      expect_no_more_notifications!(:text_methods)
+    end
+
+    it "should fire with the right class for needs" do
+      @wc_subclass.extra_assigns :use
+      expect_notification(:what => :needs, :why => :extra_assigns_changed, :class => @wc_subclass, :originating_class => @wc_subclass)
+      expect_no_more_notifications!(:needs)
+
+      @wc.extra_assigns :error
+      expect_notification(:what => :needs, :why => :extra_assigns_changed, :class => @wc, :originating_class => @wc)
+      expect_notification(:what => :needs, :why => :extra_assigns_changed, :class => @wc_subclass, :originating_class => @wc)
+      expect_no_more_notifications!(:needs)
+    end
+
+    it "should fire with the right class for run_content" do
+      @wc_subclass.around_content :around1
+      expect_notification(:what => :run_content, :why => :around_content_added, :class => @wc_subclass, :originating_class => @wc_subclass)
+      expect_no_more_notifications!(:run_content)
+
+      @wc.around_content :around2
+      expect_notification(:what => :run_content, :why => :around_content_added, :class => @wc, :originating_class => @wc)
+      expect_notification(:what => :run_content, :why => :around_content_added, :class => @wc_subclass, :originating_class => @wc)
+      expect_no_more_notifications!(:run_content)
+    end
+
+    it "should fire with the right class for tag_methods" do
+      @wc_subclass.format_output true
+      expect_notification(:what => :tag_methods, :why => :format_output_changed, :class => @wc_subclass, :originating_class => @wc_subclass)
+      expect_no_more_notifications!(:tag_methods)
+
+      @wc.format_output true
+      expect_notification(:what => :tag_methods, :why => :format_output_changed, :class => @wc, :originating_class => @wc)
+      expect_notification(:what => :tag_methods, :why => :format_output_changed, :class => @wc_subclass, :originating_class => @wc)
+      expect_no_more_notifications!(:tag_methods)
+    end
+  end
+end

data/spec/system/rendering_context_system_spec.rb

@@ -0,0 +1,83 @@
+describe "Fortitude rendering context interactions", :type => :system do
+  class RcSubclass < Fortitude::RenderingContext
+    attr_reader :widget_calls
+
+    def initialize(*args)
+      super(*args)
+      @widget_calls = [ ]
+    end
+
+    def start_widget!(widget)
+      @widget_calls << [ :start, widget ]
+    end
+
+    def end_widget!(widget)
+      @widget_calls << [ :end, widget ]
+    end
+  end
+
+  def render(widget_or_class, options = { })
+    @rc = RcSubclass.new({ })
+    super(widget_or_class, options.merge(:rendering_context => @rc))
+  end
+
+  it "should call start_widget! and end_widget! on the context when starting and ending a simple widget" do
+    wc = widget_class_with_content { p "foo" }
+    instance = wc.new
+    expect(render(instance)).to eq("<p>foo</p>")
+    expect(@rc.widget_calls).to eq([ [ :start, instance ], [ :end, instance ] ])
+  end
+
+  it "should call start_widget! and end_widget! in the right order with a deeply-nested example" do
+    bottom = widget_class do
+      needs :value
+      def content
+        text "bottom-#{value}"
+      end
+    end
+
+    mid = widget_class do
+      attr_accessor :bottom1, :bottom2
+      def content
+        widget bottom1 if bottom1
+        widget bottom2 if bottom2
+      end
+    end
+
+    top = widget_class do
+      attr_accessor :mid1, :mid2
+      def content
+        widget mid1
+        widget mid2
+      end
+    end
+
+    bottom_a1 = bottom.new(:value => 12345)
+    bottom_a2 = bottom.new(:value => 23456)
+    bottom_b2 = bottom.new(:value => 34567)
+    mid_a = mid.new
+    mid_a.bottom1 = bottom_a1
+    mid_a.bottom2 = bottom_a2
+    mid_b = mid.new
+    mid_b.bottom2 = bottom_b2
+    top = top.new
+    top.mid1 = mid_a
+    top.mid2 = mid_b
+
+    expect(render(top)).to eq("bottom-12345bottom-23456bottom-34567")
+    expect(@rc.widget_calls).to eq([
+      [ :start, top ],
+      [ :start, mid_a ],
+      [ :start, bottom_a1 ],
+      [ :end, bottom_a1 ],
+      [ :start, bottom_a2 ],
+      [ :end, bottom_a2 ],
+      [ :end, mid_a ],
+      [ :start, mid_b ],
+      [ :start, bottom_b2 ],
+      [ :end, bottom_b2 ],
+      [ :end, mid_b ],
+      [ :end, top ]
+      ])
+  end
+end