Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Test Scenario]: Carousel - (Default, with images, with videos, with Media) #8330

Closed
proeung opened this issue Feb 17, 2022 · 1 comment
Closed
Assignees
Labels
adopter: AEM used when component or pattern will be used by this adopter dev Needs some dev work DPO Consulting owner: Innovation Team used when the engineering work will be done by Hybrid Cloud with DDS engineers as consultants package: web components Work necessary for the IBM.com Library web components package severity 3 Affects minor functionality, has a workaround test: e2e used for automated e2e testing

Comments

@proeung
Copy link
Contributor

proeung commented Feb 17, 2022

Engineering info:


Component and test scenario title

Carousel - (Default, with images, with videos, and with mixture of images and videos)

Test scenario steps

Carousel - Default

  1. Check the carousel cards and ensure that the content is loaded and the whole card clickable.
  2. Ensure that the Next and Previous buttons advance the carousel.
  3. Check the content (heading, copy, and arrow link) on the carousel

Carousel - With Images

  1. Ensure that the images are rendered correctly.
  2. Ensure that the body text has the same alignment.

Carousel - With Videoes

  1. Ensure that the video thumbs and play buttons are rendered correctly.
  2. Ensure that the body text has the same alignment.
  3. Ensure that the duration text value is visible in the CTA area.

Carousel - With a mixture of images and videos (New SB Variant)

  1. Ensure that the body text has the same alignment. (see [Carousel]: mixture of dds-card and dds-card-cta causing heading alignment issue - DPO Consulting #8247)
@proeung proeung added dev Needs some dev work test: e2e used for automated e2e testing package: web components Work necessary for the IBM.com Library web components package adopter: AEM used when component or pattern will be used by this adopter follow-up: innovation team owner: Innovation Team used when the engineering work will be done by Hybrid Cloud with DDS engineers as consultants and removed package: web components Work necessary for the IBM.com Library web components package labels Feb 17, 2022
@proeung proeung changed the title [Test Scenario]: Carousel - (Default, with images, with videos, with mixture of images and videos) [Test Scenario]: Carousel - (Default, with images, with videos, with Media) Feb 21, 2022
@proeung proeung added the severity 4 Affects minor functionality, no workaround needed label Feb 21, 2022
@proeung proeung added severity 3 Affects minor functionality, has a workaround and removed severity 4 Affects minor functionality, no workaround needed labels Mar 4, 2022
@proeung
Copy link
Contributor Author

proeung commented Mar 4, 2022

Adding sev 3 to this e2e ticket as we want to make sure we have test coverage for the Carousel component which is being used on multiple templates within AEM.

kodiakhq bot pushed a commit that referenced this issue Mar 28, 2022
…ideos, with media (#8538)

### Related Ticket(s)

[[Test Scenario]: Carousel - (Default, with images, with videos, with Media)](#8330)

### Description

The original requirements can be found in the ticket. I erred a little on the verbose side, and gave each variant a full suite of tests applicable to its unique content. If I should pare back and if any are redundant, let me know.

#### Test scenario steps

Note: Accessibility and theme compliance tests were applied to all.

**Carousel - Default**

1. Check the content (heading, copy, and arrow link) are appearing on the carousel cards
2. Check the headers to make sure they all have the same height and are not set to 0px tall, providing an even vertical height for all the copy text
3. Check that the whole card is clickable
4. Ensure that the `Next` and `Previous` buttons move the carousel

**Carousel - With Images**

1. Check the content (heading, copy, and arrow link) are appearing on the carousel cards
2. Ensure that the images are rendered
2. Check the headers to make sure they all have the same height and are not set to 0px tall, providing an even vertical height for all the copy text
4. Check that the whole card is clickable
5. Ensure that the `Next` and `Previous` buttons move the carousel

**Carousel - With Videoes**

1. Check the content (heading, copy, and arrow link) are appearing on the carousel cards
2. Ensure that the video thumbs and play buttons are rendered correctly
3. Ensure that the duration text value is visible in the CTA area
2. Check the headers to make sure they all have the same height and are not set to 0px tall, providing an even vertical height for all the copy text
4. Check that the whole card is clickable
5. Ensure that the `Next` and `Previous` buttons move the carousel

**Carousel - With a mixture of images and videos (New SB Variant)**

1. Check the content (heading, copy, and arrow link) are appearing on the carousel cards
2. Ensure that the images are rendered
3. Ensure that the video thumbs and play buttons are rendered correctly
4. Ensure that the duration text value is visible in the CTA area
2. Check the headers to make sure they all have the same height and are not set to 0px tall, providing an even vertical height for all the copy text
5. Check that the whole card is clickable
6. Ensure that the `Next` and `Previous` buttons move the carousel

### Changelog

**New**

- Test for the Carousel component

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
@proeung proeung closed this as completed Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adopter: AEM used when component or pattern will be used by this adopter dev Needs some dev work DPO Consulting owner: Innovation Team used when the engineering work will be done by Hybrid Cloud with DDS engineers as consultants package: web components Work necessary for the IBM.com Library web components package severity 3 Affects minor functionality, has a workaround test: e2e used for automated e2e testing
Projects
None yet
Development

No branches or pull requests

4 participants