Skip to content

Conversation

@daipom
Copy link
Contributor

@daipom daipom commented Sep 10, 2025

Which issue(s) this PR fixes:

What this PR does / why we need it:

test_config.rb and test_plugin_helper.rb has same class name.

class ConfigTest < Test::Unit::TestCase

class ConfigTest < Test::Unit::TestCase

The test class name should be unique.

Docs Changes:
Not needed.

Release Note:
CI improvements.

@daipom daipom added this to the v1.16.10 milestone Sep 10, 2025
@daipom daipom added the CI Test/CI issues label Sep 10, 2025
**Which issue(s) this PR fixes**:
Fixes #

**What this PR does / why we need it**:

`test_config.rb` and `test_plugin_helper.rb` has same class name.

https://github.com/fluent/fluentd/blob/338050012279f401af41c95049d82cb98b2a370b/test/test_config.rb#L8

https://github.com/fluent/fluentd/blob/338050012279f401af41c95049d82cb98b2a370b/test/test_plugin_helper.rb#L5

The test class name should be unique.

**Docs Changes**:
Not needed.

**Release Note**:
CI improvements.

Signed-off-by: Shizuo Fujita <[email protected]>
Signed-off-by: Daijiro Fukuda <[email protected]>
@daipom daipom merged commit 269a0e0 into fluent:v1.16 Sep 11, 2025
40 of 41 checks passed
@daipom daipom deleted the barkport-pr5064 branch September 11, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Test/CI issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants