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.

Sign up for GitHub

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

Jump to bottom

bottle: add tests for json merging #9365

Merged
merged 1 commit into from
Dec 7, 2020
Merged

bottle: add tests for json merging #9365

merged 1 commit into from
Dec 7, 2020

Commits on Dec 7, 2020

  1. bottle: add tests for json merging 

    - Extract the json reading from the json merging: it makes the code hard to understand.
This helps for the separation of concerns: the jsons are read in one method;
and merged in a second one
- Added a first test to check the merge function

No change in behaviour was done, this change is just there to increase code coverage
and to prepare for Homebrew#9315
    @iMichka
    iMichka committed Dec 7, 2020
    Configuration menu
    Copy the full SHA
    d768b66 View commit details
    Browse the repository at this point in the history