From 3525e6f687bd5fc9948e58fb2066a3864b39176a Mon Sep 17 00:00:00 2001
From: Stefano Vuerich <vuerich@irsweb.it>
Date: Thu, 19 Nov 2020 08:40:14 +0100
Subject: [PATCH 1/2] fix: conditional getuid and getgid for Windows
 environment

---
 .../stages/templater/helpers.test.ts          | 15 +++++++-------
 .../scaffolder/stages/templater/helpers.ts    | 20 ++++++++++++++-----
 2 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
index a4441ca23f..ca838fbfaa 100644
--- a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
+++ b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
@@ -17,15 +17,9 @@ import Stream, { PassThrough } from 'stream';
 import os from 'os';
 import fs from 'fs';
 import Docker from 'dockerode';
-import { runDockerContainer } from './helpers';
+import { UserOptions, runDockerContainer} from './helpers';
 
 describe('helpers', () => {
-  if (process.platform === 'win32') {
-    // eslint-disable-next-line jest/no-focused-tests
-    it.only('should skip tests on windows', () => {
-      expect('test').not.toBe('run');
-    });
-  }
 
   const mockDocker = new Docker() as jest.Mocked<Docker>;
 
@@ -142,12 +136,17 @@ describe('helpers', () => {
         dockerClient: mockDocker,
       });
 
+      const userOptions: UserOptions = {};
+      if (process.getuid && process.getgid) {
+        userOptions.User = `${process.getuid()}:${process.getgid()}`;
+      }
+
       expect(mockDocker.run).toHaveBeenCalledWith(
         imageName,
         args,
         expect.any(Stream),
         expect.objectContaining({
-          User: `${process.getuid()}:${process.getgid()}`,
+          ...userOptions,
           Env: ['HOME=/tmp'],
         }),
       );
diff --git a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
index e71b63bfb0..42bdbc8060 100644
--- a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
+++ b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
@@ -36,6 +36,10 @@ export type RunCommandOptions = {
   logStream?: Writable;
 };
 
+export type UserOptions = {
+  User?: string;
+}
+
 /**
  * Gets the templater key to use for templating from the entity
  * @param entity Template entity
@@ -115,6 +119,16 @@ export const runDockerContainer = async ({
     });
   });
 
+  const userOptions: UserOptions = {};
+  if (process.getuid && process.getgid) {
+    // Files that are created inside the Docker container will be owned by
+    // root on the host system on non Mac systems, because of reasons. Mainly the fact that
+    // volume sharing is done using NFS on Mac and actual mounts in Linux world.
+    // So we set the user in the container as the same user and group id as the host.
+    // On Windows we don't have process.getuid nor process.getgid
+    userOptions.User = `${process.getuid()}:${process.getgid()}`;
+  }
+
   const [{ Error: error, StatusCode: statusCode }] = await dockerClient.run(
     imageName,
     args,
@@ -129,11 +143,7 @@ export const runDockerContainer = async ({
           `${await fs.promises.realpath(templateDir)}:/template`,
         ],
       },
-      // Files that are created inside the Docker container will be owned by
-      // root on the host system on non Mac systems, because of reasons. Mainly the fact that
-      // volume sharing is done using NFS on Mac and actual mounts in Linux world.
-      // So we set the user in the container as the same user and group id as the host.
-      User: `${process.getuid()}:${process.getgid()}`,
+      ...userOptions,
       // Set the home directory inside the container as something that applications can
       // write to, otherwise they will just flop and fail trying to write to /
       Env: ['HOME=/tmp'],

From 09d90484ea25b34b28b3f61eef95139385458e83 Mon Sep 17 00:00:00 2001
From: Stefano Vuerich <vuerich@irsweb.it>
Date: Thu, 19 Nov 2020 11:46:34 +0100
Subject: [PATCH 2/2] fix: runned prettier

---
 .../src/scaffolder/stages/templater/helpers.test.ts            | 3 +--
 .../src/scaffolder/stages/templater/helpers.ts                 | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
index ca838fbfaa..57e47f4c27 100644
--- a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
+++ b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.test.ts
@@ -17,10 +17,9 @@ import Stream, { PassThrough } from 'stream';
 import os from 'os';
 import fs from 'fs';
 import Docker from 'dockerode';
-import { UserOptions, runDockerContainer} from './helpers';
+import { UserOptions, runDockerContainer } from './helpers';
 
 describe('helpers', () => {
-
   const mockDocker = new Docker() as jest.Mocked<Docker>;
 
   beforeEach(() => {
diff --git a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
index 42bdbc8060..e79f2cb066 100644
--- a/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
+++ b/plugins/scaffolder-backend/src/scaffolder/stages/templater/helpers.ts
@@ -38,7 +38,7 @@ export type RunCommandOptions = {
 
 export type UserOptions = {
   User?: string;
-}
+};
 
 /**
  * Gets the templater key to use for templating from the entity