#61 Priorityqueing

[drehtisch]

Description

This allows api requests to be queued by priority.

Motivation and Context

Resolves #61

• Introduction of QueueWorker Concept
• Change Ratelimiting to fully use Async/Await with Task.ContinueWith and Release of a SemaphoreSlim once the ratelimit has passed. => Adjusted DataService interface to remove IsActionReady and WaitForAction
• Dispatching beeing Synchronous
• Usage of TaskCompletionSource instead of Async Wait Loops in both RequestQueue and Request.WaitForResponse

How Has This Been Tested?

Manually via Console & Discord and Unit Tests

Types of changes

• [x] New feature (non-breaking change which adds functionality)

• [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• [x] My code follows the code style of this project.

• [x] I have added tests to cover my changes.

• [x] All new and existing tests passed.

[codecov[bot]]

Codecov Report

Merging #67 (9be11ae) into Rel4.0 (a0ebfe3) will increase coverage by 0.39%. The diff coverage is 95.28%.

@@            Coverage Diff             @@
##           Rel4.0      #67      +/-   ##
==========================================
+ Coverage   88.10%   88.50%   +0.39%     
==========================================
  Files          28       29       +1     
  Lines        1051     1157     +106     
  Branches      111      129      +18     
==========================================
+ Hits          926     1024      +98     
- Misses        117      123       +6     
- Partials        8       10       +2     

Impacted Files	Coverage Δ
CyborgianStates/Commands/NationStatsCommand.cs	0.00% <0.00%> (ø)
CyborgianStates/Commands/PingCommand.cs	100.00% <ø> (ø)
CyborgianStates/Services/HttpDataService.cs	100.00% <ø> (ø)
...rgianStates/Services/NationStatesApiDataService.cs	97.22% <95.55%> (-2.78%)	:arrow_down:
...es/CommandHandling/NationStatesApiRequestWorker.cs	96.15% <96.15%> (ø)
CyborgianStates/CommandHandling/Request.cs	98.21% <100.00%> (-1.79%)	:arrow_down:
...borgianStates/CommandHandling/RequestDispatcher.cs	100.00% <100.00%> (ø)
...gianStates/CommandHandling/RequestPriorityQueue.cs	100.00% <100.00%> (ø)
CyborgianStates/HttpExtensions.cs	100.00% <100.00%> (ø)
CyborgianStates/Interfaces/IResponseBuilder.cs	100.00% <100.00%> (ø)
... and 4 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 a0ebfe3...9be11ae. Read the comment docs.