Build and Test

fix: Incorrect comments #1258

Triggered via pull request December 3, 2025 01:54

BenjaminMichaelis

opened #854

bmichaelis/844

Status Success

Total duration 2m 32s

Artifacts –

build-and-test.yml

on: pull_request

automerge

Matrix: build-and-test

Annotations

31 warnings

automerge

Not a dependabot PR, skipping.

build-and-test on macOS-latest: src/Chapter03.Tests/Table03.03.Tests.cs#L31

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Chapter03.Tests/Table03.03.Tests.cs#L23

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on macOS-latest: src/Chapter03.Tests/Listing03.24.Tests.cs#L9

Use 'Assert.ThrowsExactly' instead of 'Assert.ThrowsException' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0039)

build-and-test on macOS-latest: src/Chapter03.Tests/Table03.03.Tests.cs#L39

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Chapter03.Tests/Table03.03.Tests.cs#L31

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Chapter03.Tests/Table03.03.Tests.cs#L23

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on macOS-latest: src/Chapter02.Tests/Listing02.28.Tests.cs#L9

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on macOS-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on ubuntu-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on ubuntu-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on ubuntu-latest: src/Chapter02.Tests/Listing02.28.Tests.cs#L9

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on ubuntu-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on ubuntu-latest: src/Chapter04/Table04.04.CheckForNull.cs#L58

The given expression is always of the provided ('object') type

build-and-test on ubuntu-latest: src/Chapter04/TicTacToe.cs#L113

#warning: '"Same move allowed multiple times."'

build-and-test on ubuntu-latest: src/Chapter04/Table04.04.CheckForNull.cs#L58

The given expression is always of the provided ('object') type

build-and-test on ubuntu-latest: src/Chapter04/TicTacToe.cs#L113

#warning: '"Same move allowed multiple times."'

build-and-test on ubuntu-latest: src/Chapter04/Listing04.59.WarningDirective.cs#L8

#warning: '"Same move allowed multiple times."'

build-and-test on ubuntu-latest: src/Chapter04/Listing04.59.WarningDirective.cs#L8

#warning: '"Same move allowed multiple times."'

build-and-test on windows-latest: src/Chapter02.Tests/Listing02.28.Tests.cs#L9

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on windows-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on windows-latest: src/Chapter02.Tests/Listing02.28.Tests.cs#L9

Prefer 'Assert.ThrowsExactly/ThrowsExactlyAsync' over '[ExpectedException]' (https://learn.microsoft.com/dotnet/core/testing/mstest-analyzers/mstest0006)

build-and-test on windows-latest: src/Shared/Tests/CompilerAssert.cs#L128

'MSTestVerifier' is obsolete: 'https://github.com/dotnet/roslyn-sdk/blob/main/src/Microsoft.CodeAnalysis.Testing/README.md#obsolete-packages'

build-and-test on windows-latest: src/Chapter04/Table04.04.CheckForNull.cs#L58

The given expression is always of the provided ('object') type

build-and-test on windows-latest: src/Chapter04/TicTacToe.cs#L113

#warning: '"Same move allowed multiple times."'

build-and-test on windows-latest: src/Chapter04/Table04.04.CheckForNull.cs#L58

The given expression is always of the provided ('object') type

build-and-test on windows-latest: src/Chapter04/Listing04.59.WarningDirective.cs#L8

#warning: '"Same move allowed multiple times."'

build-and-test on windows-latest: src/Chapter04/TicTacToe.cs#L113

#warning: '"Same move allowed multiple times."'