search

LD4P / sinopia_server

[Deprecated - switching to MongoDB] Sinopia Back-end CRUD Service. LDP-inspired, HTTP Server taking JSON-LD resources & administrative metadata.
Apache License 2.0
1 stars 1 forks source link

add basic tests for updateResource (or updateResourceWithHttpInfo) #99

Open jmartin-sul opened 5 years ago

jmartin-sul commented 5 years ago

in particular, test updating a non-RDF resource as well as an RDF resource, i.e. have a test that provides a value of { contentType: 'application/json' } for the opts param, as well as one that gives { contentType: 'application/ld+json' }.

jermnelson commented 5 years ago

Hi @jmartin-sul, should we keep this ticket this work-cycle for M4, or can we move it to the work-cycle 2?

jmartin-sul commented 5 years ago

would be a good thing to integration test w/ trellis since we're using it in the editor, but we do seem to be using it successfully, so no objection to backlogging this for now.

should be pretty straight-forward, though.

relevant test stub: https://github.com/LD4P/sinopia_server/blob/f237647d84352da62624a69e2f228012da307473/sinopia_client/test/api/LDPApi.spec.js#L437-L446

relevant usage in editor: https://github.com/LD4P/sinopia_editor/blob/a304ea635f013e4a03cce75e3fb34c811f442a5e/src/sinopiaServer.js#L169-L174