Use Default dispatcher instead of Main in Community scope.

[rahimklaber]

The coroutineScope in the Community class uses Dispatchers.Main. Dispatchers.Main is used to run code on the UI thread and should not be used for anything else.

Code that uses the scope without specifying the Dispatcher when launching a new task (like eva), does not work on a PC, as you need another dependency to access the Main dispatcher.

kotlin-ipv8/ipv8/src/main/java/nl/tudelft/ipv8/messaging/eva/EVAProtocol.kt

Line 74 in f983ed4

scope.launch {

https://kotlinlang.org/api/kotlinx.coroutines/kotlinx-coroutines-core/kotlinx.coroutines/-dispatchers/-main.html

[github-actions]

File	Coverage [84.44%]	🍏
Community.kt	84.44%	🍏

Total Project Coverage	63.11%	🍏

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +0.21 🎉

Comparison is base (0623f77) 58.32% compared to head (6a40fa2) 58.53%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master      #79      +/-   ##
============================================
+ Coverage     58.32%   58.53%   +0.21%     
- Complexity      697      702       +5     
============================================
  Files           107      107              
  Lines          4487     4546      +59     
  Branches        634      638       +4     
============================================
+ Hits           2617     2661      +44     
- Misses         1538     1550      +12     
- Partials        332      335       +3     

Impacted Files	Coverage Δ
ipv8/src/main/java/nl/tudelft/ipv8/Community.kt	75.78% <ø> (+1.78%)	⬆️

... and 7 files with indirect coverage changes

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

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.