[ACS-6211] Fixed unit tests

Alfresco · Dec 5, 2023 · d95fb31 · d95fb31
1 parent 0994ae7
commit d95fb31
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/projects/aca-content/folder-rules/src/rule-details/actions/rule-action.ui-component.spec.ts b/projects/aca-content/folder-rules/src/rule-details/actions/rule-action.ui-component.spec.ts
@@ -74,11 +74,10 @@ describe('RuleActionUiComponent', () => {
     component.parameterConstraints = dummyConstraints;
     fixture.detectChanges();
 
-    const cardView = getPropertiesCardView();
-    expect(cardView.properties.length).toBe(0);
-
     changeMatSelectValue('mock-action-1-definition');
 
+    const cardView = getPropertiesCardView();
+
     expect(cardView.properties.length).toBe(5);
     expect(cardView.properties[0]).toBeInstanceOf(CardViewTextItemModel);
     expect(cardView.properties[1]).toBeInstanceOf(CardViewBoolItemModel);
@@ -87,19 +86,19 @@ describe('RuleActionUiComponent', () => {
     expect(cardView.properties[4]).toBeInstanceOf(CardViewSelectItemModel);
 
     changeMatSelectValue('mock-action-2-definition');
-    expect(cardView.properties.length).toBe(0);
+    expect(getPropertiesCardView()).toBeNull();
   });
 
   it('should create category-value action parameter as a text box rather than node picker', () => {
     component.actionDefinitions = [actionLinkToCategoryTransformedMock];
     component.parameterConstraints = dummyConstraints;
     fixture.detectChanges();
 
-    const cardView = getPropertiesCardView();
-    expect(cardView.properties.length).toBe(0);
-
     changeMatSelectValue('mock-action-3-definition');
 
+    const cardView = getPropertiesCardView();
+
+    expect(cardView.properties.length).toBe(1);
     expect(cardView.properties[0].icon).toBeFalsy();
     expect(cardView.properties[0].value).toBeFalsy();
     expect(cardView.properties[0]).toBeInstanceOf(CardViewTextItemModel);