Skip to content

Commit

Permalink
test: add retry attribute to CancelLongTaskShouldSucceed test (#863)
Browse files Browse the repository at this point in the history
# Motivation

Avoid potential crash during the **Common Tests**.

# Description

Implement a retry attribute so that it can retry 3 times in case of a
crash.

# Testing

None

# Impact

Avoid unexpected crash when pushing a commit on Core.

# Additional Information

None

# Checklist

- [X] My code adheres to the coding and style guidelines of the project.
- [ ] I have performed a self-review of my code.
- [ ] I have commented my code, particularly in hard-to-understand
areas.
- [ ] I have made corresponding changes to the documentation.
- [ ] I have thoroughly tested my modifications and added tests when
necessary.
- [ ] Tests pass locally and in the CI.
- [ ] I have assessed the performance impact of my modifications.
  • Loading branch information
Nico-dl05 authored Feb 25, 2025
2 parents 291a147 + f619d41 commit 5a94168
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions Common/tests/Pollster/PollsterTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -654,6 +654,7 @@ await testServiceProvider.TaskTable.GetTaskStatus(taskSubmitted,

[Test]
[Timeout(30000)]
[Retry(3)]
public async Task CancelLongTaskShouldSucceed()
{
var mockPullQueueStorage = new SimplePullQueueStorageChannel();
Expand Down

0 comments on commit 5a94168

Please sign in to comment.