snowplow / snowbridge

For replicating streams across clouds, accounts and regions
Other
15 stars 7 forks source link

Add unit tests for EventHub Target #150

Closed colmsnowplow closed 2 years ago

colmsnowplow commented 2 years ago

@TiganeteaRobert I've made this a PR because I realised that the unit_tests_rebased branch is your working branch, and there might be some disruption to your work flow if I just pop it in there. If that's not the case, feel free to merge it in or wait till I'm around tomorrow, I don't mind which.

I think it'll be easiest from here in if we create separate working branches based off unit_tests_rebased to do our wip in, then merge it once it's all done (and if we need to rebase and force push there shouldn't be a problem).

We don't need to do reviews to get into unit_tests_rebased or anything, I just think it'll be easier if we don't commit so often in there.

Not saying anything's wrong with how we've gone so far, just didn't realise it before.

Let's catch up on progress tomorrow. :)