Add integration tests #27
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
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
- 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
add integration test for mysql:8/time add integration test for mysql:8/timestamp add integration test for mysql:8/date add integration test for mysql:8/datetime add integration test for mysql:8/year
Removed the t.Parallel() call.
Run integration tests only on ubuntu-latest. Because there is no docker installed on MacOS, see here https://github.community/t5/GitHub-Actions/Why-is-Docker-not-installed-on-macOS/td-p/39364 And on Windows we get this error: vendor\github.com\ory\dockertest\docker\pkg\system\filesys_windows.go:113:24: cannot use uintptr(unsafe.Pointer(&sd[0])) (type uintptr) as type *"golang.org/x/sys/windows".SECURITY_DESCRIPTOR in assignment
Refactored for reuse of test- and expected data.
Fixed a reported issue right away.
- Updated all Go deps - Switched to dockertest/v3 - Added PG 17
fraenky8
force-pushed
the
add-integration-tests
branch
from
c6fe266 to
2ae8b7a
Compare
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.