Skip to content

Commit

Permalink
feat(NODE-5166): bump max wire version for 7.0 (#3659)
Browse files Browse the repository at this point in the history
Co-authored-by: Durran Jordan <durran@gmail.com>
  • Loading branch information
dariakp and durran committed May 4, 2023
1 parent c52a4ed commit 561229b
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions src/cmap/wire_protocol/constants.ts
@@ -1,7 +1,7 @@
export const MIN_SUPPORTED_SERVER_VERSION = '3.6';
export const MAX_SUPPORTED_SERVER_VERSION = '6.0';
export const MAX_SUPPORTED_SERVER_VERSION = '7.0';
export const MIN_SUPPORTED_WIRE_VERSION = 6;
export const MAX_SUPPORTED_WIRE_VERSION = 17;
export const MAX_SUPPORTED_WIRE_VERSION = 21;
export const OP_REPLY = 1;
export const OP_UPDATE = 2001;
export const OP_INSERT = 2002;
Expand Down
2 changes: 1 addition & 1 deletion test/unit/assorted/wire_version.test.js
Expand Up @@ -7,7 +7,7 @@ const { isHello } = require('../../mongodb');

const minCompatErrMsg = `minimum wire version ${
Number.MAX_SAFE_INTEGER - 1
}, but this version of the Node.js Driver requires at most 17`;
}, but this version of the Node.js Driver requires at most 21`;
const maxCompatErrMsg = `reports maximum wire version 1, but this version of the Node.js Driver requires at least 6`;

describe('Wire Protocol Version', () => {
Expand Down
8 changes: 4 additions & 4 deletions test/unit/cmap/wire_protocol/constants.test.js
Expand Up @@ -14,8 +14,8 @@ describe('Wire Protocol Constants', function () {
});

describe('MAX_SUPPORTED_SERVER_VERSION', function () {
it('returns 6.0', function () {
expect(MAX_SUPPORTED_SERVER_VERSION).to.equal('6.0');
it('returns 7.0', function () {
expect(MAX_SUPPORTED_SERVER_VERSION).to.equal('7.0');
});
});

Expand All @@ -26,8 +26,8 @@ describe('Wire Protocol Constants', function () {
});

describe('MAX_SUPPORTED_WIRE_VERSION', function () {
it('returns 17', function () {
expect(MAX_SUPPORTED_WIRE_VERSION).to.equal(17);
it('returns 21', function () {
expect(MAX_SUPPORTED_WIRE_VERSION).to.equal(21);
});
});
});

0 comments on commit 561229b

Please sign in to comment.