Add test for sqlite_math_functions tag

[lggruspe]

Hello, this is a continuation of #1037, which fixes #990.

Changes

• Adds test to check if sqlite math functions are compiled in
• Update github workflow to include sqlite_math_functions tag in full test

@rittneje @nkt-dk

[rittneje]

Right now you are dropping the Rows object. That means you are only checking for an error from sqlite3_prepare_v2 and not sqlite3_step. For completeness we should check both, which is why QueryRow would be preferable.

var result float64
if err := db.QueryRow(query).Scan(&result); err != nil {
    t.Errorf("invoking math function query %q: %v", query, err)
}
// For the purposes of this test, we don't care about the output, just that the query succeeded.

[lggruspe]

I see... thanks for the review :)

[mattn]

Seems okay to me.

[codecov-commenter]

Codecov Report

Base: 46.09% // Head: 46.09% // No change to project coverage 👍

Coverage data is based on head (d37b86e) compared to base (7476442).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1059   +/-   ##
=======================================
  Coverage   46.09%   46.09%           
=======================================
  Files          11       11           
  Lines        1499     1499           
=======================================
  Hits          691      691           
  Misses        669      669           
  Partials      139      139           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[rittneje]

Failure is due to unrelated issue introduced in #1085.