Skip to content

Commit

Permalink
Revert "Make AppOptions.Options required" (#612)
Browse files Browse the repository at this point in the history
  • Loading branch information
ivarne authored Apr 25, 2024
1 parent dd0de6c commit d10ab98
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
2 changes: 1 addition & 1 deletion src/Altinn.App.Core/Models/AppOptions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ public class AppOptions
/// <summary>
/// Gets or sets the list of options. Null indicates that no options are found
/// </summary>
public required List<AppOption>? Options { get; set; }
public List<AppOption>? Options { get; set; }

/// <summary>
/// Gets or sets the parameters used to generate the options.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ public void ToNameValueString_OptionParameters_ShouldConvertToHttpHeaderFormat()
{
var options = new AppOptions
{
Options = null,
Parameters = new Dictionary<string, string?>
Parameters = new Dictionary<string, string>
{
{ "lang", "nb" },
{ "level", "1" }
Expand All @@ -33,8 +32,7 @@ public void ToNameValueString_OptionParametersWithEmptyValue_ShouldConvertToHttp
{
var options = new AppOptions
{
Options = null,
Parameters = new Dictionary<string, string?>()
Parameters = new Dictionary<string, string>()
};

IHeaderDictionary headers = new HeaderDictionary
Expand All @@ -50,7 +48,6 @@ public void ToNameValueString_OptionParametersWithNullValue_ShouldConvertToHttpH
{
var options = new AppOptions
{
Options = null,
Parameters = null!
};

Expand All @@ -67,8 +64,7 @@ public void ToNameValueString_OptionParametersWithSpecialCharaters_IsValidAsHead
{
var options = new AppOptions
{
Options = null,
Parameters = new Dictionary<string, string?>
Parameters = new Dictionary<string, string>
{
{ "lang", "nb" },
{ "level", "1" },
Expand Down

0 comments on commit d10ab98

Please sign in to comment.