mlange-42 / ark / 13932290840

coverage: 99.831% (+0.05%) from 99.782%
13932290840

push

github

web-flow 
Fix removing relationship components (#212)

It was not possible to remove relationship components, as Ark complained "relations must be fully specified".
Further, it is checked whether entities actually have the relation component to set a target for.

15 of 17 new or added lines in 3 files covered. (88.24%)

8258 of 8272 relevant lines covered (99.83%)

20520.71 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

100.0
/ecs/checks.go
1 
package ecs
2 









3



import "fmt"










4












5



func (s *storage) checkRelationComponent(id ID) {




1,056✔





6



        if !s.registry.IsRelation[id.id] {




1,057✔





7



                panic(fmt.Sprintf("component with ID %d is not a relation component", id.id))




1✔





8



        }










9



}










10












11



func (s *storage) checkRelationTarget(target Entity) {




1,057✔





12



        if !target.IsZero() && !s.entityPool.Alive(target) {




1,058✔





13



                panic("can't use a dead entity as relation target, except for the zero entity")




1✔





14



        }










15



}
























    

  • 

    •