Remove redundant void function return value test

[Copilot]

Addresses review feedback on PR #214 regarding test case that unnecessarily captured and tested the return value of a void function.

Changes

• Removed test case7 in TextFieldSelectAllUseCase.test.ts that captured execute() return value
• Renumbered remaining test cases (8→7, 9→8, 10→9)

The execute function is typed to return void, making the assertion expect(result).toBeUndefined() redundant. Function behavior is already covered by existing tests that verify side effects on the TextField object.

// Before - unnecessary pattern
const result = execute(textField);
expect(result).toBeUndefined();

// After - removed (other tests cover behavior)

---

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

[Copilot]

Pull Request Overview

This pull request removes an unnecessary test case and cleans up package-lock.json metadata. The changes streamline test case numbering and remove redundant peer: true flags from package entries.

Key changes:

• Removed test case 7 which tested for undefined return value (redundant test)
• Renumbered subsequent test cases (8→7, 9→8, 10→9) for consistency
• Removed peer: true flags from multiple package entries in package-lock.json

Reviewed Changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/text/src/TextField/usecase/TextFieldSelectAllUseCase.test.ts	Removed redundant test case and renumbered remaining tests
package-lock.json	Cleaned up peer: true flags from package metadata entries

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.