You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+3-5Lines changed: 3 additions & 5 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -127,9 +127,9 @@ All pull requests must include tests to maintain 100% coverage. Coverage configu
127
127
128
128
Integration tests are contained in the [`tests/integration`](tests/integration) directory and test actual interactions with GitHub's API and webhooks.
129
129
130
-
These tests are skipped by default and must be explicitly enabled by passing `--integration` as a pytest argument. Running them requires both a GitHub App and a Personal Access Token (PAT) configured in your environment.
130
+
These tests are skipped by default and must be explicitly enabled by passing `--integration` as a pytest argument. Running them requires a GitHub App in your user account and configured in your environment.
131
131
132
-
Follow these steps to set up your environment for integration tests:
132
+
Follow these steps to get set up for integration tests:
133
133
134
134
1. Create a test GitHub App:
135
135
- Go to GitHub Developer Settings > GitHub Apps > New GitHub App
@@ -148,7 +148,6 @@ Follow these steps to set up your environment for integration tests:
148
148
3. Install the new test GitHub App on your user account by selecting "Install App" from the GitHub App admin panel.
149
149
- After installation, make note of the unique ID in the URL, e.g. `https://github.com/settings/installations/<unique ID>`
150
150
5. Configure the following environment variables:
151
-
152
151
-`TEST_ACCOUNT_NAME` - your GitHub username
153
152
-`TEST_ACCOUNT_TYPE` - user
154
153
-`TEST_APP_ID` - the App ID of the GitHub App from step 2
@@ -174,8 +173,7 @@ After setup, you can run the test suite with the integration tests enabled by pa
0 commit comments