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

Enable UI tests for new recorder #3

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Enable UI tests for new recorder #3

wants to merge 3 commits into from

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Oct 22, 2023

Enable two new UI tests.

@uhafner uhafner marked this pull request as draft October 22, 2023 20:17
@uhafner uhafner added the tests Enhancement of tests label Oct 22, 2023
@codecov
Copy link

codecov bot commented Oct 22, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@8cbba23). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage        ?   77.15%           
  Complexity      ?      722           
=======================================
  Files           ?       47           
  Lines           ?     2569           
  Branches        ?      248           
=======================================
  Hits            ?     1982           
  Misses          ?      508           
  Partials        ?       79           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner changed the title Refactor UI tests for new recorder Enable UI tests for new recorder Oct 24, 2023
private Control findRepeatableAddButtonFor(final String propertyName) {
return control(by.xpath("//div[@id='" + propertyName + "']//button[contains(@path,'-add')]"));
}

CoveragePublisher setTool(final String tool, final String pattern) {

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'tool' is never used.
rekathiru pushed a commit to intuit-build/coverage-plugin that referenced this pull request Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Enhancement of tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant