Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add postgres and sqlite repository test. #252

Merged
merged 3 commits into from
Feb 25, 2023
Merged

Conversation

pengweiqhca
Copy link
Contributor

@pengweiqhca pengweiqhca commented Feb 14, 2023

  1. Fix postgres and sqlite some bug.
  2. Test System.Data.SqlClient and MySqlConnector driver.
  3. Fix error on update using SqlProvider.PostgreSQL #242 Create tests for PostgreSql #153 Create SQLite Tests #118

1. Fix postgres and sqlite some bug.
2. Test System.Data.SqlClient and MySqlConnector driver test.
@phnx47
Copy link
Owner

phnx47 commented Feb 14, 2023

@pengweiqhca Hi, I will look PR later. Thanks!

@phnx47
Copy link
Owner

phnx47 commented Feb 15, 2023

@pengweiqhca Why we need new project MicroOrm.Dapper.Repositories.Tests2? Maybe we can use one project for tests?

@pengweiqhca
Copy link
Contributor Author

pengweiqhca commented Feb 15, 2023

@pengweiqhca Why we need new project MicroOrm.Dapper.Repositories.Tests2? Maybe we can use one project for tests?

  1. Test the different driver of mssql, mysql and sqlite.
  2. postgres bool and oracle bool not compatible in same process.

If use one project, will use different TargetFramework. Use two project or two TargetFramework?

@phnx47
Copy link
Owner

phnx47 commented Feb 16, 2023

@pengweiqhca I can check/test it only next week. Add PostgreSql to docker-compose.yml, please.

@phnx47
Copy link
Owner

phnx47 commented Feb 16, 2023

I will think about splitting tests per Provider.

@phnx47 phnx47 merged commit d978262 into phnx47:main Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

error on update using SqlProvider.PostgreSQL
2 participants