Skip to content

Commit 0caf33c

Browse files
authored
Hide FluentAssertions from new project wizard (#60)
* Hide FluentAssertions from new project wizard * Update CHANGELOG
1 parent c0b1a8a commit 0caf33c

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
## Improvements:
44

5+
* Suggestion for adding [FluentAssertions](https://github.com/fluentassertions/fluentassertions) on the new project wizard screen has been removed to avoid confusions, because FluentAssertion does not offer free use for commercial projects anymore. (#60)
56
* Step previews: do not convert options list parameter, e.g. '(option1|option2|option3)', to [string]
67

78
## Bug fixes:

Reqnroll.VisualStudio.UI/Dialogs/AddNewReqnrollProjectDialog.xaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@
251251
SelectionChanged="TestFramework_SelectionChanged" />
252252
</StackPanel>
253253

254-
<StackPanel Grid.Row="2" Grid.Column="0">
254+
<StackPanel Grid.Row="2" Grid.Column="0" Visibility="Collapsed">
255255
<CheckBox x:Name="FluentAssertionsCheckBox" Content="Add FluentAssertions library"
256256
IsChecked="{Binding FluentAssertionsIncluded}" />
257257
</StackPanel>

Reqnroll.VisualStudio/UI/ViewModels/AddNewReqnrollProjectViewModel.cs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ public class AddNewReqnrollProjectViewModel : INotifyPropertyChanged
1111
{
1212
DotNetFramework = Net8,
1313
UnitTestFramework = MsTest,
14-
FluentAssertionsIncluded = true
14+
FluentAssertionsIncluded = false
1515
};
1616
#endif
1717
private string _dotNetFramework = Net8;
@@ -27,7 +27,10 @@ public string DotNetFramework
2727
}
2828

2929
public string UnitTestFramework { get; set; } = MsTest;
30-
public bool FluentAssertionsIncluded { get; set; } = true;
30+
// FluentAssertions suggestion is temporarily hidden from the UI as it is not free for commercial use anymore.
31+
// See https://xceed.com/fluent-assertions-faq/
32+
// Maybe we could consider suggesting https://github.com/shouldly/shouldly instead.
33+
public bool FluentAssertionsIncluded { get; set; } = false;
3134
public ObservableCollection<string> TestFrameworks { get; } = new(new List<string> { "MSTest", "NUnit", "xUnit" });
3235

3336
public event PropertyChangedEventHandler PropertyChanged;

0 commit comments

Comments
 (0)