Merge pull request #1 from kenhys/add-actions

Revise CI ready and use GitHub Actions

Author: okkez

.github/workflows/ci.yml

@@ -0,0 +1,32 @@
+name: Testing on Ubuntu
+on:
+  push:
+  pull_request:
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.head_ref || github.sha }}-${{ github.workflow }}
+  cancel-in-progress: true
+
+permissions: read-all
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby:
+          - 3.2
+          - 3.3
+          - 3.4
+    name: Ruby ${{ matrix.ruby }} unit testing on ${{ matrix.os }}
+    steps:
+    - uses: actions/checkout@v4
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - name: unit testing
+      run: |
+        bundle install --jobs 4 --retry 3
+        bundle exec rake test

fluent-plugin-obsolete-plugins.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = test_files
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.14"
+  spec.add_development_dependency "bundler", "~> 2.4"
   spec.add_development_dependency "rake", "~> 12.0"
   spec.add_development_dependency "test-unit", "~> 3.0"
   spec.add_development_dependency "test-unit-rr"

lib/fluent/plugin/filter_obsolete_plugins.rb

@@ -38,7 +38,7 @@ def configure(conf)
 
         obsolete_plugins = Gem.loaded_specs.keys & @obsolete_plugins.keys
         obsolete_plugins.each do |name|
-          $log.warn("#{name} is obsolete: #{@obsolete_plugins[name].chomp}")
+          log.warn("#{name} is obsolete: #{@obsolete_plugins[name].chomp}")
         end
         if @raise_error && !obsolete_plugins.empty?
           raise Fluent::ConfigError, "Detected obsolete plugins"

test/plugin/test_filter_obsolete_plugins.rb

@@ -23,7 +23,7 @@ class ObsoletePluginsFilterTest < Test::Unit::TestCase
       d.feed({ message: "This is test message." })
     end
     assert_equal([{ message: "This is test message." }], d.filtered_records)
-    assert_equal([], $log.logs)
+    assert_equal([], d.logs)
   end
 
   test "obsolete plugins" do
@@ -42,7 +42,7 @@ class ObsoletePluginsFilterTest < Test::Unit::TestCase
       "#{@time} [warn]: fluent-plugin-tail-multiline is obsolete: Merged in in_tail in Fluentd v0.10.45. [fluent/fluentd#269](https://github.com/fluent/fluentd/issues/269)\n",
       "#{@time} [warn]: fluent-plugin-hostname is obsolete: Use [filter\\_record\\_transformer](http://docs.fluentd.org/v0.12/articles/filter_record_transformer) instead.\n"
     ]
-    assert_equal(expected_logs, $log.logs)
+    assert_equal(expected_logs, d.logs)
   end
 
   test "raise error when detect obsolete plugins" do
@@ -56,11 +56,6 @@ class ObsoletePluginsFilterTest < Test::Unit::TestCase
     ex = assert_raise(Fluent::ConfigError) do
       create_driver(CONFIG + "raise_error yes")
     end
-    expected_logs = [
-      "#{@time} [warn]: fluent-plugin-tail-multiline is obsolete: Merged in in_tail in Fluentd v0.10.45. [fluent/fluentd#269](https://github.com/fluent/fluentd/issues/269)\n",
-      "#{@time} [warn]: fluent-plugin-hostname is obsolete: Use [filter\\_record\\_transformer](http://docs.fluentd.org/v0.12/articles/filter_record_transformer) instead.\n"
-    ]
-    assert_equal(expected_logs, $log.logs)
     assert_equal("Detected obsolete plugins", ex.message)
   end