Skip to content

Commit b6d970b

Browse files
Merge pull request #2638 from saucelabs/visual-statuses-update
visual statuses update, fix broken link
2 parents a89f583 + 378143b commit b6d970b

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

docs/visual-testing/workflows/review.md

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -128,10 +128,10 @@ More information about reviewing and approving diffs can be found in [Subsequent
128128

129129
Every subsequent test execution will generate a new "Visual Build" and will be marked as ["For Review"](#visual-statuses). This is because:
130130

131-
- A new baseline snapshot could be generated for the new test execution, see also [Reviewing and Approving Baselines](#reviewing-and-approving-baselines) and [Accept Only New](#accept-only-new).
131+
- A new baseline snapshot could be generated for the new test execution, see also [Reviewing and Accepting baselines](#reviewing-and-accepting-baselines) and [Accept Only New](#accept-only-new).
132132
- You need to review the visual differences between the new snapshot and the previously accepted baseline snapshot.
133133

134-
The process of reviewing and approving the visual differences is the same as described in [Reviewing and Approving Baselines](#reviewing-and-approving-baselines).
134+
The process of reviewing and approving the visual differences is the same as described in [Reviewing and Accepting baselines](#reviewing-and-accepting-baselines).
135135

136136
The user interface including the actions you can take are described in [User Interface](#user-interface).
137137

@@ -178,10 +178,12 @@ The list below shows the available actions and their keyboard shortcuts.
178178

179179
Visual uses different statuses:
180180

181-
| Status | Description |
182-
| ----------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
183-
| In Progress | These builds have not been finished yet. They transition into one of the following states as soon as they finish. |
184-
| No Changes | The snapshots assigned to your build matched your baseline. It is considered a success because your assertions were successful. |
185-
| For Review | There were either no baselines available to compare against your uploaded snapshot or some were different from their baselines. You are supposed to review those detected diffs. As long as those changes aren't accepted, they are considered a failure state. |
186-
| Accepted | All detected changes were accepted. This is considered a success state. |
187-
| Rejected | Some of your detected changes were rejected. This is considered a failure state. |
181+
| Status | Description |
182+
|-----------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
183+
| Accepted | All detected changes were accepted. This is considered a success state. |
184+
| No Changes | The snapshots assigned to your build matched your baseline. It is considered a success because your assertions were successful. |
185+
| Running | These builds have not been finished yet. |
186+
| Queued | Some of your diffs are waiting to be generated. |
187+
| For Review | There were either no baselines available to compare against your uploaded snapshot or some were different from their baselines. You are supposed to review those detected diffs. As long as those changes aren't accepted, they are considered a failure state. |
188+
| Errored | One or more errors occurred. More information is available in the "Build Details" page. |
189+
| Rejected | Some of your detected changes were rejected. This is considered a failure state. |

docs/visual-testing/workflows/test-execution.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ Alternatively, [check out our examples](../../visual-testing.md#examples).
4747

4848
Upon executing your tests for the first time, a visual baseline is automatically created in our system. This baseline serves as the standard for all subsequent tests and matches based on the metadata as described in the [Baseline Matching](../../visual-testing.md#baseline-matching).
4949

50-
The newly created baseline needs to be reviewed and approved by a user. This can be done in the [Sauce Visual Builds](https://app.saucelabs.com/visual/builds) page. Once approved, see [Reviewing and Approving baselines](./review.md#reviewing-and-approving-baselines), the baseline is "locked" and will be used for all subsequent tests.
50+
The newly created baseline needs to be reviewed and approved by a user. This can be done in the [Sauce Visual Builds](https://app.saucelabs.com/visual/builds) page. Once approved, see [Reviewing and Accepting baselines](./review.md#reviewing-and-accepting-baselines), the baseline is "locked" and will be used for all subsequent tests.
5151

5252
## Subsequent Test Executions
5353

0 commit comments

Comments
 (0)