check the changelog bro

5 files changed, 62 insertions, 29 deletions

diff --git a/spec/component_manager_spec.rb b/spec/component_manager_spec.rb
index 203d185..d05a8a7 100644
--- a/spec/component_manager_spec.rb
+++ b/spec/component_manager_spec.rb
@@ -1,4 +1,4 @@
-require 'felflame'
+require_relative '../lib/felflame.rb'
 
 describe 'Components' do
 
@@ -20,8 +20,21 @@ describe 'Components' do
   end
 
   after :each do
-    FelFlame::Entities.each(&:delete)
-    @component_manager.each(&:delete)
+    FelFlame::Entities.reverse_each(&:delete)
+    @component_manager.reverse_each(&:delete)
+  end
+
+  it 'responds to array methods' do
+    expect(@component_manager.respond_to?(:[])).to be true
+    expect(@component_manager.respond_to?(:each)).to be true
+    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 'can delete a component' do
@@ -47,7 +60,7 @@ describe 'Components' do
 
   it 'can change params on initialization' do
     @cmp3 = @component_manager.new(param1: 'ok', param2: 10)
-    expect(@cmp3.attrs).to eq(param1: 'ok', param2: 10)
+    expect(@cmp3.to_h).to eq(param1: 'ok', param2: 10)
   end
 
 
@@ -60,16 +73,16 @@ describe 'Components' do
     expect(@cmp2.param2).to eq('def')
   end
 
-  it 'can read attrs' do
-    expect(@cmp0.attrs).to eq(param2: 'def')
-    expect(@cmp1.attrs).to eq(param2: 'def')
-    expect(@cmp2.attrs).to eq(param2: 'def')
+  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')
+    expect(@cmp1.to_h).to eq(param1: 3, param2: 'new')
   end
 
   it 'can be used as a singleton' do
diff --git a/spec/entity_manager_spec.rb b/spec/entity_manager_spec.rb
index 7bf941d..6d31dd2 100644
--- a/spec/entity_manager_spec.rb
+++ b/spec/entity_manager_spec.rb
@@ -1,4 +1,4 @@
-require 'felflame'
+require_relative '../lib/felflame.rb'
 
 #class EntitiesTest < Minitest::Test
 
@@ -24,10 +24,24 @@ describe 'Entities' do
   end
 
   after :each do
-    FelFlame::Entities.each(&:delete)
-    @component_manager.each(&:delete)
+    FelFlame::Entities.reverse_each(&:delete)
+    @component_manager.reverse_each(&:delete)
   end
 
+  it 'responds to array methods' do
+    expect(FelFlame::Entities.respond_to?(:[])).to be true
+    expect(FelFlame::Entities.respond_to?(:each)).to be true
+    expect(FelFlame::Entities.respond_to?(:filter)).to be true
+    expect(FelFlame::Entities.respond_to?(:first)).to be true
+    expect(FelFlame::Entities.respond_to?(:last)).to be true
+    expect(FelFlame::Entities.respond_to?(:somethingwrong)).to be false
+  end
+
+  it 'dont respond to missing methods' do
+    expect { FelFlame::Entities.somethingwrong }.to raise_error(NoMethodError)
+  end
+
+
   it 'won\'t add duplicate entities' do
     @ent0.add @cmp0, @cmp0, @cmp1, @cmp1
     expect(@ent0.components[@component_manager].count).to eq(2)
@@ -55,11 +69,11 @@ describe 'Entities' do
     expect(@ent1.components[@component_manager].include?(@cmp2)).to be true
   end
 
-  it 'can get id from to_i' do
-    expect(@ent0.id).to eq(@ent0.to_i)
-    expect(@ent1.id).to eq(@ent1.to_i)
-    expect(@ent2.id).to eq(@ent2.to_i)
-  end
+  #it 'can get id from to_i' do
+  #  expect(@ent0.id).to eq(@ent0.to_i)
+  #  expect(@ent1.id).to eq(@ent1.to_i)
+  #  expect(@ent2.id).to eq(@ent2.to_i)
+  #end
 
   it 'can have components removed' do
     @ent0.add @cmp0
@@ -82,16 +96,12 @@ describe 'Entities' do
     expect(@cmp2.entities).to eq([@ent0,@ent2])
     @cmp1.delete
     expect(@ent0.components).to eq({@component_manager => [@cmp0,@cmp2]})
-    @component_manager.each(&:delete)
-    $stderr.puts ('HERE HERE ' * 5)
-    $stderr.puts @component_manager
-    @component_manager.each do |component|
-      $stderr.puts component
-    end
+    @component_manager.reverse_each(&:delete)
+    expect(@component_manager.length).to eq(0)
     expect(@component_manager.empty?).to be true
     expect(@ent0.components).to eq({@component_manager => []})
     expect(@ent2.components).to eq({@component_manager => []})
-    FelFlame::Entities.each(&:delete)
+    FelFlame::Entities.reverse_each(&:delete)
     expect(FelFlame::Entities.empty?).to be true
   end
 end
diff --git a/spec/scene_manager_spec.rb b/spec/scene_manager_spec.rb
index f2aee01..65666c7 100644
--- a/spec/scene_manager_spec.rb
+++ b/spec/scene_manager_spec.rb
@@ -1,4 +1,4 @@
-require 'felflame'
+require_relative '../lib/felflame.rb'
 
 #class EntitiesTest < Minitest::Test
 
diff --git a/spec/stage_manager_spec.rb b/spec/stage_manager_spec.rb
index ed1125c..0aa8690 100644
--- a/spec/stage_manager_spec.rb
+++ b/spec/stage_manager_spec.rb
@@ -1,4 +1,4 @@
-require 'felflame'
+require_relative '../lib/felflame.rb'
 
 #class EntitiesTest < Minitest::Test
 
diff --git a/spec/system_manager_spec.rb b/spec/system_manager_spec.rb
index f9df439..3d2e3dc 100644
--- a/spec/system_manager_spec.rb
+++ b/spec/system_manager_spec.rb
@@ -1,4 +1,4 @@
-require 'felflame'
+require_relative '../lib/felflame.rb'
 
 describe 'Components' do
 
@@ -73,6 +73,16 @@ describe 'Components' do
     expect(second.health).to eq(init2 - (multiple * 10))
   end
 
+  it 'can clear triggers from components and systems' do
+    @cmp0 = @component_manager.new
+    @system.trigger_when_added @cmp0
+    expect(@cmp0.addition_triggers.length).to eq(1)
+    expect(@system.addition_triggers.length).to eq(1)
+    expect(@cmp0.delete).to be true
+    expect(@cmp0.addition_triggers.length).to eq(0)
+    expect(@system.addition_triggers.length).to eq(0)
+  end
+
   it 'can trigger when a single Component is added' do
     @cmp0 = @component_manager.new
     @cmp1 = @component_manager.new health: 20
@@ -326,7 +336,7 @@ describe 'Components' do
     expect(@cmp0.health).to eq(10)
     expect(@cmp1.health).to eq(20)
   end
-  
+
   it 'can clear addition_trigger, with component' do
     @cmp0 = @component_manager.new health: 10
     @cmp1 = @component_manager.new health: 20
@@ -369,7 +379,7 @@ describe 'Components' do
     expect(@cmp0.health).to eq(10)
     expect(@cmp1.health).to eq(20)
   end
-  
+
   it 'can clear removal_trigger, with component' do
     @cmp0 = @component_manager.new health: 10
     @cmp1 = @component_manager.new health: 20