SpiNNakerManchester / SpiNNUtils

Utility classes and functions for SpiNNaker projects
Apache License 2.0
1 stars 5 forks source link

runs an intergation test in linux, windows and macos #272

Closed Christian-B closed 5 months ago

Christian-B commented 5 months ago

Supports https://github.com/SpiNNakerManchester/sPyNNaker/pull/1455

python_actions.yml with separate coverage-package, pylint-packages, mypy-packages

A better error in excuatable_finder.

Support for a logs.sqlite3 file download from github arficats

could not just set c_logs_dict to the downloaed file due ot: A downloaded does not have write permissions sqlite3 way of doing read only is to use a file url, which does not work in github actions Changing file permissions cross platform is messy

So the fix is to copy an log.sqlite from cfg external_binaries

Note if there are two log.sqlite3 files the one in external_binaries is ignored. (Also means it is only copied on the first use)

There was a test for unittest this but it failed on windows as temp dir tried to delete before the read locks was removed. Functionality tested by spynnaker

LogSqlLiteDatabase needs various methods supers ect as Convertor the other LogSqlLiteDatabase is run when config files are not available.

coveralls commented 5 months ago

Coverage Status

coverage: 88.119% (-0.2%) from 88.27% when pulling 6df3d1b71ebb54825d503ce1bb3250bbc4f1bcd5 on use_build into 1a1f5c1180341d2a9565411b73ed31e7bc529516 on master.