fragile 0.0.3 → 0.0.4

data/lib/fragile/plugin/rss_input.rb

@@ -1,21 +1,36 @@
-# coding: utf-8
-require "rss"
-
-module Fragile
-  module Plugin
-    class RssInput
-      def initialize(config)
-        @url = config[:url]
-      end
-
-      def call(data=[])
-        rss = RSS::Parser.parse(@url)
-        urls = rss.items.map do |item|
-          { :title => item.title, :link => item.link }
-        end
-        data + urls
-      end
-    end
-  end
-end
-
+# coding: utf-8
+require "rss"
+
+module Fragile
+  module Plugin
+    class RssInput
+      module FeedItemNormalizer
+        def title
+          super.content
+        rescue
+          super
+        end
+
+        def link 
+          super.href
+        rescue
+          super
+        end
+      end
+
+      def initialize(config)
+        @url = config[:url]
+      end
+
+      def call(data=[])
+        rss = RSS::Parser.parse(@url)
+        urls = rss.items.map do |item|
+          item.extend FeedItemNormalizer
+          { :title => item.title, :link => item.link }
+        end
+        data + urls
+      end
+    end
+  end
+end
+

data/lib/fragile/plugin/select_filter.rb

@@ -1,18 +1,18 @@
-# coding: utf-8
-
-module Fragile
-  module Plugin
-    class SelectFilter
-      def initialize(config)
-        @proc = config[:proc]
-      end
-
-      def call(data)
-        data.select do |v|
-          @proc.call(v)
-        end
-      end
-    end
-  end
-end
-
+# coding: utf-8
+
+module Fragile
+  module Plugin
+    class SelectFilter
+      def initialize(config)
+        @proc = config[:proc]
+      end
+
+      def call(data)
+        data.select do |v|
+          @proc.call(v)
+        end
+      end
+    end
+  end
+end
+

data/lib/fragile/plugin_manager.rb

@@ -1,38 +1,38 @@
-# coding: utf-8
-# Name::      Fragile
-# Author::    tnakamura <http://d.hatena.ne.jp/griefworker>
-# Created::   Jun 15, 2012
-# Updated::   Jun 15, 2012
-# Copyright:: tnakamura Copyright (c) 2012
-# License::   Licensed under the MIT LICENSE.
-
-module Fragile
-  module Plugin; end
-
-  class PluginError < Exception
-  end
-
-  module PluginManager
-    def create_plugin(plugin, config)
-      if plugin.instance_of?(Class)
-        # クラスなら直接 new する
-        plugin.new(config)
-      else
-        # 文字列かシンボルならクラスを取得して new する
-        plugin_name = classify(plugin.to_s)
-        create_plugin_instance(plugin_name, config)
-      end
-    end
-
-    private
-    def create_plugin_instance(class_name, config)
-      klass = Fragile::Plugin.const_get(class_name)
-      klass.new(config)
-    end
-
-    def classify(name)
-      name.split("_").map{ |s| s.capitalize }.join
-    end
-  end
-end
-
+# coding: utf-8
+# Name::      Fragile
+# Author::    tnakamura <http://d.hatena.ne.jp/griefworker>
+# Created::   Jun 15, 2012
+# Updated::   Jun 15, 2012
+# Copyright:: tnakamura Copyright (c) 2012
+# License::   Licensed under the MIT LICENSE.
+
+module Fragile
+  module Plugin; end
+
+  class PluginError < Exception
+  end
+
+  module PluginManager
+    def create_plugin(plugin, config)
+      if plugin.instance_of?(Class)
+        # クラスなら直接 new する
+        plugin.new(config)
+      else
+        # 文字列かシンボルならクラスを取得して new する
+        plugin_name = classify(plugin.to_s)
+        create_plugin_instance(plugin_name, config)
+      end
+    end
+
+    private
+    def create_plugin_instance(class_name, config)
+      klass = Fragile::Plugin.const_get(class_name)
+      klass.new(config)
+    end
+
+    def classify(name)
+      name.split("_").map{ |s| s.capitalize }.join
+    end
+  end
+end
+

