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

Added support for UUID property and ability to delete/update stub by UUID #87

Merged
merged 1 commit into from
Apr 11, 2018

Conversation

azagniotov
Copy link
Owner

@azagniotov azagniotov commented Jan 22, 2018

  • Added support for optional uuid property in YAML config
    • Added ability to GET stub by UUID by making a request to Admin portal API
    • Added ability to DELETE stub by UUID by making a request to Admin portal API
    • Added ability to UPDATE stub by UUID by making a request to Admin portal API
  • Displaying UUID value in popup dialogs in Admin console when [view] YAML source button is clicked
  • Revisited HTTP response codes of Admin portal APIs that manage stubs (delete, get & update APIs)
  • Upgraded from Jetty 9.4.8.v20171121 to 9.4.9.v20180320
  • Using ehCache v3.5.2 for internal caching
  • Moved from XMLUnit v1.x to XMLUnit v2.x for a more optimized XML evaluation
  • Upon parsing of YAML config, Compiling regex patterns using different flag combinations:
    • Pattern.MULTILINE
    • Pattern.MULTILINE | Pattern.DOTALL
    • Pattern.LITERAL (as a fallback if any of the aforementioned compilations failed)

@azagniotov azagniotov mentioned this pull request Jan 22, 2018
@codecov-io
Copy link

codecov-io commented Jan 22, 2018

Codecov Report

Merging #87 into master will increase coverage by 0.19%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
+ Coverage      99%   99.19%   +0.19%     
==========================================
  Files          35       33       -2     
  Lines        1100      994     -106     
  Branches      180      169      -11     
==========================================
- Hits         1089      986     -103     
+ Misses          4        3       -1     
+ Partials        7        5       -2
Impacted Files Coverage Δ
...ithub/azagniotov/stubby4j/utils/DateTimeUtils.java 100% <100%> (ø)
...j/handlers/strategy/admin/GetHandlingStrategy.java 95.83% <100%> (+1.38%) ⬆️
.../github/azagniotov/stubby4j/stubs/StubMatcher.java 100% <100%> (+1.03%) ⬆️
...b/azagniotov/stubby4j/stubs/StubHttpLifecycle.java 97.5% <100%> (+0.16%) ⬆️
.../github/azagniotov/stubby4j/utils/StringUtils.java 97.4% <100%> (+0.14%) ⬆️
...j/handlers/strategy/admin/PutHandlingStrategy.java 100% <100%> (ø) ⬆️
.../handlers/strategy/admin/PostHandlingStrategy.java 100% <100%> (ø) ⬆️
...niotov/stubby4j/yaml/ConfigurableYAMLProperty.java 100% <100%> (ø) ⬆️
...andlers/strategy/admin/DeleteHandlingStrategy.java 100% <100%> (ø) ⬆️
...io/github/azagniotov/stubby4j/yaml/YamlParser.java 100% <100%> (ø)
... and 2 more

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 1bb5eb7...e010437. Read the comment docs.

@azagniotov azagniotov changed the title Added support for uuid property in YAML config WiP - Added support for uuid property in YAML config Mar 19, 2018
@azagniotov azagniotov force-pushed the adding_uuid_to_stubs branch 2 times, most recently from c0296f7 to 9cc54d0 Compare March 26, 2018 00:22
@azagniotov azagniotov changed the title WiP - Added support for uuid property in YAML config Added support for UUID property and ability to delete/update stub by UUID Mar 26, 2018
@azagniotov azagniotov self-assigned this Mar 26, 2018
@azagniotov azagniotov force-pushed the adding_uuid_to_stubs branch 20 times, most recently from 6e828a1 to dc6be0c Compare April 1, 2018 10:02
@azagniotov azagniotov force-pushed the adding_uuid_to_stubs branch 3 times, most recently from fdc1c4d to 7c6b326 Compare April 2, 2018 02:08
@azagniotov azagniotov force-pushed the adding_uuid_to_stubs branch 2 times, most recently from d670656 to a8c1074 Compare April 3, 2018 17:52
@azagniotov azagniotov force-pushed the adding_uuid_to_stubs branch 2 times, most recently from b8079e4 to 6968b1a Compare April 11, 2018 00:36
@azagniotov azagniotov merged commit 10b16e2 into master Apr 11, 2018
@azagniotov azagniotov deleted the adding_uuid_to_stubs branch April 11, 2018 06:37
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

2 participants