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

chore(tooltip): Added tests for gux-tooltip-title component #471

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

jason-evans-genesys
Copy link
Collaborator

@jason-evans-genesys jason-evans-genesys commented May 21, 2024

✅ Closes: COMUI-1408

Let me know if there are any other tests you think I should write for gux-truncate and gux-tooltip-title.

@jason-evans-genesys jason-evans-genesys self-assigned this May 21, 2024
Copy link

@321gillian
Copy link
Collaborator

Any reason you're going for just e2e tests alone here?

@jason-evans-genesys
Copy link
Collaborator Author

Any reason you're going for just e2e tests alone here?

It looks like some of the tests to test for tooltip visibility wouldn't work in a spec file without breaking the gux-tooltip-text component at runtime. I saw that sometimes components in the spark repo don't have a spec file and rely solely on a e2e test file. I thought perhaps this could be one of those components where write all tests within an e2e test file made sense.

@jason-evans-genesys jason-evans-genesys changed the title chore(tooltip): Added tests for gux-tooltip-title component (WIP) chore(tooltip): Added tests for gux-tooltip-title component May 23, 2024
@@ -86,7 +86,10 @@ export class GuxTruncate {
}}
>
<span
class="gux-truncate-slot-container"
class={{
'gux-overflow-hidden': this.needsTruncation(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this is backwards compatible.

✅ Closes: COMUI-1408

chore(truncate): Added tests for checking if tooltip is visible or not

✅ Closes: COMUI-1408

chore(tooltip): Added visibility tests for gux-tooltip within a gux-tooltip-text tag

✅ Closes: COMUI-1408

chore(truncate): Removed unneeded page is defined expect statement in tests

✅ Closes: COMUI-1408

chore(truncate): Added more tests

✅ Closes: COMUI-1408

chore(truncate): Removed console log statement

✅ Closes: COMUI-1408

chore(truncate): Added truncation text check test

✅ Closes: COMUI-1408

chore(tooltip): Added test for truncation for tooltip-title component

✅ Closes: COMUI-1408

chore(tooltip): Added more tests for tooltip-title component

✅ Closes: COMUI-1408

chore(tooltip): Fixed some snapshot tests for tooltip-title

✅ Closes: COMUI-1408
@jason-evans-genesys jason-evans-genesys merged commit 715a54f into main Jun 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants