Added unit tests#150
Conversation
|
@Patryk-MM Project approved! 😄✅ And well done for completing the challenges! 👏 I love the UI/UX. Super smooth and user friendly. You leveraged Spectre Console really well. And you also added unit tests! 🤩 Feedback 🔍The README says the app uses User Secrets for the database connection string, but the DbContext currently has a hardcoded SQL Server connection string in OnConfiguring. It would be better to inject configuration into the context and read the connection string from configuration/user secrets consistently. Better yet, have a look into the Options pattern for configuration, as it's recommended by Microsoft and it's widely adopted in the industry.
🧑🏫Overall well done! 👏👏✋🏻Looking forward to seeing your next projects! |


No description provided.