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

respond with 409 when settlement engine id is dupe on new account #416 #432

Merged
merged 3 commits into from
Nov 15, 2019

Conversation

theotherian
Copy link
Collaborator

Signed-off-by: Ian Simpson ian.g.simpson@gmail.com

Signed-off-by: Ian Simpson <ian.g.simpson@gmail.com>
@codecov
Copy link

codecov bot commented Nov 15, 2019

Codecov Report

Merging #432 into master will increase coverage by <.01%.
The diff coverage is 12.5%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #432      +/-   ##
============================================
+ Coverage     61.71%   61.72%   +<.01%     
- Complexity      826      827       +1     
============================================
  Files           247      248       +1     
  Lines          4676     4690      +14     
  Branches        180      182       +2     
============================================
+ Hits           2886     2895       +9     
- Misses         1718     1722       +4     
- Partials         72       73       +1
Impacted Files Coverage Δ Complexity Δ
...s/AccountSettlementEngineAlreadyExistsProblem.java 0% <0%> (ø) 0 <0> (?)
...dger/connector/accounts/DefaultAccountManager.java 15.15% <16.66%> (+15.15%) 2 <1> (+2) ⬆️
...server/spring/settings/link/IlpOverHttpConfig.java 95% <0%> (-5%) 6% <0%> (-1%)

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 f642e1b...db7e1f9. Read the comment docs.

…ctor into is/respond-with-409-on-dupe-settlement

Signed-off-by: Ian Simpson <ian.g.simpson@gmail.com>

# Conflicts:
#	connector-server/src/test/java/org/interledger/connector/server/spring/controllers/admin/AccountSettingsSpringBootTest.java
Signed-off-by: Ian Simpson <ian.g.simpson@gmail.com>
Copy link
Collaborator

@sappenin sappenin left a comment

Choose a reason for hiding this comment

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

LGTM!

@theotherian theotherian merged commit 6e84a02 into master Nov 15, 2019
@theotherian theotherian deleted the is/respond-with-409-on-dupe-settlement branch November 15, 2019 20:13
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.

None yet

3 participants