Add the ability to use an array of versions in Parser.satisfies()

pull/466/head
Stas Tolpekin 3 years ago
parent f09411489c
commit 9f4429a564

@ -360,7 +360,7 @@ class Parser {
allDefinitions.forEach((key) => {
const currentDefinition = checkTree[key];
if (typeof currentDefinition === 'string') {
if (typeof currentDefinition === 'string' || Array.isArray(currentDefinition)) {
browsers[key] = currentDefinition;
browsersCounter += 1;
} else if (typeof currentDefinition === 'object') {
@ -399,7 +399,7 @@ class Parser {
const matchingDefinition = Utils.find(browserNames, name => (this.isBrowser(name, true)));
if (matchingDefinition !== void 0) {
return this.compareVersion(browsers[matchingDefinition]);
return this.compareVersions(browsers[matchingDefinition]);
}
}
@ -423,6 +423,19 @@ class Parser {
return browserNameLower === defaultBrowserName;
}
/**
* Check if browser version equals the version or equals one of versions
* @param {(string|string[])} versionsOrVersion versions strings array or version string
* @returns {boolean}
*/
compareVersions(versionsOrVersion) {
if (typeof versionsOrVersion === 'string') {
return this.compareVersion(versionsOrVersion);
}
return versionsOrVersion.some(version => this.compareVersion(version));
}
compareVersion(version) {
let expectedResults = [0];
let comparableVersion = version;

@ -80,7 +80,7 @@ test('Skip parsing shouldn\'t parse', (t) => {
});
test('Parser.satisfies should make simple comparisons', (t) => {
// also covers Parser.compareVersion() method
// also covers Parser.compareVersion() and Parser.compareVersions() methods
t.is(parser.satisfies({ opera: '>42' }), true);
t.is(parser.satisfies({ opera: '<44' }), true);
t.is(parser.satisfies({ opera: '=43.0.2442.1165' }), true);
@ -96,6 +96,12 @@ test('Parser.satisfies should make simple comparisons', (t) => {
t.is(parser.satisfies({ opera: '~43' }), true);
});
test('Parser.satisfies should make comparisons with array of versions', (t) => {
// also covers Parser.compareVersion() and Parser.compareVersions() methods
t.is(parser.satisfies({ opera: ['~42', '~43'] }), true);
t.is(parser.satisfies({ opera: ['~40', '~41'] }), false);
});
test('Parser.satisfies should make complex comparison', (t) => {
t.is(parser.satisfies({
macos: {

Loading…
Cancel
Save