test: add unit test for ChaosHub output mapping - #5549
Conversation
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
There was a problem hiding this comment.
Pull request overview
Adds a new unit test file for the ChaosHub ChaosFaults view to validate some of the rendered fault details and ensure generated fault links include the chartName query parameter, addressing part of the mapping behavior described in #5535.
Changes:
- Introduces
ChaosFaults.test.tsxcovering basic fault name/description rendering. - Adds an assertion that the fault card link URL contains
chartName=<chart metadata.name>.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hey @Ju-Min-Kyung |
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
Yes, I've pushed an update that adds multi-chart mapping coverage and tag-based filtering tests. |
|
@Ju-Min-Kyung could you please fix the CI failures? |
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
d2c56e6 to
ea10d0f
Compare
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
It looks like Codacy is analyzing stale code. The latest commit (61fc9ee) already contains the fixes, but Codacy is still reporting the previous 17 issues. Could you please refresh the analysis? |
Signed-off-by: Ju-Min-Kyung <alsrud7767@naver.com>
0eaaff9 to
5c3a790
Compare
Proposed changes
Fixes #5535
Added unit tests for the ChaosFaults component
chartNameparameterTypes of changes
What types of changes does your code introduce to Litmus? Put an
xin the boxes that applyChecklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.