exercism / exercism/csharp

Bank Account (Unit Tests)

Aperta
#2,406 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C#
Stelle
427
Fork
383
Merge medio
2g 2h
PR unite (30g)
5

Descrizione

I am not sure if I am doing ti right, but the tests indicate that the cancellation token should be passed from a given source.

We received the following error when we ran your code:

BankAccountTests.cs(158,23): warning xUnit1051: Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)

I am sure the test case with the Tasks is the one with the issue, as indicated by the error message.

`csharp
[Fact(Skip = "Remove this Skip property to run this test")]
public async Task Can_handle_concurrent_transactions()
{
var account = new BankAccount();
account.Open();
for (int i = 0; i < 500; i++)
{
var tasks = new List();
tasks.Add(Task.Factory.StartNew(() =>
{
for (int j = 0; j < 100; j++)
{
account.Deposit(1m);
account.Withdraw(1m);
}
}));
await Task.WhenAll(tasks.ToArray());
}
Assert.Equal(0m, account.Balance);
}
`

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.