Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Oct 14, 2025

This commit adds transformers for Jasmine's spying functionality.

Coverage includes:

  • spyOn, spyOnProperty, jasmine.createSpy, and jasmine.createSpyObj
  • Spy strategies (and.returnValue, and.callFake, etc.) are mapped to Vitest's 'mock*' equivalents.
  • Inspection of spy calls (spy.calls.reset, spy.calls.mostRecent).

@clydin clydin added the target: major This PR is targeted for the next major release label Oct 14, 2025
…mine-to-vitest schematic

This commit adds transformers for Jasmine's spying functionality (https://jasmine.github.io/tutorials/spying_on_properties).

Coverage includes:
- spyOn, spyOnProperty, jasmine.createSpy, and jasmine.createSpyObj
- Spy strategies (and.returnValue, and.callFake, etc.) are mapped to Vitest's 'mock*' equivalents.
- Inspection of spy calls (spy.calls.reset, spy.calls.mostRecent).
@clydin clydin force-pushed the refactor/jasmine-vitest-3 branch from 562d061 to 953f921 Compare October 14, 2025 11:48
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 14, 2025
@clydin clydin requested a review from alan-agius4 October 14, 2025 13:14
@alan-agius4 alan-agius4 added action: global presubmit The PR is in need of a google3 global presubmit action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer action: global presubmit The PR is in need of a google3 global presubmit labels Oct 14, 2025
@clydin clydin merged commit d0d2a17 into angular:main Oct 14, 2025
34 checks passed
@clydin clydin deleted the refactor/jasmine-vitest-3 branch October 14, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @schematics/angular detected: feature PR contains a feature commit target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants