1 files changed, 78 insertions, 32 deletions

diff --git a/spec/component_manager_spec.rb b/spec/component_manager_spec.rb
index d99d744..9bf9b9e 100644
--- a/spec/component_manager_spec.rb
+++ b/spec/component_manager_spec.rb
@@ -1,16 +1,19 @@
-require 'felflame'
+# frozen_string_literal: true
 
-describe 'Components' do
+require_relative '../lib/felflame'
 
-  #let :component_manager do
+describe 'Components' do
+  # let :component_manager do
   #  @component_manager ||= FelFlame::Components.new('TestComponents', :param1, param2: 'def')
-  #end
+  # end
 
   before :all do
     @component_manager ||= FelFlame::Components.new('TestComponents', :param1, param2: 'def')
   end
 
   before :each do
+    @orig_stderr = $stderr
+    $stderr = StringIO.new
     @ent0 = FelFlame::Entities.new
     @ent1 = FelFlame::Entities.new
     @ent2 = FelFlame::Entities.new
@@ -20,36 +23,81 @@ describe 'Components' do
   end
 
   after :each do
-    FelFlame::Entities.each(&:delete)
-    @component_manager.each(&:delete)
+    $stderr = @orig_stderr
+    FelFlame::Entities.reverse_each(&:delete)
+    @component_manager.reverse_each(&:delete)
+  end
+
+  it 'can get a single entity' do
+    @cmp0.entity
+    $stderr.rewind
+    $stderr.string.chomp.should eq("This component belongs to NO entities but you called the method that is intended for components belonging to a single entity.\nYou may have a bug in your logic.")
+    @ent0.add @cmp0
+    expect(@cmp0.entity).to eq(@ent0)
+    @ent1.add @cmp0
+    @cmp0.entity
+    $stderr.rewind
+    $stderr.string.chomp.should eq("This component belongs to MANY entities but you called the method that is intended for components belonging to a single entity.\nYou may have a bug in your logic.")
+  end
+
+  it 'responds to array methods' do
+    expect(@component_manager.respond_to?(:[])).to be true
+    expect(@component_manager.respond_to?(:each)).to be true
+    @component_manager.each do |component|
+      expect(component.respond_to?(:param1)).to be true
+    end
+    expect(@component_manager.respond_to?(:filter)).to be true
+    expect(@component_manager.respond_to?(:first)).to be true
+    expect(@component_manager.respond_to?(:last)).to be true
+    expect(@component_manager.respond_to?(:somethingwrong)).to be false
+  end
+
+  it 'dont respond to missing methods' do
+    expect { @component_manager.somethingwrong }.to raise_error(NoMethodError)
+  end
+
+  it 'Component module responds to array methods' do
+    expect(FelFlame::Components.respond_to?(:[])).to be true
+    expect(FelFlame::Components.respond_to?(:each)).to be true
+    FelFlame::Components.each do |component_manager|
+      expect(component_manager.respond_to?(:addition_triggers)).to be true
+    end
+    expect(FelFlame::Components.respond_to?(:filter)).to be true
+    expect(FelFlame::Components.respond_to?(:first)).to be true
+    expect(FelFlame::Components.respond_to?(:last)).to be true
+    expect(FelFlame::Components.respond_to?(:somethingwrong)).to be false
+  end
+
+  it 'Component module doesnt respond to missing methods' do
+    expect { FelFlame::Components.somethingwrong }.to raise_error(NoMethodError)
   end
 
   it 'can delete a component' do
-    component_id = @cmp1.id
+    # component_id = @cmp1.id
     @ent0.add @cmp1
-
+    length = @component_manager.length
     expect(@cmp1.delete).to be true
-    expect(@cmp1.id).to be_nil
-    expect(@component_manager[component_id]).to be_nil
+    expect(@component_manager.length).to eq(length - 1)
+    # expect(@cmp1.id).to be_nil
+    # expect(@component_manager[component_id]).to be_nil
     expect(@cmp1.entities).to eq([])
   end
 
-  it 'can iterate over all component managers' do
-    all_components = FelFlame::Components.constants
+  it 'can iterate component managers' do
+    all_components_symbols = FelFlame::Components.constants
+    all_components = all_components_symbols.map do |symbol|
+      FelFlame::Components.const_get symbol
+    end
+    expect(all_components).to eq(FF::Components.each.to_a)
     expect(all_components.length).to be > 0
     expect(FelFlame::Components.each).to be_an Enumerator
-    FelFlame::Components.each do |component_manager|
-      all_components.delete component_manager.to_s.to_sym
-    end
-    expect(all_components).to eq([])
   end
 
   it 'can change params on initialization' do
     @cmp3 = @component_manager.new(param1: 'ok', param2: 10)
-    expect(@cmp3.attrs).to eq(param1: 'ok', param2: 10, id: @cmp3.id)
+    expect(@cmp3.to_h).to eq(param1: 'ok', param2: 10)
   end
 
-
   it 'sets default params correctly' do
     expect(@cmp0.param1).to be_nil
     expect(@cmp0.param2).to eq('def')
@@ -59,28 +107,26 @@ describe 'Components' do
     expect(@cmp2.param2).to eq('def')
   end
 
-  it 'can read attrs' do
-    expect(@cmp0.attrs).to eq(param2: 'def', id: 0)
-    expect(@cmp1.attrs).to eq(param2: 'def', id: 1)
-    expect(@cmp2.attrs).to eq(param2: 'def', id: 2)
+  it 'can read attributes' do
+    expect(@cmp0.to_h).to eq(param2: 'def')
+    expect(@cmp1.to_h).to eq(param2: 'def')
+    expect(@cmp2.to_h).to eq(param2: 'def')
   end
 
   it 'can set attrs' do
     expect(@cmp0.param1 = 4).to eq(4)
     expect(@cmp1.update_attrs(param1: 3, param2: 'new')).to eq(param1: 3, param2: 'new')
-    expect(@cmp1.attrs).to eq(param1: 3, param2: 'new', id: 1)
+    expect(@cmp1.to_h).to eq(param1: 3, param2: 'new')
   end
 
-  it 'can be accessed' do
-    expect(@cmp0).to eq(@component_manager[0])
-    expect(@cmp1).to eq(@component_manager[1])
-    expect(@cmp2).to eq(@component_manager[2])
+  it 'can be used as a singleton' do
+    expect(@component_manager.first).to eq(@cmp0)
   end
 
-  it 'can get id from to_i' do
-    expect(@cmp0.id).to eq(@cmp0.to_i)
-    expect(@cmp1.id).to eq(@cmp1.to_i)
-    expect(@cmp2.id).to eq(@cmp2.to_i)
+  it 'can be accessed' do
+    expect(@component_manager[0].respond_to?(:param1)).to eq(true)
+    expect(@component_manager[1].respond_to?(:param1)).to eq(true)
+    expect(@component_manager[2].respond_to?(:param1)).to eq(true)
   end
 
   it 'cant overwrite exiting component managers' do
@@ -89,7 +135,7 @@ describe 'Components' do
   end
 
   it 'can\'t create an attribute when its name is an existing method' do
-    expect { FelFlame::Components.new('TestComponent2', :id) }.to raise_error(NameError)
+    # expect { FelFlame::Components.new('TestComponent2', :id) }.to raise_error(NameError)
     expect { FelFlame::Components.new('TestComponent2', :addition_triggers) }.to raise_error(NameError)
     expect { FelFlame::Components.new('TestComponent2', :removal_triggers) }.to raise_error(NameError)
     expect { FelFlame::Components.new('TestComponent2', :attr_triggers) }.to raise_error(NameError)