Add integration tests#27
Merged
Merged
Conversation
fraenky8
force-pushed
the
add-integration-tests
branch
2 times, most recently
from
8ed7b24 to
dc0b516
Compare
fraenky8
force-pushed
the
add-integration-tests
branch
2 times, most recently
from
cced0de to
dde5b73
Compare
fraenky8
force-pushed
the
add-integration-tests
branch
8 times, most recently
from
2d68f74 to
ca5a6fe
Compare
fraenky8
force-pushed
the
add-integration-tests
branch
from
b8405e4 to
0ad27dd
Compare
fraenky8
force-pushed
the
add-integration-tests
branch
from
f7fa270 to
c6fe266
Compare
fraenky8
force-pushed
the
add-integration-tests
branch
from
c6fe266 to
2ae8b7a
Compare
- Used MySQL 8 as first candidate - Added build flags - Added test data for temporal data types - Added and vendor'ed new dependencies
removed also the // +build !integration flags of the unit tests.
- support multiple SQL data files - the output directory is a temporary one - make the tests running in parallel add integration test for postgres:10/int2 add integration test for postgres:10/int4 add integration test for postgres:10/int8 add integration test for postgres:10/smallint add integration test for postgres:10/integer add integration test for postgres:10/bigint add integration test for postgres:10/smallserial add integration test for postgres:10/serial add integration test for postgres:10/bigserial add integration test for postgres:10/serial2 add integration test for postgres:10/serial4 add integration test for postgres:10/serial8
add integration test for postgres:10/varchar add integration test for postgres:10/character varying add integration test for postgres:10/character add integration test for postgres:10/char
add integration test for postgres:10/float add integration test for postgres:10/float4 add integration test for postgres:10/float8 add integration test for postgres:10/numeric add integration test for postgres:10/decimal add integration test for postgres:10/real add integration test for postgres:10/double precision
add integration test for postgres:10/time add integration test for postgres:10/timestamp add integration test for postgres:10/date add integration test for postgres:10/time without time zone add integration test for postgres:10/timestamp without time zone add integration test for postgres:10/time with time zone add integration test for postgres:10/timestamp with time zone add integration test for postgres:10/timestamptz
add integration test for mysql:8/integer add integration test for mysql:8/bigint add integration test for mysql:8/mediumint add integration test for mysql:8/smallint add integration test for mysql:8/tinyint
add integration test for mysql:8/tinytext add integration test for mysql:8/text add integration test for mysql:8/mediumtext add integration test for mysql:8/longtext
add integration test for mysql:8/tinyblob add integration test for mysql:8/blob add integration test for mysql:8/mediumblob add integration test for mysql:8/longblob
add integration test for mysql:8/char add integration test for mysql:8/varchar add integration test for mysql:8/binary add integration test for mysql:8/varbinary
add integration test for mysql:8/numeric add integration test for mysql:8/double add integration test for mysql:8/float add integration test for mysql:8/real add integration test for mysql:8/double precision
Refactored the test and helpers and added resetting of data etc. Now test containers are reused by different tests and reset in between. Also changes the test directory layout structure to have an intermediate directory per test, so that each test can own its directory.
With help of 🤖
Such that testdata is now shared and each test can declare expected.
And renamed a local settings variable in all other tests.
Regenerated the files in the integration tests.
To be able to run the integration tests with -race flag. Seems to be recommended to not mix-and-match coverage with it.
fraenky8
force-pushed
the
add-integration-tests
branch
from
eadde83 to
3359863
Compare
Detected and fixed by 🤖
Added a new user table to test various initialisms.
Fixed a long-standing bug where the table name itself was not proper (title) cased. Rearranged and grouped some vars. Also converted resource expiration to a proper time.Time value.
fraenky8
force-pushed
the
add-integration-tests
branch
from
1dcb3c0 to
3c8003a
Compare
- Use in-memory tmpfs - Loosen the logging/crash resistance via flags - Close the DB connections after each test
fraenky8
force-pushed
the
add-integration-tests
branch
from
11adb06 to
1d8266e
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 298 out of 3278 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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 aims to add integration tests for all supported database types.