caprover-api 0.0.22 → 0.0.23

package/dist/api/ApiManager.js

@@ -78,12 +78,19 @@ class ApiManager {
     }
     saveTheme(oldName, theme) {
         const http = this.http;
-        return Promise.resolve() //
-            .then(http.fetch(http.POST, '/user/system/themes/update', {
+        const payload = {
             oldName,
             name: theme.name,
             content: theme.content,
-        }));
+        };
+        if (theme.extra !== undefined) {
+            payload.extra = theme.extra;
+        }
+        if (theme.headEmbed !== undefined) {
+            payload.headEmbed = theme.headEmbed;
+        }
+        return Promise.resolve() //
+            .then(http.fetch(http.POST, '/user/system/themes/update', payload));
     }
     deleteTheme(themeName) {
         const http = this.http;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "caprover-api",
-    "version": "0.0.22",
+    "version": "0.0.23",
     "description": "API client for CapRover",
     "types": "dist/index.d.ts",
     "main": "dist/index.js",

package/src/api/ApiManager.ts

@@ -145,15 +145,28 @@ export default class ApiManager {
 
     saveTheme(oldName: string, theme: CapRoverTheme): Promise<{}> {
         const http = this.http
-
-        return Promise.resolve() //
-            .then(
-                http.fetch(http.POST, '/user/system/themes/update', {
-                    oldName,
-                    name: theme.name,
-                    content: theme.content,
-                })
-            )
+        const payload: {
+            oldName: string
+            name: string
+            content: string
+            extra?: string
+            headEmbed?: string
+        } = {
+            oldName,
+            name: theme.name,
+            content: theme.content,
+        }
+
+        if (theme.extra !== undefined) {
+            payload.extra = theme.extra
+        }
+
+        if (theme.headEmbed !== undefined) {
+            payload.headEmbed = theme.headEmbed
+        }
+
+        return Promise.resolve() //
+            .then(http.fetch(http.POST, '/user/system/themes/update', payload))
     }
 
     deleteTheme(themeName: string): Promise<{}> {

package/test/ApiManager.test.js

@@ -101,3 +101,48 @@ test('executeGenericApiCommand accepts PATCH', async () => {
 
     assert.equal(requests[0].method, 'PATCH')
 })
+
+test('saveTheme sends supplied writable theme fields', async () => {
+    const { api, requests } = createApiWithRequestRecorder()
+
+    await api.saveTheme('old-theme', {
+        name: 'new-theme',
+        content: 'theme-content',
+        extra: '{"siderTheme":"dark"}',
+        headEmbed: '<meta name="theme-marker" content="test">',
+        builtIn: true,
+    })
+
+    assert.deepEqual(requests, [
+        {
+            method: 'POST',
+            endpoint: '/user/system/themes/update',
+            data: {
+                oldName: 'old-theme',
+                name: 'new-theme',
+                content: 'theme-content',
+                extra: '{"siderTheme":"dark"}',
+                headEmbed: '<meta name="theme-marker" content="test">',
+            },
+            bodyType: 'json',
+        },
+    ])
+})
+
+test('saveTheme omits unspecified optional theme fields', async () => {
+    const { api, requests } = createApiWithRequestRecorder()
+
+    await api.saveTheme('old-theme', {
+        name: 'new-theme',
+        content: 'theme-content',
+    })
+
+    assert.deepEqual(requests[0].data, {
+        oldName: 'old-theme',
+        name: 'new-theme',
+        content: 'theme-content',
+    })
+    assert.equal('extra' in requests[0].data, false)
+    assert.equal('headEmbed' in requests[0].data, false)
+    assert.equal(JSON.stringify(requests[0].data).includes('undefined'), false)
+})