fix: Re-initiate subscription telemetry on _reset() to prevent data loss

src/api/telemetry/TelemetryCollection.js

@@ -562,17 +562,16 @@ export default class TelemetryCollection extends EventEmitter {
 
   /**
    * Reset the telemetry data of the collection, and re-request
-   * historical telemetry
+   * historical telemetry and re-initiate subscriptions
    * @private
-   *
-   * @todo handle subscriptions more granually
    */
   _reset() {
     this.boundedTelemetry = [];
     this.futureBuffer = [];
 
     this.emit('clear');
     this._requestHistoricalTelemetry();
+    this._initiateSubscriptionTelemetry();
   }
 
   /**

src/api/telemetry/TelemetryCollectionSpec.js

@@ -96,4 +96,25 @@ describe('Telemetry Collection', () => {
 
     expect(telemetryCollection._warn).not.toHaveBeenCalled();
   });
+
+  it('Re-initiates subscription telemetry when _reset is called', () => {
+    mockMetadata.values = [
+      {
+        key: 'utc',
+        name: 'Timestamp',
+        format: 'utc',
+        hints: {
+          domain: 1
+        }
+      }
+    ];
+
+    const telemetryCollection = openmct.telemetry.requestCollection(domainObject);
+    telemetryCollection.load();
+
+    spyOn(telemetryCollection, '_initiateSubscriptionTelemetry');
+    telemetryCollection._reset();
+
+    expect(telemetryCollection._initiateSubscriptionTelemetry).toHaveBeenCalled();
+  });
 });