Merge pull request #177 from sasjs/cli.spec-fix

fix(regexp): fixed newLine utility
sasjs · Mar 15, 2022 · 85fa446 · 85fa446
2 parents eec480e + 2f0e73f
commit 85fa446
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/formatter/regexp.spec.ts b/src/formatter/regexp.spec.ts
@@ -1,5 +1,4 @@
 import { newLine } from './'
-import { isWindows } from '../utils'
 
 describe('newLine', () => {
   it('should return correct new line regexp entry for windows', () => {

diff --git a/src/formatter/regexp.ts b/src/formatter/regexp.ts
@@ -1,3 +1,5 @@
-import { isWindows } from '../'
+// FIXME: if isWindows is used, it breaks cli.spec.ts because Logger is undefined
+// import { isWindows } from '../'
+// export const newLine = () => (isWindows() ? '\\r\\n' : '\\n')
 
-export const newLine = () => (isWindows() ? '\\r\\n' : '\\n')
+export const newLine = () => (process.platform === 'win32' ? '\\r\\n' : '\\n')