Backtrace-unity update 2.0.0 (#8)

## Version 2.0.0
- Backtrace-Unity plugin will set `"Destroy object on new scene"` by default to false.
- Backtrace stack trace improvements,
- `BacktraceDatabase` retry method now respect correctly `BacktraceDatabase` `retryInterval` property,
- New `Backtrace Configuration` won't override existing `Backtrace Configuration` in configuration directory. 
- Backtrace-Unity plugin tests now won't override some files in Backtrace-Database unit tests,
- Backtrace-Unity plugin now allows you to setup client side deduplication rules via `Fingerprint`. By using this field you can limit reporting of an error that occurs many times over a few frames. 
- Backtrace report limit watcher feature now will validate limits before BacktraceReport creation.
- `BacktraceClient` and `BacktraceDatabase` now expose `Reload` method. You can use this method do dynamically change `BacktraceClient`/`BacktraceDatabase` configurations.

Author: konraddysput

.gitignore

@@ -1,13 +1,30 @@
-[Ll]ibrary/
-[Tt]emp/
-[Oo]bj/
-[Bb]uild/
-[Bb]uilds/
-Assets/AssetStoreTools*
+# This .gitignore file should be placed at the root of your Unity project directory
+#
+# Get latest from https://github.com/github/gitignore/blob/master/Unity.gitignore
+#
+/[Ll]ibrary/
+/[Tt]emp/
+/[Oo]bj/
+/[Bb]uild/
+/[Bb]uilds/
+/[Ll]ogs/
+/[Mm]emoryCaptures/
+
+# Asset meta data should only be ignored when the corresponding asset is also ignored
+!/[Aa]ssets/**/*.meta
+
+# Uncomment this line if you wish to ignore the asset store tools plugin
+# /[Aa]ssets/AssetStoreTools*
+
+# Autogenerated Jetbrains Rider plugin
+/[Aa]ssets/Plugins/Editor/JetBrains*
 
 # Visual Studio cache directory
 .vs/
 
+# Gradle cache directory
+.gradle/
+
 # Autogenerated VS/MD/Consulo solution and project files
 ExportedObj/
 .consulo/
@@ -22,15 +39,21 @@ ExportedObj/
 *.booproj
 *.svd
 *.pdb
+*.mdb
 *.opendb
+*.VC.db
 
 # Unity3D generated meta files
 *.pidb.meta
 *.pdb.meta
+*.mdb.meta
 
-# Unity3D Generated File On Crash Reports
+# Unity3D generated file on crash reports
 sysinfo.txt
 
 # Builds
 *.apk
 *.unitypackage
+
+# Crashlytics generated file
+crashlytics-build.properties

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Backtrace Unity Release Notes
 
+## Version 2.0.0
+- Backtrace-Unity plugin will set `"Destroy object on new scene"` by default to false.
+- Backtrace stack trace improvements,
+- `BacktraceDatabase` retry method now respect correctly `BacktraceDatabase` `retryInterval` property,
+- New `Backtrace Configuration` won't override existing `Backtrace Configuration` in configuration directory. 
+- Backtrace-Unity plugin tests now won't override some files in Backtrace-Database unit tests,
+- Backtrace-Unity plugin now allows you to setup client side deduplication rules via `Fingerprint`. By using this field you can limit reporting of an error that occurs many times over a few frames. 
+- Backtrace report limit watcher feature now will validate limits before BacktraceReport creation.
+- `BacktraceClient` and `BacktraceDatabase` now expose `Reload` method. You can use this method do dynamically change `BacktraceClient`/`BacktraceDatabase` configurations.
+
 ## Version 1.1.5 - 09.01.2019
 - Added support to DontDestroyOnLoad property. Right now users might use this property to store `BacktraceClient`/`BacktraceDatabase` instances between all game scenes.
 - Added more attributes to `BacktraceReport` object,

Editor/Menu/BacktraceClientConfigurationEditor.cs

@@ -1,5 +1,6 @@
 #if UNITY_EDITOR
 using Backtrace.Unity.Model;
+using Backtrace.Unity.Types;
 using System.IO;
 using UnityEditor;
 using UnityEngine;
@@ -13,7 +14,9 @@ public class BacktraceClientConfigurationEditor : UnityEditor.Editor
         public const string LABEL_REPORT_PER_MIN = "Reports per minute";
         public const string LABEL_HANDLE_UNHANDLED_EXCEPTION = "Handle unhandled exceptions";
         public const string LABEL_IGNORE_SSL_VALIDATION = "Ignore SSL validation";
-        public const string LABEL_DONT_DESTROY_BACKTRACE_ON_SCENE_LOAD = "Don't destroy Backtrace client on scene load";
+        public const string LABEL_DEDUPLICATION_RULES = "Deduplication rules";
+        public const string LABEL_DONT_DESTROY_BACKTRACE_ON_SCENE_LOAD = "Don't destroy on scene load";
+
 
 
 
@@ -29,10 +32,12 @@ public override void OnInspectorGUI()
             {
                 EditorGUILayout.HelpBox("Detected different pattern of url. Please make sure its a valid Backtrace url!", MessageType.Warning);
             }
+
+            settings.DestroyOnLoad = EditorGUILayout.Toggle(LABEL_DONT_DESTROY_BACKTRACE_ON_SCENE_LOAD, settings.DestroyOnLoad);
             settings.ReportPerMin = EditorGUILayout.IntField(LABEL_REPORT_PER_MIN, settings.ReportPerMin);
             settings.HandleUnhandledExceptions = EditorGUILayout.Toggle(LABEL_HANDLE_UNHANDLED_EXCEPTION, settings.HandleUnhandledExceptions);
             settings.IgnoreSslValidation = EditorGUILayout.Toggle(LABEL_IGNORE_SSL_VALIDATION, settings.IgnoreSslValidation);
-            settings.DestroyOnLoad = EditorGUILayout.Toggle(LABEL_DONT_DESTROY_BACKTRACE_ON_SCENE_LOAD, settings.DestroyOnLoad);
+            settings.DeduplicationStrategy = (DeduplicationStrategy)EditorGUILayout.EnumPopup(LABEL_DEDUPLICATION_RULES, settings.DeduplicationStrategy);
         }
     }
 

Editor/Menu/BacktraceClientEditor.cs

@@ -19,7 +19,7 @@ public override void OnInspectorGUI()
                     false);
             if (component.Configuration != null)
             {
-                BacktraceConfigurationEditor editor = (BacktraceConfigurationEditor)CreateEditor(component.Configuration);
+                var editor = (BacktraceConfigurationEditor)CreateEditor(component.Configuration);
                 editor.OnInspectorGUI();
                 if (component.Configuration.Enabled && component.gameObject.GetComponent<BacktraceDatabase>() == null)
                 {

Editor/Menu/BacktraceConfigurationEditor.cs

@@ -13,6 +13,8 @@ public class BacktraceConfigurationEditor : UnityEditor.Editor
         public const string LABEL_HANDLE_UNHANDLED_EXCEPTION = "Handle unhandled exceptions";
         public const string LABEL_ENABLE_DATABASE = "Enable Database";
         public const string LABEL_IGNORE_SSL_VALIDATION = "Ignore SSL validation";
+        public const string LABEL_DEDUPLICATION_RULES = "Deduplication rules";
+
         public const string LABEL_DESTROY_CLIENT_ON_SCENE_LOAD = "Destroy client on new scene load";
 
         public const string LABEL_PATH = "Backtrace database path";
@@ -44,11 +46,15 @@ public override void OnInspectorGUI()
             SerializedProperty sslValidation = serializedObject.FindProperty("IgnoreSslValidation");
             EditorGUILayout.PropertyField(sslValidation, new GUIContent(LABEL_IGNORE_SSL_VALIDATION));
 
+
+            SerializedProperty deduplicationStrategy = serializedObject.FindProperty("DeduplicationStrategy");
+            EditorGUILayout.PropertyField(deduplicationStrategy, new GUIContent(LABEL_DEDUPLICATION_RULES));
+
             SerializedProperty destroyOnLoad = serializedObject.FindProperty("DestroyOnLoad");
             EditorGUILayout.PropertyField(destroyOnLoad, new GUIContent(LABEL_DESTROY_CLIENT_ON_SCENE_LOAD));
 
             SerializedProperty enabled = serializedObject.FindProperty("Enabled");
-            EditorGUILayout.PropertyField(enabled, new GUIContent(LABEL_ENABLE_DATABASE));
+            EditorGUILayout.PropertyField(enabled, new GUIContent(LABEL_ENABLE_DATABASE));            
 
             if (enabled.boolValue)
             {

Editor/Menu/BacktraceMenu.cs

@@ -1,14 +1,17 @@
 #if UNITY_EDITOR
+using Backtrace.Unity.Model;
 using System.IO;
+using System.Linq;
 using UnityEditor;
 using UnityEngine;
-using Backtrace.Unity.Model;
 
 namespace Backtrace.Unity.Editor
 {
     public class BacktraceMenu : MonoBehaviour
     {
-        public const string DEFAULT_CLIENT_CONFIGURATION_NAME = "Backtrace Configuration.asset";
+        public const string DEFAULT_CONFIGURATION_NAME = "Backtrace Configuration";
+        public const string DEFAULT_EXTENSION_NAME = ".asset";
+        public const string DEFAULT_CLIENT_CONFIGURATION_NAME = DEFAULT_CONFIGURATION_NAME + DEFAULT_EXTENSION_NAME;
 
         [MenuItem("Assets/Backtrace/Configuration", false, 1)]
         public static void CreateClientConfigurationFile()
@@ -19,7 +22,7 @@ public static void CreateClientConfigurationFile()
         private static void CreateAsset<T>(string fileName) where T : ScriptableObject
         {
             T asset = ScriptableObject.CreateInstance<T>();
-            string currentProjectPath = AssetDatabase.GetAssetPath(Selection.activeObject);
+            var currentProjectPath = AssetDatabase.GetAssetPath(Selection.activeObject);
             if (string.IsNullOrEmpty(currentProjectPath))
             {
                 currentProjectPath = "Assets";
@@ -28,11 +31,32 @@ private static void CreateAsset<T>(string fileName) where T : ScriptableObject
             {
                 currentProjectPath = Path.GetDirectoryName(currentProjectPath);
             }
-            AssetDatabase.CreateAsset(asset, "Assets/" + fileName);
-            AssetDatabase.SaveAssets();
-
             var destinationPath = Path.Combine(currentProjectPath, fileName);
-            AssetDatabase.MoveAsset("Assets/" + fileName, destinationPath);
+            if (File.Exists(destinationPath))
+            {
+                var files = Directory.GetFiles(currentProjectPath);
+                var lastFileIndex = files
+                    .Where(n =>
+                        Path.GetFileNameWithoutExtension(n).StartsWith(DEFAULT_CONFIGURATION_NAME) &&
+                        Path.GetExtension(n) == DEFAULT_EXTENSION_NAME)
+                        .Select(n =>
+                        {
+                            int startIndex = n.IndexOf('(') + 1;
+                            int endIndex = n.IndexOf(')');
+                            if (startIndex != 0 && endIndex != -1 && int.TryParse(n.Substring(startIndex, endIndex - startIndex), out int result))
+                            {
+                                return result;
+                            }
+                            return 0;
+                        })
+                        .DefaultIfEmpty().Max();
+
+                lastFileIndex++;
+                destinationPath = Path.Combine(currentProjectPath, $"{DEFAULT_CONFIGURATION_NAME}({lastFileIndex}){DEFAULT_EXTENSION_NAME}");
+            }
+            Debug.Log($"Generating new Backtrace configuration file available in path: {destinationPath}");
+            AssetDatabase.CreateAsset(asset, destinationPath);
+            AssetDatabase.SaveAssets();
             Selection.activeObject = asset;
         }
     }

Editor/Tests/BacktraceBaseTest.cs

@@ -0,0 +1,41 @@
+using UnityEngine;
+using System.Collections;
+using Backtrace.Unity;
+using Backtrace.Unity.Model;
+using NUnit.Framework;
+
+public class BacktraceBaseTest : MonoBehaviour
+{
+    protected GameObject GameObject;
+    protected BacktraceClient BacktraceClient;
+    protected void BeforeSetup()
+    {
+        Debug.unityLogger.logEnabled = false;
+        GameObject = new GameObject();
+        GameObject.SetActive(false);
+        BacktraceClient = GameObject.AddComponent<BacktraceClient>();
+    }
+
+    protected void AfterSetup(bool refresh = true)
+    {
+        if (refresh)
+        {
+            BacktraceClient.Refresh();
+        }
+        GameObject.SetActive(true);
+    }
+
+    protected BacktraceConfiguration GetBasicConfiguration()
+    {
+        var configuration = ScriptableObject.CreateInstance<BacktraceConfiguration>();
+        configuration.ServerUrl = "https://submit.backtrace.io/test/token/json";
+        configuration.DestroyOnLoad = true;
+        return configuration;
+    }
+
+    [TearDown]
+    public void Cleanup()
+    {
+        DestroyImmediate(GameObject);
+    }
+}