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

Browser console logs #3092

Merged
merged 3 commits into from
Aug 24, 2022
Merged

Browser console logs #3092

merged 3 commits into from
Aug 24, 2022

Conversation

ikalinin1
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 22, 2022

Codecov Report

Merging #3092 (8eb10fc) into master (91b3687) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3092      +/-   ##
============================================
- Coverage     89.36%   89.35%   -0.01%     
  Complexity     6112     6112              
============================================
  Files           825      825              
  Lines         16721    16717       -4     
  Branches       1082     1081       -1     
============================================
- Hits          14942    14938       -4     
  Misses         1648     1648              
  Partials        131      131              
Impacted Files Coverage Δ
...rg/vividus/selenium/logging/BrowserLogManager.java 100.00% <100.00%> (ø)
...va/org/vividus/steps/ui/web/JsValidationSteps.java 100.00% <100.00%> (ø)

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


=== Browser logs steps

This set of steps allows you to validate the https://developer.mozilla.org/en-US/docs/Web/API/console[browser's console logging messages].
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
This set of steps allows you to validate the https://developer.mozilla.org/en-US/docs/Web/API/console[browser's console logging messages].
This set of steps allows to validate the https://developer.mozilla.org/en-US/docs/Web/API/console[browser console logging messages].


This set of steps allows you to validate the https://developer.mozilla.org/en-US/docs/Web/API/console[browser's console logging messages].

:log-levels: List of the comma-separated mesagges' levels. The supported levels are: ERRORS, WARNINGS.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
:log-levels: List of the comma-separated mesagges' levels. The supported levels are: ERRORS, WARNINGS.
:log-levels: List of the comma-separated messages levels. The supported levels are: ERRORS, WARNINGS.


==== Validate log entries absence

Step validates the absence of log entries of desired level in the browser's console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Step validates the absence of log entries of desired level in the browser's console.
Validates the absence of log entries of the desired level in the browser console.


==== Validate specific log entries absence

Step validates absence of specific log entries of desired level in the browser's console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Step validates absence of specific log entries of desired level in the browser's console.
Validates the absence of specific log entries of the desired level in the browser console.

Then there are no browser console $logLevels by regex '$pattern'
----
* `$logLevels` - {log-levels}
* `$pattern` - The pattern to match log entry message.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* `$pattern` - The pattern to match log entry message.
* `$pattern` - The regular expression to match log entry messages.


==== Validate specific log entries presence

Step validates the presence of specific log entries of desired level in the browser's console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Step validates the presence of specific log entries of desired level in the browser's console.
Validates the presence of specific log entries of the desired level in the browser console.

Then there are browser console $logLevels by regex '$pattern'
----
* `$logLevels` - {log-levels}
* `$pattern` - The pattern to match log entry message.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* `$pattern` - The pattern to match log entry message.
* `$pattern` - The regular expression to match log entry messages.

@sonarcloud
Copy link

sonarcloud bot commented Aug 24, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@valfirst valfirst merged commit 352eb91 into master Aug 24, 2022
@valfirst valfirst deleted the simplify branch August 24, 2022 08:03
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.

3 participants