os.name is now os.raw_desc (#305)

getsentry · Sep 9, 2021 · 4f1c861 · 4f1c861
1 parent 2d72a16
commit 4f1c861
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### Features
 
+- Operating System reported as raw_description and parsed by Sentry ([#305](https://github.com/getsentry/sentry-unity/pull/305))
 - Release & Environment now sync with native options ([#298](https://github.com/getsentry/sentry-unity/pull/298))
 - Bump Sentry .NET SDK 3.9.0 ([#299](https://github.com/getsentry/sentry-unity/pull/299))
   - [changelog](https://github.com/getsentry/sentry-dotnet/blob/3.9.0/CHANGELOG.md)

diff --git a/src/Sentry.Unity/UnityEventProcessor.cs b/src/Sentry.Unity/UnityEventProcessor.cs
@@ -79,8 +79,7 @@ private void PopulateApp(App app)
 
         private void PopulateOperatingSystem(OperatingSystem operatingSystem)
         {
-            // TODO: Will move to raw_description once parsing is done in Sentry
-            operatingSystem.Name = _mainThreadData.OperatingSystem;
+            operatingSystem.RawDescription = _mainThreadData.OperatingSystem;
         }
 
         private void PopulateDevice(Device device)

diff --git a/test/Sentry.Unity.Tests/UnityEventProcessorTests.cs b/test/Sentry.Unity.Tests/UnityEventProcessorTests.cs
@@ -389,7 +389,7 @@ public IEnumerator Process_OperatingSystemProtocol_Assigned()
             sut.Process(sentryEvent);
 
             // assert
-            Assert.AreEqual(_sentryMonoBehaviour.SentrySystemInfo.OperatingSystem, sentryEvent.Contexts.OperatingSystem.Name);
+            Assert.AreEqual(_sentryMonoBehaviour.SentrySystemInfo.OperatingSystem, sentryEvent.Contexts.OperatingSystem.RawDescription);
         }
 
         [UnityTest]