Add comprehensive tests, benchmarks, and improve documentation and infrastructure - #8
Merged
Conversation
This commit includes several major improvements to the application structure: 1. Created new packages under internal/app/ for better code organization: - internal/app/lifecycle/ with startup.go and shutdown.go for common server lifecycle management - internal/app/web-server/ with server.go and routes.go for web server logic - internal/app/grpc-server/ with server.go for gRPC server logic - internal/app/grpc-client/ with client.go for gRPC client logic 2. Updated main.go files to use the new packages: - cmd/web-server/main.go now uses webserver package - cmd/grpc-server/main.go now uses grpcserver package - cmd/grpc-client/main.go now uses grpcclient package 3. Fixed Swagger documentation basePath issue: - Removed '/api' prefix from @router annotations in handler files - This resolves the duplicate basePath issue where Swagger UI was showing /api/api instead of /api - Updated docs/swagger.json, docs/swagger.yaml, and docs/docs.go accordingly 4. Improved route organization: - Extracted route setup to internal/app/web-server/routes.go - Made the code more modular and maintainable
…ive validation checks
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This PR introduces significant improvements to the project by adding comprehensive tests across multiple layers, performance benchmarks, and infrastructure enhancements: