mirror of
https://github.com/hedgedoc/hedgedoc.git
synced 2025-05-13 06:34:39 -04:00
NotesService: updateNoteByIdOrAlias
should return the new note
Fixes #702 Signed-off-by: David Mehren <git@herrmehren.de>
This commit is contained in:
parent
a14056dbc9
commit
22e14fb706
2 changed files with 3 additions and 1 deletions
|
@ -195,6 +195,7 @@ export class NotesService {
|
||||||
revisions.push(Revision.create(noteContent, noteContent));
|
revisions.push(Revision.create(noteContent, noteContent));
|
||||||
note.revisions = Promise.resolve(revisions);
|
note.revisions = Promise.resolve(revisions);
|
||||||
await this.noteRepository.save(note);
|
await this.noteRepository.save(note);
|
||||||
|
return this.toNoteDto(note);
|
||||||
}
|
}
|
||||||
|
|
||||||
async getNoteMetadata(noteIdOrAlias: string): Promise<NoteMetadataDto> {
|
async getNoteMetadata(noteIdOrAlias: string): Promise<NoteMetadataDto> {
|
||||||
|
|
|
@ -109,7 +109,7 @@ describe('Notes', () => {
|
||||||
|
|
||||||
it(`PUT /notes/{note}`, async () => {
|
it(`PUT /notes/{note}`, async () => {
|
||||||
await notesService.createNote('This is a test note.', 'test4');
|
await notesService.createNote('This is a test note.', 'test4');
|
||||||
await request(app.getHttpServer())
|
const response = await request(app.getHttpServer())
|
||||||
.put('/notes/test4')
|
.put('/notes/test4')
|
||||||
.set('Content-Type', 'text/markdown')
|
.set('Content-Type', 'text/markdown')
|
||||||
.send('New note text')
|
.send('New note text')
|
||||||
|
@ -117,6 +117,7 @@ describe('Notes', () => {
|
||||||
await expect(
|
await expect(
|
||||||
(await notesService.getNoteDtoByIdOrAlias('test4')).content,
|
(await notesService.getNoteDtoByIdOrAlias('test4')).content,
|
||||||
).toEqual('New note text');
|
).toEqual('New note text');
|
||||||
|
expect(response.body.content).toEqual('New note text');
|
||||||
|
|
||||||
// check if a missing note correctly returns 404
|
// check if a missing note correctly returns 404
|
||||||
await request(app.getHttpServer())
|
await request(app.getHttpServer())
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue