Skip to content

test: migrate tests from vite-plus#101

Merged
branchseer merged 2 commits intomainfrom
01-13-test_migrate_tests_from_vite-plus
Jan 14, 2026
Merged

test: migrate tests from vite-plus#101
branchseer merged 2 commits intomainfrom
01-13-test_migrate_tests_from_vite-plus

Conversation

@branchseer
Copy link
Member

@branchseer branchseer commented Jan 13, 2026

  • Migrate tests from vite-plus
  • Add inline shell comments (# explanation) to all e2e test steps so
    they appear in generated snapshots for better readability
  • Fix shared-caching-inputs: correct comments to reflect actual cache
    behavior (script2 gets cache hit from script1's identical command)
  • Fix vite-task-smoke: correct comments (main.js IS an input file)
  • Fix individual-cache-for-envs: add vite.config.json with envs config
    so different env values actually get separate cache entries
  • Fix colon-in-name: rename test from "task with underscore in name"
    to "read file with colon in name" and delete stale snapshot
  • Add file header comments explaining test purpose

Copy link
Member Author

branchseer commented Jan 13, 2026

@branchseer branchseer marked this pull request as ready for review January 13, 2026 15:12
@branchseer branchseer force-pushed the 01-13-test_migrate_tests_from_vite-plus branch from 76761da to 391d270 Compare January 13, 2026 15:16
@branchseer branchseer changed the title migration tests from vite-plus test: migrate tests from vite-plus Jan 13, 2026
@branchseer branchseer force-pushed the 01-13-test_migrate_tests_from_vite-plus branch from 391d270 to 9fbff13 Compare January 13, 2026 15:35
Copy link
Member Author

branchseer commented Jan 14, 2026

Merge activity

  • Jan 14, 3:53 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 14, 3:54 AM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 14, 3:55 AM UTC: @branchseer merged this pull request with Graphite.

@branchseer branchseer changed the base branch from 01-13-fix_plan_add_shell_arguments_for_shell_fallback_execution to graphite-base/101 January 14, 2026 03:53
@branchseer branchseer changed the base branch from graphite-base/101 to main January 14, 2026 03:53
branchseer and others added 2 commits January 14, 2026 03:54
- Add inline shell comments (# explanation) to all e2e test steps so
  they appear in generated snapshots for better readability
- Fix shared-caching-inputs: correct comments to reflect actual cache
  behavior (script2 gets cache hit from script1's identical command)
- Fix vite-task-smoke: correct comments (main.js IS an input file)
- Fix individual-cache-for-envs: add vite.config.json with envs config
  so different env values actually get separate cache entries
- Fix colon-in-name: rename test from "task with underscore in name"
  to "read file with colon in name" and delete stale snapshot
- Add file header comments explaining test purpose

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@branchseer branchseer force-pushed the 01-13-test_migrate_tests_from_vite-plus branch from 9fbff13 to bbab48f Compare January 14, 2026 03:54
@branchseer branchseer merged commit e164719 into main Jan 14, 2026
6 checks passed
@branchseer branchseer deleted the 01-13-test_migrate_tests_from_vite-plus branch January 14, 2026 03: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