data/lib/fragile/version.rb

@@ -1,11 +1,11 @@
-# coding: utf-8
-# Name::      Fragile
-# Author::    tnakamura <http://d.hatena.ne.jp/griefworker>
-# Created::   Jun 15, 2012
-# Updated::   Jun 23, 2012
-# Copyright:: tnakamura Copyright (c) 2012
-# License::   Licensed under the MIT LICENSE.
-
-module Fragile
-  VERSION = "0.0.3"
-end
+# coding: utf-8
+# Name::      Fragile
+# Author::    tnakamura <http://d.hatena.ne.jp/griefworker>
+# Created::   Jun 15, 2012
+# Updated::   Jun 23, 2012
+# Copyright:: tnakamura Copyright (c) 2013
+# License::   Licensed under the MIT LICENSE.
+
+module Fragile
+  VERSION = "0.0.4"
+end

data/{test/dsl_test.rb → spec/dsl_spec.rb}

@@ -1,18 +1,17 @@
-# coding: utf-8
-require_relative "test_helper"
-
-describe "DSL" do
-  describe "#pipeline" do
-    include Fragile::DSL
-
-    after do
-      Fragile.application.pipelines.clear
-    end
-
-    it "パイプラインを登録できるべき" do
-      pipeline "foo" do;end
-      assert_equal true, Fragile.application.pipelines.has_key?("foo")
-    end
-  end
-end
-
+# coding: utf-8
+require_relative "spec_helper"
+
+describe Fragile::DSL do
+  describe "#pipeline" do
+    include Fragile::DSL
+
+    after do
+      Fragile.application.pipelines.clear
+    end
+
+    it "パイプラインを登録できるべき" do
+      pipeline "foo" do;end
+      expect(Fragile.application.pipelines.has_key?("foo")).to be_true
+    end
+  end
+end

data/{test/pipeline_manager_test.rb → spec/pipeline_manager_spec.rb}

