Conversation
✅ Deploy Preview for rstest-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR adds documentation about the priority order for timeout configuration in Rstest. The documentation clarifies how the testing framework determines which timeout value to use when multiple sources specify timeout values.
- Adds priority order documentation for both test timeouts and hook timeouts
- Documents the 5-level priority hierarchy from most specific (per-test configuration) to least specific (default value)
- Provides bilingual documentation in both English and Chinese
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| website/docs/en/config/test/test-timeout.mdx | Adds English documentation for test timeout priority order with 5 levels of configuration hierarchy |
| website/docs/en/config/test/hook-timeout.mdx | Adds English documentation for hook timeout priority order with 5 levels of configuration hierarchy |
| website/docs/zh/config/test/test-timeout.mdx | Adds Chinese documentation for test timeout priority order with complete markdown links |
| website/docs/zh/config/test/hook-timeout.mdx | Adds Chinese documentation for hook timeout priority order with complete markdown links |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Rstest determines the test timeout based on the following priority order: | ||
|
|
||
| 1. [timeout](/api/test-api/test#test-1) configuration in each test case | ||
| 2. Timeout configured in the test file via [`rstest.setConfig({ testTimeout })`] |
There was a problem hiding this comment.
The markdown link is incomplete and missing the URL reference. The link text is present but the URL in parentheses is missing. This should follow the same pattern as line 30 and include the full link like: (/api/rstest/utilities#rstestsetconfig)
| 2. Timeout configured in the test file via [`rstest.setConfig({ testTimeout })`] | |
| 2. Timeout configured in the test file via [`rstest.setConfig({ testTimeout })`](/api/rstest/utilities#rstestsetconfig) |
| Rstest determines the hook timeout based on the following priority order: | ||
|
|
||
| 1. [timeout](/api/test-api/hooks) configuration in each test case | ||
| 2. Timeout configured in the test file via [`rstest.setConfig({ hookTimeout })`] |
There was a problem hiding this comment.
The markdown link is incomplete and missing the URL reference. The link text is present but the URL in parentheses is missing. This should follow the same pattern as line 30 and include the full link like: (/api/rstest/utilities#rstestsetconfig)
| 2. Timeout configured in the test file via [`rstest.setConfig({ hookTimeout })`] | |
| 2. Timeout configured in the test file via [`rstest.setConfig({ hookTimeout })`](/api/rstest/utilities#rstestsetconfig) |
Summary
docs: add timeout priority order
Related Links
Checklist