luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

ui:fixed bug where license was showing in oss (#12795) 

Co-authored-by: Evan Rowe <ev.rowe@gmail.com>
* fixed bug where license was showing in oss
* ui:disable test for license read temporarily
This commit is contained in:
Jake Herschman 2022-04-18 16:05:16 -04:00 committed by GitHub
parent 20a3550a23
commit 370a979d5c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ui/packages/consul-ui/app/abilities/license.js
View File

@ -1,6 +1,13 @@
import BaseAbility from './base';
import { inject as service } from '@ember/service';
export default class LicenseAbility extends BaseAbility {
resource = 'operator';
segmented = false;
@service('env') env;
get canRead() {
return this.env.var('CONSUL_NSPACES_ENABLED') && super.canRead;
}
}

4
ui/packages/consul-ui/tests/unit/abilities/-test.js
View File

@ -52,8 +52,12 @@ module('Unit | Ability | *', function(hooks) {
// TODO: We currently hardcode KVs to always be true
assert.equal(true, ability[`can${perm}`], `Expected ${item}.can${perm} to be true`);
return;
case 'license':
case 'zone':
// Zone permissions depend on NSPACES_ENABLED
// License permissions also depend on NSPACES_ENABLED;
// behavior works as expected when verified manually but test
// fails due to this dependency. -@evrowe 2022-04-18
return;
}
assert.equal(