[PR #2732] [MERGED] Separate pjdfstest tests by command to make the log units smaller #2751

New issue

Closed

opened 2026-03-04 02:07:07 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:07:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2732
Author: @ggtakec
Created: 9/8/2025
Status: ✅ Merged
Merged: 9/8/2025
Merged by: @gaul

Base: master ← Head: update/test_pjdfstest

📝 Commits (1)

c1ceac7 Separate pjdfstest tests by command to make the log units smaller

📊 Changes

1 file changed (+131 additions, -34 deletions)

View changed files

📝 test/integration-test-main.sh (+131 -34)

📄 Description

Relevant Issue (if applicable)

n/a

Details

Currently, when a Github Actions(CI) test fails, we use filter-suite-log.sh to extract only the failed test items from the log.
(The execution log of small-integration-test.sh is huge, and if we were to output that log in the event of a failure, it would not be viewable on github.com. So that, we use filter-suite-log.sh.)

I recently noticed that this failure log has been getting larger.
This was occurring because the test_pjdfstest test did not have a describe statement.
(filter-suite-log.sh detects and filters strings output to the log using describe statements.)

(1) Therefore, this PR adds this describe statement.
(2) Also, the test_pjdfstest test is a single test item, which is a large unit. Therefore, I have also separated the test items within test_pjdfstest.

This PR allows us to separate the items and logs in the event of a test failure into smaller units.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/2732 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 9/8/2025 **Status:** ✅ Merged **Merged:** 9/8/2025 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `update/test_pjdfstest` --- ### 📝 Commits (1) - [`c1ceac7`](https://github.com/s3fs-fuse/s3fs-fuse/commit/c1ceac70d5b9cfa4f227f8afa715935eb7dcc5aa) Separate pjdfstest tests by command to make the log units smaller ### 📊 Changes **1 file changed** (+131 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `test/integration-test-main.sh` (+131 -34) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details Currently, when a Github Actions(CI) test fails, we use `filter-suite-log.sh` to extract only the failed test items from the log. _(The execution log of `small-integration-test.sh` is huge, and if we were to output that log in the event of a failure, it would not be viewable on github.com. So that, we use `filter-suite-log.sh`.)_ I recently noticed that this failure log has been getting larger. This was occurring because the `test_pjdfstest` test did not have a `describe` statement. _(`filter-suite-log.sh` detects and filters strings output to the log using `describe` statements.)_ (1) Therefore, this PR adds this `describe` statement. (2) Also, the `test_pjdfstest` test is a single test item, which is a large unit. Therefore, I have also separated the test items within `test_pjdfstest`. This PR allows us to separate the items and logs in the event of a test failure into smaller units. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>