From 00854e37785858c0b187ee6f74f6ceeb7792f46d Mon Sep 17 00:00:00 2001
From: Jane Sandberg <sandbej@linnbenton.edu>
Date: Wed, 3 Jul 2019 09:46:04 -0700
Subject: [PATCH] LP1830432: Make sure that unit tests have an org unit
selected
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
.../org-family-select.component.spec.ts | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/Open-ILS/src/eg2/src/app/share/org-family-select/org-family-select.component.spec.ts b/Open-ILS/src/eg2/src/app/share/org-family-select/org-family-select.component.spec.ts
index 3e7117cb31..f93b386216 100644
--- a/Open-ILS/src/eg2/src/app/share/org-family-select/org-family-select.component.spec.ts
+++ b/Open-ILS/src/eg2/src/app/share/org-family-select/org-family-select.component.spec.ts
@@ -26,9 +26,8 @@ describe('Component: OrgFamilySelect', () => {
beforeEach(() => {
// stub of OrgService for testing
- // with a very simple org structure:
- // 1 is the root note
- // 2 is its child
+ // with a super simple org structure:
+ // 1 is the root note, with no children
orgServiceStub = {
root: () => {
return {
@@ -42,7 +41,7 @@ describe('Component: OrgFamilySelect', () => {
a: [],
classname: 'aou',
_isfieldmapper: true,
- children: () => Array(2 - ouId) };
+ children: () => Array() };
}
};
cookieServiceStub = {};
@@ -59,6 +58,7 @@ describe('Component: OrgFamilySelect', () => {
fixture = TestBed.createComponent(OrgFamilySelectComponent);
component = fixture.componentInstance;
component.domId = 'family-test';
+ component.selectedOrgId = 1;
fixture.detectChanges();
});
@@ -97,12 +97,19 @@ describe('Component: OrgFamilySelect', () => {
it('disables includeAncestors checkbox when root OU is chosen', () => {
fixture.whenStable().then(() => {
- component.selectedOrgId = 1;
fixture.detectChanges();
includeAncestors = fixture.debugElement.query(By.css('#family-test-include-ancestors'));
expect(includeAncestors.nativeElement.disabled).toBe(true);
});
});
+ it('disables includeAncestors checkbox when OU has no children', () => {
+ fixture.whenStable().then(() => {
+ fixture.detectChanges();
+ includeDescendants = fixture.debugElement.query(By.css('#family-test-include-descendants'));
+ expect(includeDescendants.nativeElement.disabled).toBe(true);
+ });
+ });
+
});
--
2.11.0