add test case && fix test fail

Author: githoniel

dist/index.js

@@ -2,8 +2,6 @@ const sectionExpr = /^\[(.*)\]/,
     commentExpr = /[;#](?: )?(.+)/,
     lineExpr = /(^\s*[;#])|(^\[[^\]]*\])|(^.+$)/,
     quotedExpr = /^(\s*['"]).+$/,
-    lineBreak = typeof process !== 'undefined' &&
-        process.platform === 'win32' ? '\r\n' : '\n',
     lineTypes = {
         blank: 0,
         comment: 1,
@@ -240,13 +238,14 @@ class Ini {
         }, new Ini());
     }
 
-    constructor(text = '') {
+    constructor(text = '', lineBreak) {
         if (typeof text !== 'string')
             throw new Error('Input must be a string.');
+        this.lineBreak = lineBreak || this.determineLineBreak(text)
         this.sections = [];
         let currentSection = this.globals = new IniSection();
         if (text.length === 0) return;
-        text.split(lineBreak).forEach(line => {
+        text.split(this.lineBreak).forEach(line => {
             if (isSectionLine(line)) {
                 currentSection = new IniSection(line);
                 this.sections.push(currentSection);
@@ -256,6 +255,26 @@ class Ini {
         });
     }
 
+    determineLineBreak(text) {
+        if(text === '') {
+            return typeof process !== 'undefined' &&
+                process.platform === 'win32' ? '\r\n' : '\n'
+        } else {
+            let lineBreak
+            if(['\r\n', '\n'].some((t) => {
+                if (text.split(t).length > 1) {
+                    lineBreak = t
+                    return true
+                }
+            })) {
+                return lineBreak
+            } else {
+                return typeof process !== 'undefined' &&
+                    process.platform === 'win32' ? '\r\n' : '\n'
+            }
+        }
+    }
+
     getSection(name) {
         return this.sections.find(section => section.name === name);
     }
@@ -287,12 +306,12 @@ class Ini {
                     (!options.removeCommentLines || !isCommentLine(line));
             }).map(line => line.text);
             if (!lines.length) return;
-            str += lines.join(lineBreak);
+            str += lines.join(this.lineBreak);
             if (index === sections.length - 1) return;
             let lastLine = lines[lines.length - 1];
             if (options.blankLineBeforeSection && !!lastLine.trim())
-                str += lineBreak;
-            str += lineBreak;
+                str += this.lineBreak;
+            str += this.lineBreak;
         });
         return str;
     }

src/ini.js

@@ -30,9 +30,9 @@ class Ini {
     }
 
     constructor(text = '', lineBreak) {
-        this.lineBreak = lineBreak || this.determineLineBreak(text)
         if (typeof text !== 'string')
             throw new Error('Input must be a string.');
+        this.lineBreak = lineBreak || this.determineLineBreak(text)
         this.sections = [];
         let currentSection = this.globals = new IniSection();
         if (text.length === 0) return;

test/ini.test.js

@@ -173,4 +173,22 @@ describe('Ini', function() {
             });
         });
     });
+
+    describe('linebreak', function() {
+        it('should accept linebreak', () => {
+            const newIni = new Ini('[x]\r\nb=3', '\r\n')
+            expect(newIni.sections[0].lines.length).toBe(2)
+
+            const newIni2 = new Ini('[x]\nb=3', '\r\n')
+            expect(newIni2.sections[0].lines.length).toBe(1)
+        })
+
+        it('should determind linebreak by text', () => {
+            const newIni = new Ini('[x]\r\nb=3')
+            expect(newIni.sections[0].lines.length).toBe(2)
+
+            const newIni2 = new Ini('[x]\nb=3')
+            expect(newIni2.sections[0].lines.length).toBe(2)
+        })
+    }) 
 });