eschutho opened a new pull request, #35329: URL: https://github.com/apache/superset/pull/35329
### SUMMARY Added comprehensive testing structure guidelines to LLMS.md that explain why and how to use `test()` instead of `describe()` and `it()`, following the "avoid nesting when testing" principle championed by Kent C. Dodds. This guideline promotes: - Better test isolation - each test stands alone - Improved readability - flat structure with descriptive test names - Easier test execution - simpler to run individual tests - Reduced cognitive overhead - less nested context switching ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF N/A - Documentation only change ### TESTING INSTRUCTIONS 1. Review the updated LLMS.md file 2. Verify the guidelines are clear and comprehensive 3. Check that the example provided effectively illustrates the pattern ### ADDITIONAL INFORMATION - [ ] Has associated issue: No - [ ] Required feature flags: No - [ ] Changes UI: No - [ ] Includes DB Migration: No - [ ] Introduces new feature or API: No - [ ] Removes existing feature or API: No This aligns with the ongoing testing strategy migration mentioned in the document and provides concrete guidance for developers writing tests in the Superset codebase. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
