@protontech/drive-sdk 0.1.2 → 0.2.0

package/src/internal/upload/telemetry.test.ts

@@ -14,7 +14,7 @@ describe('UploadTelemetry', () => {
 
     beforeEach(() => {
         mockTelemetry = {
-            logEvent: jest.fn(),
+            recordMetric: jest.fn(),
             getLogger: jest.fn().mockReturnValue({
                 info: jest.fn(),
                 warn: jest.fn(),
@@ -34,7 +34,7 @@ describe('UploadTelemetry', () => {
         const error = new Error('Failed');
         await uploadTelemetry.uploadInitFailed(parentNodeUid, error, 1000);
 
-        expect(mockTelemetry.logEvent).toHaveBeenCalledWith({
+        expect(mockTelemetry.recordMetric).toHaveBeenCalledWith({
             eventName: 'upload',
             volumeType: 'own_volume',
             uploadedSize: 0,
@@ -48,7 +48,7 @@ describe('UploadTelemetry', () => {
         const error = new Error('Failed');
         await uploadTelemetry.uploadFailed(revisionUid, error, 500, 1000);
 
-        expect(mockTelemetry.logEvent).toHaveBeenCalledWith({
+        expect(mockTelemetry.recordMetric).toHaveBeenCalledWith({
             eventName: 'upload',
             volumeType: 'own_volume',
             uploadedSize: 500,
@@ -61,7 +61,7 @@ describe('UploadTelemetry', () => {
     it('should log successful upload (excludes error)', async () => {
         await uploadTelemetry.uploadFinished(revisionUid, 1000);
 
-        expect(mockTelemetry.logEvent).toHaveBeenCalledWith({
+        expect(mockTelemetry.recordMetric).toHaveBeenCalledWith({
             eventName: 'upload',
             volumeType: 'own_volume',
             uploadedSize: 1000,
@@ -71,7 +71,7 @@ describe('UploadTelemetry', () => {
 
     describe('detect error category', () => {
         const verifyErrorCategory = (error: string) => {
-            expect(mockTelemetry.logEvent).toHaveBeenCalledWith(
+            expect(mockTelemetry.recordMetric).toHaveBeenCalledWith(
                 expect.objectContaining({
                     error,
                 }),
@@ -81,7 +81,7 @@ describe('UploadTelemetry', () => {
         it('should ignore ValidationError', async () => {
             const error = new ValidationError('Validation error');
             await uploadTelemetry.uploadFailed(revisionUid, error, 500, 1000);
-            expect(mockTelemetry.logEvent).not.toHaveBeenCalled();
+            expect(mockTelemetry.recordMetric).not.toHaveBeenCalled();
         });
 
         it('should ignore AbortError', async () => {
@@ -89,7 +89,7 @@ describe('UploadTelemetry', () => {
             error.name = 'AbortError';
             await uploadTelemetry.uploadFailed(revisionUid, error, 500, 1000);
 
-            expect(mockTelemetry.logEvent).not.toHaveBeenCalled();
+            expect(mockTelemetry.recordMetric).not.toHaveBeenCalled();
         });
 
         it('should detect "rate_limited" error for RateLimitedError', async () => {

package/src/internal/upload/telemetry.ts

@@ -23,7 +23,7 @@ export class UploadTelemetry {
     }
 
     logBlockVerificationError(retryHelped: boolean) {
-        this.telemetry.logEvent({
+        this.telemetry.recordMetric({
             eventName: 'blockVerificationError',
             retryHelped,
         });
@@ -89,7 +89,7 @@ export class UploadTelemetry {
             this.logger.error('Failed to get metric volume type', error);
         }
 
-        this.telemetry.logEvent({
+        this.telemetry.recordMetric({
             eventName: 'upload',
             volumeType,
             ...options,

package/src/telemetry.ts

@@ -66,7 +66,7 @@ export interface MetricHandler<T extends MetricEvent> {
  * const logger = telemetry.getLogger('myLogger');
  * logger.debug('Debug message');
  *
- * telemetry.logEvent({ name: 'somethingHappened', value: 42 });
+ * telemetry.recordMetric({ name: 'somethingHappened', value: 42 });
  *
  * const logs = memoryLogHandler.getLogs();
  * // Process logs
@@ -91,7 +91,7 @@ export class Telemetry<T extends MetricEvent> {
         return new Logger(name, this.logFilter, this.logHandlers);
     }
 
-    logEvent(event: T): void {
+    recordMetric(event: T): void {
         const metric = {
             time: new Date(),
             event,

package/src/tests/telemetry.ts

@@ -4,6 +4,6 @@ import { getMockLogger } from './logger';
 export function getMockTelemetry(): ProtonDriveTelemetry {
     return {
         getLogger: getMockLogger,
-        logEvent: jest.fn(),
+        recordMetric: jest.fn(),
     };
 }

package/src/transformers.ts

@@ -18,7 +18,8 @@ type InternalPartialNode = Pick<
     | 'name'
     | 'keyAuthor'
     | 'nameAuthor'
-    | 'directMemberRole'
+    | 'directRole'
+    | 'membership'
     | 'type'
     | 'mediaType'
     | 'isShared'
@@ -84,7 +85,8 @@ export function convertInternalNode(node: InternalPartialNode): PublicMaybeNode
         parentUid: node.parentUid,
         keyAuthor: node.keyAuthor,
         nameAuthor: node.nameAuthor,
-        directMemberRole: node.directMemberRole,
+        directRole: node.directRole,
+        membership: node.membership,
         type: node.type,
         mediaType: node.mediaType,
         isShared: node.isShared,