Inherited required test (#1207)

* remove unused bindlingflag in RequiredSubscriptionsInfo

* Added test for inherited requires

Author: zeroZshadow

test-project/Assets/EditmodeTests/Subscriptions/InjectionCriteriaTests.cs

@@ -114,7 +114,29 @@ public void Injection_happens_if_all_constraints_are_satisfied()
             Assert.IsNotNull(compositeBehaviour.Writer);
         }
 
+        [Test]
+        public void Injection_happens_if_inherited_constraints_are_satisfied()
+        {
+            ReceiveSystem.Update();
+
+            createdGameObject = CreateAndLinkGameObjectWithComponent<InheritanceBehaviour>(EntityId);
+            var inheritanceBehaviour = createdGameObject.GetComponent<InheritanceBehaviour>();
+
+            Assert.IsTrue(inheritanceBehaviour.enabled);
+            Assert.IsNotNull(inheritanceBehaviour.OwnReader);
+        }
+
 #pragma warning disable 649
+        private class InheritanceBehaviour : BaseBehaviour
+        {
+            public PositionReader OwnReader => Reader;
+        }
+
+        private class BaseBehaviour : MonoBehaviour
+        {
+            [Require] protected PositionReader Reader;
+        }
+
         private class PositionReaderBehaviour : MonoBehaviour
         {
             [Require] public PositionReader Reader;

workers/unity/Packages/io.improbable.gdk.core/Subscriptions/RequiredSubscriptionsInfo.cs

@@ -11,8 +11,7 @@ internal class RequiredSubscriptionsInfo
 
         public RequiredSubscriptionsInfo(Type type)
         {
-            var fields = type.GetFields(BindingFlags.Instance | BindingFlags.Public | BindingFlags.GetField |
-                BindingFlags.NonPublic);
+            var fields = type.GetFields(BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic);
 
             RequiredFields = new List<FieldInfo>();