Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

Fixes lint errors introduced during the Jasmine to vitest migration that blocked CI builds.

Changes

Parsing Errors

  • Removed malformed empty property syntax in mock objects from tree-selection.service.spec.ts and tree-selection.spec.ts
  • Jasmine's createSpyObj('tree', ['']) with empty method array incorrectly migrated to invalid object literal syntax

Before (invalid):

mockTree = {
    : vi.fn().mockName("tree."),
    selection: IgxTreeSelectionType.BiState
};

After:

mockTree = {
    selection: IgxTreeSelectionType.BiState
};

Auto-fixable Style Violations

  • Applied npm run lint --fix to resolve 117 brace-style violations across 47 test files

Impact

  • Lint errors: 119 → 0
  • Remaining warnings (148) are non-blocking and pre-existing

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: ChronosSF <2188411+ChronosSF@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor tests by moving to Vitest fix(lint): resolve parsing errors and style violations from vitest migration Jan 21, 2026
Copilot AI requested a review from ChronosSF January 21, 2026 14:17
@ChronosSF ChronosSF marked this pull request as ready for review January 21, 2026 14:50
@ChronosSF ChronosSF merged commit b080e7e into sstoychev/migrate-vitest Jan 21, 2026
@ChronosSF ChronosSF deleted the copilot/sub-pr-16781 branch January 21, 2026 14:51
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