feat: #970 add tests for adding forest client number

[NickSaglioni]

• Adding tests for the ForestClientInput component;
• Adding tests for the ForestClientCard component;
• Refactoring LoadingState to have getter and setter for the loading state;

[ianliuwk1019]

Hi @NickSaglioni , could you also add a minor update on frontend readme about what types of tests we currently do, what tools we use, and how to run test locally? Thank you.

[NickSaglioni]

Hi @NickSaglioni , could you also add a minor update on frontend readme about what types of tests we currently do, what tools we use, and how to run test locally? Thank you.

Sure, I'll do that.

[ianliuwk1019]

Hi Nick, I am not sure what this comment means, maybe its left over from the past.
Could you check and cleanup if not making sense.
Thank you.

[ianliuwk1019]

Hi Nick, a bit confusing on this part, should be simple, could you check?
Is this logic unnecessary? The previous expect already make sure it has a emitted property, right?

[ianliuwk1019]

Hi @NickSaglioni , I have add some comments, hope not too much for you to tackle .
Since this is probably the first frontend test file, there are many tests to be done here on this component. I appreciate you spent time on this to set some common examples for later test writing.

[MCatherine1994]

Looks pretty good overall, just some small things left, almost there! thanks so much Nick and Ian!!!

[sonarcloud]

Quality Gate passed for 'nr-forests-access-management_admin'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud