Skip to content

Commit

Permalink
feat(MongoBinaryDownloadUrl::translatePlatform): remove translation o…
Browse files Browse the repository at this point in the history
…f "sunos"

fixes #661

BREAKING CHANGE:
potentially breaking change removing "sunos" from translation
  • Loading branch information
hasezoey committed Aug 17, 2022
1 parent 068cce5 commit 31060b2
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
Expand Up @@ -505,14 +505,6 @@ export class MongoBinaryDownloadUrl implements MongoBinaryDownloadUrlOpts {
case 'linux':
case 'elementary OS':
return 'linux';
case 'sunos':
deprecate(
() => {},
'mongodb-memory-server will fully drop support for sunos in 9.0',
'MMS002'
)();

return 'sunos5';
default:
throw new UnknownPlatformError(platform);
}
Expand Down
Expand Up @@ -1386,5 +1386,17 @@ describe('MongoBinaryDownloadUrl', () => {
});
expect(du.translatePlatform('win32')).toBe('windows');
});

it('should throw when using "sunos"', () => {
const du = new MongoBinaryDownloadUrl({
platform: 'win32', // mock to use later call for actual testing
arch: 'x64',
version: '0.0.0',
os: {
os: 'win32',
},
});
expect(() => du.translatePlatform('sunos')).toThrowError(UnknownPlatformError);
});
});
});

0 comments on commit 31060b2

Please sign in to comment.