Fix: Update E2E iteration tests to use valid PR with fresh artifacts#249
Merged
pedropaulovc merged 1 commit intoJan 17, 2026
Merged
Conversation
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>
|
🔗 Azure DevOps Mirror: PR #83 |
pedropaulovc
temporarily deployed
to
azdo-mirror
GitHub Actions
Inactive
CodjiFlo Iteration TrackingIterations captured: 1 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:
|
Coverage Report
File CoverageNo changed files found. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes failing E2E iteration tests by updating test references from an expired PR (#97) with 90-day old artifacts to a fresh test PR (#11) in a new dedicated E2E test repository. The changes update test configuration and file references to work with the new test infrastructure.
Changes:
- Updated E2E test references from PR #97 to PR #11 in the new
codjiflo-e2e-test-repo - Changed file references from
action/action.ymltotarget-file.ymlto match the new test data structure
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
e2e/prod/iteration-mode/iteration-tabs-ui.spec.ts |
Updated test configuration to use PR #11 in the new E2E test repository |
e2e/prod/iteration-mode/iteration-file-status.spec.ts |
Updated test configuration and all file references from action/action.yml to target-file.yml to match the new test PR structure |
Preview Deployment🚀 Preview: https://pr-249.codjiflo.vza.net/pedropaulovc/codjiflo/249 Updated for commit d724686 |
pedropaulovc
deleted the
fix/update-e2e-iteration-tests-to-use-valid-pr
branch
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.
Problem
The E2E iteration tests in prod mode were failing because they referenced PR #97, which has an expired artifact (90-day GitHub artifact retention).
Solution
Created a dedicated test infrastructure with fresh, valid iteration data:
New Test Infrastructure
Action Repository:
E2E Test Repository:
file-first-modified-later-iteration/directoryTest PR: pedropaulovc/codjiflo-e2e-test-repo#11
other-file.jsonlytarget-file.yml(exists in base, first change in PR)Changes
e2e/prod/iteration-mode/iteration-file-status.spec.tsaction/action.ymltotarget-file.ymle2e/prod/iteration-mode/iteration-tabs-ui.spec.tsTest Results
✅ All 34 E2E prod tests passing consistently
Documentation
Test scenario documentation: file-first-modified-later-iteration/README.md
🤖 Generated with Claude Code
🔍 Review in CodjiFlo