-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
⚒️ Testing | Setup unit tests #411
base: main
Are you sure you want to change the base?
Conversation
<Project Sdk="Microsoft.NET.Sdk"> | ||
|
||
<PropertyGroup> | ||
<TargetFramework>netcoreapp3.1</TargetFramework> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At that point we cannot target .net5+ because Rg.Plugins
package has a dependency on .NET Framework 4.7.2 https://www.nuget.org/packages/Rg.Plugins.Popup . As we are gonna use Xamarin. Mopups
(a fork of Rg.Plugins
) with MAUI, I will target .net7 after the merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes:
SSW.Rewards.Mobile/src/maui/SSW.Rewards.Mobile/SSW.Rewards.Mobile/SSW.Rewards.Mobile.csproj
Line 103 in 0527ae3
<PackageReference Include="Mopups" Version="1.1.0" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hopefully this PR will come out of draft tomorrow :)
var nextSelectedIndex = Sut.Items.IndexOf(Sut.SelectedItem) + 1; | ||
|
||
// Act | ||
var raisedEvent = Assert.Raises<int>(handler => Sut.ScrollToRequested += handler, handler => Sut.ScrollToRequested -= handler, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason didn't manage to get it working with FluentAssertions
😕 With Assert
it works fine, but if anyone knows how to do that via FluentAssertions
- please let me know.
This PR: