Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 2, 2025

Expands test coverage for the NES (Name Element Symbol) rename validator from 11 to 43 test cases.

New test scenarios

  • Inheritance: Deep class hierarchies, interface chains, multiple inheritance (extends + implements)
  • Member types: Properties, parameters, accessors, arrow function params
  • Visibility modifiers: Static, private, protected members
  • Scoping: Nested functions, module exports, outer scope conflicts
  • Type constructs: Optional properties, readonly members, const enums

Test format

Tests use annotation comments in the fixture file:

class ChildClass extends ParentClass {
    //// { "title": "Deep inheritance - no rename to grandparent", "oldName": "childMethod", "newName": "grandMethod", "expected": "no" }
    childMethod(): void { }
}
Original prompt

Can you write more tests for me

The user has attached the following file paths as relevant context:

  • src\extension\typescriptContext\serverPlugin\src\node\test\nes.spec.ts
  • src\extension\typescriptContext\serverPlugin\fixtures\nes\p1\source\test.ts
  • .github\instructions\vitest-unit-tests.instructions.md
  • .github\copilot-instructions.md

Created from VS Code.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 2, 2025 09:52
Co-authored-by: dbaeumer <1931590+dbaeumer@users.noreply.github.com>
Co-authored-by: dbaeumer <1931590+dbaeumer@users.noreply.github.com>
Copilot AI changed the title [WIP] Add more tests for Nes functionality Add comprehensive NES rename validation tests Dec 2, 2025
Copilot AI requested a review from dbaeumer December 2, 2025 09:55
Copilot finished work on behalf of dbaeumer December 2, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants