Skip to content

Commit f85e275

Browse files
fix: Major update label must be displayed only if user-defined version is greater or equal to installed version
1 parent f81bf1d commit f85e275

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

src/PackageInfo.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {
22
coerce,
33
diff,
44
gt,
5+
gte,
56
maxSatisfying,
67
prerelease,
78
ReleaseType,
@@ -98,7 +99,8 @@ export class PackageInfo {
9899
return Boolean(
99100
versionLatest &&
100101
versionInstalled &&
101-
diff(versionLatest, versionInstalled) === "major"
102+
diff(versionLatest, versionInstalled) === "major" &&
103+
gte(versionLatest, versionInstalled)
102104
)
103105
}
104106

src/extension.test.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,17 @@ describe("package diagnostics", () => {
253253
)
254254
})
255255

256+
it("valid dependency, with major already installed must not show 'major' tooltip", async () => {
257+
const { decorations, diagnostics } = await vscodeSimulator({
258+
packageJson: { dependencies: { "npm-outdated": "^1.0.0" } },
259+
packagesInstalled: { "npm-outdated": "2.0.0" },
260+
packagesRepository: { "npm-outdated": ["1.0.0", "1.0.1", "2.0.0"] },
261+
})
262+
263+
expect(diagnostics[0]?.message).toContain("available: 1.0.1")
264+
expect(decorations[0]).not.toContain("(attention: major update!)")
265+
})
266+
256267
it("dependency name is invalid", async () => {
257268
const { decorations, diagnostics } = await vscodeSimulator({
258269
packageJson: { dependencies: { "invalid!": "^1.0.0" } },

0 commit comments

Comments
 (0)