fixed the % update issue in copyjobs (#2399)

Co-authored-by: Sakshi Gupta <sakshig@microsoft.com>
This commit is contained in:
sakshigupta12feb
2026-03-02 17:37:08 +05:30
committed by GitHub
parent 204444b878
commit b478f2732c
2 changed files with 9 additions and 1 deletions

View File

@@ -395,6 +395,14 @@ describe("CopyJobUtils", () => {
expect(result).toBe(false); expect(result).toBe(false);
}); });
it("should return false for different completion percentage", () => {
const jobs1 = [createMockJob("job1", "Running")];
const jobs2 = [{ ...createMockJob("job1", "Running"), CompletionPercentage: 75 }];
const result = CopyJobUtils.isEqual(jobs1, jobs2);
expect(result).toBe(false);
});
it("should return true for empty arrays", () => { it("should return true for empty arrays", () => {
const result = CopyJobUtils.isEqual([], []); const result = CopyJobUtils.isEqual([], []);
expect(result).toBe(true); expect(result).toBe(true);

View File

@@ -142,7 +142,7 @@ export function isEqual(prevJobs: CopyJobType[], newJobs: CopyJobType[]): boolea
if (!newJob) { if (!newJob) {
return false; return false;
} }
return prevJob.Status === newJob.Status; return prevJob.Status === newJob.Status && prevJob.CompletionPercentage === newJob.CompletionPercentage;
}); });
} }