mattn / go-sqlite3 / 1074
54%

DEFAULT BRANCH: master
Ran
Jobs 12
Files 12
Run time 74min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
1074

push

travis-ci

web-flow 
Add mock versions of SQLiteDriver and SQLiteConn for +build !cgo (#819)

My app can use PostgreSQL and – optionally – SQLite. I would like to be
able to compile the app without cgo when SQLite isn't used, as this
removes the need for a C compiler which makes builds easier and faster,
especially for end-users.

In the simple case, this is not a problem go-sqlite3 already provides a
simple non-cgo mock so it compiles and gives a runtime error if you try
to use it anyway.

However, now I'd like to register a function for my SQLite connection to
match a PostgreSQL function like so:

	sql.Register("sqlite3_custom", &sqlite3.SQLiteDriver{
		ConnectHook: func(conn *sqlite3.SQLiteConn) error {
			return conn.RegisterFunc("pow", pow, true); err != nil {
		},
	})

But this makes it quite hard to keep the same logic since it refers to
types that don't exist with CGO_ENABLED=0. I will need to create a db.go
with `+build !cgo` and db_cgo.go with `+buid cgo` which duplicates all
the logic but with the sqlite hooks. In my case, this actually affects
quite a lot; for example I have a helper function which connects and
runs migrations and whatnot which looks like:

	type ConnectOptions struct {
		Connect    string // Connect string.
		Schema     []byte // Database schema to create on startup.
		Migrate    *Migrate
		SQLiteHook func(*sqlite3.SQLiteConn) error
	}

And I'd have to have two versions of that, too. You could perhaps do
something with interfaces, but because the sql.Register() call above
references the `sqlite3.SQLiteDriver.ConnectHook` struct field that's
not so straightforward (and wrapping stuff in interfaces probably won't
do much for the general clarity either).

This simplifies all of that by providing some common types that may be
used when setting up a SQLite connectin. I renamed the
`SQLiteDriverMock` to `&SQLiteDriver` for this reason. As far as I can
tell in my testing, this has no real downsides (but perha... (continued)

1012 of 1895 relevant lines covered (53.4%)

1544225.44 hits per line

Jobs
ID Job ID Ran Files Coverage
1 1074.1 12
53.29
 Travis Job 1074.1
2 1074.2 12
53.43
 Travis Job 1074.2
3 1074.3 12
53.43
 Travis Job 1074.3
4 1074.4 12
53.63
 Travis Job 1074.4
5 1074.5 12
53.63
 Travis Job 1074.5
6 1074.6 12
53.43
 Travis Job 1074.6
7 1074.7 12
53.29
 Travis Job 1074.7
8 1074.8 12
53.43
 Travis Job 1074.8
9 1074.9 12
53.43
 Travis Job 1074.9
10 1074.10 12
53.43
 Travis Job 1074.10
11 1074.11 12
53.63
 Travis Job 1074.11
12 1074.12 12
53.63
 Travis Job 1074.12
Source Files on build 1074