Skip to content

test(e2e): verify full repository Git history - #1087

Draft
saurya wants to merge 5 commits into
mainfrom
test/ohe-member-full-git-history
Draft

saurya wants to merge 5 commits into
mainfrom
test/ohe-member-full-git-history

Conversation

@saurya

@saurya saurya commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Enable the authenticated OHE member's full Git history setting from App settings.
  • Launch a configured repository conversation and verify git rev-list --count HEAD meets the configured multi-commit minimum.
  • Restore the original disabled toggle after deleting the conversation.
  • Use selectors and routes checked against OpenHands/enterprise@d603dc3b4e85b249638154c0016c5f082a76989f and OpenHands/OpenHands@2e1502f39d8f7357fca35c6c18cc2c0dadcf0da3.

Environment contract

  • BASE_URL: required non-production OHE target.
  • e2e_tests/fixtures/auth.json: authenticated OHE member storage state with full Git history initially disabled.
  • TEST_FULL_HISTORY_REPOSITORY: readable repository with multiple commits.
  • TEST_FULL_HISTORY_PROVIDER: configured provider (github, gitlab, azure_devops, bitbucket, or bitbucket_data_center).
  • TEST_FULL_HISTORY_MIN_COMMITS: integer minimum expected from the repository, at least 2.
  • A working LLM profile capable of running the deterministic git rev-list prompt.

Missing configuration throws actionable errors; the test does not skip and does not print credentials.

Cleanup

A finally block deletes the created app conversation and restores the original git_full_clone setting through the authenticated settings API, even after assertion failure. The source repository is never modified.

Validation

  • npm ci was completed once for the shared harness dependency tree.
  • npm run lint -- tests/member-full-git-history.spec.ts
  • BASE_URL=https://release-under-test.invalid npx playwright test tests/member-full-git-history.spec.ts --project=chromium --list
  • One added spec file, one top-level test, one commit parented directly to current main.

AI disclosure: This pull request was created by an AI agent (OpenHands) on behalf of the user.

Co-authored-by: openhands <openhands@all-hands.dev>
@github-actions github-actions Bot added the type: test Adding or fixing tests label Aug 12, 2026
tofarr and others added 4 commits August 14, 2026 16:10
Co-authored-by: openhands <openhands@all-hands.dev>
Run this spec with the stable returning-user fixture supplied by the current Playwright project topology.

Co-authored-by: openhands <openhands@all-hands.dev>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: test Adding or fixing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants