From 0581ec11d79806c117a27fa51033fbe5874d4173 Mon Sep 17 00:00:00 2001
From: Deepak Mardi <deepakmardi22@gmail.com>
Date: Fri, 5 Sep 2025 12:21:37 +0530
Subject: [PATCH 1/2] add shared label tests

---
 src/TodoistApi.labels.test.ts | 91 +++++++++++++++++++++++++++++++++++
 1 file changed, 91 insertions(+)

diff --git a/src/TodoistApi.labels.test.ts b/src/TodoistApi.labels.test.ts
index 61d0617..009125f 100644
--- a/src/TodoistApi.labels.test.ts
+++ b/src/TodoistApi.labels.test.ts
@@ -167,4 +167,95 @@ describe('TodoistApi label endpoints', () => {
             expect(result).toEqual(true)
         })
     })
+
+    describe('getSharedLabels', () => {
+        test('calls getSharedLabels with expected parameters', async () => {
+            const mockResponse = { results: ['shared1', 'shared2'], nextCursor: 'abc' }
+            const requestMock = setupRestClientMock(mockResponse)
+            const api = getTarget()
+
+            await api.getSharedLabels({
+                omitPersonal: true,
+                limit: 10,
+                cursor: 'abc',
+            })
+
+            expect(requestMock).toBeCalledTimes(1)
+            expect(requestMock).toBeCalledWith(
+                'GET',
+                getSyncBaseUri(),
+                'labels/shared',
+                DEFAULT_AUTH_TOKEN,
+                { omitPersonal: true, limit: 10, cursor: 'abc' },
+            )
+        })
+
+        test('returns result from rest client', async () => {
+            const mockResponse = { results: ['shared1', 'shared2'], nextCursor: 'abc' }
+            setupRestClientMock(mockResponse)
+            const api = getTarget()
+
+            const result = await api.getSharedLabels()
+
+            expect(result).toEqual(mockResponse)
+        })
+    })
+
+    describe('renameSharedLabel', () => {
+        test('calls renameSharedLabel with expected parameters', async () => {
+            const args = { name: 'old', newName: 'new' }
+            const requestMock = setupRestClientMock(null)
+            const api = getTarget()
+
+            await api.renameSharedLabel(args)
+
+            expect(requestMock).toBeCalledTimes(1)
+            expect(requestMock).toBeCalledWith(
+                'POST',
+                getSyncBaseUri(),
+                'labels/shared/rename',
+                DEFAULT_AUTH_TOKEN,
+                args,
+            )
+        })
+
+        test('returns success result from rest client', async () => {
+            const args = { name: 'old', newName: 'new' }
+            setupRestClientMock(null)
+            const api = getTarget()
+
+            const result = await api.renameSharedLabel(args)
+
+            expect(result).toBe(true)
+        })
+    })
+
+    describe('removeSharedLabel', () => {
+        test('calls removeSharedLabel with expected parameters', async () => {
+            const args = { name: 'toremove' }
+            const requestMock = setupRestClientMock(null)
+            const api = getTarget()
+
+            await api.removeSharedLabel(args)
+
+            expect(requestMock).toBeCalledTimes(1)
+            expect(requestMock).toBeCalledWith(
+                'POST',
+                getSyncBaseUri(),
+                'labels/shared/remove',
+                DEFAULT_AUTH_TOKEN,
+                args,
+            )
+        })
+
+        test('returns success result from rest client', async () => {
+            const args = { name: 'toremove' }
+            setupRestClientMock(null)
+            const api = getTarget()
+
+            const result = await api.removeSharedLabel(args)
+
+            expect(result).toBe(true)
+        })
+    })
 })

From feb612a82c04f5a4fa5361aa86048ec3712e1fc3 Mon Sep 17 00:00:00 2001
From: Deepak Mardi <deepakmardi22@gmail.com>
Date: Fri, 5 Sep 2025 15:33:18 +0530
Subject: [PATCH 2/2] refactor: use shared label endpoint constants in label
 tests

---
 src/TodoistApi.labels.test.ts | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/TodoistApi.labels.test.ts b/src/TodoistApi.labels.test.ts
index 009125f..c346083 100644
--- a/src/TodoistApi.labels.test.ts
+++ b/src/TodoistApi.labels.test.ts
@@ -1,6 +1,12 @@
 import { TodoistApi } from '.'
 import { DEFAULT_AUTH_TOKEN, DEFAULT_LABEL, DEFAULT_REQUEST_ID } from './testUtils/testDefaults'
-import { getSyncBaseUri, ENDPOINT_REST_LABELS } from './consts/endpoints'
+import {
+    getSyncBaseUri,
+    ENDPOINT_REST_LABELS,
+    ENDPOINT_REST_LABELS_SHARED,
+    ENDPOINT_REST_LABELS_SHARED_RENAME,
+    ENDPOINT_REST_LABELS_SHARED_REMOVE,
+} from './consts/endpoints'
 import { setupRestClientMock } from './testUtils/mocks'
 
 function getTarget() {
@@ -184,7 +190,7 @@ describe('TodoistApi label endpoints', () => {
             expect(requestMock).toBeCalledWith(
                 'GET',
                 getSyncBaseUri(),
-                'labels/shared',
+                ENDPOINT_REST_LABELS_SHARED,
                 DEFAULT_AUTH_TOKEN,
                 { omitPersonal: true, limit: 10, cursor: 'abc' },
             )
@@ -213,7 +219,7 @@ describe('TodoistApi label endpoints', () => {
             expect(requestMock).toBeCalledWith(
                 'POST',
                 getSyncBaseUri(),
-                'labels/shared/rename',
+                ENDPOINT_REST_LABELS_SHARED_RENAME,
                 DEFAULT_AUTH_TOKEN,
                 args,
             )
@@ -242,7 +248,7 @@ describe('TodoistApi label endpoints', () => {
             expect(requestMock).toBeCalledWith(
                 'POST',
                 getSyncBaseUri(),
-                'labels/shared/remove',
+                ENDPOINT_REST_LABELS_SHARED_REMOVE,
                 DEFAULT_AUTH_TOKEN,
                 args,
             )