Fix for concurrent fetching bugs#1227

Merged

designatednerd merged 14 commits intomasterfrom

May 27, 2020

Commits on May 27, 2020

pull out identifier into its own variable when setting up task
designatednerd
committed
update transport setup in star wars server tests to not recreate the session during a test
designatednerd
committed
test creating a bunch of identical requests at once creates different request IDs
designatednerd
committed
get tests failing more consistently using concurrentperform
designatednerd
committed
update test to hit HTTP bin direclty
designatednerd
committed
key of the URLSessionTask rather than the ID
designatednerd
committed
prevent race condition on lazy var
designatednerd
committed
add mutating function to atomic
designatednerd
committed
centralize to task data structure, use mutating to handle locking better.
designatednerd
committed
go back to using the task identifier since fixing concurrency seems to have made that actually work properly
designatednerd
committed
rm debugging print 🤦‍♀️
designatednerd
committed
clear all tasks when `URLSessionClient` gets deinited
designatednerd
committed
assertion failure if there's no task data found for a given URLSessionTask
designatednerd
committed
validate that all task IDs created are unique
designatednerd
committed

⚡