Skip to content

Commit e1e4f79

Browse files
sampaiodiegoclaude
andcommitted
test: remove redundant schema validation tests from custom-sounds
These tests validated AJV schema rejection (empty _id, regex injection via query object/bracket syntax) which is now enforced at build time by the new API.v1.get pattern with compiled validators. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent f2f42c4 commit e1e4f79

1 file changed

Lines changed: 0 additions & 18 deletions

File tree

apps/meteor/tests/end-to-end/api/custom-sounds.ts

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -211,10 +211,6 @@ describe('[CustomSounds]', () => {
211211
await request.get(api('custom-sounds.getOne')).set(credentials).query({ _id: 'invalid-id' }).expect(404);
212212
});
213213

214-
it('should return bad request if the _id length is not more than one', async () => {
215-
await request.get(api('custom-sounds.getOne')).set(credentials).query({ _id: '' }).expect(400);
216-
});
217-
218214
it('should return the custom sound successfully', async () => {
219215
await request
220216
.get(api('custom-sounds.getOne'))
@@ -230,20 +226,6 @@ describe('[CustomSounds]', () => {
230226
});
231227
});
232228

233-
it('should reject regex injection via query object', async () => {
234-
await request
235-
.get(api('custom-sounds.getOne'))
236-
.set(credentials)
237-
.query({
238-
_id: { $regex: '.*' },
239-
})
240-
.expect(400);
241-
});
242-
243-
it('should reject regex injection via bracket syntax', async () => {
244-
await request.get(api('custom-sounds.getOne')).set(credentials).query('_id[$regex]=.*').expect(400);
245-
});
246-
247229
it('should reject encoded regex injection attempt', async () => {
248230
await request
249231
.get(api('custom-sounds.getOne'))

0 commit comments

Comments
 (0)