@cloud-cli/vm 1.2.0 → 1.3.1

package/dist/index.d.ts

@@ -7,6 +7,7 @@ interface NameAndPathOption {
 }
 declare function list(): Promise<string[]>;
 declare function ls(options: NameAndPathOption): Promise<string[]>;
+declare function fixPermissions(options: NameOption): Promise<boolean>;
 declare function rm(options: NameAndPathOption): Promise<boolean>;
 declare function cat(options: NameAndPathOption): Promise<string>;
 declare function show(options: NameOption): Promise<{
@@ -26,5 +27,6 @@ declare const _default: {
     ls: typeof ls;
     rm: typeof rm;
     cat: typeof cat;
+    fixPermissions: typeof fixPermissions;
 };
 export default _default;

package/dist/index.js

@@ -23,7 +23,13 @@ async function ls(options) {
     const { name, path = '' } = options;
     const root = await getVolumeMountpoint(name);
     const files = await exec('ls', ['-1', join(root, path)]);
-    return files.stdout.trim().split('\n');
+    return files.stdout.trim().split('\n').filter(Boolean);
+}
+async function fixPermissions(options) {
+    const { name } = options;
+    const root = await getVolumeMountpoint(name);
+    const result = await exec('chmod', ['-R', 'a+w', root]);
+    return result.ok;
 }
 async function rm(options) {
     const { name, path } = options;
@@ -82,4 +88,4 @@ async function prune() {
     await exec('docker', ['volume', 'prune']);
     return '';
 }
-export default { add, remove, list, show, prune, ls, rm, cat };
+export default { add, remove, list, show, prune, ls, rm, cat, fixPermissions };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cloud-cli/vm",
-  "version": "1.2.0",
+  "version": "1.3.1",
   "description": "Docker volume management",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/index.spec.ts

@@ -48,6 +48,17 @@ describe('volume manager', () => {
     });
   });
 
+  describe('fixPermissions', () => {
+    it('should fix write permissions on volume folder', async () => {
+      const outputs = [inspectOutput, ''];
+      execMocks.exec = jest.fn().mockImplementation(() => ({ ok: true, stdout: outputs.shift() }));
+
+      await expect(vm.fixPermissions({ name: 'test' })).resolves.toEqual(true);
+      expect(exec.exec).toHaveBeenCalledWith('docker', ['volume', 'inspect', 'test']);
+      expect(exec.exec).toHaveBeenCalledWith('chmod', ['-R', 'a+w', '/var/lib/docker/volumes/test/_data']);
+    });
+  });
+
   describe('ls', () => {
     it('should list files of a volume and path', async () => {
       const outputs = [inspectOutput, 'dir', inspectOutput, 'a.txt\nb.txt'];

package/src/index.ts

@@ -41,7 +41,15 @@ async function ls(options: NameAndPathOption) {
   const root = await getVolumeMountpoint(name);
   const files = await exec('ls', ['-1', join(root, path)]);
 
-  return files.stdout.trim().split('\n');
+  return files.stdout.trim().split('\n').filter(Boolean);
+}
+
+async function fixPermissions(options: NameOption) {
+  const { name } = options;
+  const root = await getVolumeMountpoint(name);
+  const result = await exec('chmod', ['-R', 'a+w', root]);
+
+  return result.ok;
 }
 
 async function rm(options: NameAndPathOption) {
@@ -122,4 +130,4 @@ async function prune() {
   return '';
 }
 
-export default { add, remove, list, show, prune, ls, rm, cat };
+export default { add, remove, list, show, prune, ls, rm, cat, fixPermissions };