Skip to content
Snippets Groups Projects
Unverified Commit cec37bb2 authored by Rhys Arkins's avatar Rhys Arkins Committed by GitHub
Browse files

fix(pep440): fix exception matching two ranges (#28827)

parent dc301be3
No related branches found
No related tags found
No related merge requests found
......@@ -37,6 +37,14 @@ describe('modules/versioning/pep440/index', () => {
expect(pep440.equals(a, b)).toBe(expected);
});
it.each`
a | b | expected
${'1.0'} | ${'>=1.0.0'} | ${true}
${'>=3.8'} | ${'>=3.9'} | ${false}
`('matches($a, $b) === $expected', ({ a, b, expected }) => {
expect(pep440.matches(a, b)).toBe(expected);
});
it.each`
version | isSingle
${'1.2.3'} | ${true}
......
......@@ -16,7 +16,7 @@ export const supportedRangeStrategies: RangeStrategy[] = [
const {
compare: sortVersions,
satisfies: matches,
satisfies,
valid,
validRange,
explain,
......@@ -74,6 +74,10 @@ export { isVersion, matches };
const equals = (version1: string, version2: string): boolean =>
isVersion(version1) && isVersion(version2) && eq(version1, version2);
function matches(version: string, range: string): boolean {
return isVersion(version) && isValid(range) && satisfies(version, range);
}
export const api: VersioningApi = {
equals,
getMajor,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment