Skip to content
This repository was archived by the owner on Aug 1, 2025. It is now read-only.

Conversation

mkondratek
Copy link
Contributor

@mkondratek mkondratek commented Jul 25, 2025

Let's adjust e2e tests first before removing more code (https://github.com/sourcegraph/cody/pull/8189)

Test plan

  • green CI
  • manually tested (Cody should offer enterprise sign in only)

Copy link
Member

@thenamankumar thenamankumar left a comment

Choose a reason for hiding this comment

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

LGTM, on track with that I was removing as part of https://github.com/sourcegraph/cody/pull/8205

I have closed that one already for now. Lets continue with the cleanup in this.

@thenamankumar thenamankumar force-pushed the mk/chore/fix-e2e-tests branch from c066d16 to 4f6771c Compare July 29, 2025 10:59
@mkondratek mkondratek force-pushed the mk/chore/fix-e2e-tests branch 2 times, most recently from c726157 to d755c6e Compare July 30, 2025 12:20
testWithGitRemote,
} from './helpers'

testWithGitRemote('initial context - self-serve repo', async ({ page, sidebar }) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this one was dotcom specific

@mkondratek mkondratek force-pushed the mk/chore/fix-e2e-tests branch from d755c6e to 4e33045 Compare July 30, 2025 12:23
},
})

// Mock additional endpoints needed for enterprise context (from mockEnterpriseRepoMapping)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that one was not configured properly before.

@mkondratek mkondratek merged commit b23f67c into main Jul 30, 2025
21 of 22 checks passed
@mkondratek mkondratek deleted the mk/chore/fix-e2e-tests branch July 30, 2025 18:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants