make getTransaction return transaction on scope regardlesss of sampling decision

Author: lobsterkatie

packages/hub/src/scope.ts

@@ -212,11 +212,7 @@ export class Scope implements ScopeInterface {
    * @inheritDoc
    */
   public getTransaction(): Transaction | undefined {
-    const span = this.getSpan() as Span & { spanRecorder: { spans: Span[] } };
-    if (span && span.spanRecorder && span.spanRecorder.spans[0]) {
-      return span.spanRecorder.spans[0] as Transaction;
-    }
-    return undefined;
+    return this.getSpan() as Transaction;
   }
 
   /**

packages/tracing/test/hub.test.ts

@@ -32,6 +32,16 @@ describe('Hub', () => {
       expect(hub.getScope()?.getTransaction()).toBe(transaction);
     });
 
+    it('should find an unsampled transaction which has been set on the scope', () => {
+      const hub = new Hub(new BrowserClient({ tracesSampleRate: 1 }));
+      const transaction = hub.startTransaction({ name: 'dogpark', sampled: false });
+      hub.configureScope(scope => {
+        scope.setSpan(transaction);
+      });
+
+      expect(hub.getScope()?.getTransaction()).toBe(transaction);
+    });
+
     it("should not find an open transaction if it's not on the scope", () => {
       const hub = new Hub(new BrowserClient({ tracesSampleRate: 1 }));
       hub.startTransaction({ name: 'dogpark' });