DSingh0304 commented on PR #3286:
URL: 
https://github.com/apache/apisix-dashboard/pull/3286#issuecomment-3789133394

   > Hi @DSingh0304, could you add a test to fix this issue?
   
   The fix is defensive:
   
   - It prevents crashes if corrupted data somehow reaches the frontend
   - The existing e2e test routes.crud all fields.spec.ts already tests the 
vars field with valid data.
   - Since APISIX API validates data strictly, invalid data can't be created 
via the API for e2e testing.
   - The project doesn't have unit test infrastructure for component-level 
testing.
   
   However, if you'd like me to add a test, I can create an e2e test that mocks 
the API response to simulate corrupted data reaching the frontend. This would 
demonstrate that the components handle invalid data gracefully, though it would 
deviate from the existing test pattern where all tests use real API calls for 
creation and deletion.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to