[Host.Kafka] Fix and improve commit logic · zarusz/SlimMessageBus@8bf1a5c

Failed test were found and 'fail-on-error' option is set to ***

Process completed with exit code 1.

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3, actions/setup-java@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L10

Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)

src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L22

Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)

src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L64

Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)

src/Tests/SlimMessageBus.Host.Test/Helpers/ReflectionUtilsTests.cs#L78

Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)

src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L65

Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)

src/SlimMessageBus.Host.Kafka/Consumer/KafkaPartitionConsumer.cs#L40

Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)

src/SlimMessageBus.Host.Kafka/KafkaMessageBus.cs#L164

Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)

src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L131

Either remove or fill this block of code. (https://rules.sonarsource.com/csharp/RSPEC-108)

src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L162

Await CancelAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)

src/SlimMessageBus.Host.Kafka/KafkaMessageBus.cs#L117

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Host.Kafka] Fix and improve commit logic #1105

build

[Host.Kafka] Fix and improve commit logic #1105

Jobs

Run details

Annotations

This job failed

Re-running jobs...