fix: postman by force using compiled js instead of tsx (#786)

* fix: test different branch

* fix: clean cache

* fix: add debug logging

* fix: add debug logging

* fix: add debug logging

* fix: add debug logging

* fix: force compiled js instead of relying on tsx

* fix: remove ref

* fix: tests

---------

Co-authored-by: svcAPLBot <[email protected]>

Author: CasLubbers

.github/workflows/postman.yml

@@ -55,7 +55,8 @@ jobs:
           nvm use
           node -v
           npm install
-          npm run server > $GITHUB_WORKSPACE/core.log 2>&1 &
+          npm run compile
+          NODE_PATH="/usr/local/lib/node_modules" npm run server > $GITHUB_WORKSPACE/core.log 2>&1 &
       - name: Start api
         run: |
           npm install

src/otomi-stack.ts

@@ -2,7 +2,8 @@ import { CoreV1Api, User as k8sUser, KubeConfig, V1ObjectReference } from '@kube
 import Debug from 'debug'
 
 import { getRegions, ObjectStorageKeyRegions } from '@linode/api-v4'
-import { existsSync, pathExists, rmSync, unlink } from 'fs-extra'
+import { existsSync, rmSync } from 'fs'
+import { pathExists, unlink } from 'fs-extra'
 import { readdir, readFile, writeFile } from 'fs/promises'
 import { generate as generatePassword } from 'generate-password'
 import { cloneDeep, filter, isEmpty, map, mapValues, merge, omit, pick, set, unset } from 'lodash'

src/utils/workloadUtils.test.ts

@@ -1,5 +1,6 @@
 // workloadUtils.test.ts
 import axios from 'axios'
+import * as fs from 'fs'
 import * as fsExtra from 'fs-extra'
 import * as fsPromises from 'fs/promises'
 import path from 'path'
@@ -24,13 +25,15 @@ jest.mock('fs/promises', () => ({
   writeFile: jest.fn(),
   readdir: jest.fn(),
 }))
-jest.mock('fs-extra', () => ({
-  readFile: jest.fn(),
+jest.mock('fs', () => ({
   existsSync: jest.fn(),
   mkdirSync: jest.fn(),
   renameSync: jest.fn(),
   rmSync: jest.fn(),
 }))
+jest.mock('fs-extra', () => ({
+  readFile: jest.fn(),
+}))
 jest.mock('simple-git', () => ({
   __esModule: true,
   default: jest.fn(() => ({
@@ -354,7 +357,7 @@ describe('sparseCloneChart', () => {
     // Set up environment variables for tests
     process.env = { ...originalEnv, GIT_USER: 'git-user', GIT_PASSWORD: 'git-password' }
     // Mock necessary function responses
-    ;(fsExtra.existsSync as jest.Mock).mockReturnValue(false)
+    ;(fs.existsSync as jest.Mock).mockReturnValue(false)
   })
 
   afterEach(() => {
@@ -390,16 +393,16 @@ describe('sparseCloneChart', () => {
     )
 
     expect(result).toBe(true)
-    expect(fsExtra.mkdirSync).toHaveBeenCalledWith(localHelmChartsDir, { recursive: true })
-    expect(fsExtra.mkdirSync).toHaveBeenCalledWith(`${localHelmChartsDir}-newChart`, { recursive: true })
+    expect(fs.mkdirSync).toHaveBeenCalledWith(localHelmChartsDir, { recursive: true })
+    expect(fs.mkdirSync).toHaveBeenCalledWith(`${localHelmChartsDir}-newChart`, { recursive: true })
     expect(mockGit.clone).toHaveBeenCalledTimes(2) // Once for catalog repo, once for chart repo
     expect(mockGit.listRemote).toHaveBeenCalled()
     expect(mockGit.raw).toHaveBeenCalledWith(['sparse-checkout', 'init', '--cone'])
     expect(mockGit.raw).toHaveBeenCalledWith(['sparse-checkout', 'set', 'main/bitnami/cassandra/'])
     expect(mockGit.checkout).toHaveBeenCalled()
-    expect(fsExtra.renameSync).toHaveBeenCalled()
-    expect(fsExtra.rmSync).toHaveBeenCalledWith(`${localHelmChartsDir}-newChart`, { recursive: true, force: true })
-    expect(fsExtra.rmSync).toHaveBeenCalledWith(`${localHelmChartsDir}/${chartTargetDirName}/.git`, {
+    expect(fs.renameSync).toHaveBeenCalled()
+    expect(fs.rmSync).toHaveBeenCalledWith(`${localHelmChartsDir}-newChart`, { recursive: true, force: true })
+    expect(fs.rmSync).toHaveBeenCalledWith(`${localHelmChartsDir}/${chartTargetDirName}/.git`, {
       recursive: true,
       force: true,
     })
@@ -491,7 +494,7 @@ describe('sparseCloneChart', () => {
       listRemote: jest.fn().mockResolvedValue(''),
     }
     ;(simpleGit as jest.Mock).mockReturnValue(mockGit)
-    ;(fsExtra.existsSync as jest.Mock).mockReturnValueOnce(false)
+    ;(fs.existsSync as jest.Mock).mockReturnValueOnce(false)
 
     await sparseCloneChart(
       gitRepositoryUrl,
@@ -504,7 +507,7 @@ describe('sparseCloneChart', () => {
       allowTeams,
     )
 
-    expect(fsExtra.mkdirSync).toHaveBeenCalledWith(localHelmChartsDir, { recursive: true })
+    expect(fs.mkdirSync).toHaveBeenCalledWith(localHelmChartsDir, { recursive: true })
   })
 
   test('returns false if git provider detection fails', async () => {
@@ -535,7 +538,7 @@ describe('fetchWorkloadCatalog', () => {
   beforeEach(() => {
     jest.clearAllMocks()
     process.env = { ...originalEnv, GIT_USER: 'git-user', GIT_PASSWORD: 'git-password' }
-    ;(fsExtra.existsSync as jest.Mock).mockReturnValue(false)
+    ;(fs.existsSync as jest.Mock).mockReturnValue(false)
 
     // Mock directory structure
     const files = ['.git', 'chart1', 'chart2', 'README.md', 'rbac.yaml']
@@ -597,7 +600,7 @@ describe('fetchWorkloadCatalog', () => {
 
     const result = await fetchWorkloadCatalog(url, helmChartsDir, 'admin')
 
-    expect(fsExtra.mkdirSync).toHaveBeenCalledWith(helmChartsDir, { recursive: true })
+    expect(fs.mkdirSync).toHaveBeenCalledWith(helmChartsDir, { recursive: true })
     expect(mockGit.clone).toHaveBeenCalledWith(
       'https://git-user:[email protected]/otomi/charts.git',
       helmChartsDir,
@@ -780,7 +783,7 @@ describe('chartRepo', () => {
     expect(mockGit.raw).toHaveBeenCalledWith(['sparse-checkout', 'init', '--cone'])
     expect(mockGit.raw).toHaveBeenCalledWith(['sparse-checkout', 'set', 'charts/my-chart'])
     expect(mockGit.checkout).toHaveBeenCalledWith('main')
-    expect(fsExtra.renameSync).toHaveBeenCalledWith(path.join(localPath, 'charts/my-chart'), finalDestinationPath)
+    expect(fs.renameSync).toHaveBeenCalledWith(path.join(localPath, 'charts/my-chart'), finalDestinationPath)
   })
 
   test('addConfig method sets git config', async () => {

src/utils/workloadUtils.ts

@@ -1,6 +1,7 @@
 import axios from 'axios'
 import Debug from 'debug'
-import { existsSync, mkdirSync, readFile, renameSync, rmSync } from 'fs-extra'
+import { existsSync, mkdirSync, renameSync, rmSync } from 'fs'
+import { readFile } from 'fs-extra'
 import { readdir, writeFile } from 'fs/promises'
 import path from 'path'
 import simpleGit, { SimpleGit } from 'simple-git'