changed(tracing): always clear timeout when new activity starts within idle timeout

Author: datbth

packages/tracing/src/idletransaction.ts

@@ -221,7 +221,10 @@ export class IdleTransaction extends Transaction {
       // Remember timestampWithMs is in seconds, timeout is in ms
       const end = timestampWithMs() + timeout / 1000;
 
-      setTimeout(() => {
+      if (this._initTimeout) {
+        clearTimeout(this._initTimeout);
+      }
+      this._initTimeout = setTimeout(() => {
         if (!this._finished) {
           this.setTag(FINISH_REASON_TAG, IDLE_TRANSACTION_FINISH_REASONS[1]);
           this.finish(end);

packages/tracing/test/idletransaction.test.ts

@@ -188,6 +188,8 @@ describe('IdleTransaction', () => {
     it('does not finish if a activity is started', () => {
       const transaction = new IdleTransaction({ name: 'foo', startTimestamp: 1234 }, hub, DEFAULT_IDLE_TIMEOUT);
       transaction.initSpanRecorder(10);
+      const span = transaction.startChild({});
+      span.finish();
       transaction.startChild({});
 
       jest.advanceTimersByTime(DEFAULT_IDLE_TIMEOUT);