Skip to content

fix tests - #18

Open
royneary wants to merge 1 commit into
kiranandcode:masterfrom
royneary:fix-tests
Open

royneary wants to merge 1 commit into
kiranandcode:masterfrom
royneary:fix-tests

Conversation

@royneary

@royneary royneary commented Apr 26, 2026

Copy link
Copy Markdown
Contributor
  1. Make tests compile with caqti >= 2.0.0
  2. Fix an sqlite error message (I'm using sqlite 3.50.4)
  3. Use postgresql's peer authentication method. I couldn't figure out how to specify a postgres password, so I removed localhost:5432 from the postgresql URIs. Now, if a postgres user and database with the same name as my unix user exists, authentication works without password. I create the postgres user like this:
    CREATE USER christian CREATEDB;
    CREATE DATABASE christian OWNER christian;
    
  4. Allow running dune test without the --profile=test flag by removing all the (enabled_if (= %{profile} test)) conditions. Please let me know if these conditions are still important.

- make tests compile with caqti >= 2.0.0
- fix an sqlite error message
- use postgresql's peer authentication method
- allow running 'dune test' without the --profile=test flag
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.

1 participant