WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Commit ea0addc

Browse files
shawn-hurleyclaude
andcommitted
Fix documentation coverage inconsistencies
Updated test documentation to accurately reflect current test coverage: - Updated test count from 9 to 18 functions - Updated location type coverage from 8 to 15 (100%) - Updated query count from 15 to 40+ unique queries - Fixed project name: customers-tomcat → customers-tomcat-legacy - Added missing test functions to summary tables (TestDefaultSearch, TestEnumConstantSearch, TestReturnTypeSearch, TestVariableDeclarationSearch, TestPackageDeclarationSearch, TestFieldDeclarationSearch, TestMethodDeclarationSearch, TestAnnotatedElementMatching, TestFilePathFiltering) Addresses CodeRabbit documentation review comments. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 3f92bd6 commit ea0addc

File tree

2 files changed

+16
-5
lines changed

2 files changed

+16
-5
lines changed

java-analyzer-bundle.test/docs/README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ Complete integration test documentation:
4141
- Architecture (JDT.LS server, LSP client, test framework)
4242
- Component details (Go LSP client, test framework, helpers)
4343
- Running tests (locally, containers, CI/CD)
44-
- Test scenarios for all 8 tested location types
44+
- Test scenarios for all 15 location types (100% coverage)
4545
- Test assertions and verification
4646
- Troubleshooting guide
4747
- Adding new tests
@@ -139,7 +139,7 @@ Detailed overview of the Java test projects:
139139

140140
---
141141

142-
### Phase 2: Go Integration Tests (16 functions)
142+
### Phase 2: Go Integration Tests (18 functions)
143143
**Location**: `../integration/`
144144
**Framework**: Go test with real JDT.LS server
145145
**Coverage**: Actual search results, symbol verification, migration patterns
@@ -161,6 +161,8 @@ Detailed overview of the Java test projects:
161161
- `TestMethodDeclarationSearch` (location 13)
162162
- `TestClassDeclarationSearch` (location 14)
163163
- `TestCustomersTomcatLegacy` (migration patterns)
164+
- `TestAnnotatedElementMatching` (Priority 1: annotation attributes)
165+
- `TestFilePathFiltering` (Priority 1: file path filtering)
164166

165167
**Run**: `make phase2` or `cd ../integration && go test -v`
166168

java-analyzer-bundle.test/docs/query-reference.md

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ This document provides a comprehensive overview of all search queries tested by
44

55
## Quick Reference
66

7-
**Total Tests**: 9 test functions covering 8 location types
7+
**Total Tests**: 18 test functions covering 15 location types (100% coverage)
88
**Test Projects**:
99
- `test-project`: Systematic patterns for all location types
10-
- `customers-tomcat`: Real-world Spring MVC application with migration targets
10+
- `customers-tomcat-legacy`: Real-world Spring MVC application with migration targets
1111

1212
---
1313

@@ -422,17 +422,26 @@ Some location types accept simple names:
422422

423423
| Test Function | Location Types | Queries | Assertions |
424424
|---------------|----------------|---------|------------|
425+
| `TestDefaultSearch` | 0 | 3 | Cross-location search verification |
425426
| `TestInheritanceSearch` | 1 | 3 | Verify specific classes found |
426427
| `TestMethodCallSearch` | 2 | 2 | Verify call sites and count |
427428
| `TestConstructorCallSearch` | 3 | 2 | Verify instantiation count |
428429
| `TestAnnotationSearch` | 4 | 1 | Verify annotation on target |
429430
| `TestImplementsTypeSearch` | 5 | 1 | Verify implementing class |
431+
| `TestEnumConstantSearch` | 6 | 2 | Verify enum constant usage |
432+
| `TestReturnTypeSearch` | 7 | 3 | Verify method return types |
430433
| `TestImportSearch` | 8 | 1 | Verify import statements |
434+
| `TestVariableDeclarationSearch` | 9 | 3 | Verify local variable declarations |
431435
| `TestTypeSearch` | 10 | 1 | Verify type reference count |
436+
| `TestPackageDeclarationSearch` | 11 | 2 | Verify package declarations |
437+
| `TestFieldDeclarationSearch` | 12 | 3 | Verify field declarations |
438+
| `TestMethodDeclarationSearch` | 13 | 4 | Verify method declarations |
432439
| `TestClassDeclarationSearch` | 14 | 1 | Verify class declaration |
433440
| `TestCustomersTomcatLegacy` | 4, 8 | 3 | Migration pattern verification |
441+
| `TestAnnotatedElementMatching` | 4 | 4 | Annotation attributes matching |
442+
| `TestFilePathFiltering` | 9 | 2 | File path filtering with includedPaths |
434443

435-
**Total Query Executions**: 15 unique search queries across 9 test functions
444+
**Total Query Executions**: 40+ unique search queries across 18 test functions covering all 15 location types
436445

437446
---
438447

0 commit comments

Comments
 (0)