Fix for issue #468 includes passing test (#469)

Author: diberry

lib/js-yaml/dumper.js

@@ -735,7 +735,7 @@ function writeNode(state, level, object, block, compact, iskey) {
         }
       }
     } else if (type === '[object Array]') {
-      var arrayLevel = (state.noArrayIndent) ? level - 1 : level;
+      var arrayLevel = (state.noArrayIndent && (level > 0)) ? level - 1 : level;
       if (block && (state.dump.length !== 0)) {
         writeBlockSequence(state, arrayLevel, state.dump, compact);
         if (duplicate) {

test/issues/0468.js

@@ -0,0 +1,29 @@
+'use strict';
+
+
+var assert = require('assert');
+var yaml = require('../..');
+
+test('should not indent arrays an extra level when disabled', function () {
+  /* eslint-disable max-len */
+  var output = yaml.dump(
+    [
+      {
+        a: 'a_val',
+        b: 'b_val'
+      },
+      {
+        a: 'a2_val',
+        items: [
+          {
+            a: 'a_a_val',
+            b: 'a_b_val'
+          }
+        ]
+      }
+    ],
+    { noArrayIndent: true }
+  );
+  var expected = '- a: a_val\n  b: b_val\n- a: a2_val\n  items:\n  - a: a_a_val\n    b: a_b_val\n';
+  assert.strictEqual(output, expected);
+});