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

Rework XStream initialization #352

Merged
merged 4 commits into from
Jan 25, 2021
Merged

Rework XStream initialization #352

merged 4 commits into from
Jan 25, 2021

Commits on Jan 21, 2021

  1. fix: synchronize XStream initialization 

    Due to the tests now running in parallel, we sometimes had
ConcurrentModificationExceptions since the global XStream instance was
initialized in parallel from different threads.

Now all initialization points are locked.
    @xeruf
    xeruf committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    dd02991 View commit details
    Browse the repository at this point in the history

  2. fix(plugin): move xstream initialization synchronization to tests

    @xeruf
    xeruf committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    ae28278 View commit details
    Browse the repository at this point in the history

  3. rework(XStream): replace global XStream instance by XStreamProvider

    @xeruf
    xeruf committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    5931cd7 View commit details
    Browse the repository at this point in the history

  4. test(sdk/testConfig): initialize test XStream instance via ProjectLis… 

    …tener

Thus we save the synchronization overhead.
    @xeruf
    xeruf committed Jan 21, 2021
    Configuration menu
    Copy the full SHA
    c2365c8 View commit details
    Browse the repository at this point in the history