feat: Improve telemetry for editor type and license type (#1828)

The editor type is currently embedded inside the event name. That
complicates data analysis.

This PR fixes this issue, and adds data on the active license type
(`Free`, `Pro`, `Inactive`, and others).

**Note:** These tests are, and always will be flaky. This is
unavoidable. The goal of these tests isn't to get `100%` success rate,
the goal is to improve the current success rate from `~0%` to `~70%`.

### How Has This Been Tested:

- Checked input properties at the time of `sendTelemetryEvent` call
- All properties match the expected value

### How Has This Change Been Documented:

Not applicable

Author: latekvo

packages/vscode-extension/src/extension.ts

@@ -120,7 +120,10 @@ export async function activate(context: ExtensionContext) {
   }
 
   migrateOldConfiguration();
-  getTelemetryReporter().sendTelemetryEvent(`extension:activated:${getEditorType()}`);
+
+  getTelemetryReporter().sendTelemetryEvent(`extension:activated`, {
+    editorType: getEditorType(),
+  });
 
   commands.executeCommand("setContext", "RNIDE.sidePanelIsClosed", false);
 

packages/vscode-extension/src/project/project.ts

@@ -477,6 +477,11 @@ export class Project implements Disposable, ProjectInterface, DeviceSessionsMana
         status: LicenseStatus.Inactive,
         featuresAvailability: DefaultFeaturesAvailability,
       });
+
+      this.telemetry.sendTelemetry("license:detected", {
+        licenseStatus: LicenseStatus.Inactive,
+      });
+
       return;
     }
 
@@ -492,6 +497,10 @@ export class Project implements Disposable, ProjectInterface, DeviceSessionsMana
     }
 
     this.licenseStateManager.updateState(newState);
+
+    this.telemetry.sendTelemetry("license:detected", {
+      licenseStatus: newState.status,
+    });
   };
 
   // #endregion License