Skip to content

Conversation

simonw
Copy link
Owner

@simonw simonw commented May 8, 2025

@simonw simonw added the ci label May 8, 2025
@simonw
Copy link
Owner Author

simonw commented May 8, 2025

Ouch, looks like it really doesn't like that ancient SQLite version:

undefined symbol: sqlite3_create_window_function

Traceback (most recent call last):
  File "<string>", line 1, in <module>
    import sqlite3; print(sqlite3.sqlite_version)
    ^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.13.3/x64/lib/python3.13/sqlite3/__init__.py", line 57, in <module>
    from sqlite3.dbapi2 import *
  File "/opt/hostedtoolcache/Python/3.13.3/x64/lib/python3.13/sqlite3/dbapi2.py", line 27, in <module>
    from _sqlite3 import *
ImportError: /opt/hostedtoolcache/Python/3.13.3/x64/lib/python3.13/lib-dynload/_sqlite3.cpython-313-x86_64-linux-gnu.so: undefined symbol: sqlite3_create_window_function
Error: Process completed with exit code 1.

Copy link

codecov bot commented May 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.64%. Comparing base (0aefbb6) to head (d83d7a0).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #654      +/-   ##
==========================================
- Coverage   95.65%   95.64%   -0.02%     
==========================================
  Files           8        8              
  Lines        2874     2867       -7     
==========================================
- Hits         2749     2742       -7     
  Misses        125      125              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@simonw simonw force-pushed the test-sqlite-versions branch from 290f250 to ba74194 Compare May 8, 2025 00:14
@simonw
Copy link
Owner Author

simonw commented May 8, 2025

Now this fails:

        else:
>           return self.conn.execute(sql)
E           sqlite3.OperationalError: no such table: pragma_function_list
sqlite_utils/db.py:535: OperationalError
=========================== short test summary info ============================
FAILED tests/test_plugins.py::test_prepare_connection - sqlite3.OperationalError: no such table: pragma_function_list
================== 1 failed, 988 passed, 22 skipped in 16.14s ==================

@simonw
Copy link
Owner Author

simonw commented May 8, 2025

I think this is wrong:

select distinct name from pragma_function_list

Should be:

select distinct name from pragma_function_list()

@simonw
Copy link
Owner Author

simonw commented May 8, 2025

Turns out other tests also skip if no pragma_function_list(), so I should do that.

@simonw simonw marked this pull request as ready for review May 8, 2025 00:38
@simonw simonw merged commit 0e4e270 into main May 8, 2025
86 checks passed
@simonw simonw deleted the test-sqlite-versions branch May 8, 2025 00:49
simonw added a commit that referenced this pull request May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant