search

xplorfin / moneysocket-go

A *highly experimental* golang implementation of moneysocket
MIT License
3 stars 1 forks source link

test out locks to eliminate race conditions #23

Closed jakesylvestre closed 3 years ago

jakesylvestre commented 3 years ago

There seem to be a number of race conditions in existing tests with certain runs (e.g. https://github.com/xplorfin/moneysocket-go/runs/2073052679) getting to consumer nexus where as my runs don't hit the rendezvous.

This attempts to partially address that, but isn't really working so far

coveralls commented 3 years ago

Coverage Status

Coverage decreased (-1.6%) to 58.576% when pulling 08c8186bf1a438853df4a1f1c059b8af694320a9 on racetest into 57d19aa031dc44068a430710f4769836027f1f88 on master.

jakesylvestre commented 3 years ago

The race condition referenced in the original issue was not a rendezvous issue and was resolved in #36