diff options
author | Ruy Adorno <ruyadorno@hotmail.com> | 2020-12-09 23:14:44 +0300 |
---|---|---|
committer | isaacs <i@izs.me> | 2020-12-11 22:29:02 +0300 |
commit | 7ff6efbb866591b2330b967215cef8146dff3ebf (patch) | |
tree | c9478be5d869a142fb42f6ab44484338c8a745c5 /tap-snapshots | |
parent | c243e3b9d9bda0580a0fc1b3e230b4d47412176e (diff) |
fix: lib/team.js tweaks and tests
- Removes unnecessary colon when listing 0 users/teams
- Removes unimplemented `npm team edit` placeholder
- Adds `test/lib/team.js` tests
- Fixes: https://github.com/npm/statusboard/issues/176
PR-URL: https://github.com/npm/cli/pull/2314
Credit: @ruyadorno
Close: #2314
Reviewed-by: @isaacs
Diffstat (limited to 'tap-snapshots')
-rw-r--r-- | tap-snapshots/test-lib-team.js-TAP.test.js | 85 |
1 files changed, 85 insertions, 0 deletions
diff --git a/tap-snapshots/test-lib-team.js-TAP.test.js b/tap-snapshots/test-lib-team.js-TAP.test.js new file mode 100644 index 000000000..73123ee1a --- /dev/null +++ b/tap-snapshots/test-lib-team.js-TAP.test.js @@ -0,0 +1,85 @@ +/* IMPORTANT + * This snapshot file is auto-generated, but designed for humans. + * It should be checked into source control and tracked carefully. + * Re-generate by setting TAP_SNAPSHOT=1 and running tests. + * Make sure to inspect the output below. Do not ignore changes! + */ +'use strict' +exports[`test/lib/team.js TAP team add <scope:team> <user> --parseable > should output success result for parseable add user 1`] = ` +foo npmcli:developers added +` + +exports[`test/lib/team.js TAP team add <scope:team> <user> default output > should output success result for add user 1`] = ` +foo added to @npmcli:developers +` + +exports[`test/lib/team.js TAP team create <scope:team> --parseable > should output parseable success result for create team 1`] = ` +npmcli:newteam created +` + +exports[`test/lib/team.js TAP team create <scope:team> default output > should output success result for create team 1`] = ` ++@npmcli:newteam +` + +exports[`test/lib/team.js TAP team destroy <scope:team> --parseable > should output parseable result for destroy team 1`] = ` +npmcli:newteam deleted +` + +exports[`test/lib/team.js TAP team destroy <scope:team> default output > should output success result for destroy team 1`] = ` +-@npmcli:newteam +` + +exports[`test/lib/team.js TAP team ls <scope:team> --parseable > should list users for a parseable scope:team 1`] = ` +darcyclarke +isaacs +nlf +ruyadorno +` + +exports[`test/lib/team.js TAP team ls <scope:team> default output > should list users for a given scope:team 1`] = ` + +@npmcli:developers has 4 users: +darcyclarke isaacs nlf ruyadorno +` + +exports[`test/lib/team.js TAP team ls <scope:team> no users > should list no users for a given scope 1`] = ` + +@npmcli:developers has 0 users +` + +exports[`test/lib/team.js TAP team ls <scope:team> single user > should list single user for a given scope 1`] = ` + +@npmcli:developers has 1 user: +foo +` + +exports[`test/lib/team.js TAP team ls <scope> --parseable > should list teams for a parseable scope 1`] = ` +npmcli:designers +npmcli:developers +npmcli:product +` + +exports[`test/lib/team.js TAP team ls <scope> default output > should list teams for a given scope 1`] = ` + +@npmcli has 3 teams: +@npmcli:designers @npmcli:developers @npmcli:product +` + +exports[`test/lib/team.js TAP team ls <scope> no teams > should list no teams for a given scope 1`] = ` + +@npmcli has 0 teams +` + +exports[`test/lib/team.js TAP team ls <scope> single team > should list single team for a given scope 1`] = ` + +@npmcli has 1 team: +@npmcli:developers +` + +exports[`test/lib/team.js TAP team rm <scope:team> <user> --parseable > should output parseable result for remove user 1`] = ` +foo npmcli:newteam removed +` + +exports[`test/lib/team.js TAP team rm <scope:team> <user> default output > should output success result for remove user 1`] = ` +foo removed from @npmcli:newteam +` |