Angular testing using Jasmine and Karma

Question

I am trying to improve code coverage of Angular app. In code coverage it is mentioned that if else condition is not covered. Could anyone tell me how to do that? Feel free to ask for more code details.

public searchByText(textVal: any): void {
        let matchedEquipments = [];
        // **
        if (this.model.searchText.length > 1) {
            matchedEquipments = this.refineByText(textVal, this.equipments);
        } else {
            matchedEquipments = this.equipments;
        }
        // **
        matchedEquipments = this.refineByPlant(this.model.plants, matchedEquipments);
        matchedEquipments = this.refineByPlantIsland(this.model.plantIslands, matchedEquipments);
        matchedEquipments = this.refineByProcess(this.model.processes, matchedEquipments);
        matchedEquipments = this.refineByIndustry(this.model.divisions, matchedEquipments);
        this.displayClientData(matchedEquipments);
        this.updateSearchCounters(SelectionFilter.FreeText);
}

Spec:

it('verify the result with search', async(() => {
        equipmentSelectionComponent.searchByText("123");
        //equipmentSelectionComponent.model.searchText = "123";
        expect(equipmentSelectionComponent.matchedData.length).toBeGreaterThan(0);
}));

No, I mean that's how you can test the other case. Look at the condition, think about the possible routes through your logic. — jonrsharpe
– jonrsharpe, Commented Aug 18, 2020 at 15:48
can i write one case with condition success and one for else? — Suruchi Babbar
– Suruchi Babbar, Commented Aug 18, 2020 at 15:54

Shashank Vivek · Accepted Answer · 2020-08-26 12:48:15Z

You need write more than just 1 it block to test this:

it('should call "refineByText" when search Text is there', async(() => {
   spyOn(equipmentSelectionComponent,'refineByText').and.callThrough();
   equipmentSelectionComponent.model.searchText = "some Val";
   equipmentSelectionComponent.equipments = ["item1"]
   equipmentSelectionComponent.searchByText("123");
   expect(equipmentSelectionComponent.refineByText).toHaveBeenCalledWith("123",["item1"]); // <-- this will check true condition
   // I dont see "matchedData" in your provided code so I dont know about this check.
   expect(equipmentSelectionComponent.matchedData.length).toBeGreaterThan(0);
   // spy and check other function calls as well just like I did for refineByText
}));

it('should not call "refineByText" when search Text is empty', async(() => {
   spyOn(equipmentSelectionComponent,'refineByText').and.callThrough();
   equipmentSelectionComponent.model.searchText = "";
   equipmentSelectionComponent.searchByText("123");
   expect(equipmentSelectionComponent.refineByText).not.toHaveBeenCalled(); // <-- this will check true condition
   // similarly use use "toHaveBeenCalledWith(val) for other functions
}));

I wouold recommend you to read testing with spies and also some more basic testing ways of angular. This would help you to get more familiar with unit testing mindset. Feel free to clap as well !! :)

Rajat · Accepted Answer · 2020-08-20 01:00:58Z

0

Below test cases should be able to include the coverage for your test case

if condition : should assign to matchedEquipments by modifying the equipments when searchText has value

Expectation :
```
 refineByText toHaveBeenCalledWith(textVal, this.equipments)
```
else condition : should assign equipment to matchedEquipments when searchText has no value

Expectation :
```
 refineByText not.toHaveBeenCalledWith(textVal, this.equipments)
```

answered Aug 20, 2020 at 1:00

Rajat

4371 gold badge6 silver badges17 bronze badges

Collectives™ on Stack Overflow

Angular testing using Jasmine and Karma

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related