ref: Rename send on transport to captureEvent

Author: HazAT

packages/browser/src/backend.ts

@@ -161,7 +161,7 @@ export class BrowserBackend implements Backend {
       }
     }
 
-    return this.transport.send(event);
+    return this.transport.captureEvent(event);
   }
 
   /**

packages/browser/src/transports/base.ts

@@ -15,7 +15,7 @@ export abstract class BaseTransport implements Transport {
   /**
    * @inheritDoc
    */
-  public async send(_: SentryEvent): Promise<SentryResponse> {
-    throw new SentryError('Transport Class has to implement `send` method');
+  public async captureEvent(_: SentryEvent): Promise<SentryResponse> {
+    throw new SentryError('Transport Class has to implement `captureEvent` method');
   }
 }

packages/browser/src/transports/beacon.ts

@@ -10,7 +10,7 @@ export class BeaconTransport extends BaseTransport {
   /**
    * @inheritDoc
    */
-  public async send(event: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(event: SentryEvent): Promise<SentryResponse> {
     const data = serialize(event);
 
     const result = global.navigator.sendBeacon(this.url, data);

packages/browser/src/transports/fetch.ts

@@ -11,7 +11,7 @@ export class FetchTransport extends BaseTransport {
   /**
    * @inheritDoc
    */
-  public async send(event: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(event: SentryEvent): Promise<SentryResponse> {
     const defaultOptions: RequestInit = {
       body: serialize(event),
       method: 'POST',

packages/browser/src/transports/xhr.ts

@@ -7,7 +7,7 @@ export class XHRTransport extends BaseTransport {
   /**
    * @inheritDoc
    */
-  public async send(event: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(event: SentryEvent): Promise<SentryResponse> {
     return new Promise<SentryResponse>((resolve, reject) => {
       const request = new XMLHttpRequest();
 

packages/browser/test/backend.test.ts

@@ -4,7 +4,7 @@ import { BrowserBackend } from '../src/backend';
 import { BaseTransport } from '../src/transports';
 
 class SimpleTransport extends BaseTransport {
-  public async send(_: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(_: SentryEvent): Promise<SentryResponse> {
     return {
       status: Status.fromHttpCode(200),
     };
@@ -34,7 +34,7 @@ describe('BrowserBackend', () => {
       }
     });
 
-    it('should call send() on provided transport', async () => {
+    it('should call captureEvent() on provided transport', async () => {
       backend = new BrowserBackend({ dsn, transport: SimpleTransport });
       const status = await backend.sendEvent(testEvent);
       expect(status.status).equal(Status.Success);

packages/browser/test/transports/base.test.ts

@@ -6,13 +6,13 @@ const testDsn = 'https://[email protected]/42';
 class SimpleTransport extends BaseTransport {}
 
 describe('BaseTransport', () => {
-  it('doesnt provide send() implementation', async () => {
-    const transport = new SimpleTransport({ dsn: testDsn });
+  it('doesnt provide captureEvent() implementation', async () => {
+    const transport = new SimpleTransport({ dsn: testDSN });
 
     try {
-      await transport.send({});
+      await transport.captureEvent({});
     } catch (e) {
-      expect(e.message).equal('Transport Class has to implement `send` method');
+      expect(e.message).equal('Transport Class has to implement `captureEvent` method');
     }
   });
 

packages/browser/test/transports/beacon.test.ts

@@ -29,11 +29,11 @@ describe('BeaconTransport', () => {
     expect(transport.url).equal(transportUrl);
   });
 
-  describe('send()', async () => {
+  describe('captureEvent()', async () => {
     it('sends a request to Sentry servers', async () => {
       sendBeacon.returns(true);
 
-      return transport.send(payload).then(res => {
+      return transport.captureEvent(payload).then(res => {
         expect(res.status).equal(Status.Success);
         expect(sendBeacon.calledOnce).equal(true);
         expect(sendBeacon.calledWith(transportUrl, JSON.stringify(payload))).equal(true);
@@ -43,7 +43,7 @@ describe('BeaconTransport', () => {
     it('rejects with failed status', async () => {
       sendBeacon.returns(false);
 
-      return transport.send(payload).catch(res => {
+      return transport.captureEvent(payload).catch(res => {
         expect(res.status).equal(Status.Failed);
         expect(sendBeacon.calledOnce).equal(true);
         expect(sendBeacon.calledWith(transportUrl, JSON.stringify(payload))).equal(true);

packages/browser/test/transports/fetch.test.ts

@@ -29,13 +29,13 @@ describe('FetchTransport', () => {
     expect(transport.url).equal(transportUrl);
   });
 
-  describe('send()', async () => {
+  describe('captureEvent()', async () => {
     it('sends a request to Sentry servers', async () => {
       const response = { status: 200 };
 
       fetch.returns(Promise.resolve(response));
 
-      return transport.send(payload).then(res => {
+      return transport.captureEvent(payload).then(res => {
         expect(res.status).equal(Status.Success);
         expect(fetch.calledOnce).equal(true);
         expect(
@@ -53,7 +53,7 @@ describe('FetchTransport', () => {
 
       fetch.returns(Promise.reject(response));
 
-      return transport.send(payload).catch(res => {
+      return transport.captureEvent(payload).catch(res => {
         expect(res.status).equal(403);
         expect(fetch.calledOnce).equal(true);
         expect(

packages/browser/test/transports/xhr.test.ts

@@ -30,11 +30,11 @@ describe('XHRTransport', () => {
     expect(transport.url).equal(transportUrl);
   });
 
-  describe('send()', async () => {
+  describe('captureEvent()', async () => {
     it('sends a request to Sentry servers', async () => {
       server.respondWith('POST', transportUrl, [200, {}, '']);
 
-      return transport.send(payload).then(res => {
+      return transport.captureEvent(payload).then(res => {
         expect(res.status).equal(Status.Success);
         const request = server.requests[0];
         expect(server.requests.length).equal(1);
@@ -46,7 +46,7 @@ describe('XHRTransport', () => {
     it('rejects with non-200 status code', done => {
       server.respondWith('POST', transportUrl, [403, {}, '']);
 
-      transport.send(payload).catch(res => {
+      transport.captureEvent(payload).catch(res => {
         expect(res.status).equal(403);
 
         const request = server.requests[0];

packages/node/src/backend.ts

@@ -105,7 +105,7 @@ export class NodeBackend implements Backend {
           : new HTTPSTransport(transportOptions);
     }
 
-    return this.transport.send(event);
+    return this.transport.captureEvent(event);
   }
 
   /**

packages/node/src/transports/base.ts

@@ -77,7 +77,7 @@ export abstract class BaseTransport implements Transport {
   /**
    * @inheritDoc
    */
-  public async send(_: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(_: SentryEvent): Promise<SentryResponse> {
     throw new SentryError('Transport Class has to implement `send` method');
   }
 }

packages/node/src/transports/http.ts

@@ -13,7 +13,7 @@ export class HTTPTransport extends BaseTransport {
   /**
    * @inheritDoc
    */
-  public async send(event: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(event: SentryEvent): Promise<SentryResponse> {
     return this.sendWithModule(http, event);
   }
 }

packages/node/src/transports/https.ts

@@ -13,7 +13,7 @@ export class HTTPSTransport extends BaseTransport {
   /**
    * @inheritDoc
    */
-  public async send(event: SentryEvent): Promise<SentryResponse> {
+  public async captureEvent(event: SentryEvent): Promise<SentryResponse> {
     return this.sendWithModule(https, event);
   }
 }

packages/types/src/index.ts

@@ -222,7 +222,7 @@ export interface TransportOptions {
 
 /** JSDoc */
 export interface Transport {
-  send(event: SentryEvent): Promise<SentryResponse>;
+  captureEvent(event: SentryEvent): Promise<SentryResponse>;
 }
 
 /** JSDoc */