ci: enable test assets and cache them

[Tachi107]

As suggested in #63 (comment)

[wwylele]

Nice! Where is the test asset uploaded?

[Tachi107]

It’s stored in a short-lived GitHub server somewhere, I guess :) The caching action uses a caching api provided by GitHub which should be documented somewhere, but I didn’t really investigate. Data gets deleted after being unused for 7 days. Is it something we should care about?

…

Il giorno 6 ago 2025, alle ore 20:55, Weiyi Wang ***@***.***> ha scritto:  wwylele left a comment (wwylele/teakra#64) Nice! Where is the test asset uploaded? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

[wwylele]

This repo doesn't seem it can maintain activity once a week. So the cache will be deleted?

[Tachi107]

This repo doesn't seem it can maintain activity once a week. So the cache will be deleted?

I believe so, yes. And the file will get re-downloaded. Alternatively, would it make sense to store the file in the repo with git-lfs?

[wwylele]

git-lfs sounds good to me. It is also good for archiving. I don't think I have used my quota at all

[Tachi107]

I cannot add git-lfs files to the project myself. The way I did so was:

git lfs install
mkdir tests/data
touch tests/data/.keep
git add tests/data/.keep
git commit -m 'test: create tests/data directory'
mv build/teaklite2_tests_result tests/data/
git lfs track --filename tests/data/teaklite2_tests_result
git add .gitattributes
git commit -m 'test: track teaklite2_tests_result with git-lfs'
git add tests/data/teaklite2_tests_result
git commit -m 'test: add teaklite2_tests_result'
git lfs ls-files
baffcd4f80 * tests/data/teaklite2_tests_result

[wwylele]

I have uploaded it to https://github.com/wwylele/teakra/tree/master/src/test_verifier/data. I realized this has been downloading from someone else' server in

teakra/src/test_verifier/CMakeLists.txt

Lines 18 to 25 in a0364b6

	if(NOT EXISTS "${TEAKRA_TEST_ASSETS_DIR}/teaklite2_tests_result")
	message(STATUS "Downloading required samples...")
	file(DOWNLOAD
	"https://liushuyu.b-cdn.net/teaklite2_tests_result_20181208"
	"${TEAKRA_TEST_ASSETS_DIR}/teaklite2_tests_result"
	EXPECTED_HASH SHA256=${ASSET_SHA256SUM}
	SHOW_PROGRESS
	)

. We should change that line and redirect it to the in-repo file, though I don't have set up to test this locally

[Tachi107]

Hi again! I've reworked with patch set, combining Git LFS with GitHub Actions caching. This way, multiple repeated runs will download the big test asset from the cache instead of LFS storage. The cache is (supposed to be) faster, and LFS bandwidth is limited.

I've also disabled Git LFS downloads by default, since most users are unlikely to want to download that test asset, and it takes a long time to do so

[wwylele]

Thanks!