Skip to content

Commit a13d1ba

Browse files
committed
fix: error message
1 parent 8d2d8ab commit a13d1ba

File tree

4 files changed

+16
-8
lines changed

4 files changed

+16
-8
lines changed

src/utils.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -501,7 +501,9 @@ export function hasAtomicOperators(
501501
}
502502
}
503503
if (allUndefined) {
504-
throw new MongoInvalidArgumentError('All atomic operators provided have undefined values.');
504+
throw new MongoInvalidArgumentError(
505+
'Update operations require that all atomic operators have defined values, but none were provided.'
506+
);
505507
}
506508
}
507509

test/integration/crud/bulk.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ describe('Bulk', function () {
6666
)
6767
.catch(error => error);
6868
expect(error.message).to.include(
69-
'All atomic operators provided have undefined values.'
69+
'Update operations require that all atomic operators have defined values, but none were provided'
7070
);
7171
});
7272
});
@@ -88,7 +88,7 @@ describe('Bulk', function () {
8888
)
8989
.catch(error => error);
9090
expect(error.message).to.include(
91-
'All atomic operators provided have undefined values.'
91+
'Update operations require that all atomic operators have defined values, but none were provided'
9292
);
9393
});
9494
});

test/integration/crud/client_bulk_write.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ describe('Client Bulk Write', function () {
5757
)
5858
.catch(error => error);
5959
expect(error.message).to.include(
60-
'All atomic operators provided have undefined values.'
60+
'Update operations require that all atomic operators have defined values, but none were provided'
6161
);
6262
});
6363
});
@@ -82,7 +82,7 @@ describe('Client Bulk Write', function () {
8282
)
8383
.catch(error => error);
8484
expect(error.message).to.include(
85-
'All atomic operators provided have undefined values.'
85+
'Update operations require that all atomic operators have defined values, but none were provided'
8686
);
8787
});
8888
});

test/integration/crud/crud_api.test.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -913,7 +913,9 @@ describe('CRUD API', function () {
913913
const error = await collection
914914
.updateOne({ a: 1 }, { $set: undefined, $unset: undefined }, { ignoreUndefined: true })
915915
.catch(error => error);
916-
expect(error.message).to.include('All atomic operators provided have undefined values.');
916+
expect(error.message).to.include(
917+
'Update operations require that all atomic operators have defined values, but none were provided'
918+
);
917919
});
918920
}
919921
);
@@ -937,7 +939,9 @@ describe('CRUD API', function () {
937939
const error = await collection
938940
.updateMany({ a: 1 }, { $set: undefined, $unset: undefined }, { ignoreUndefined: true })
939941
.catch(error => error);
940-
expect(error.message).to.include('All atomic operators provided have undefined values.');
942+
expect(error.message).to.include(
943+
'Update operations require that all atomic operators have defined values, but none were provided'
944+
);
941945
});
942946
}
943947
);
@@ -965,7 +969,9 @@ describe('CRUD API', function () {
965969
{ ignoreUndefined: true }
966970
)
967971
.catch(error => error);
968-
expect(error.message).to.include('All atomic operators provided have undefined values.');
972+
expect(error.message).to.include(
973+
'Update operations require that all atomic operators have defined values, but none were provided'
974+
);
969975
});
970976
}
971977
);

0 commit comments

Comments
 (0)