Add Roslyn analyzer for pubsub #1448
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Introduced a new diagnostic analyzer `WorkflowActivityAnalyzer` to check if workflow activities are registered in the DI container. Updated the solution to include new projects for analyzers and their tests. Added unit tests to verify analyzer functionality and updated documentation to reflect the new release and rules. Included necessary package references and streamlined testing setup. Signed-off-by: Nils Gruson <[email protected]>
Introduce `WorkflowRegistrationAnalyzer` and `WorkflowActivityAnalyzer` to validate workflow and activity registrations in the DI container. Implement corresponding code fix providers to suggest automatic fixes. Add tests to ensure functionality and robustness of the Dapr workflow framework. Signed-off-by: Nils Gruson <[email protected]>
Signed-off-by: Nils Gruson <[email protected]>
Added `<IncludeBuildOutput>false</IncludeBuildOutput>` to `Dapr.Workflow.Analyzers.csproj` to prevent the build output from being included as a library dependency. This change includes a comment for clarity and aligns with existing NuGet package properties. Signed-off-by: Nils Gruson <[email protected]>
Updated the WorkflowRegistrationCodeFixProvider to include logic for obtaining the semantic model and extracting workflow type names. Added methods for finding and creating the AddDaprWorkflow invocation. Modified the Utilities class to reference WebApplication. Expanded test coverage in WorkflowRegistrationCodeFixProviderTests with new cases for handling AddDaprWorkflow invocations, including standard and top-level statements. Signed-off-by: Nils Gruson <[email protected]>
- Introduced new projects for `Dapr.Pubsub.Analyzers` and `Dapr.Pubsub.Analyzers.Test` in the solution. - Updated solution configuration to include new analyzers for Debug and Release builds. - Added analyzer rule DAPR2001 to enforce calling `MapSubscribeHandler` for Dapr subscriptions. - Implemented `MapSubscribeHandlerCodeFixProvider` for automatic code fixes. - Developed `SubscriptionAnalyzer` to validate subscription method usage. - Added unit tests for the new analyzer and code fix provider. - Updated test utilities and configurations to support new functionality. Signed-off-by: Nils Gruson <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Added an analyzer to check if
MapSubscribeHandleris called in app startup when Dapr programmatic subscriptions are setup.Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1426
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: