alphagov / notifications-net-client

.NET client for the GOV.UK Notify API
https://www.nuget.org/packages/GovukNotify/
MIT License
25 stars 20 forks source link

Update integration tests to wait before checking SMS cost data #189

Closed klssmith closed 1 month ago

klssmith commented 1 month ago

What problem does the pull request solve?

The tests which check if the costDetails properties for an SMS notification are not null have been failing at times because those fields are null. Those properties aren't ready immediately, but we can check if they are by checking whether isCostDataReady is true and adding in a wait if the cost data is not ready yet.

Checklist