Skip to content

test(e2e): parse a 98-page fixture once and clone it for every test - #120

Merged
Rl0007 merged 20 commits into
mainfrom
feat/e2e-parse-once
Oct 7, 2026
Merged

Rl0007 merged 20 commits into
mainfrom
feat/e2e-parse-once

Conversation

@Rl0007

@Rl0007 Rl0007 commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor
  1. Every e2e run parsed the fixture several times (seed plus fresh imports in smoke, publish and tree), and the 6-page fixture was too small to show real-document problems.
  2. The suite now parses a 98-page CC BY OpenStax chapter once per run and gives each test an API clone of it; tests read section titles from the seeded outline, and the tests for bugs fixed this week are normal tests again.
  3. The CI worker now starts without CI: frappe runs every job in test mode when CI is set, which turned Local models off for parse, remediation and the assistant.
Verification

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@Rl0007
Rl0007 merged commit 29e6491 into main Oct 7, 2026
11 checks passed
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.

1 participant