Skip to content

Commit 28cffd1

Browse files
authored
docs: Update contributing guide for E2E tests (#21763)
Removed some Verdaccio references - we don't use it no more
1 parent b72a360 commit 28cffd1

1 file changed

Lines changed: 3 additions & 17 deletions

File tree

‎CONTRIBUTING.md‎

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,11 @@ Note: you must run `yarn build` before `yarn test` will work.
7575

7676
## Running E2E Tests Locally
7777

78-
E2E tests verify SDK behavior in real-world framework scenarios using a local npm registry (Verdaccio).
78+
E2E tests verify SDK behavior in real-world framework scenarios.
7979

8080
### Prerequisites
8181

82-
1. **Docker**: Required to run the Verdaccio registry container
82+
1. **Docker**: Required for some E2E tests
8383
2. **Volta with pnpm support**: Enable pnpm in Volta by setting `VOLTA_FEATURE_PNPM=1` in your environment. See [Volta pnpm docs](https://docs.volta.sh/advanced/pnpm).
8484

8585
### Step-by-Step Instructions
@@ -122,17 +122,6 @@ E2E tests verify SDK behavior in real-world framework scenarios using a local np
122122

123123
### Common Issues and Troubleshooting
124124

125-
#### Packages install from public npm instead of Verdaccio
126-
127-
Every E2E test application **must** have an `.npmrc` file with:
128-
129-
```
130-
@sentry:registry=http://127.0.0.1:4873
131-
@sentry-internal:registry=http://127.0.0.1:4873
132-
```
133-
134-
Without this, pnpm will fetch packages from the public npm registry instead of the local Verdaccio instance, causing tests to use outdated/published versions instead of your local changes.
135-
136125
#### Tests fail after making SDK changes
137126

138127
Make sure to rebuild tarballs:
@@ -144,15 +133,12 @@ yarn build:tarball
144133

145134
#### Docker-related issues
146135

147-
- Ensure Docker daemon is running
148-
- Check that port 4873 is not in use by another process
149-
- Try stopping and removing existing Verdaccio containers
136+
- Ensure Docker daemon is running, some E2E tests require Docker
150137

151138
#### Debugging test failures
152139

153140
1. Check browser console logs for SDK initialization errors
154141
2. Enable debug mode in the test app's Sentry config: `debug: true`
155-
3. Verify packages are installed from Verdaccio by checking the version in `node_modules/@sentry/*/package.json`
156142

157143
For more details, see [dev-packages/e2e-tests/README.md](dev-packages/e2e-tests/README.md).
158144

0 commit comments

Comments
 (0)