MicrosoftLearning / AZ400-DesigningandImplementingMicrosoftDevOpsSolutions

AZ-400 Course Repository for Labs and Demos.
https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/
MIT License
518 stars 963 forks source link

Labs have warnings in them when deploying #542

Closed limeman40 closed 5 months ago

limeman40 commented 6 months ago

Contact Details

nicholas.clark@kdrp.com

What should we change?

Fixed all the warning in your solution `

[warning]tests/FunctionalTests/Web/WebPageHelpers.cs(25,16): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/WebPageHelpers.cs(25,16): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Controllers/OrderControllerIndex.cs(26,32): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Controllers/OrderControllerIndex.cs(26,32): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(55,43): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(55,43): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(55,43): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(55,43): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/BasketPageCheckout.cs(48,52): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/BasketPageCheckout.cs(48,52): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/BasketPageCheckout.cs(48,52): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/BasketPageCheckout.cs(48,52): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(95,43): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(95,43): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(95,43): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/IndexTest.cs(95,43): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/CheckoutTest.cs(65,44): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/CheckoutTest.cs(65,44): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj]

[warning]tests/FunctionalTests/Web/Pages/Basket/CheckoutTest.cs(65,44): Warning CS8602: Dereference of a possibly null reference.

/home/vsts/work/1/s/tests/FunctionalTests/Web/Pages/Basket/CheckoutTest.cs(65,44): warning CS8602: Dereference of a possibly null reference. [/home/vsts/work/1/s/tests/FunctionalTests/FunctionalTests.csproj] FunctionalTests -> /home/vsts/work/1/s/tests/FunctionalTests/bin/Debug/net7.0/FunctionalTests.dll

[warning]tests/PublicApiIntegrationTests/ProgramTest.cs(10,55): Warning CS8618: Non-nullable field '_application' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.

/home/vsts/work/1/s/tests/PublicApiIntegrationTests/ProgramTest.cs(10,55): warning CS8618: Non-nullable field '_application' must contain a non-null value when exiting constructor. Consider declaring the field as nullable. [/home/vsts/work/1/s/tests/PublicApiIntegrationTests/PublicApiIntegrationTests.csproj] PublicApiIntegrationTests -> /home/vsts/work/1/s/tests/PublicApiIntegrationTests/bin/Debug/net7.0/PublicApiIntegrationTests.dll `

There are over 39 warning in this solution this is truly bad. Your examples should be clean of warning and errors.

Lab

Other

Relevant screenshots

image

LuizMacedo commented 5 months ago

Fix by https://github.com/MicrosoftLearning/eShopOnWeb/pull/81. Thanks!