only use scope transaction name if event doesn't already have a transaction

Author: lobsterkatie

packages/hub/src/scope.ts

@@ -465,7 +465,7 @@ export class Scope implements ScopeInterface {
       event.level = this._level;
     }
     if (this._transactionName) {
-      event.transaction = this._transactionName;
+      event.transaction = event.transaction || this._transactionName;
     }
 
     // We want to set the trace context for normal events only if there isn't already

packages/hub/test/scope.test.ts

@@ -302,14 +302,14 @@ describe('Scope', () => {
       });
     });
 
-    test('scope transaction should have priority over event transaction', async () => {
+    test("scope transaction shouldn't overwrite existing event transaction", async () => {
       expect.assertions(1);
       const scope = new Scope();
       scope.setTransactionName('/abc');
       const event: Event = {};
       event.transaction = '/cdf';
       return scope.applyToEvent(event).then(processedEvent => {
-        expect(processedEvent!.transaction).toEqual('/abc');
+        expect(processedEvent!.transaction).toEqual('/cdf');
       });
     });