These are chat archives for damianh/Cedar

22nd
Jun 2015
Damian Hickey
@damianh
Jun 22 2015 08:00
you should try maintain neventstores dialect based sql abstractions :trollface:
Dan Barua
@danbarua
Jun 22 2015 08:04
sod that thank you very much
Damian Hickey
@damianh
Jun 22 2015 08:10
@danbarua Am testing out other append edge cases here https://github.com/damianh/Cedar.EventStore/tree/append-other-scenarios
WIP
These were the failing tests I was talking about.
Dan Barua
@danbarua
Jun 22 2015 08:18
ah cool
i noticed there was no append-to-existing-stream in mssql2008
hm, should i cherry pick the commits where you've added tests and then implement them?
or just copy and paste in and sort out on merge..
Damian Hickey
@damianh
Jun 22 2015 08:21
Merge it in should be fine, that branch is ahead of master
Dan Barua
@danbarua
Jun 22 2015 08:22
cool will do
Damian Hickey
@damianh
Jun 22 2015 08:22
I'm messing around there, so quality may be so-so
Or, you can hold off and wait to see if I get the scenarios working
or .. you could poke about, I'd always appreciated 2nd view on this.
Dan Barua
@danbarua
Jun 22 2015 08:23
When_delete_stream_with_a_non_matching_expected_version_then_should_throw
that actually tests deleting a stream with no events
so i tweaked that test to include a couple of events
Damian Hickey
@damianh
Jun 22 2015 08:24
yep, see, I'm missing stuff. Sometimes I just commit my WIP so I can move to a different machine or pick it up later.
Dan Barua
@danbarua
Jun 22 2015 08:25
.ncrunchproject files should be kept out of git right?
Damian Hickey
@damianh
Jun 22 2015 08:25
I don't think so
they're not .user files
I commit them because I have to re-configure ncrunch every single checkout
Dan Barua
@danbarua
Jun 22 2015 08:25
ok
Damian Hickey
@damianh
Jun 22 2015 08:26
for ncrunch to work with GES, you have to configure it to copy some files from the packages dir
There are no user specific settings there afaik.
should be portable/
Dan Barua
@danbarua
Jun 22 2015 08:27
ok cool
been a while
I've missed nCrunch
can't convince mgmt to shell out for it
new NewStreamEvent(Guid.NewGuid(), "data", "meta"),
needs to be
new NewStreamEvent(Guid.NewGuid(), "\"data\"", "\"meta\""),
since i'm using a db with proper support for json
:trollface:
Dan Barua
@danbarua
Jun 22 2015 08:49
FluentAssertions.Execution.AssertionFailedException: Expected string to be "Delete stream failed due to WrongExpectedVersion. Stream: stream, Expected version: 100." with a length of 88, but "Delete stream failed due to WrongExpectedVersion. Stream: C82E3D7279EFA3ECA576370AF952C815D48CE41F, Expected version: 100." has a length of 122.
would it make more sense to show the consumer/dev facing stream id?
than the hashed id?
wait
answered my own question
Dan Barua
@danbarua
Jun 22 2015 08:56
1 failing test left
Damian Hickey
@damianh
Jun 22 2015 08:59
I just bought ncrunch myself.
Dan Barua
@danbarua
Jun 22 2015 09:00
i'm sorely tempted
Damian Hickey
@damianh
Jun 22 2015 09:00
The ballache of going through procurement is not worth it.
If the solution is ncrunch compatible, it's worth my happiness.
João Bragança
@thefringeninja
Jun 22 2015 09:10
hah! i see your commit building on our ci server @danbarua
Dan Barua
@danbarua
Jun 22 2015 09:10
nice, will probably fail - using myget unstable feed for npgsql
using custom feeds, esp. auth'ed ones has been a huge ballache for us
oh and it relies on a pg db called "cedar_tests"
Damian Hickey
@damianh
Jun 22 2015 09:15
yeah couldn't resolve the reference
dm, will sort out in a bit
I'll have to install pgsql on the build agent though
Dan Barua
@danbarua
Jun 22 2015 09:18
added nuget.config, seems to work
Damian Hickey
@damianh
Jun 22 2015 09:19
cool
Dan Barua
@danbarua
Jun 22 2015 09:19
after much pulling hair out we ended up putting plain-text creds in nuget.config
such a massive ballache getting CI working with auth'ed nuget feeds
Damian Hickey
@damianh
Jun 22 2015 09:20
Yeah... so I started a project to help that - an API gateway of sorts. Proxies auth'd feeds anon.
?
Dan Barua
@danbarua
Jun 22 2015 09:34
right that's my OSS gardening done for the day, on to devops stuff
Damian Hickey
@damianh
Jun 22 2015 09:34
No that is for something else.
I want to see how my owin apps behave when "load balanced"
without the ballache of setting up nginx and shit
Just 2 or more application instances and a router
@danbarua thx for the work thus far. Will be on this later in the week/