mirror of
https://github.com/hedgedoc/hedgedoc.git
synced 2024-11-29 19:24:38 -05:00
NotesE2ETest: Add GET /api/v2/notes/{note}/media test
Signed-off-by: Philip Molares <philip.molares@udo.edu>
This commit is contained in:
parent
935d5cd548
commit
4aca8ada4c
1 changed files with 52 additions and 0 deletions
|
@ -29,11 +29,15 @@ import { MockAuthGuard } from '../../src/auth/mock-auth.guard';
|
||||||
import { UsersService } from '../../src/users/users.service';
|
import { UsersService } from '../../src/users/users.service';
|
||||||
import { User } from '../../src/users/user.entity';
|
import { User } from '../../src/users/user.entity';
|
||||||
import { UsersModule } from '../../src/users/users.module';
|
import { UsersModule } from '../../src/users/users.module';
|
||||||
|
import { promises as fs } from 'fs';
|
||||||
|
import { MediaService } from '../../src/media/media.service';
|
||||||
|
|
||||||
describe('Notes', () => {
|
describe('Notes', () => {
|
||||||
let app: INestApplication;
|
let app: INestApplication;
|
||||||
let notesService: NotesService;
|
let notesService: NotesService;
|
||||||
|
let mediaService: MediaService;
|
||||||
let user: User;
|
let user: User;
|
||||||
|
let user2: User;
|
||||||
let content: string;
|
let content: string;
|
||||||
let forbiddenNoteId: string;
|
let forbiddenNoteId: string;
|
||||||
|
|
||||||
|
@ -69,8 +73,10 @@ describe('Notes', () => {
|
||||||
app = moduleRef.createNestApplication();
|
app = moduleRef.createNestApplication();
|
||||||
await app.init();
|
await app.init();
|
||||||
notesService = moduleRef.get(NotesService);
|
notesService = moduleRef.get(NotesService);
|
||||||
|
mediaService = moduleRef.get(MediaService);
|
||||||
const userService = moduleRef.get(UsersService);
|
const userService = moduleRef.get(UsersService);
|
||||||
user = await userService.createUser('hardcoded', 'Testy');
|
user = await userService.createUser('hardcoded', 'Testy');
|
||||||
|
user2 = await userService.createUser('hardcoded2', 'Max Mustermann');
|
||||||
content = 'This is a test note.';
|
content = 'This is a test note.';
|
||||||
});
|
});
|
||||||
|
|
||||||
|
@ -322,6 +328,52 @@ describe('Notes', () => {
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('GET /notes/{note}/media', () => {
|
||||||
|
it('works', async () => {
|
||||||
|
const note = await notesService.createNote(content, 'test9', user);
|
||||||
|
const extraNote = await notesService.createNote(content, 'test10', user);
|
||||||
|
const httpServer = app.getHttpServer();
|
||||||
|
const response = await request(httpServer)
|
||||||
|
.get(`/notes/${note.id}/media/`)
|
||||||
|
.expect('Content-Type', /json/)
|
||||||
|
.expect(200);
|
||||||
|
expect(response.body).toHaveLength(0);
|
||||||
|
|
||||||
|
const testImage = await fs.readFile('test/public-api/fixtures/test.png');
|
||||||
|
const url0 = await mediaService.saveFile(testImage, 'hardcoded', note.id);
|
||||||
|
const url1 = await mediaService.saveFile(
|
||||||
|
testImage,
|
||||||
|
'hardcoded',
|
||||||
|
extraNote.id,
|
||||||
|
);
|
||||||
|
|
||||||
|
const responseAfter = await request(httpServer)
|
||||||
|
.get(`/notes/${note.id}/media/`)
|
||||||
|
.expect('Content-Type', /json/)
|
||||||
|
.expect(200);
|
||||||
|
expect(responseAfter.body).toHaveLength(1);
|
||||||
|
expect(responseAfter.body[0].url).toEqual(url0);
|
||||||
|
expect(responseAfter.body[0].url).not.toEqual(url1);
|
||||||
|
});
|
||||||
|
it('fails, when note does not exist', async () => {
|
||||||
|
await request(app.getHttpServer())
|
||||||
|
.get(`/notes/i_dont_exist/media/`)
|
||||||
|
.expect('Content-Type', /json/)
|
||||||
|
.expect(404);
|
||||||
|
});
|
||||||
|
it("fails, when user can't read note", async () => {
|
||||||
|
const note = await notesService.createNote(
|
||||||
|
'This is a test note.',
|
||||||
|
'test11',
|
||||||
|
user2,
|
||||||
|
);
|
||||||
|
await request(app.getHttpServer())
|
||||||
|
.get(`/notes/${note.id}/media/`)
|
||||||
|
.expect('Content-Type', /json/)
|
||||||
|
.expect(401);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
afterAll(async () => {
|
afterAll(async () => {
|
||||||
await app.close();
|
await app.close();
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue