Tests (#13)

* Add listener tests, fix Vector2 GameEvent naming

* Rename test class

* Add game event listener test class, rename test methods

* Update local var naming

* Make API more flexible, rename raise method, update tests to match real use-cases

* Fixup link in README

* Add initial changelog

* Put changes into proper place

* Add mutable object test, update readme, update(?) quality settings, make mutable obj handler more flexible

* Remove unnecessary Debug.Log statements

* Copy over README to package docs

Author: Edvinas01

CHANGELOG.md

@@ -4,7 +4,20 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.4.0] - 2020-10-07
+## [0.5.0] - 2020-10-07
+
+### Added
+- Tests for game events and their listeners.
+- Tests for mutable objects.
+
+### Changed
+- **Renamed** event and listener trigger functions (`OnGameEvent` -> `RaiseGameEvent`).
+- **Renamed** `GameEvents.Vector2.Vector3GameEvent` to `GameEvents.Vector2.Vector2GameEvent`, fixed typo.
+- Made `MutableObjectHandler` more extensible.
+- Exposed `GameEvent` and `OnGameEvent` on listeners via properties.
+- `MutableObject` instances are now reset using `SceneManager.activeSceneChanged`.
+
+## [0.4.0] - 2020-10-06
 
 ### Added
 - `MutableObjectHandler` which would automate the process of resetting `MutableObjects`.

Documentation~/README.md

@@ -31,7 +31,7 @@ Available game events:
   width="100%" alt="Example usage of Mutable Objects"
 />
 
-Mutable objects are used for storing and editing data on `ScriptableObject` assets at runtime. This data can be referenced, observed and used as a bridge by various scripts. Mutable objects are useful in situations where `ScriptableObject` data needs to be reset when a new level loads.
+Mutable objects are used for storing and editing data on `ScriptableObject` assets at runtime. This data can be referenced, observed and used as a bridge by various scripts. Mutable objects are useful in situations where `ScriptableObject` data needs to be reset when the [active scene changes](https://docs.unity3d.com/ScriptReference/SceneManagement.SceneManager-activeSceneChanged.html).
 
 Available mutable objects:
 - `MutableBool` - encapsulates a `bool` value.
@@ -104,3 +104,5 @@ public class MutableCustom : MutableObject
     }
 }
 ```
+
+[Unity Package Manager]: https://docs.unity3d.com/Packages/[email protected]/manual/index.html

Editor/GameEvents/Vector2/Vector2GameEventEditor.cs

@@ -4,9 +4,9 @@
 
 namespace GameEvents.Vector2
 {
-    [CustomEditor(typeof(Vector3GameEvent))]
+    [CustomEditor(typeof(Vector2GameEvent))]
     public class Vector2GameEventEditor
-        : ArgumentGameEventEditor<Vector3GameEvent, UnityEngine.Vector2>
+        : ArgumentGameEventEditor<Vector2GameEvent, UnityEngine.Vector2>
     {
         protected override UnityEngine.Vector2 DrawArgumentField(UnityEngine.Vector2 value)
         {

Runtime/GameEvents/Game/GameEvent.cs

@@ -29,7 +29,7 @@ public void RaiseGameEvent()
                     Debug.Log($"Raise event: {name}, listener: {listener}");
                 }
 
-                listener.OnGameEvent();
+                listener.RaiseGameEvent();
             }
         }
 

Runtime/GameEvents/Generic/ArgumentGameEvent.cs

@@ -28,7 +28,7 @@ public void RaiseGameEvent(TArgument argument)
                     Debug.Log($"Raise event: {name}, listener: {listener}, argument: {argument}");
                 }
 
-                listener.OnGameEvent(argument);
+                listener.RaiseGameEvent(argument);
             }
         }
 

Runtime/GameEvents/Generic/ArgumentGameEventListener.cs

@@ -18,10 +18,28 @@ public abstract class ArgumentGameEventListener<TGameEvent, TUnityEvent, TArgume
         [Tooltip("Called when the listener is triggered with an argument")]
         private TUnityEvent onGameEvent = default;
 
+        /// <summary>
+        ///     Get or set the underlying GameEvent.
+        /// </summary>
+        public TGameEvent GameEvent
+        {
+            get => gameEvent;
+            set => gameEvent = value;
+        }
+
+        /// <summary>
+        ///     Get or set the underlying UnityEvent.
+        /// </summary>
+        public TUnityEvent OnGameEvent
+        {
+            get => onGameEvent;
+            set => onGameEvent = value;
+        }
+
         /// <summary>
         ///     Trigger this listener with an argument.
         /// </summary>
-        public void OnGameEvent(TArgument argument)
+        public void RaiseGameEvent(TArgument argument)
         {
             onGameEvent.Invoke(argument);
         }

Runtime/GameEvents/Generic/BaseGameEventListener.cs

@@ -15,6 +15,29 @@ public abstract class BaseGameEventListener<TGameEvent>
         [Tooltip("Called when the listener is triggered with an argument")]
         private UnityEvent onGameEvent = default;
 
+        /// <summary>
+        ///     Get or set the underlying GameEvent.
+        /// </summary>
+        public TGameEvent GameEvent
+        {
+            get => gameEvent;
+            set => gameEvent = value;
+        }
+
+        /// <summary>
+        ///     Get or set the underlying UnityEvent.
+        /// </summary>
+        public UnityEvent OnGameEvent
+        {
+            get => onGameEvent;
+            set => onGameEvent = value;
+        }
+
+        public void RaiseGameEvent()
+        {
+            onGameEvent.Invoke();
+        }
+
         private void OnEnable()
         {
             if (gameEvent == null)
@@ -36,10 +59,5 @@ private void OnDisable()
 
             gameEvent.UnregisterListener(this);
         }
-
-        public void OnGameEvent()
-        {
-            onGameEvent.Invoke();
-        }
     }
 }

Runtime/GameEvents/Generic/IArgumentGameEventListener.cs

@@ -5,6 +5,6 @@ public interface IArgumentGameEventListener<in TArgument>
         /// <summary>
         ///     Trigger this listener with an argument.
         /// </summary>
-        void OnGameEvent(TArgument argument);
+        void RaiseGameEvent(TArgument argument);
     }
 }

Runtime/GameEvents/Generic/IGameEventListener.cs

@@ -5,6 +5,6 @@ public interface IGameEventListener
         /// <summary>
         ///     Trigger this listener.
         /// </summary>
-        void OnGameEvent();
+        void RaiseGameEvent();
     }
 }

Runtime/GameEvents/Vector2/Vector3GameEvent.cs renamed to Runtime/GameEvents/Vector2/Vector2GameEvent.cs

@@ -4,7 +4,7 @@
 namespace GameEvents.Vector2
 {
     [CreateAssetMenu(fileName = "Vector2GameEvent", menuName = "Game Events/Vector2 Game Event")]
-    public class Vector3GameEvent : ArgumentGameEvent<UnityEngine.Vector2>
+    public class Vector2GameEvent : ArgumentGameEvent<UnityEngine.Vector2>
     {
     }
 }