caleb-brown/unity-atoms
1
0
Fork 0
mirror of https://github.com/unity-atoms/unity-atoms.git synced 2025-01-22 08:08:51 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix Remove Tag from AtomTags and Fix Remove Tag Test (#102)

Browse Source
This commit is contained in:
Lucas Ribeiro 2020-01-23 17:09:27 -03:00 committed by Adam Ramberg
parent a3db8f13e9
commit 82e78f51d3
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Packages/Tags/Runtime/AtomTags.cs
View File

@ -141,7 +141,7 @@ namespace UnityAtoms.Tags
public void RemoveTag(string tag)
{
if (tag == null) return;
if (_sortedTags.ContainsKey(tag)) return;
if (!_sortedTags.ContainsKey(tag)) return;
_sortedTags.Remove(tag);
Tags = new ReadOnlyList<StringConstant>(_sortedTags.Values);

4
Packages/Tags/Tests/Runtime/AtomTagsTests.cs
View File

@ -94,9 +94,11 @@ namespace UnityAtoms.Tags.Tests
using (new ProfilerMarker("MySystem.RemoveTag").Auto())
{
atomicTags.RemoveTag("b");
atomicTags.RemoveTag(newConstant.Value);
}
Assert.AreEqual(random_tags_raw.Count, atomicTags.Tags.Count);
Assert.IsFalse(atomicTags.HasTag(newConstant.Value));
using (new ProfilerMarker("MySystem.AddTagAgain").Auto())
{