Bank Account (Unit Tests)

[igakigongo]

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<Task>(); 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); }

[github-actions]

Hello. Thanks for opening an issue on Exercism 🙂

At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.

This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!

If you're interested in learning more about this auto-responder, please read this blog post.

[ErikSchierboom]

Could you share your code please?