@@ -1,94 +1,94 @@
-# coding: utf-8
-require_relative "test_helper"
-
-class TestApp
-  include Fragile::PipelineManager
-end
-
-class TestPlugin
-  def initialize(config)
-    @config = config
-  end
-
-  def call(data)
-    @config[:called] = true
-  end
-end
-
-describe "PipelineManager" do
-  describe "#define_pipeline" do
-    before do
-      @manager = TestApp.new
-    end
-
-    it "パイプラインを登録できるべき" do
-      @manager.define_pipeline :foo do;end
-      assert_equal true, @manager.pipelines.has_key?("foo")
-    end
-  end
-
-  describe "#pipeline_exist?" do
-    describe "存在しないパイプライン名のとき" do
-      before do
-        @manager = TestApp.new
-      end
-
-      it "false を返すべき" do
-        assert_equal false, @manager.pipeline_exist?("foo")
-      end
-    end
-
-    describe "存在するパイプライン名のとき" do
-      before do
-        @manager = TestApp.new
-        @manager.define_pipeline "bar" do;end
-      end
-
-      it "true を返すべき" do
-        assert_equal true, @manager.pipeline_exist?("bar")
-      end
-    end
-  end
-
-  describe "#pipelines" do
-    describe "パイプラインが登録されていないとき" do
-      before do
-        @manager = TestApp.new
-      end
-
-      it "空であるべき" do
-        assert_equal true, @manager.pipelines.empty?
-      end
-    end
-  end
-
-  describe "#run_pipeline" do
-    describe "登録されていないパイプラインを指定したとき" do
-      before do
-        @manager = TestApp.new
-      end
-
-      it "PipelineError が発生するべき" do
-        assert_raises Fragile::PipelineError do
-          @manager.run_pipeline("hoge")
-        end
-      end
-    end
-
-    describe "登録されているパイプラインを指定したとき" do
-      before do
-        @config = config = {}
-        @manager = TestApp.new
-        @manager.define_pipeline "hoge" do
-          use TestPlugin, config
-        end
-      end
-
-      it "パイプラインを実行できるべき" do
-        @manager.run_pipeline("hoge")
-        assert_equal true, @config[:called]
-      end
-    end
-  end
-end
-
+# coding: utf-8
+require_relative "spec_helper"
+
+class TestApp
+  include Fragile::PipelineManager
+end
+
+class TestPlugin
+  def initialize(config)
+    @config = config
+  end
+
+  def call(data)
+    @config[:called] = true
+  end
+end
+
+describe Fragile::PipelineManager do
+  describe "#define_pipeline" do
+    before do
+      @manager = TestApp.new
+    end
+
+    it "パイプラインを登録できるべき" do
+      @manager.define_pipeline :foo do;end
+      expect(@manager.pipelines.has_key?("foo")).to be_true
+    end
+  end
+
+  describe "#pipeline_exist?" do
+    describe "存在しないパイプライン名のとき" do
+      before do
+        @manager = TestApp.new
+      end
+
+      it "false を返すべき" do
+        expect(@manager.pipeline_exist?("foo")).to be_false
+      end
+    end
+
+    describe "存在するパイプライン名のとき" do
+      before do
+        @manager = TestApp.new
+        @manager.define_pipeline "bar" do;end
+      end
+
+      it "true を返すべき" do
+        expect(@manager.pipeline_exist?("bar")).to be_true
+      end
+    end
+  end
+
+  describe "#pipelines" do
+    describe "パイプラインが登録されていないとき" do
+      before do
+        @manager = TestApp.new
+      end
+
+      it "空であるべき" do
+        expect(@manager.pipelines).to be_empty
+      end
+    end
+  end
+
+  describe "#run_pipeline" do
+    describe "登録されていないパイプラインを指定したとき" do
+      before do
+        @manager = TestApp.new
+      end
+
+      it "PipelineError が発生するべき" do
+        expect {
+          @manager.run_pipeline("hoge")
+        }.to raise_error(Fragile::PipelineError)
+      end
+    end
+
+    describe "登録されているパイプラインを指定したとき" do
+      before do
+        @config = config = {}
+        @manager = TestApp.new
+        @manager.define_pipeline "hoge" do
+          use TestPlugin, config
+        end
+      end
+
+      it "パイプラインを実行できるべき" do
+        @manager.run_pipeline("hoge")
+        expect(@config[:called]).to be_true
+      end
+    end
+  end
+end
+

data/{test/pipeline_test.rb → spec/pipeline_spec.rb}

