Skip to content

test: migrate progressEventReducer test to vitest#10741

Merged
jasonsaayman merged 2 commits intoaxios:v1.xfrom
shaanmajid:test/migrate-progress-event-reducer
Apr 17, 2026
Merged

test: migrate progressEventReducer test to vitest#10741
jasonsaayman merged 2 commits intoaxios:v1.xfrom
shaanmajid:test/migrate-progress-event-reducer

Conversation

@shaanmajid
Copy link
Copy Markdown
Contributor

@shaanmajid shaanmajid commented Apr 16, 2026

Remove last stray Mochatest/ file -> vitest tests/; removes old test/ dir.


Summary by cubic

Migrated the last mocha test to vitest and removed the old test/ directory to standardize the test runner. This completes the move to vitest.

Description

  • Switched to vitest (describe, it, expect) from Node assert.
  • Renamed/moved test to tests/unit/helpers/progressEventReducer.test.js; removed legacy test/ directory.
  • Synced with v1.x; no logic changes.
  • Update docs to reference vitest only; remove mocha mentions.

Testing

  • Modified existing unit test; assertions equivalent and coverage unchanged.
  • No new tests needed.

Written for commit e155b1c. Summary will update on new commits.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@jasonsaayman jasonsaayman self-requested a review as a code owner April 17, 2026 08:22
@jasonsaayman jasonsaayman merged commit 9ed4f58 into axios:v1.x Apr 17, 2026
22 checks passed
@shaanmajid shaanmajid deleted the test/migrate-progress-event-reducer branch April 17, 2026 19:50
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