fix: removing label without providing user (#804)

Author: 2ndalpha

packages/javascript-api/src/lib/services/ticket/ticket.service.spec.ts

@@ -1514,10 +1514,10 @@ describe('Ticket service', function () {
       expect(() => (TicketService.removeLabel as any)(12345)).toThrow();
     });
 
-    it('throws an error when the user is missing', function () {
+    it('does not throw an error when the user is missing', function () {
       expect(() =>
         (TicketService.removeLabel as any)(12345, 'LABEL'),
-      ).toThrow();
+      ).not.toThrow();
     });
 
     it('does not throw an error when the user is a number', function () {

packages/javascript-api/src/lib/services/ticket/ticket.ts

@@ -259,6 +259,11 @@ interface CallNextRequest extends TicketCallRequest {
   lines: string;
 }
 
+interface LabelRemoveRequest {
+  value: string;
+  user?: string;
+}
+
 type TicketCreationResponse = Pick<Ticket, 'id'>;
 
 export function search(search: TicketSearchCriteria): Promise<Array<Ticket>> {
@@ -548,10 +553,10 @@ export function setLabels(
 export function removeLabel(
   ticket: IdOrObject<Ticket>,
   label: string,
-  user: IdOrObject<User>,
+  user?: IdOrObject<User>,
 ): Promise<'success'> {
   const ticketId = extractId(ticket);
-  const userId = extractId(user);
+  const userId = user ? extractId(user) : undefined;
 
   if (!ticketId || typeof ticketId !== 'string') {
     throw new Error(ERROR_NO_TICKET_ID);
@@ -561,17 +566,15 @@ export function removeLabel(
     throw new Error('No label given.');
   }
 
-  if (!userId || typeof userId !== 'string') {
-    throw new Error('No user given');
-  }
-
-  const body = {
+  const requestBody: LabelRemoveRequest = {
     value: label,
-    user: userId,
   };
+  if (userId) {
+    requestBody.user = userId;
+  }
 
   return ApiBase.request(`v1/tickets/${ticketId}/labels/remove`, {
-    body: body,
+    body: requestBody,
     method: 'POST',
   }).then((response: { result: 'success' }) => response.result);
 }