search

samvera / ldp

Linked Data Platform.rb client
Other
16 stars 15 forks source link

added and fixed some tests, made it raise an exception on failed updates #4

Closed bmaddy closed 10 years ago

bmaddy commented 10 years ago

A test was catching the wrong exception (it was hiding a method not found).

I also made save return false and save! raise on an unsuccessful update.

bmaddy commented 10 years ago

@cbeer, are you comfortable merging this in? If not, let me know and I'll do a few monkey patches to get access to the last_response value in the fedora_lens thing I've been working on.

cbeer commented 10 years ago

Sorry, lost track of it. I thought I added you as a collaborator too, so feel free to nudge or assume lazy consensus and merge without me.

bmaddy commented 10 years ago

Heh, no prob. Thanks Chris!

On Wed, Mar 19, 2014 at 9:55 AM, Chris Beer notifications@github.comwrote:

Sorry, lost track of it. I thought I added you as a collaborator too, so feel free to nudge or assume lazy consensus and merge without me.

Reply to this email directly or view it on GitHubhttps://github.com/cbeer/ldp/pull/4#issuecomment-38059881 .