[PR #103] [MERGED] Test refactor and reverify #261

New issue

Closed

opened 2026-03-02 04:13:07 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:13:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/git-ai-project/git-ai/pull/103
Author: @acunniffe
Created: 10/13/2025
Status: ✅ Merged
Merged: 10/13/2025
Merged by: @acunniffe

Base: main ← Head: feat/test-refactor

📝 Commits (6)

b2eaec2 helpers
1dd3f2c migrate simple additions
7382e37 cherry pick refactored
045569b test refactor
4dfce4f switch to random dep for windows
d3b209e comment out bad test

📊 Changes

16 files changed (+2390 additions, -3656 deletions)

View changed files

📝 Cargo.lock (+60 -3)
📝 Cargo.toml (+1 -0)
📝 tests/blame_flags.rs (+160 -353)
📝 tests/cherry_pick.rs (+177 -361)
📝 tests/merge_rebase.rs (+141 -98)
📝 tests/rebase.rs (+598 -1588)
➕ tests/repos/snapshots/blame_flags__repos__test_file__assert_blame_snapshot.snap (+5 -0)
➕ tests/repos/snapshots/cherry_pick__repos__test_file__assert_blame_snapshot.snap (+5 -0)
➕ tests/repos/snapshots/merge_rebase__repos__test_file__assert_blame_snapshot.snap (+5 -0)
➕ tests/repos/snapshots/rebase__repos__test_file__assert_blame_snapshot.snap (+5 -0)
➕ tests/repos/snapshots/reset__repos__test_file__assert_blame_snapshot.snap (+5 -0)
➕ tests/repos/snapshots/simple_additions__repos__test_file__assert_blame_snapshot.snap (+5 -0)
📝 tests/repos/test_file.rs (+322 -0)
📝 tests/repos/test_repo.rs (+128 -38)
📝 tests/reset.rs (+392 -534)
📝 tests/simple_additions.rs (+381 -681)

📄 Description

Rewrote test rig to make it easier to understand which assertions to make.

Migrated the biggest history rewriting test suites and manually verified useful assertions

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/git-ai-project/git-ai/pull/103 **Author:** [@acunniffe](https://github.com/acunniffe) **Created:** 10/13/2025 **Status:** ✅ Merged **Merged:** 10/13/2025 **Merged by:** [@acunniffe](https://github.com/acunniffe) **Base:** `main` ← **Head:** `feat/test-refactor` --- ### 📝 Commits (6) - [`b2eaec2`](https://github.com/git-ai-project/git-ai/commit/b2eaec2bcd813261c5b9524920597870c32894ef) helpers - [`1dd3f2c`](https://github.com/git-ai-project/git-ai/commit/1dd3f2cb494db6014e951461bad086f8f591a7f2) migrate simple additions - [`7382e37`](https://github.com/git-ai-project/git-ai/commit/7382e378e84b4221ed660bcb2a085d0da90fe075) cherry pick refactored - [`045569b`](https://github.com/git-ai-project/git-ai/commit/045569bbc4c2dc31f4dcfc3005d4da462223d441) test refactor - [`4dfce4f`](https://github.com/git-ai-project/git-ai/commit/4dfce4fca74588094c6e4ef4143d302b819ba7e8) switch to random dep for windows - [`d3b209e`](https://github.com/git-ai-project/git-ai/commit/d3b209eae686605c7d8966d49b4dce0fa759f308) comment out bad test ### 📊 Changes **16 files changed** (+2390 additions, -3656 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+60 -3) 📝 `Cargo.toml` (+1 -0) 📝 `tests/blame_flags.rs` (+160 -353) 📝 `tests/cherry_pick.rs` (+177 -361) 📝 `tests/merge_rebase.rs` (+141 -98) 📝 `tests/rebase.rs` (+598 -1588) ➕ `tests/repos/snapshots/blame_flags__repos__test_file__assert_blame_snapshot.snap` (+5 -0) ➕ `tests/repos/snapshots/cherry_pick__repos__test_file__assert_blame_snapshot.snap` (+5 -0) ➕ `tests/repos/snapshots/merge_rebase__repos__test_file__assert_blame_snapshot.snap` (+5 -0) ➕ `tests/repos/snapshots/rebase__repos__test_file__assert_blame_snapshot.snap` (+5 -0) ➕ `tests/repos/snapshots/reset__repos__test_file__assert_blame_snapshot.snap` (+5 -0) ➕ `tests/repos/snapshots/simple_additions__repos__test_file__assert_blame_snapshot.snap` (+5 -0) 📝 `tests/repos/test_file.rs` (+322 -0) 📝 `tests/repos/test_repo.rs` (+128 -38) 📝 `tests/reset.rs` (+392 -534) 📝 `tests/simple_additions.rs` (+381 -681) </details> ### 📄 Description Rewrote test rig to make it easier to understand which assertions to make. Migrated the biggest history rewriting test suites and manually verified useful assertions --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>