@@ -1,106 +1,108 @@
-# coding: utf-8
-require_relative "test_helper"
-
-describe "Pipeline" do
-  describe "#name" do
-    before do
-      @pipeline = Fragile::Pipeline.new("test")
-    end
-
-    it "設定した値を取得できるべき" do
-      assert_equal "test", @pipeline.name
-    end
-  end
-
-  describe "#retry_count" do
-    before do
-      @pipeline = Fragile::Pipeline.new("test")
-    end
-
-    it "値を設定できるべき" do
-      @pipeline.retry_count 5
-
-      actual = nil
-      @pipeline.instance_eval { actual = @retry_count }
-      assert_equal 5, actual
-    end
-  end
-
-  describe "#use" do
-    class TestPlugin
-      attr_accessor :config
-
-      def initialize(config)
-        @config = config
-      end
-
-      def call(data)
-      end
-    end
-
-    before do
-      @pipeline = Fragile::Pipeline.new("test")
-    end
-
-    it "プラグインを設定できるべき" do
-      @pipeline.use TestPlugin
-
-      actual_count = nil
-      @pipeline.instance_eval { actual_count = @plugins.count }
-      assert_equal 1, actual_count
-    end
-  end
-
-  describe "#run" do
-    class TestPlugin
-      attr_accessor :config
-
-      def initialize(config)
-        @config = config
-        @max_error_count = config[:max_error_count] || 0
-        @error_count = 0
-      end
-
-      def call(data)
-        if @error_count < @max_error_count
-          @error_count += 1
-          raise "Error #{@error_count}"
-        end
-        @config[:called] = true
-      end
-    end
-
-    it "パイプラインを実行できるべき" do
-      @data = {}
-      @pipeline = Fragile::Pipeline.new("test")
-      @pipeline.use TestPlugin, @data
-
-      @pipeline.run
-
-      assert_equal true, @data[:called]
-    end
-
-    it "@retry_count 回リトライできるべき" do
-      @data = {:max_error_count => 3}
-      @pipeline = Fragile::Pipeline.new("test")
-      @pipeline.use TestPlugin, @data
-      @pipeline.retry_count 2
-
-      @pipeline.run
-
-      assert_equal true, @data[:called]
-    end
-
-    it "@retry_count 回以上失敗すると PipelineError が発生するべき" do
-      @data = {:max_error_count => 4}
-      @pipeline = Fragile::Pipeline.new("test")
-      @pipeline.use TestPlugin, @data
-      @pipeline.retry_count 2
-
-      assert_raises Fragile::PipelineError do
-        @pipeline.run
-      end
-    end
-  end
-end
-
+# coding: utf-8
+require_relative "spec_helper"
+
+describe Fragile::Pipeline do
+  describe "#name" do
+    before do
+      @pipeline = Fragile::Pipeline.new("test")
+    end
+
+    it "設定した値を取得できるべき" do
+      expect(@pipeline.name).to eq("test")
+    end
+  end
+
+  describe "#retry_count" do
+    before do
+      @pipeline = Fragile::Pipeline.new("test")
+    end
+
+    it "値を設定できるべき" do
+      @pipeline.retry_count 5
+
+      actual = nil
+      @pipeline.instance_eval { actual = @retry_count }
+
+      expect(actual).to eq(5)
+    end
+  end
+
+  describe "#use" do
+    class TestPlugin
+      attr_accessor :config
+
+      def initialize(config)
+        @config = config
+      end
+
+      def call(data)
+      end
+    end
+
+    before do
+      @pipeline = Fragile::Pipeline.new("test")
+    end
+
+    it "プラグインを設定できるべき" do
+      @pipeline.use TestPlugin
+
+      actual_count = nil
+      @pipeline.instance_eval { actual_count = @plugins.count }
+
+      expect(actual_count).to eq(1)
+    end
+  end
+
+  describe "#run" do
+    class TestPlugin
+      attr_accessor :config
+
+      def initialize(config)
+        @config = config
+        @max_error_count = config[:max_error_count] || 0
+        @error_count = 0
+      end
+
+      def call(data)
+        if @error_count < @max_error_count
+          @error_count += 1
+          raise "Error #{@error_count}"
+        end
+        @config[:called] = true
+      end
+    end
+
+    it "パイプラインを実行できるべき" do
+      @data = {}
+      @pipeline = Fragile::Pipeline.new("test")
+      @pipeline.use TestPlugin, @data
+
+      @pipeline.run
+
+      expect(@data[:called]).to be_true
+    end
+
+    it "@retry_count 回リトライできるべき" do
+      @data = {:max_error_count => 3}
+      @pipeline = Fragile::Pipeline.new("test")
+      @pipeline.use TestPlugin, @data
+      @pipeline.retry_count 2
+
+      @pipeline.run
+
+      expect(@data[:called]).to be_true
+    end
+
+    it "@retry_count 回以上失敗すると PipelineError が発生するべき" do
+      @data = {:max_error_count => 4}
+      @pipeline = Fragile::Pipeline.new("test")
+      @pipeline.use TestPlugin, @data
+      @pipeline.retry_count 2
+
+      expect{
+        @pipeline.run
+      }.to raise_error(Fragile::PipelineError)
+    end
+  end
+end
+