ExternalVendorCode/openmct
1
0
Fork 0
mirror of https://github.com/nasa/openmct.git synced 2025-04-09 04:14:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #758 from nasa/error-message-753

Browse Source 
[Tree] Don't assume context capability is present
This commit is contained in:
Andrew Henry 2016-03-16 15:33:59 -07:00
commit b0cf9bbd29
2 changed files with 18 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
platform/commonUI/general/src/ui/TreeNodeView.js
View File

@ -105,13 +105,13 @@ define([
};
function getIdPath(domainObject) {
var context = domainObject && domainObject.getCapability('context');
function getId(domainObject) {
return domainObject.getId();
}
return domainObject ?
domainObject.getCapability('context').getPath().map(getId) :
[];
return context ? context.getPath().map(getId) : [];
}
TreeNodeView.prototype.value = function (domainObject) {

15
platform/commonUI/general/test/ui/TreeViewSpec.js
View File

@ -206,6 +206,21 @@ define([
});
});
describe("when a context-less object is selected", function () {
beforeEach(function () {
var testCapabilities = makeGenericCapabilities(),
mockDomainObject =
makeMockDomainObject('xyz', {}, testCapabilities);
delete testCapabilities.context;
treeView.value(mockDomainObject);
});
it("clears all selection state", function () {
var selected = $(treeView.elements()[0]).find('.selected');
expect(selected.length).toEqual(0);
});
});
describe("when children contain children", function () {
beforeEach(function () {
var newCapabilities = makeGenericCapabilities(),