diff --git a/imports/_test/image.ts b/imports/_test/image.ts index 3a8ce1518..aa24b87ad 100644 --- a/imports/_test/image.ts +++ b/imports/_test/image.ts @@ -48,7 +48,7 @@ const _xorClient = ( b: Uint8ClampedArray, ): Uint8ClampedArray => { const n = a.length; - assert.equal(b.length, n); + assert.strictEqual(b.length, n); const delta = new Uint8ClampedArray(n); for (let i = 0; i < n; ++i) { // eslint-disable-next-line no-bitwise @@ -97,25 +97,25 @@ const _assertSameDimensions = async (a: Sharp, b: Sharp): Promise => { hasAlpha: bHasAlpha, } = await b.metadata(); - assert.equal( + assert.strictEqual( aWidth, bWidth, `Images have different widths: ${aWidth} !== ${bWidth}`, ); - assert.equal( + assert.strictEqual( aHeight, bHeight, `Images have different heights: ${aHeight} !== ${bHeight}`, ); - assert.equal( + assert.strictEqual( aChannels, bChannels, `Images have different number of channels: ${aChannels} !== ${bChannels}`, ); - assert.equal( + assert.strictEqual( aHasAlpha, bHasAlpha, `Images have different alpha channel settings: ${aHasAlpha} !== ${bHasAlpha}`, diff --git a/imports/api/endpoint/allergies/changeColor.tests.ts b/imports/api/endpoint/allergies/changeColor.tests.ts index d433f01a9..73500df2c 100644 --- a/imports/api/endpoint/allergies/changeColor.tests.ts +++ b/imports/api/endpoint/allergies/changeColor.tests.ts @@ -21,16 +21,16 @@ server(__filename, () => { const {upsertedId: allergyId} = await newAllergy({userId}); - assert.equal(await Allergies.find({}).countAsync(), 1); - assert.equal(await Allergies.find({_id: allergyId}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({_id: allergyId}).countAsync(), 1); const {color} = await findOneOrThrow(Allergies, {_id: allergyId}); - assert.equal(color, undefined); + assert.strictEqual(color, undefined); const expected = '#fff'; await invoke(changeAllergyColor, {userId}, [allergyId, expected]); - assert.equal(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); assert.deepInclude(await Allergies.findOneAsync({_id: allergyId}), { color: expected, @@ -42,11 +42,11 @@ server(__filename, () => { const {upsertedId: allergyId} = await newAllergy({userId}); - assert.equal(await Allergies.find({}).countAsync(), 1); - assert.equal(await Allergies.find({_id: allergyId}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({_id: allergyId}).countAsync(), 1); const {color} = await findOneOrThrow(Allergies, {_id: allergyId}); - assert.equal(color, undefined); + assert.strictEqual(color, undefined); const expected = '#fff'; @@ -59,7 +59,7 @@ server(__filename, () => { /not-authorized/, ); - assert.equal(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); assert.notDeepInclude(await Allergies.findOneAsync({_id: allergyId}), { color: expected, @@ -71,10 +71,10 @@ server(__filename, () => { const {upsertedId: allergyId} = await newAllergy({userId}); - assert.equal(await Allergies.find({}).countAsync(), 1); - assert.equal(await Allergies.find({_id: allergyId}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({_id: allergyId}).countAsync(), 1); const {color} = await findOneOrThrow(Allergies, {_id: allergyId}); - assert.equal(color, undefined); + assert.strictEqual(color, undefined); const expected = '#fff'; @@ -87,7 +87,7 @@ server(__filename, () => { /not-found/, ); - assert.equal(await Allergies.find({}).countAsync(), 1); + assert.strictEqual(await Allergies.find({}).countAsync(), 1); assert.notDeepInclude(await Allergies.findOneAsync({_id: allergyId}), { color: expected, diff --git a/imports/api/endpoint/appointments/beginConsultation.tests.ts b/imports/api/endpoint/appointments/beginConsultation.tests.ts index 5a1f1401f..78446c05f 100644 --- a/imports/api/endpoint/appointments/beginConsultation.tests.ts +++ b/imports/api/endpoint/appointments/beginConsultation.tests.ts @@ -26,12 +26,15 @@ server(__filename, () => { const appointmentId = await newAppointment({userId}); - assert.equal(await Appointments.find({}).countAsync(), 1); - assert.equal(await Appointments.find({_id: appointmentId}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); + assert.strictEqual( + await Appointments.find({_id: appointmentId}).countAsync(), + 1, + ); await invoke(appointmentsBeginConsultation, {userId}, [appointmentId]); - assert.equal(await Appointments.find({}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); assert.deepInclude(await Appointments.findOneAsync({_id: appointmentId}), { isDone: true, diff --git a/imports/api/endpoint/appointments/cancel.tests.ts b/imports/api/endpoint/appointments/cancel.tests.ts index 75b3e648c..72a3c0de8 100644 --- a/imports/api/endpoint/appointments/cancel.tests.ts +++ b/imports/api/endpoint/appointments/cancel.tests.ts @@ -27,12 +27,15 @@ server(__filename, () => { const appointmentId = await newAppointment({userId}); - assert.equal(await Appointments.find({}).countAsync(), 1); - assert.equal(await Appointments.find({_id: appointmentId}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); + assert.strictEqual( + await Appointments.find({_id: appointmentId}).countAsync(), + 1, + ); await invoke(appointmentsCancel, {userId}, [appointmentId, '', '']); - assert.equal(await Appointments.find({}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); assert.deepInclude(await Appointments.findOneAsync({_id: appointmentId}), { isCancelled: true, diff --git a/imports/api/endpoint/appointments/remove.tests.ts b/imports/api/endpoint/appointments/remove.tests.ts index 766dd61f4..559cba29e 100644 --- a/imports/api/endpoint/appointments/remove.tests.ts +++ b/imports/api/endpoint/appointments/remove.tests.ts @@ -26,12 +26,15 @@ server(__filename, () => { const appointmentId = await newAppointment({userId}); - assert.equal(await Appointments.find({}).countAsync(), 1); - assert.equal(await Appointments.find({_id: appointmentId}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); + assert.strictEqual( + await Appointments.find({_id: appointmentId}).countAsync(), + 1, + ); await invoke(appointmentsRemove, {userId}, [appointmentId]); - assert.equal(await Appointments.find({}).countAsync(), 0); + assert.strictEqual(await Appointments.find({}).countAsync(), 0); }); it("cannot remove other user's consultation", async () => { diff --git a/imports/api/endpoint/appointments/reschedule.tests.ts b/imports/api/endpoint/appointments/reschedule.tests.ts index e573127a2..9680e4697 100644 --- a/imports/api/endpoint/appointments/reschedule.tests.ts +++ b/imports/api/endpoint/appointments/reschedule.tests.ts @@ -89,7 +89,7 @@ server(__filename, () => { const before = await findOneOrThrow(Appointments); - assert.equal(before._id, appointmentId); + assert.strictEqual(before._id, appointmentId); assert.sameDeepMembers( dropIds(await Availability.find().fetchAsync()), expected(before), @@ -101,7 +101,7 @@ server(__filename, () => { const after = await findOneOrThrow(Appointments); - assert.equal(after._id, appointmentId); + assert.strictEqual(after._id, appointmentId); assert.sameDeepMembers( dropIds(await Availability.find().fetchAsync()), expected(after), diff --git a/imports/api/endpoint/appointments/schedule.tests.ts b/imports/api/endpoint/appointments/schedule.tests.ts index 48ae9c27f..4d363b12a 100644 --- a/imports/api/endpoint/appointments/schedule.tests.ts +++ b/imports/api/endpoint/appointments/schedule.tests.ts @@ -65,7 +65,7 @@ server(__filename, () => { it('creates associated patient', async () => { const userId = randomUserId(); - assert.equal(await Patients.findOneAsync(), undefined); + assert.strictEqual(await Patients.findOneAsync(), undefined); const patientFields = { firstname: 'Jane', diff --git a/imports/api/endpoint/appointments/uncancel.tests.ts b/imports/api/endpoint/appointments/uncancel.tests.ts index 20de2d87a..4c9a92b18 100644 --- a/imports/api/endpoint/appointments/uncancel.tests.ts +++ b/imports/api/endpoint/appointments/uncancel.tests.ts @@ -27,12 +27,15 @@ server(__filename, () => { const appointmentId = await newAppointment({userId}); - assert.equal(await Appointments.find({}).countAsync(), 1); - assert.equal(await Appointments.find({_id: appointmentId}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); + assert.strictEqual( + await Appointments.find({_id: appointmentId}).countAsync(), + 1, + ); await invoke(appointmentsCancel, {userId}, [appointmentId, '', '']); - assert.equal(await Appointments.find({}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); assert.deepInclude(await Appointments.findOneAsync({_id: appointmentId}), { isCancelled: true, @@ -40,7 +43,7 @@ server(__filename, () => { await invoke(appointmentsUncancel, {userId}, [appointmentId]); - assert.equal(await Appointments.find({}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); assert.deepInclude(await Appointments.findOneAsync({_id: appointmentId}), { isCancelled: false, diff --git a/imports/api/endpoint/books/csv.tests.ts b/imports/api/endpoint/books/csv.tests.ts index 5aed8161f..8dacb2dcb 100644 --- a/imports/api/endpoint/books/csv.tests.ts +++ b/imports/api/endpoint/books/csv.tests.ts @@ -134,7 +134,7 @@ server(__filename, () => { maxRows, ]); - assert.equal(result, expected); + assert.strictEqual(result, expected); }); it('cannot download csv file if not logged in', async () => { diff --git a/imports/api/endpoint/books/rename.tests.ts b/imports/api/endpoint/books/rename.tests.ts index 1f6039ca8..bd3f2a52f 100644 --- a/imports/api/endpoint/books/rename.tests.ts +++ b/imports/api/endpoint/books/rename.tests.ts @@ -42,7 +42,7 @@ server(__filename, () => { consultationAId, ); - assert.equal(updatedBookNumber, newBookNumber); + assert.strictEqual(updatedBookNumber, newBookNumber); }); it('renaming a book updates its consultations', async () => { @@ -95,7 +95,7 @@ server(__filename, () => { ]) { // eslint-disable-next-line no-await-in-loop const {book} = await findOneOrThrow(Consultations, consultationId); - assert.equal(book, newBookNumber); + assert.strictEqual(book, newBookNumber); } }); @@ -177,19 +177,19 @@ server(__filename, () => { ]) { // eslint-disable-next-line no-await-in-loop const {book} = await findOneOrThrow(Consultations, consultationId); - assert.equal(book, newBookNumber); + assert.strictEqual(book, newBookNumber); } for (const consultationId of [consultationDId]) { // eslint-disable-next-line no-await-in-loop const {book} = await findOneOrThrow(Consultations, consultationId); - assert.equal(book, oldBookNumber); + assert.strictEqual(book, oldBookNumber); } for (const consultationId of [consultationEId, consultationFId]) { // eslint-disable-next-line no-await-in-loop const {book} = await findOneOrThrow(Consultations, consultationId); - assert.equal(book, `${oldBookNumber}x`); + assert.strictEqual(book, `${oldBookNumber}x`); } }); }); diff --git a/imports/api/endpoint/consultations/attach.tests.ts b/imports/api/endpoint/consultations/attach.tests.ts index f71caaf80..8ee2caa95 100644 --- a/imports/api/endpoint/consultations/attach.tests.ts +++ b/imports/api/endpoint/consultations/attach.tests.ts @@ -34,29 +34,29 @@ server(__filename, () => { const {_id: uploadAId} = await newUpload({userId}); const {_id: uploadBId} = await newUpload({userId}); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); await invoke(consultationsAttach, {userId}, [consultationAId, uploadAId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); assert.deepInclude( await Attachments.findOneAsync({ @@ -67,7 +67,7 @@ server(__filename, () => { }, ); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationBId, }), @@ -76,16 +76,16 @@ server(__filename, () => { await invoke(consultationsAttach, {userId}, [consultationBId, uploadBId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); assert.deepInclude( await Attachments.findOneAsync({ @@ -118,7 +118,7 @@ server(__filename, () => { const {_id: uploadId} = await newUpload({userId}); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationId, }), diff --git a/imports/api/endpoint/consultations/detach.tests.ts b/imports/api/endpoint/consultations/detach.tests.ts index 05cc4d5d0..63605ecbd 100644 --- a/imports/api/endpoint/consultations/detach.tests.ts +++ b/imports/api/endpoint/consultations/detach.tests.ts @@ -40,18 +40,18 @@ server(__filename, () => { await invoke(consultationsDetach, {userId}, [consultationAId, uploadAId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationAId, }), @@ -69,25 +69,25 @@ server(__filename, () => { await invoke(consultationsDetach, {userId}, [consultationBId, uploadBId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationAId, }), undefined, ); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationBId, }), @@ -111,7 +111,7 @@ server(__filename, () => { await invoke(consultationsDetach, {userId}, [consultationId, uploadId]); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationId, }), @@ -120,7 +120,7 @@ server(__filename, () => { await invoke(consultationsDetach, {userId}, [consultationId, uploadId]); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToConsultations': consultationId, }), diff --git a/imports/api/endpoint/consultations/insert.tests.ts b/imports/api/endpoint/consultations/insert.tests.ts index fedbba506..c5a220060 100644 --- a/imports/api/endpoint/consultations/insert.tests.ts +++ b/imports/api/endpoint/consultations/insert.tests.ts @@ -52,7 +52,7 @@ server(__filename, () => { const bookName = books.name(datetime, book) as NormalizedLine; - assert.equal( + assert.strictEqual( await Books.findOneAsync({name: bookName, owner: userId}), undefined, ); @@ -77,7 +77,10 @@ server(__filename, () => { const patientId = await newPatient({userId}); - assert.equal(await Availability.find({owner: userId}).countAsync(), 0); + assert.strictEqual( + await Availability.find({owner: userId}).countAsync(), + 0, + ); await newConsultation( {userId}, @@ -86,6 +89,9 @@ server(__filename, () => { }, ); - assert.equal(await Availability.find({owner: userId}).countAsync(), 0); + assert.strictEqual( + await Availability.find({owner: userId}).countAsync(), + 0, + ); }); }); diff --git a/imports/api/endpoint/consultations/remove.tests.ts b/imports/api/endpoint/consultations/remove.tests.ts index a4dbcf2e5..4ab84f292 100644 --- a/imports/api/endpoint/consultations/remove.tests.ts +++ b/imports/api/endpoint/consultations/remove.tests.ts @@ -25,16 +25,16 @@ server(__filename, () => { {patientId}, ); - assert.equal( + assert.strictEqual( await Consultations.find({_id: consultationId}).countAsync(), 1, ); - assert.equal(await Consultations.find({patientId}).countAsync(), 1); + assert.strictEqual(await Consultations.find({patientId}).countAsync(), 1); await invoke(consultationsRemove, {userId}, [consultationId]); - assert.equal(await Consultations.find({patientId}).countAsync(), 0); - assert.equal( + assert.strictEqual(await Consultations.find({patientId}).countAsync(), 0); + assert.strictEqual( await Consultations.find({_id: consultationId}).countAsync(), 0, ); @@ -77,28 +77,28 @@ server(__filename, () => { it("detaches removed consultation's attachments", async () => { const userId = randomUserId(); - assert.equal(await Patients.find({}).countAsync(), 0); + assert.strictEqual(await Patients.find({}).countAsync(), 0); const patientId = await newPatient({userId}); - assert.equal(await Patients.find({}).countAsync(), 1); + assert.strictEqual(await Patients.find({}).countAsync(), 1); - assert.equal(await Consultations.find({}).countAsync(), 0); + assert.strictEqual(await Consultations.find({}).countAsync(), 0); const {insertedId: consultationId} = await newConsultation( {userId}, {patientId}, ); - assert.equal(await Consultations.find({}).countAsync(), 1); + assert.strictEqual(await Consultations.find({}).countAsync(), 1); - assert.equal(await Uploads.collection.find({}).countAsync(), 0); + assert.strictEqual(await Uploads.collection.find({}).countAsync(), 0); const uploadA = await newUpload({userId}); const uploadB = await newUpload({userId}); - assert.equal(await Uploads.collection.find({}).countAsync(), 2); - assert.equal( + assert.strictEqual(await Uploads.collection.find({}).countAsync(), 2); + assert.strictEqual( await Uploads.collection .find({ 'meta.attachedToConsultations': consultationId, @@ -106,7 +106,7 @@ server(__filename, () => { .countAsync(), 0, ); - assert.equal( + assert.strictEqual( await Uploads.collection .find({'meta.attachedToPatients': patientId}) .countAsync(), @@ -116,10 +116,10 @@ server(__filename, () => { await invoke(consultationsAttach, {userId}, [consultationId, uploadA._id]); await invoke(consultationsAttach, {userId}, [consultationId, uploadB._id]); - assert.equal(await Patients.find({}).countAsync(), 1); - assert.equal(await Consultations.find({}).countAsync(), 1); - assert.equal(await Uploads.collection.find({}).countAsync(), 2); - assert.equal( + assert.strictEqual(await Patients.find({}).countAsync(), 1); + assert.strictEqual(await Consultations.find({}).countAsync(), 1); + assert.strictEqual(await Uploads.collection.find({}).countAsync(), 2); + assert.strictEqual( await Uploads.collection .find({ 'meta.attachedToConsultations': consultationId, @@ -127,7 +127,7 @@ server(__filename, () => { .countAsync(), 2, ); - assert.equal( + assert.strictEqual( await Uploads.collection .find({'meta.attachedToPatients': patientId}) .countAsync(), @@ -136,8 +136,8 @@ server(__filename, () => { await invoke(consultationsRemove, {userId}, [consultationId]); - assert.equal(await Uploads.collection.find({}).countAsync(), 2); - assert.equal( + assert.strictEqual(await Uploads.collection.find({}).countAsync(), 2); + assert.strictEqual( await Uploads.collection .find({ 'meta.attachedToConsultations': consultationId, @@ -145,13 +145,13 @@ server(__filename, () => { .countAsync(), 0, ); - assert.equal( + assert.strictEqual( await Uploads.collection .find({'meta.attachedToPatients': patientId}) .countAsync(), 0, ); - assert.equal(await Consultations.find({}).countAsync(), 0); - assert.equal(await Patients.find({}).countAsync(), 1); + assert.strictEqual(await Consultations.find({}).countAsync(), 0); + assert.strictEqual(await Patients.find({}).countAsync(), 1); }); }); diff --git a/imports/api/endpoint/consultations/restoreAppointment.tests.ts b/imports/api/endpoint/consultations/restoreAppointment.tests.ts index 8dfca3e02..bfe42b2f0 100644 --- a/imports/api/endpoint/consultations/restoreAppointment.tests.ts +++ b/imports/api/endpoint/consultations/restoreAppointment.tests.ts @@ -24,7 +24,7 @@ server(__filename, () => { await invoke(restoreAppointment, {userId}, [appointmentId]); - assert.equal(await Appointments.find({}).countAsync(), 1); + assert.strictEqual(await Appointments.find({}).countAsync(), 1); assert.deepInclude(await Appointments.findOneAsync({_id: appointmentId}), { isDone: false, diff --git a/imports/api/endpoint/consultations/transfer.tests.ts b/imports/api/endpoint/consultations/transfer.tests.ts index d3068b1c5..e44c68829 100644 --- a/imports/api/endpoint/consultations/transfer.tests.ts +++ b/imports/api/endpoint/consultations/transfer.tests.ts @@ -23,22 +23,22 @@ server(__filename, () => { {patientId: patientAId}, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 0, ); await invoke(consultationsTransfer, {userId}, [consultationId, patientBId]); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientAId}).countAsync(), 0, ); - assert.equal( + assert.strictEqual( await Consultations.find({patientId: patientBId}).countAsync(), 1, ); diff --git a/imports/api/endpoint/consultations/update.tests.ts b/imports/api/endpoint/consultations/update.tests.ts index 512063474..c6b47ce04 100644 --- a/imports/api/endpoint/consultations/update.tests.ts +++ b/imports/api/endpoint/consultations/update.tests.ts @@ -79,7 +79,7 @@ server(__filename, () => { const price = 100; const paid = 0; - assert.equal(await Consultations.findOneAsync(), undefined); + assert.strictEqual(await Consultations.findOneAsync(), undefined); const {insertedId: consultationId} = await newConsultation( {userId}, @@ -110,19 +110,28 @@ server(__filename, () => { const patientId = await newPatient({userId}); - assert.equal(await Availability.find({owner: userId}).countAsync(), 0); + assert.strictEqual( + await Availability.find({owner: userId}).countAsync(), + 0, + ); const {insertedId: consultationId} = await newConsultation( {userId}, {patientId}, ); - assert.equal(await Availability.find({owner: userId}).countAsync(), 0); + assert.strictEqual( + await Availability.find({owner: userId}).countAsync(), + 0, + ); const newFields = newConsultationFormData({patientId}); await invoke(update, {userId}, [consultationId, newFields]); - assert.equal(await Availability.find({owner: userId}).countAsync(), 0); + assert.strictEqual( + await Availability.find({owner: userId}).countAsync(), + 0, + ); }); }); diff --git a/imports/api/endpoint/documents/delete.tests.ts b/imports/api/endpoint/documents/delete.tests.ts index a9d3de388..9687a446a 100644 --- a/imports/api/endpoint/documents/delete.tests.ts +++ b/imports/api/endpoint/documents/delete.tests.ts @@ -13,23 +13,23 @@ server(__filename, () => { it('cannot delete a document when not logged in', async () => { const userId = randomUserId(); const documentId = await newDocument({userId}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentDelete, {userId: undefined!}, [documentId]), /not-authorized/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot delete a document owned by another user', async () => { const userId = randomUserId(); const documentId = await newDocument({userId: `${userId}x`}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentDelete, {userId}, [documentId]), /not-found/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot delete a document that does not exist', async () => { @@ -45,28 +45,28 @@ server(__filename, () => { const documentAId = await newDocument({userId}); const documentBId = await newDocument({userId}); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); await invoke(documentDelete, {userId}, [documentAId]); - assert.equal(await Documents.find().countAsync(), 2); - assert.equal(await Documents.find({deleted: true}).countAsync(), 1); - assert.equal( + assert.strictEqual(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 1); + assert.strictEqual( await Documents.find({_id: documentAId, deleted: true}).countAsync(), 1, ); await invoke(documentDelete, {userId}, [documentBId]); - assert.equal(await Documents.find().countAsync(), 2); - assert.equal(await Documents.find({deleted: true}).countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 2); }); it('is idempotent', async () => { const userId = randomUserId(); const documentId = await newDocument({userId}); - assert.equal(await Documents.find({deleted: true}).countAsync(), 0); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 0); await invoke(documentDelete, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 1); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 1); await invoke(documentDelete, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 1); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 1); }); }); diff --git a/imports/api/endpoint/documents/fetch.tests.ts b/imports/api/endpoint/documents/fetch.tests.ts index b0ed12757..7ce12827d 100644 --- a/imports/api/endpoint/documents/fetch.tests.ts +++ b/imports/api/endpoint/documents/fetch.tests.ts @@ -46,7 +46,7 @@ server(__filename, () => { }, ]); - assert.equal( + assert.strictEqual( await invoke(documentFetch, {userId}, [documentId!]), exampleMedidocReport.contents, ); diff --git a/imports/api/endpoint/documents/insert.tests.ts b/imports/api/endpoint/documents/insert.tests.ts index 3a08e14cd..cd0ac1af8 100644 --- a/imports/api/endpoint/documents/insert.tests.ts +++ b/imports/api/endpoint/documents/insert.tests.ts @@ -25,7 +25,7 @@ server(__filename, () => { /not-authorized/, ); - assert.equal(await Documents.find().countAsync(), 0); + assert.strictEqual(await Documents.find().countAsync(), 0); }); it('cannot insert a document linked to a non-existing patient', async () => { @@ -41,7 +41,7 @@ server(__filename, () => { /not-found/, ); - assert.equal(await Documents.find().countAsync(), 0); + assert.strictEqual(await Documents.find().countAsync(), 0); }); it('cannot insert a document linked to a patient that is not owned', async () => { @@ -58,7 +58,7 @@ server(__filename, () => { /not-found/, ); - assert.equal(await Documents.find().countAsync(), 0); + assert.strictEqual(await Documents.find().countAsync(), 0); }); it('can insert a document linked to a patient that is owned', async () => { @@ -71,7 +71,7 @@ server(__filename, () => { }, ]); - assert.equal( + assert.strictEqual( await Documents.find().countAsync(), exampleHealthoneLab.count, ); @@ -96,11 +96,11 @@ server(__filename, () => { }, ]); - assert.equal( + assert.strictEqual( await Documents.find().countAsync(), exampleHealthoneReport.count, ); - assert.equal(await Documents.find({patientId}).countAsync(), 1); + assert.strictEqual(await Documents.find({patientId}).countAsync(), 1); }); it('set patientId overwrites patient matching', async () => { @@ -115,11 +115,11 @@ server(__filename, () => { }, ]); - assert.equal( + assert.strictEqual( await Documents.find().countAsync(), exampleHealthoneReport.count, ); - assert.equal( + assert.strictEqual( await Documents.find({patientId}).countAsync(), exampleHealthoneReport.count, ); @@ -137,11 +137,11 @@ server(__filename, () => { }, ]); - assert.equal( + assert.strictEqual( await Documents.find().countAsync(), exampleHealthoneReport.count, ); - assert.equal( + assert.strictEqual( await Documents.find({patientId: {$exists: true}}).countAsync(), 0, ); diff --git a/imports/api/endpoint/documents/link.tests.ts b/imports/api/endpoint/documents/link.tests.ts index 227a867df..47c5b454b 100644 --- a/imports/api/endpoint/documents/link.tests.ts +++ b/imports/api/endpoint/documents/link.tests.ts @@ -24,8 +24,8 @@ server(__filename, () => { const documentAId = await newDocument({userId}); const documentBId = await newDocument({userId}); - assert.equal(await Patients.find().countAsync(), 3); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 3); + assert.strictEqual(await Documents.find().countAsync(), 2); await invoke(documentsLink, invocation, [documentAId, patientAId]); await invoke(documentsLink, invocation, [documentBId, patientBId]); @@ -37,8 +37,8 @@ server(__filename, () => { assert.deepNestedInclude(await Documents.findOneAsync(documentBId), { patientId: patientBId, }); - assert.equal(await Patients.find().countAsync(), 3); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 3); + assert.strictEqual(await Documents.find().countAsync(), 2); }); it('is idempotent', async () => { @@ -48,7 +48,7 @@ server(__filename, () => { const documentId = await newDocument({userId}); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId, }), diff --git a/imports/api/endpoint/documents/restore.tests.ts b/imports/api/endpoint/documents/restore.tests.ts index 4f8ad1366..c53938390 100644 --- a/imports/api/endpoint/documents/restore.tests.ts +++ b/imports/api/endpoint/documents/restore.tests.ts @@ -14,23 +14,23 @@ server(__filename, () => { it('cannot restore a document when not logged in', async () => { const userId = randomUserId(); const documentId = await newDocument({userId}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentRestore, {userId: undefined!}, [documentId]), /not-authorized/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot restore a document owned by another user', async () => { const userId = randomUserId(); const documentId = await newDocument({userId: `${userId}x`}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentRestore, {userId}, [documentId]), /not-found/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot restore a document that does not exist', async () => { @@ -49,20 +49,20 @@ server(__filename, () => { await invoke(documentDelete, {userId}, [documentAId]); await invoke(documentDelete, {userId}, [documentBId]); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); await invoke(documentRestore, {userId}, [documentAId]); - assert.equal(await Documents.find().countAsync(), 2); - assert.equal( + assert.strictEqual(await Documents.find().countAsync(), 2); + assert.strictEqual( await Documents.find({_id: documentAId, deleted: false}).countAsync(), 1, ); - assert.equal( + assert.strictEqual( await Documents.find({_id: documentBId, deleted: true}).countAsync(), 1, ); await invoke(documentRestore, {userId}, [documentBId]); - assert.equal(await Documents.find().countAsync(), 2); - assert.equal(await Documents.find({deleted: false}).countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find({deleted: false}).countAsync(), 2); }); it('is idempotent', async () => { @@ -70,15 +70,15 @@ server(__filename, () => { const documentId = await newDocument({userId}); await invoke(documentRestore, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 0); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 0); await invoke(documentDelete, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 1); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 1); await invoke(documentRestore, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 0); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 0); await invoke(documentRestore, {userId}, [documentId]); - assert.equal(await Documents.find({deleted: true}).countAsync(), 0); + assert.strictEqual(await Documents.find({deleted: true}).countAsync(), 0); }); }); diff --git a/imports/api/endpoint/documents/superdelete.tests.ts b/imports/api/endpoint/documents/superdelete.tests.ts index 233810dc8..799bf7b70 100644 --- a/imports/api/endpoint/documents/superdelete.tests.ts +++ b/imports/api/endpoint/documents/superdelete.tests.ts @@ -13,24 +13,24 @@ server(__filename, () => { it('cannot superdelete a document when not logged in', async () => { const userId = randomUserId(); const documentId = await newDocument({userId}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentSuperdelete, {userId: undefined!}, [documentId]), /not-authorized/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot superdelete a document owned by another user', async () => { const userId = randomUserId(); const documentId = await newDocument({userId: `${userId}x`}); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); await throws( async () => invoke(documentSuperdelete, {userId}, [documentId]), /not-found/, ); - assert.equal(await Documents.find().countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); }); it('cannot superdelete a document that does not exist', async () => { @@ -46,12 +46,18 @@ server(__filename, () => { const documentAId = await newDocument({userId}); const documentBId = await newDocument({userId}); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); await invoke(documentSuperdelete, {userId}, [documentAId]); - assert.equal(await Documents.find().countAsync(), 1); - assert.equal(await Documents.find({_id: documentAId}).countAsync(), 0); - assert.equal(await Documents.find({_id: documentBId}).countAsync(), 1); + assert.strictEqual(await Documents.find().countAsync(), 1); + assert.strictEqual( + await Documents.find({_id: documentAId}).countAsync(), + 0, + ); + assert.strictEqual( + await Documents.find({_id: documentBId}).countAsync(), + 1, + ); await invoke(documentSuperdelete, {userId}, [documentBId]); - assert.equal(await Documents.find().countAsync(), 0); + assert.strictEqual(await Documents.find().countAsync(), 0); }); }); diff --git a/imports/api/endpoint/documents/unlink.tests.ts b/imports/api/endpoint/documents/unlink.tests.ts index a0f4272bf..5ce30ad4a 100644 --- a/imports/api/endpoint/documents/unlink.tests.ts +++ b/imports/api/endpoint/documents/unlink.tests.ts @@ -24,10 +24,10 @@ server(__filename, () => { await invoke(documentsUnlink, {userId}, [documentAId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId: patientAId, }), @@ -63,17 +63,17 @@ server(__filename, () => { await invoke(documentsUnlink, {userId}, [documentBId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId: patientAId, }), undefined, ); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId: patientBId, }), @@ -90,7 +90,7 @@ server(__filename, () => { await invoke(documentsUnlink, {userId}, [documentId]); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId: {$exists: true}, }), @@ -99,7 +99,7 @@ server(__filename, () => { await invoke(documentsUnlink, {userId}, [documentId]); - assert.equal( + assert.strictEqual( await Documents.findOneAsync({ patientId: {$exists: true}, }), diff --git a/imports/api/endpoint/patients/attach.tests.ts b/imports/api/endpoint/patients/attach.tests.ts index 1e8b02a67..898e1dca2 100644 --- a/imports/api/endpoint/patients/attach.tests.ts +++ b/imports/api/endpoint/patients/attach.tests.ts @@ -24,8 +24,8 @@ server(__filename, () => { const {_id: uploadAId} = await newUpload({userId}); const {_id: uploadBId} = await newUpload({userId}); - assert.equal(await Patients.find().countAsync(), 3); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 3); + assert.strictEqual(await Attachments.find().countAsync(), 2); await invoke(patientsAttach, invocation, [patientAId, uploadAId]); await invoke(patientsAttach, invocation, [patientBId, uploadBId]); @@ -37,8 +37,8 @@ server(__filename, () => { assert.deepNestedInclude(await Attachments.findOneAsync(uploadBId), { 'meta.attachedToPatients': [patientBId], }); - assert.equal(await Patients.find().countAsync(), 3); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 3); + assert.strictEqual(await Attachments.find().countAsync(), 2); }); it('is idempotent', async () => { @@ -48,7 +48,7 @@ server(__filename, () => { const {_id: uploadId} = await newUpload({userId}); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientId, }), diff --git a/imports/api/endpoint/patients/detach.tests.ts b/imports/api/endpoint/patients/detach.tests.ts index 15a622639..a6167d895 100644 --- a/imports/api/endpoint/patients/detach.tests.ts +++ b/imports/api/endpoint/patients/detach.tests.ts @@ -28,10 +28,10 @@ server(__filename, () => { await invoke(patientsDetach, {userId}, [patientAId, uploadAId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientAId, }), @@ -49,17 +49,17 @@ server(__filename, () => { await invoke(patientsDetach, {userId}, [patientBId, uploadBId]); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientAId, }), undefined, ); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientBId, }), @@ -78,7 +78,7 @@ server(__filename, () => { await invoke(patientsDetach, {userId}, [patientId, uploadId]); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientId, }), @@ -87,7 +87,7 @@ server(__filename, () => { await invoke(patientsDetach, {userId}, [patientId, uploadId]); - assert.equal( + assert.strictEqual( await Attachments.findOneAsync({ 'meta.attachedToPatients': patientId, }), diff --git a/imports/api/endpoint/patients/insert.tests.ts b/imports/api/endpoint/patients/insert.tests.ts index 4ed3a2919..451d2cfb7 100644 --- a/imports/api/endpoint/patients/insert.tests.ts +++ b/imports/api/endpoint/patients/insert.tests.ts @@ -36,7 +36,7 @@ server(__filename, () => { await newPatient({userId}, {allergies, doctors, insurances}); - assert.equal(await Patients.find({owner: userId}).countAsync(), 1); + assert.strictEqual(await Patients.find({owner: userId}).countAsync(), 1); assert.sameDeepMembers( await Allergies.find().mapAsync(({displayName, name}) => ({ diff --git a/imports/api/endpoint/patients/merge.tests.ts b/imports/api/endpoint/patients/merge.tests.ts index 3bee36570..ac7e08764 100644 --- a/imports/api/endpoint/patients/merge.tests.ts +++ b/imports/api/endpoint/patients/merge.tests.ts @@ -56,7 +56,7 @@ server(__filename, () => { // create an irrelevant document await newDocument(invocation); - assert.equal(await Patients.find().countAsync(), 3); + assert.strictEqual(await Patients.find().countAsync(), 3); const patientA = await Patients.findOneAsync(patientAId); const patientB = await Patients.findOneAsync(patientBId); @@ -78,14 +78,14 @@ server(__filename, () => { const newPatientId = await invoke(patientsMerge, invocation, parameters); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Consultations.find().countAsync(), 2); - assert.equal(await Attachments.find().countAsync(), 2); - assert.equal(await Documents.find().countAsync(), 2); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Consultations.find().countAsync(), 2); + assert.strictEqual(await Attachments.find().countAsync(), 2); + assert.strictEqual(await Documents.find().countAsync(), 2); const mergedPatient = await Patients.findOneAsync(newPatientId); - assert.equal(mergedPatient!.firstname, patientB!.firstname); + assert.strictEqual(mergedPatient!.firstname, patientB!.firstname); const expectedAttachments = setLike( [uploadA, uploadB].map(({meta, ...rest}) => rest), @@ -103,10 +103,10 @@ server(__filename, () => { const consultationA = await Consultations.findOneAsync(consultationAId); - assert.equal(consultationA!.patientId, newPatientId); + assert.strictEqual(consultationA!.patientId, newPatientId); documentA = await Documents.findOneAsync(documentA!._id); - assert.equal(documentA!.patientId, newPatientId); + assert.strictEqual(documentA!.patientId, newPatientId); }); }); diff --git a/imports/api/endpoint/patients/remove.tests.ts b/imports/api/endpoint/patients/remove.tests.ts index 19a66897e..9dc40e610 100644 --- a/imports/api/endpoint/patients/remove.tests.ts +++ b/imports/api/endpoint/patients/remove.tests.ts @@ -23,7 +23,7 @@ server(__filename, () => { await invoke(patientsRemove, invocation, [patientId]); - assert.equal(await Patients.find().countAsync(), 0); + assert.strictEqual(await Patients.find().countAsync(), 0); }); it("cannot delete someone else's patient", async () => { @@ -52,17 +52,17 @@ server(__filename, () => { {patientId: patientBId, datetime: new Date()}, ); - assert.equal(await Patients.find().countAsync(), 2); - assert.equal(await Consultations.find().countAsync(), 3); + assert.strictEqual(await Patients.find().countAsync(), 2); + assert.strictEqual(await Consultations.find().countAsync(), 3); await invoke(patientsRemove, {userId}, [patientAId]); - assert.equal(await Patients.find().countAsync(), 1); - assert.equal(await Consultations.find().countAsync(), 1); + assert.strictEqual(await Patients.find().countAsync(), 1); + assert.strictEqual(await Consultations.find().countAsync(), 1); await invoke(patientsRemove, {userId}, [patientBId]); - assert.equal(await Patients.find().countAsync(), 0); - assert.equal(await Consultations.find().countAsync(), 0); + assert.strictEqual(await Patients.find().countAsync(), 0); + assert.strictEqual(await Consultations.find().countAsync(), 0); }); }); diff --git a/imports/api/endpoint/patients/update.tests.ts b/imports/api/endpoint/patients/update.tests.ts index cc7d12897..2a0c58fa6 100644 --- a/imports/api/endpoint/patients/update.tests.ts +++ b/imports/api/endpoint/patients/update.tests.ts @@ -61,7 +61,7 @@ server(__filename, () => { {allergies, doctors, insurances}, ]); - assert.equal(await Patients.find({owner: userId}).countAsync(), 1); + assert.strictEqual(await Patients.find({owner: userId}).countAsync(), 1); assert.includeDeepMembers( await Allergies.find().mapAsync(({displayName, name}) => ({ diff --git a/imports/api/endpoint/settings/reset.tests.ts b/imports/api/endpoint/settings/reset.tests.ts index 564fdbfe6..fa86724ac 100644 --- a/imports/api/endpoint/settings/reset.tests.ts +++ b/imports/api/endpoint/settings/reset.tests.ts @@ -15,15 +15,15 @@ server(__filename, () => { const key = 'currency'; const value = 123_819_832; - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); await invoke(update, {userId}, [key, value]); - assert.equal(get(userId, key), value); + assert.strictEqual(get(userId, key), value); await invoke(reset, {userId}, [key]); - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); }); it('is idempotent', async () => { @@ -32,23 +32,23 @@ server(__filename, () => { const key = 'x'; const value = 123_819_832; - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); await invoke(reset, {userId}, [key]); - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); await invoke(update, {userId}, [key, value]); - assert.equal(get(userId, key), value); + assert.strictEqual(get(userId, key), value); await invoke(reset, {userId}, [key]); - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); await invoke(reset, {userId}, [key]); - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); }); it('cannot reset a setting if not logged in', async () => { @@ -70,13 +70,13 @@ server(__filename, () => { await invoke(update, {userId}, [key, value]); await invoke(update, {userId: `${userId}x`}, [key, valueX]); - assert.equal(get(userId, key), value); - assert.equal(get(`${userId}x`, key), valueX); + assert.strictEqual(get(userId, key), value); + assert.strictEqual(get(`${userId}x`, key), valueX); await invoke(reset, {userId}, [key]); - assert.equal(get(`${userId}x`, key), valueX); + assert.strictEqual(get(`${userId}x`, key), valueX); await invoke(reset, {userId}, [key]); - assert.equal(get(`${userId}x`, key), valueX); + assert.strictEqual(get(`${userId}x`, key), valueX); }); }); diff --git a/imports/api/endpoint/settings/update.tests.ts b/imports/api/endpoint/settings/update.tests.ts index 3d366d14c..fde8b3697 100644 --- a/imports/api/endpoint/settings/update.tests.ts +++ b/imports/api/endpoint/settings/update.tests.ts @@ -14,11 +14,11 @@ server(__filename, () => { const key = 'x'; const value = 123_819_832; - assert.equal(get(userId, key), defaults[key]); + assert.strictEqual(get(userId, key), defaults[key]); await invoke(update, {userId}, [key, value]); - assert.equal(get(userId, key), value); + assert.strictEqual(get(userId, key), value); }); it('can update a setting', async () => { @@ -30,11 +30,11 @@ server(__filename, () => { await invoke(update, {userId}, [key, value]); - assert.equal(get(userId, key), value); + assert.strictEqual(get(userId, key), value); await invoke(update, {userId}, [key, updated]); - assert.equal(get(userId, key), updated); + assert.strictEqual(get(userId, key), updated); }); it('cannot set a setting if not logged in', async () => { @@ -70,6 +70,6 @@ server(__filename, () => { await invoke(update, {userId}, [key, value]); - assert.equal(get(`${userId}x`, key), defaults[key]); + assert.strictEqual(get(`${userId}x`, key), defaults[key]); }); }); diff --git a/imports/api/transaction/TransactionDriver.tests.ts b/imports/api/transaction/TransactionDriver.tests.ts index 0cc91a402..fb0a9bdf8 100644 --- a/imports/api/transaction/TransactionDriver.tests.ts +++ b/imports/api/transaction/TransactionDriver.tests.ts @@ -29,7 +29,7 @@ server(__filename, () => { const Tests = defineCollection('test-8h9e8w89hf98239'); it2('insertOne', (db) => async () => { - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); const result = await db.insertOne(Tests, {test: 'test'}); assert.deepInclude(result, { acknowledged: true, @@ -46,7 +46,7 @@ server(__filename, () => { const x = randomId(); const y = randomId(); const z = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); const result = await db.insertMany(Tests, [{x}, {y}, {z}]); assert.deepInclude(result, { acknowledged: true, @@ -63,28 +63,28 @@ server(__filename, () => { const y = randomId(); const z = randomId(); const w = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); const expected = {y}; const actual = await db.findOne(Tests, expected); assert.deepEqual(dropId(actual), expected); const shoudBeNull = await db.findOne(Tests, {w}); - assert.equal(shoudBeNull, null); + assert.strictEqual(shoudBeNull, null); }); it2('fetch', (db) => async () => { const x = randomId(); const y = randomId(); const z = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); const expected = [{x}, {y}, {z}]; const actual = await db.fetch(Tests, {}); assert.sameDeepMembers(dropIds(actual), expected); @@ -95,36 +95,36 @@ server(__filename, () => { const y = randomId(); const z = randomId(); const w = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); assert.deepInclude(await db.deleteOne(Tests, {w}), { acknowledged: true, deletedCount: 0, }); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); assert.deepInclude(await db.deleteOne(Tests, {x}), { acknowledged: true, deletedCount: 1, }); - assert.equal(await Tests.find().countAsync(), 2); + assert.strictEqual(await Tests.find().countAsync(), 2); assert.deepInclude(await db.deleteOne(Tests, {}), { acknowledged: true, deletedCount: 1, }); - assert.equal(await Tests.find().countAsync(), 1); + assert.strictEqual(await Tests.find().countAsync(), 1); assert.deepInclude(await db.deleteOne(Tests, {}), { acknowledged: true, deletedCount: 1, }); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); assert.deepInclude(await db.deleteOne(Tests, {}), { acknowledged: true, deletedCount: 0, }); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); }); it2('deleteMany', (db) => async () => { @@ -132,31 +132,31 @@ server(__filename, () => { const y = randomId(); const z = randomId(); const w = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); assert.deepInclude(await db.deleteMany(Tests, {w}), { acknowledged: true, deletedCount: 0, }); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); assert.deepInclude(await db.deleteMany(Tests, {x}), { acknowledged: true, deletedCount: 1, }); - assert.equal(await Tests.find().countAsync(), 2); + assert.strictEqual(await Tests.find().countAsync(), 2); assert.deepInclude(await db.deleteMany(Tests, {}), { acknowledged: true, deletedCount: 2, }); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); assert.deepInclude(await db.deleteMany(Tests, {}), { acknowledged: true, deletedCount: 0, }); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); }); it2('updateOne', (db) => async () => { @@ -164,11 +164,11 @@ server(__filename, () => { const y = randomId(); const z = randomId(); const w = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); const op1 = await db.updateOne(Tests, {}, {$set: {w}}); assert.deepInclude(op1, { acknowledged: true, @@ -192,7 +192,7 @@ server(__filename, () => { const x = randomId(); const y = randomId(); const z = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); const result = await db.updateOne(Tests, {x}, {$set: {y}}, {upsert: true}); const {_id} = await Tests.findOneAsync(); assert.deepInclude(result, { @@ -201,7 +201,7 @@ server(__filename, () => { upsertedCount: 1, upsertedId: _id, }); - assert.equal(await Tests.find().countAsync(), 1); + assert.strictEqual(await Tests.find().countAsync(), 1); assert.deepInclude( await db.updateOne(Tests, {x}, {$set: {z}}, {upsert: true}), { @@ -210,7 +210,7 @@ server(__filename, () => { upsertedCount: 0, }, ); - assert.equal(await Tests.find().countAsync(), 1); + assert.strictEqual(await Tests.find().countAsync(), 1); assert.deepInclude( await db.updateOne(Tests, {x}, {$set: {z}}, {upsert: true}), { @@ -219,7 +219,7 @@ server(__filename, () => { upsertedCount: 0, }, ); - assert.equal(await Tests.find().countAsync(), 1); + assert.strictEqual(await Tests.find().countAsync(), 1); const expected = [{x, y, z}]; const actual = await Tests.find().fetchAsync(); assert.sameDeepMembers(dropIds(actual), expected); @@ -230,11 +230,11 @@ server(__filename, () => { const y = randomId(); const z = randomId(); const w = randomId(); - assert.equal(await Tests.find().countAsync(), 0); + assert.strictEqual(await Tests.find().countAsync(), 0); await db.insertOne(Tests, {x}); await db.insertOne(Tests, {y}); await db.insertOne(Tests, {z}); - assert.equal(await Tests.find().countAsync(), 3); + assert.strictEqual(await Tests.find().countAsync(), 3); const result = await db.updateMany(Tests, {}, {$set: {w}}); assert.deepInclude(result, { acknowledged: true, diff --git a/imports/lib/blob/dataURL.tests.ts b/imports/lib/blob/dataURL.tests.ts index 700ce08d4..7b96715fd 100644 --- a/imports/lib/blob/dataURL.tests.ts +++ b/imports/lib/blob/dataURL.tests.ts @@ -14,6 +14,6 @@ isomorphic(__filename, () => { assert.instanceOf(blob, Blob); const result = await blobToDataURL(blob); - assert.equal(result, url); + assert.strictEqual(result, url); }); }); diff --git a/server/api/healthcheck/index.app-tests.ts b/server/api/healthcheck/index.app-tests.ts index b904d96b6..3cec6ba45 100644 --- a/server/api/healthcheck/index.app-tests.ts +++ b/server/api/healthcheck/index.app-tests.ts @@ -11,7 +11,7 @@ server(__filename, () => { it('works', async () => { const response = await request(app).get(`/`); - assert.equal(response.status, 200); - assert.equal(response.text, 'OK'); + assert.strictEqual(response.status, 200); + assert.strictEqual(response.text, 'OK'); }); }); diff --git a/server/api/ics/index.app-tests.ts b/server/api/ics/index.app-tests.ts index 7818df423..4a411810c 100644 --- a/server/api/ics/index.app-tests.ts +++ b/server/api/ics/index.app-tests.ts @@ -56,8 +56,8 @@ server(__filename, () => { .get(`/appointments/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 422); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 422); + assert.strictEqual(response.text, ''); }); it('cannot download past month consultations list with invalid token', async () => { @@ -67,8 +67,8 @@ server(__filename, () => { .get(`/consultations/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 422); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 422); + assert.strictEqual(response.text, ''); }); it('cannot download past month and upcoming appointments list with revoked token', async () => { @@ -86,8 +86,8 @@ server(__filename, () => { .get(`/appointments/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 404); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 404); + assert.strictEqual(response.text, ''); }); it('cannot download past month consultations list with revoked token', async () => { @@ -105,8 +105,8 @@ server(__filename, () => { .get(`/consultations/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 404); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 404); + assert.strictEqual(response.text, ''); }); it('cannot download past month and upcoming appointments list with bad permissions', async () => { @@ -120,8 +120,8 @@ server(__filename, () => { .get(`/appointments/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 404); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 404); + assert.strictEqual(response.text, ''); }); it('cannot download past month consultations list with bad permissions', async () => { @@ -135,8 +135,8 @@ server(__filename, () => { .get(`/consultations/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 404); - assert.equal(response.text, ''); + assert.strictEqual(response.status, 404); + assert.strictEqual(response.text, ''); }); it('can download past month and upcoming appointments list with token', async () => { @@ -201,20 +201,20 @@ server(__filename, () => { .get(`/appointments/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 200); + assert.strictEqual(response.status, 200); const calendar = parse(response.text); const events = getEvents(calendar); - assert.equal(events.length, 2); + assert.strictEqual(events.length, 2); for (const [event, appointment] of zip( events, [appointmentA, appointmentB].sort(prop(decreasing, 'lastModifiedAt')), )) { - assert.equal(event.isRecurring(), false); - assert.equal(event.description, patient.phone); + assert.strictEqual(event.isRecurring(), false); + assert.strictEqual(event.description, patient.phone); assert( isSameDatetime( event.startDate.toJSDate(), @@ -227,7 +227,10 @@ server(__filename, () => { setMilliseconds(appointment.end, 0), ), ); - assert.equal(event.summary, `${patient.lastname} ${patient.firstname}`); + assert.strictEqual( + event.summary, + `${patient.lastname} ${patient.firstname}`, + ); assert( isSameDatetime( event._firstProp('created').toJSDate(), @@ -240,7 +243,7 @@ server(__filename, () => { setMilliseconds(appointment.lastModifiedAt, 0), ), ); - assert.equal( + assert.strictEqual( event._firstProp('url'), absoluteURL(`/consultation/${appointment._id}`), ); @@ -348,18 +351,18 @@ server(__filename, () => { .get(`/consultations/${token}/events.ics`) .set('Accept', 'text/calendar'); - assert.equal(response.status, 200); + assert.strictEqual(response.status, 200); const events = getEvents(parse(response.text)); - assert.equal(events.length, 2); + assert.strictEqual(events.length, 2); for (const [event, consultation] of zip( events, [consultationB, consultationA].sort(prop(decreasing, 'lastModifiedAt')), )) { - assert.equal(event.isRecurring(), false); - assert.equal(event.description, patient.phone); + assert.strictEqual(event.isRecurring(), false); + assert.strictEqual(event.description, patient.phone); assert( isSameDatetime( event.startDate.toJSDate(), @@ -372,7 +375,10 @@ server(__filename, () => { setMilliseconds(consultation.end, 0), ), ); - assert.equal(event.summary, `${patient.lastname} ${patient.firstname}`); + assert.strictEqual( + event.summary, + `${patient.lastname} ${patient.firstname}`, + ); assert( isSameDatetime( event._firstProp('created').toJSDate(), @@ -385,7 +391,7 @@ server(__filename, () => { setMilliseconds(consultation.lastModifiedAt, 0), ), ); - assert.equal( + assert.strictEqual( event._firstProp('url'), absoluteURL(`/consultation/${consultation._id}`), ); diff --git a/server/api/rateLimit.tests.ts b/server/api/rateLimit.tests.ts index a4a504a99..1ce3cb896 100644 --- a/server/api/rateLimit.tests.ts +++ b/server/api/rateLimit.tests.ts @@ -273,7 +273,7 @@ server(__filename, () => { // eslint-disable-next-line no-await-in-loop const response = await send(); - assert.equal(response.text, 'OK'); + assert.strictEqual(response.text, 'OK'); } }).timeout(3000); @@ -322,11 +322,11 @@ server(__filename, () => { assert.isAtMost(elapsed, duration * 1000); if (isEven) { - assert.equal(response.status, 200); - assert.equal(response.text, 'OK'); + assert.strictEqual(response.status, 200); + assert.strictEqual(response.text, 'OK'); } else { - assert.equal(response.status, 429); - assert.equal(response.text, 'Too Many Requests'); + assert.strictEqual(response.status, 429); + assert.strictEqual(response.text, 'Too Many Requests'); } } });