docs: Update test cases #155
Conversation
Other Columns tests assert the expected mark up. I think it is safe to presume that if the mark up is correct, the front-end website will render correctly in the browser.
dcalhoun
force-pushed
the
docs/update-test-cases
branch
from
ad259d1 to
fc67438
Compare
This was automated in the following: WordPress/gutenberg#38571
This was automated in the following: WordPress/gutenberg#38571
This was automated in the following: wordpress-mobile/gutenberg-mobile#5826
dcalhoun
force-pushed
the
docs/update-test-cases
branch
from
33777fa to
67e7668
Compare
Manually or automatically testing animations feels like a low return on effort required.
Manually or automatically testing animations feels like a low return on effort required.
|
@fluiddot removing the color settings animation test cases is aggressive. Feel free to push back if you disagree. |
There was a problem hiding this comment.
Testing animations — whether manually or automatically — seems like a low return of value for the effort required.
There was a problem hiding this comment.
I completely agree, I don't see them as a critical flow that needs to be tested.
|
|
||
| ##### TC010 | ||
|
|
||
| ### Columns percentage mechanism - custom percentage values are rendered correctly on frontend |
There was a problem hiding this comment.
Other automated Columns tests assert the expected mark up. I think it is safe to presume that if the mark up is correct, the front-end website will render correctly in the browser.
There was a problem hiding this comment.
Yep, as pointed out, we are already checking the HTML content related to the column's width (reference). So this test case is not really adding value to the test coverage 👍 .
There was a problem hiding this comment.
These test cases were automated in WordPress/gutenberg#38571.
| @@ -28,21 +28,6 @@ Use Simple WordPress.com site with a Premium plan. Free or Personal plans only a | |||
|
|
|||
| <img src="../../test-cases/resources/../resources/videopress-upload-video-4.png" width="360"> | |||
|
|
|||
| ##### TC002 | |||
There was a problem hiding this comment.
This test case was automated in wordpress-mobile/gutenberg-mobile#5826.
There was a problem hiding this comment.
I completely agree, I don't see them as a critical flow that needs to be tested.
|
|
||
| ##### TC010 | ||
|
|
||
| ### Columns percentage mechanism - custom percentage values are rendered correctly on frontend |
There was a problem hiding this comment.
Yep, as pointed out, we are already checking the HTML content related to the column's width (reference). So this test case is not really adding value to the test coverage 👍 .
| * Tap the block settings icon ⚙️ :gear: | ||
| * Tap `Link to` setting | ||
| * For each of the settings | ||
| * <img src="../../test-cases/resources/gallery-settings-link-to.png" width="360"> |
There was a problem hiding this comment.
This image is only referenced here so we can also remove it.
Update test cases to note known issues and remove automated or low-value test cases.