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

refactor: Remove hard-coded URLs from BDD tests #1422

Merged
merged 1 commit into from
Aug 12, 2022

Conversation

bstasyszyn
Copy link
Contributor

@bstasyszyn bstasyszyn commented Aug 11, 2022

Refactor BDD tests to remove hard-coded URLs for local host mappings, anchor origins. Also use variables in the feature files instead of repeating the entire URL many times.

closes #1420

Signed-off-by: Bob Stasyszyn Bob.Stasyszyn@securekey.com

Refactor BDD tests to remove hard-coded URLs for local host mappings, anchor origins. Also use variables in the feature files instead of repeating the entire URL multiple times.

closes trustbloc#1420

Signed-off-by: Bob Stasyszyn <Bob.Stasyszyn@securekey.com>
@cla-bot cla-bot bot added the cla-signed label Aug 11, 2022
@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #1422 (78c53f5) into main (f56e6ee) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1422      +/-   ##
==========================================
+ Coverage   87.95%   88.00%   +0.05%     
==========================================
  Files         206      206              
  Lines       19221    19227       +6     
==========================================
+ Hits        16905    16920      +15     
+ Misses       1484     1477       -7     
+ Partials      832      830       -2     
Impacted Files Coverage Δ
pkg/vcsigner/signer.go 87.62% <0.00%> (-1.85%) ⬇️
pkg/observer/pubsub.go 94.49% <0.00%> (ø)
...nkset/generator/didorbgenerator/didorbgenerator.go 78.62% <0.00%> (+0.60%) ⬆️
pkg/vct/proofmonitoring/monitoring.go 86.72% <0.00%> (+2.65%) ⬆️
...pub/service/inbox/httpsubscriber/httpsubscriber.go 97.77% <0.00%> (+8.88%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bstasyszyn bstasyszyn merged commit adb3687 into trustbloc:main Aug 12, 2022
@bstasyszyn bstasyszyn deleted the 1420 branch August 12, 2022 13:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor BDD tests to remove hard-coded URLs
2 participants