Test: File first modified in iteration 2#11
Open
pedropaulovc wants to merge 3 commits into
Open
Conversation
This sets up the base state for testing that files first modified in iteration 2 are correctly marked as Modified (not Added). See file-first-modified-later-iteration/README.md for details.
pedropaulovc
force-pushed
the
test/file-modified-in-iteration-2
branch
from
40ba9af to
7df6938
Compare
There was a problem hiding this comment.
Pull request overview
This PR sets up the first iteration of a test scenario that validates correct file status tracking when files are modified in later iterations. The test verifies that files existing in the PR base but first modified in iteration 2 (not iteration 1) are correctly marked as "Modified" rather than "Added".
Changes:
- Modified
other-file.jsto update existing function and add a new helper function in iteration 1 - Sets up test data structure for E2E testing of iteration file status tracking
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pedropaulovc
added a commit
to vezzadev/codjiflo
that referenced
this pull request
Jan 17, 2026
Previously, these tests referenced PR #97 in pedropaulovc/codjiflo, but the artifact for that PR has expired (90-day retention). Changes: - Created new test repository: pedropaulovc/codjiflo-e2e-test-repo - Created standalone action repository: pedropaulovc/codjiflo-action - Created test PR #11 with proper iteration scenario: - Iteration 1: Modified other-file.js only - Iteration 2: Modified target-file.yml (exists in base, first change in PR) - Updated iteration-file-status.spec.ts to use PR #11 - Updated iteration-tabs-ui.spec.ts to use PR #11 - Updated file references from action/action.yml to target-file.yml Test Results: All 34 E2E prod tests passing consistently Artifact: vezzadev/codjiflo-e2e-test-repo#11 Action: https://github.com/pedropaulovc/codjiflo-action Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
pedropaulovc
added a commit
to vezzadev/codjiflo
that referenced
this pull request
Jan 17, 2026
Previously, these tests referenced PR #97 in pedropaulovc/codjiflo, but the artifact for that PR has expired (90-day retention). Changes: - Created new test repository: pedropaulovc/codjiflo-e2e-test-repo - Created standalone action repository: pedropaulovc/codjiflo-action - Created test PR #11 with proper iteration scenario: - Iteration 1: Modified other-file.js only - Iteration 2: Modified target-file.yml (exists in base, first change in PR) - Updated iteration-file-status.spec.ts to use PR #11 - Updated iteration-tabs-ui.spec.ts to use PR #11 - Updated file references from action/action.yml to target-file.yml Test Results: All 34 E2E prod tests passing consistently Artifact: vezzadev/codjiflo-e2e-test-repo#11 Action: https://github.com/pedropaulovc/codjiflo-action Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
pedropaulovc
force-pushed
the
test/file-modified-in-iteration-2
branch
from
7df6938 to
656505e
Compare
pedropaulovc
force-pushed
the
test/file-modified-in-iteration-2
branch
from
7df6938 to
656505e
Compare
CodjiFlo Iteration TrackingIterations captured: 2 What is this?This comment is automatically updated by the CodjiFlo GitHub Action to enable force-push resilient code review with iteration tracking. The artifact referenced above contains iteration data that the CodjiFlo frontend uses to:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Test Scenario
Iteration 1 (current): Modifies
other-file.jsonlyIteration 2 (coming): Will modify
target-file.ymlfor the first timeExpected Behavior
When viewing iteration 2,
target-file.ymlshould show as:This tests the base equivalence fix for iteration tracking.
See:
file-first-modified-later-iteration/README.md🔍 Review in CodjiFlo