Amended "insertNewLine..." formatting options (cont')

edge/createFormattingOptions.js

@@ -37,19 +37,19 @@ const schema = {
 			`
 		}
 	},
-	insertNewLineAroundBlock: {
+	insertNewLineAroundProperties: {
 		description: '',
-		type: 'boolean',
+		oneOf: [true, false],
 		default: true,
 	},
-	insertNewLineAroundProperties: {
+	insertNewLineAroundOthers: {
 		description: '',
-		type: 'boolean',
+		oneOf: [true, false, 'root', 'nested'],
 		default: true,
 	},
-	insertNewLineAroundOthers: {
+	insertNewLineAroundBlock: {
 		description: '',
-		type: 'boolean',
+		oneOf: [true, false, 'root', 'nested'],
 		default: true,
 	},
 	insertNewLineBetweenSelectors: {
@@ -290,8 +290,7 @@ function createFormattingOptions(options = {}) {
 					hash[name] = options[name]
 				}
 			} catch (ex) {
-				ex.message += ` at "${name}".` 
-				throw ex
+				throw new Error(ex.message + ` at "${name}".`)
 			}
 			return hash
 		}, {})
@@ -300,7 +299,20 @@ function createFormattingOptions(options = {}) {
 
 function verify(data, info) {
 	if (info.oneOf !== undefined) {
-		info.oneOf.some(item => _.isObject(item) ? verify(data, item) : (item === data))
+		const matchAnyValue = info.oneOf.some(item => {
+			if (_.isObject(item)) {
+				try {
+					return verify(data, item)
+				} catch (ex) {
+					return false
+				}
+			} else {
+				return item === data
+			}
+		})
+		if (matchAnyValue === false) {
+			throw new Error(`Expected ${data} to be one of the defined values`)
+		}
 
 	} else if (info.type === 'integer') {
 		if (_.isInteger(data) === false) {

edge/format.js

@@ -231,21 +231,39 @@ function format(content, options = {}) {
 				}
 			})
 
+			const checkIf = (value) => {
+				if (value === true) {
+					return true
+				}
+
+				if (options.wrapMode) {
+					return _.some(originalBaseIndent) ? value === 'nested' : value === 'root'
+				} else {
+					return inputNode instanceof Stylus.nodes.Root ? value === 'root' : value === 'nested'
+				}
+
+				return false
+			}
+
 			// Insert CSS body
 			outputBuffer.append(_.chain(groups)
 				.map(group => {
 					const nodeType = getType(group[0])
 
 					let newLineAround = ''
 					if (
-						nodeType === 'Block' && options.insertNewLineAroundBlock ||
-						nodeType === 'Property' && options.insertNewLineAroundProperties ||
-						nodeType === 'Other' && options.insertNewLineAroundOthers
+						nodeType === 'Block' && checkIf(options.insertNewLineAroundBlock) ||
+						nodeType === 'Property' && checkIf(options.insertNewLineAroundProperties) ||
+						nodeType === 'Other' && checkIf(options.insertNewLineAroundOthers)
 					) {
 						newLineAround = options.newLineChar
 					}
 
-					return _.compact([newLineAround, group.map(node => travel(inputNode, node, childIndentLevel)), newLineAround])
+					return _.compact([
+						newLineAround,
+						group.map(node => travel(inputNode, node, childIndentLevel)).join(''),
+						newLineAround
+					])
 				})
 				.flatten()
 				.reject((text, rank, list) => text === options.newLineChar && (
@@ -257,8 +275,6 @@ function format(content, options = {}) {
 				.value()
 			)
 
-			// _.repeat(options.newLineChar, indentLevel === 0 || originalBaseIndent === '' ? options.insertNewLineBetweenOuterGroups : options.insertNewLineBetweenInnerGroups)
-
 			// Insert the bottom comment(s)
 			const bottomCommentNodes = tryGetSingleLineCommentNodesOnTheBottomOf(_.last(nonCommentNodes))
 			if (bottomCommentNodes) {
@@ -754,9 +770,9 @@ function format(content, options = {}) {
 			}
 			if (inputNode.nodes.length > 0) {
 				if (inputNode.type) {
-				outputBuffer.append(' and ')
-			}
-			outputBuffer.append(inputNode.nodes.map(node => travel(inputNode, node, indentLevel, true)).join(' and '))
+					outputBuffer.append(' and ')
+				}
+				outputBuffer.append(inputNode.nodes.map(node => travel(inputNode, node, indentLevel, true)).join(' and '))
 			}
 
 		} else if (inputNode instanceof Stylus.nodes.Feature) {
@@ -821,6 +837,7 @@ function format(content, options = {}) {
 			if (options.insertSemicolons) {
 				outputBuffer.append(';')
 			}
+			outputBuffer.append(options.newLineChar)
 
 		} else if (inputNode instanceof Stylus.nodes.Comment && inputNode.str.startsWith('//')) { // In case of single-line comments
 			if (insideExpression === false) {

spec/function/output.styl

@@ -1,7 +1,6 @@
 add(a, b) {
 	a = unit(a, px);
 	b = unit(b, px);
-
 	a + b;
 }
 
@@ -14,7 +13,6 @@ func(x, y = unit(a, px)) {
 }
 
 func(x: 10, y: 25);
-
 alias = func;
 
 multi-returned-value-func() {

spec/hash/output.styl

@@ -8,7 +8,6 @@ foo = {
 	a2: { c2: true },
 	a3: {}
 };
-
 foo['0'] = bar;
 
 body {

spec/operator/output.styl

@@ -12,15 +12,13 @@ body {
 	range = 1...5;
 	math = 1 + 2 - 3 * (4 / 5) % 6 ** 7;
 	color = color is defined ? unit(num, 'px') : white;
-
 	color ?= white;
 	color ?= white;
 
 	border-color: #fff is a 'rgba';
 	background-color: #f00 - rgba(100, 0, 0, 0.5);
 
 	name = 'blue';
-
 	lookup('light-' + name);
 
 	font-size: num em;

spec/option-insert-new-line-around-any-1/formattingOptions.json

@@ -1,4 +1,6 @@
 {
+	"insertNewLineAroundProperties": true,
+	"insertNewLineAroundOthers": true,
 	"insertNewLineAroundBlock": true,
 	"newLineChar": "\r\n"
 }

spec/option-insert-new-line-around-any-1/input.styl

@@ -1,16 +1,25 @@
 .class1
-  display none
+  mixin()
+    display func()
+  func()
+    1+2
+  var = func()
+  mixin()
+  margin 0
+  padding 0
 
 .class2
-  display none
+  margin 0
+  padding 0
 
   .class3
     display none
+  .class4
+    display none
 
 mixin()
-  display block
-
-var = 1
-
+  display func()
 func()
-  1+2
+  1+2
+var = func()
+mixin()

spec/option-insert-new-line-around-any-1/output.styl

@@ -1,17 +1,39 @@
-body {
-	display: none;
+.class1 {
+	mixin() {
+		display: func();
+	}
+
+	func() {
+		1 + 2;
+	}
+
+	var = func();
+	mixin();
+
+	margin: 0;
+	padding: 0;
+}
+
+.class2 {
+	margin: 0;
+	padding: 0;
 
-	.class1 {
+	.class3 {
+		display: none;
+	}
+
+	.class4 {
 		display: none;
 	}
 }
 
 mixin() {
-	display: block;
+	display: func();
 }
 
-var = 1;
-
 func() {
 	1 + 2;
-}
+}
+
+var = func();
+mixin();

spec/option-insert-new-line-around-any-2/formattingOptions.json

@@ -0,0 +1,6 @@
+{
+	"insertNewLineAroundProperties": false,
+	"insertNewLineAroundOthers": false,
+	"insertNewLineAroundBlock": true,
+	"newLineChar": "\r\n"
+}

spec/option-insert-new-line-around-any-2/input.styl

@@ -0,0 +1,26 @@
+.class1
+  mixin()
+    display func()
+  func()
+    1+2
+  var = func()
+  mixin()
+
+  margin 0
+  padding 0
+
+.class2
+  margin 0
+  padding 0
+
+  .class3
+    display none
+  .class4
+    display none
+
+mixin()
+  display func()
+func()
+  1+2
+var = func()
+mixin()

spec/option-insert-new-line-around-any-2/output.styl

@@ -0,0 +1,38 @@
+.class1 {
+	mixin() {
+		display: func();
+	}
+
+	func() {
+		1 + 2;
+	}
+
+	var = func();
+	mixin();
+	margin: 0;
+	padding: 0;
+}
+
+.class2 {
+	margin: 0;
+	padding: 0;
+
+	.class3 {
+		display: none;
+	}
+
+	.class4 {
+		display: none;
+	}
+}
+
+mixin() {
+	display: func();
+}
+
+func() {
+	1 + 2;
+}
+
+var = func();
+mixin();

spec/option-insert-new-line-around-any-3/formattingOptions.json

@@ -0,0 +1,6 @@
+{
+	"insertNewLineAroundProperties": false,
+	"insertNewLineAroundOthers": false,
+	"insertNewLineAroundBlock": false,
+	"newLineChar": "\r\n"
+}

spec/option-insert-new-line-around-any-3/input.styl

@@ -0,0 +1,25 @@
+.class1
+  mixin()
+    display func()
+  func()
+    1+2
+  var = func()
+  mixin()
+  margin 0
+  padding 0
+
+.class2
+  margin 0
+  padding 0
+
+  .class3
+    display none
+  .class4
+    display none
+
+mixin()
+  display func()
+func()
+  1+2
+var = func()
+mixin()