Skip to content
New issue

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

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

Already on GitHub? # to your account

[engine] Add ability to load aliases from resources #2492

Merged
merged 1 commit into from
Feb 18, 2022

Conversation

uarlouski
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Feb 16, 2022

Codecov Report

Merging #2492 (e26d7e6) into master (79cbea6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2492   +/-   ##
=========================================
  Coverage     96.59%   96.59%           
- Complexity     5474     5475    +1     
=========================================
  Files           774      774           
  Lines         15748    15750    +2     
  Branches       1043     1043           
=========================================
+ Hits          15211    15213    +2     
  Misses          414      414           
  Partials        123      123           
Impacted Files Coverage Δ
...java/org/vividus/spring/ExtendedConfiguration.java 89.09% <100.00%> (+0.41%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79cbea6...e26d7e6. Read the comment docs.

@uarlouski uarlouski force-pushed the jbehave-5-0-0-alpha-25 branch from 37ba59f to 0e9f3aa Compare February 16, 2022 17:30
@uarlouski uarlouski force-pushed the jbehave-5-0-0-alpha-25 branch from 0e9f3aa to b62d3a1 Compare February 16, 2022 17:32
Scenario: Verify aliases loaded from resources

!-- Vividus step
Then `1` matches `\d+`
Copy link
Member

Choose a reason for hiding this comment

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

is it possible to use a different language as an alias? f

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Does it make sense to create an issue for this? @valfirst @uarlouski

Copy link
Member Author

Choose a reason for hiding this comment

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

@uarlouski uarlouski force-pushed the jbehave-5-0-0-alpha-25 branch from 7fb6a38 to e26d7e6 Compare February 18, 2022 07:14
@sonarqubecloud
Copy link

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 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@valfirst valfirst merged commit c106d31 into master Feb 18, 2022
@valfirst valfirst deleted the jbehave-5-0-0-alpha-25 branch February 18, 2022 17:02
VitaliaPiliuhina pushed a commit to VitaliaPiliuhina/vividus that referenced this pull request Apr 8, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants