CS8602: Dereference of a possibly null reference

With the tests migrated to .NET 6, we have improved nullability annotations on the core libraries, and thus the tests now recognize that `ViewData` can, potentially, be nullable. As a result, the tests are updated to account for this possibility.

Author: JeremyCaney

OnTopic.AspNetCore.Mvc.Tests/TopicViewComponentTest.cs

@@ -121,7 +121,7 @@ public async Task Menu_Invoke_ReturnsNavigationViewModel() {
 
       var result                = await viewComponent.InvokeAsync().ConfigureAwait(false);
       var concreteResult        = result as ViewViewComponentResult;
-      var model                 = concreteResult?.ViewData.Model as NavigationViewModel<NavigationTopicViewModel>;
+      var model                 = concreteResult?.ViewData?.Model as NavigationViewModel<NavigationTopicViewModel>;
 
       Assert.NotNull(model);
       Assert.Equal(_topic.GetWebPath(), model?.CurrentWebPath);
@@ -147,7 +147,7 @@ public async Task Menu_Invoke_ReturnsConfiguredNavigationRoot() {
 
       var result                = await viewComponent.InvokeAsync().ConfigureAwait(false);
       var concreteResult        = result as ViewViewComponentResult;
-      var model                 = concreteResult?.ViewData.Model as NavigationViewModel<NavigationTopicViewModel>;
+      var model                 = concreteResult?.ViewData?.Model as NavigationViewModel<NavigationTopicViewModel>;
 
       Assert.NotNull(model);
       Assert.Equal(webPath, model?.CurrentWebPath);
@@ -194,7 +194,7 @@ public async Task PageLevelNavigation_Invoke_ReturnsNavigationViewModel() {
 
       var result                = await viewComponent.InvokeAsync().ConfigureAwait(false);
       var concreteResult        = result as ViewViewComponentResult;
-      var model                 = concreteResult?.ViewData.Model as NavigationViewModel<NavigationTopicViewModel>;
+      var model                 = concreteResult?.ViewData?.Model as NavigationViewModel<NavigationTopicViewModel>;
 
       Assert.NotNull(model);
       Assert.Equal(_topic.GetWebPath(), model?.CurrentWebPath);
@@ -222,7 +222,7 @@ public async Task PageLevelNavigation_Invoke_ReturnsNull() {
 
       var result                = await viewComponent.InvokeAsync().ConfigureAwait(false);
       var concreteResult        = result as ViewViewComponentResult;
-      var model                 = concreteResult?.ViewData.Model as NavigationViewModel<NavigationTopicViewModel>;
+      var model                 = concreteResult?.ViewData?.Model as NavigationViewModel<NavigationTopicViewModel>;
 
       Assert.NotNull(model);
       Assert.Equal(webPath, model?.CurrentWebPath);
@@ -250,7 +250,7 @@ public async Task PageLevelNavigation_InvokeWithNullTopic_ReturnsNull()
 
       var result = await viewComponent.InvokeAsync().ConfigureAwait(false);
       var concreteResult = result as ViewViewComponentResult;
-      var model = concreteResult?.ViewData.Model as NavigationViewModel<NavigationTopicViewModel>;
+      var model = concreteResult?.ViewData?.Model as NavigationViewModel<NavigationTopicViewModel>;
 
       Assert.NotNull(model);
       Assert.Equal(String.Empty, model?.CurrentWebPath);