add tests for group reset and option uniqueness per group

yargs · Jan 9, 2016 · de5818a · de5818a
1 parent d387a3d
commit de5818a
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 0 deletions.
diff --git a/test/usage.js b/test/usage.js
@@ -1200,6 +1200,34 @@ describe('usage tests', function () {
         'Missing', 'required', 'argument:', 'y'
       ])
     })
+
+    it('resets groups for a command handler, respecting order', function () {
+      var r = checkUsage(function () {
+        return yargs(['upload', '-h'])
+          .command('upload', 'upload something', function (yargs, argv) {
+            argv = yargs
+              .option('q', {
+                type: 'boolean',
+                group: 'Flags:'
+              })
+              .help('h').group('h', 'Global Flags:')
+              .wrap(null)
+              .argv
+          })
+          .help('h').group('h', 'Global Flags:')
+          .wrap(null)
+          .argv
+      })
+
+      r.logs[0].split('\n').should.deep.equal([
+        'Flags:',
+        '  -q  [boolean]',
+        '',
+        'Global Flags:',
+        '  -h  Show help  [boolean]',
+        ''
+      ])
+    })
   })
 
   describe('epilogue', function () {
@@ -1747,6 +1775,30 @@ describe('usage tests', function () {
         ''
       ])
     })
+
+    it('only displays a duplicated option once per group', function () {
+      var r = checkUsage(function () {
+        return yargs(['-h'])
+          .help('h')
+          .group(['batman', 'batman'], 'Heroes:')
+          .group('robin', 'Heroes:')
+          .option('robin', {
+            group: 'Heroes:'
+          })
+          .wrap(null)
+          .argv
+      })
+
+      r.logs[0].split('\n').should.deep.equal([
+        'Heroes:',
+        '  --batman',
+        '  --robin',
+        '',
+        'Options:',
+        '  -h  Show help  [boolean]',
+        ''
+      ])
+    })
   })
 
   describe('cjk', function () {

diff --git a/test/yargs.js b/test/yargs.js
@@ -213,6 +213,7 @@ describe('yargs dsl tests', function () {
         .string('foo')
         .choices('foo', ['bar', 'baz'])
         .implies('foo', 'snuh')
+        .group('foo', 'Group:')
         .strict()
         .exitProcess(false)  // defaults to true.
         .env('YARGS')
@@ -242,6 +243,7 @@ describe('yargs dsl tests', function () {
       expect(y.getStrict()).to.equal(false)
       expect(y.getDemanded()).to.deep.equal({})
       expect(y.getCommandHandlers()).to.deep.equal({})
+      expect(y.getGroups()).to.deep.equal({})
     })
   })