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

Add callsite and selector expression to the overlapped warning message #7386

Closed
AndreyBelym opened this issue Nov 17, 2022 · 1 comment · Fixed by #7399
Closed

Add callsite and selector expression to the overlapped warning message #7386

AndreyBelym opened this issue Nov 17, 2022 · 1 comment · Fixed by #7399
Assignees
Labels
TYPE: enhancement The accepted proposal for future implementation.

Comments

@AndreyBelym
Copy link
Contributor

No description provided.

@AndreyBelym AndreyBelym added the TYPE: enhancement The accepted proposal for future implementation. label Nov 17, 2022
@Aleksey28 Aleksey28 self-assigned this Nov 30, 2022
AndreyBelym pushed a commit that referenced this issue Dec 2, 2022
…7399)

<!--
Thank you for your contribution.

Before making a PR, please read our contributing guidelines at

https://github.com/DevExpress/testcafe/blob/master/CONTRIBUTING.md#code-contribution

We recommend creating a *draft* PR, so that you can mark it as 'ready
for review' when you are done.
-->

[closes #7386 ]

## Purpose
Add `callsite` to the warning with the overlapping element.

## Approach
1. Add `callsite` to warning
2. Fix test


![image](https://user-images.githubusercontent.com/32869530/204814857-cba9819f-12a6-4817-aafd-cf7226718c3b.png)


## References
#7386

## Pre-Merge TODO
- [ ] Write tests for your proposed changes
- [ ] Make sure that existing tests do not fail
@github-actions
Copy link

github-actions bot commented Dec 6, 2022

Release v2.1.1-alpha.2 addresses this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TYPE: enhancement The accepted proposal for future implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants