Remove custom grouping

Author: rubenvanassche

src/Enums/GroupingTypes.php

@@ -2,6 +2,7 @@
 
 namespace Facade\FlareClient\Enums;
 
+/** @deprecated  */
 class GroupingTypes
 {
     const TOP_FRAME = 'topFrame';

src/Flare.php

@@ -230,8 +230,6 @@ public function createReportFromMessage(string $message, string $logLevel): Repo
             $this->applicationPath
         );
 
-        $report->groupByException();
-
         return $this->applyMiddlewareToReport($report);
     }
 

src/Report.php

@@ -60,7 +60,7 @@ class Report
     private $openFrameIndex;
 
     /** @var string */
-    private $groupBy;
+    private $groupBy ;
 
     public static function createForThrowable(Throwable $throwable, ContextInterface $context, ?string $applicationPath = null): self
     {
@@ -221,13 +221,15 @@ public function userProvidedContext(array $userProvidedContext)
         return $this;
     }
 
+    /** @deprecated  */
     public function groupByTopFrame()
     {
         $this->groupBy = GroupingTypes::TOP_FRAME;
 
         return $this;
     }
 
+    /** @deprecated  */
     public function groupByException()
     {
         $this->groupBy = GroupingTypes::EXCEPTION;
@@ -270,7 +272,6 @@ public function toArray()
             'stage' => $this->stage,
             'message_level' => $this->messageLevel,
             'open_frame_index' => $this->openFrameIndex,
-            'group_by' => $this->groupBy ?? GroupingTypes::TOP_FRAME,
             'application_path' => $this->applicationPath,
         ];
     }

tests/ReportTest.php

@@ -60,13 +60,13 @@ public function it_can_create_a_report_with_custom_grouping_type()
     }
 
     /** @test */
-    public function it_groups_by_top_frame_as_a_default()
+    public function it_does_not_group_by_default()
     {
         $report = Report::createForMessage('this is a message', 'Log', new ConsoleContext());
 
         $reportData = $report->toArray();
 
-        $this->assertSame($reportData['group_by'], GroupingTypes::TOP_FRAME);
+        $this->assertNull($reportData['group_by']);
     }
 
     /** @test */

tests/__snapshots__/FlareTest__it_can_report_an_exception__1.yml

@@ -14,5 +14,5 @@ context:
 stage: null
 message_level: null
 open_frame_index: null
-group_by: topFrame
+group_by: null
 application_path: null

tests/__snapshots__/ReportTest__it_can_create_a_report__1.yml

@@ -14,5 +14,5 @@ context:
 stage: null
 message_level: null
 open_frame_index: null
-group_by: topFrame
+group_by: null
 application_path: null

tests/__snapshots__/ReportTest__it_can_create_a_report_for_a_string_message__1.yml

@@ -14,5 +14,5 @@ context:
 stage: null
 message_level: null
 open_frame_index: 0
-group_by: topFrame
+group_by: null
 application_path: null

tests/__snapshots__/ReportTest__it_can_create_a_report_with_glows__1.yml

@@ -21,5 +21,5 @@ context:
 stage: null
 message_level: null
 open_frame_index: null
-group_by: topFrame
+group_by: null
 application_path: null