Mistakenly, GetFloat was used where GetInt was needed. Fixed it.

Extended test case to ReflexionGraph.
uni-bremen-agst · May 31, 2024 · 5b3c093 · 5b3c093
1 parent 7e30849
commit 5b3c093
Showing 1 changed file with 16 additions and 4 deletions.
diff --git a/Assets/SEETests/TestGraph.cs b/Assets/SEETests/TestGraph.cs
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using NUnit.Framework;
+using SEE.Tools.ReflexionAnalysis;
 
 namespace SEE.DataModel.DG
 {
@@ -775,8 +776,7 @@ public void TestDeleteTree()
         /// <summary>
         /// Tests <see cref="Graph.SubgraphBy"/>.
         /// </summary>
-        [Test]
-        public void TestSubgraph()
+        private void TestSubgraph<T>() where T : Graph, new()
         {
             const string floatAttribute = "float";
             const string intAttribute = "int";
@@ -788,7 +788,7 @@ public void TestSubgraph()
             const string stringAttributeValue = "hello, world";
             const bool toggleAttributeValue = true;
 
-            Graph g = NewEmptyGraph();
+            Graph g = new T();
             g.SetFloat(floatAttribute, floatAttributeValue);
             g.SetInt(intAttribute, intAttributeValue);
             g.SetString(stringAttribute, stringAttributeValue);
@@ -800,12 +800,24 @@ public void TestSubgraph()
 
             Graph sg = g.SubgraphBy(x => x is Node || (x is Edge e && e.Type == "set"));
             Assert.AreEqual(floatAttributeValue, sg.GetFloat(floatAttribute));
-            Assert.AreEqual(intAttributeValue, sg.GetFloat(intAttribute));
+            Assert.AreEqual(intAttributeValue, sg.GetInt(intAttribute));
             Assert.AreEqual(stringAttributeValue, sg.GetString(stringAttribute));
             Assert.IsTrue(sg.HasToggle(toggleAttribute));
 
             Assert.AreEqual(2, sg.NodeCount);
             Assert.AreEqual(1, sg.EdgeCount);
         }
+
+        [Test]
+        public void TestSubgraphGraph()
+        {
+            TestSubgraph<Graph>();
+        }
+
+        [Test]
+        public void TestSubgraphReflexionGraph()
+        {
+            TestSubgraph<ReflexionGraph>();
+        }
     }
 }