influxdb-rails 1.0.0.beta2 → 1.0.1.beta2

data/spec/support/rails4/app.rb

@@ -1,44 +0,0 @@
-require "action_controller/railtie"
-require "active_record"
-
-app = Class.new(Rails::Application)
-app.config.secret_key_base = "1234567890abcdef1234567890abcdef"
-app.config.secret_token = "1234567890abcdef1234567890abcdef"
-app.config.session_store :cookie_store, key: "_myapp_session"
-app.config.active_support.deprecation = :log
-app.config.eager_load = false
-app.config.root = __dir__
-Rails.backtrace_cleaner.remove_silencers!
-app.initialize!
-
-app.routes.draw do
-  resources :widgets
-end
-
-InfluxDB::Rails.configure do |config|
-end
-
-ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
-ActiveRecord::Schema.define do
-  create_table :widgets, force: true do |t|
-    t.string :title
-
-    t.timestamps
-  end
-end
-
-class Widget < ActiveRecord::Base; end
-class ApplicationController < ActionController::Base; end
-class WidgetsController < ApplicationController
-  prepend_view_path File.join(__dir__, "..", "views")
-
-  def index
-    Widget.create!(title: "test")
-  end
-
-  def new
-    1 / 0
-  end
-end
-
-Object.const_set(:ApplicationHelper, Module.new)

data/spec/unit/backtrace_spec.rb

@@ -1,85 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails::Backtrace do
-  before do
-    @raw_backtrace = [
-      "/var/www/current/app/models/foo.rb:10:in `bar'",
-      "/var/www/current/app/models/foo.rb:19:in `baz'",
-      "/var/www/current/app/models/foo.rb:32:in `<main>'"
-    ]
-
-    @backtrace = InfluxDB::Rails::Backtrace.new(@raw_backtrace)
-  end
-
-  it "should accept an exception into the initializer" do
-    expect(@backtrace.lines).not_to be_empty
-    expect(@backtrace.lines.count).to eq(3)
-  end
-
-  it "should correctly parse lines into their elements" do
-    line = @backtrace.lines.first
-
-    expect(line.file).to eq("/var/www/current/app/models/foo.rb")
-    expect(line.number).to eq("10")
-    expect(line.method).to eq("bar")
-  end
-
-  describe "#to_a" do
-    it "should return an array of lines" do
-      expect(@backtrace.to_a.is_a?(Array)).to be_truthy
-    end
-  end
-
-  context "nil backtrace" do
-    before do
-      @raw_backtrace = nil
-      @backtrace = InfluxDB::Rails::Backtrace.new(@raw_backtrace)
-    end
-
-    it "should accept an exception into the initializer" do
-      expect(@backtrace.lines).to be_empty
-      expect(@backtrace.lines.count).to eq(0)
-    end
-
-    describe "#to_a" do
-      it "should return an array of lines" do
-        expect(@backtrace.to_a.is_a?(Array)).to be_truthy
-      end
-    end
-  end
-
-  describe "backtrace filters" do
-    before do
-      InfluxDB::Rails.configure do |config|
-        config.application_root = "/var/www/current"
-      end
-    end
-
-    it "should apply a single default backtrace filter correctly" do
-      filtered_backtrace = InfluxDB::Rails::Backtrace.new(@raw_backtrace)
-
-      line = filtered_backtrace.lines.first
-      expect(line.file).to eq("[APP_ROOT]/app/models/foo.rb")
-    end
-
-    it "should all default backtrace filters correctly" do
-      extended_backtrace = @raw_backtrace.dup
-      extended_backtrace << "#{Gem.path.first}/lib/foo_gem.rb:1:in `blah'"
-
-      filtered_backtrace = InfluxDB::Rails::Backtrace.new(extended_backtrace)
-      expect(filtered_backtrace.lines.first.file).to eq("[APP_ROOT]/app/models/foo.rb")
-      expect(filtered_backtrace.lines.last.file).to eq("[GEM_ROOT]/lib/foo_gem.rb")
-    end
-
-    it "should allow the addition of custom backtrace filters" do
-      InfluxDB::Rails.configure do |config|
-        config.backtrace_filters << ->(line) { line.gsub(/foo/, "F00") }
-      end
-
-      filtered_backtrace = InfluxDB::Rails::Backtrace.new(@raw_backtrace)
-
-      line = filtered_backtrace.lines.first
-      expect(line.file).to eq("[APP_ROOT]/app/models/F00.rb")
-    end
-  end
-end

data/spec/unit/exception_presenter_spec.rb

@@ -1,23 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails::ExceptionPresenter do
-  before do
-    begin
-      1 / 0
-    rescue ZeroDivisionError => e
-      @exception = e
-    end
-  end
-
-  describe ".new" do
-    it "should create a new ExceptionPresenter" do
-      exception_presenter = InfluxDB::Rails::ExceptionPresenter.new(@exception)
-      expect(exception_presenter).to be_a(InfluxDB::Rails::ExceptionPresenter)
-    end
-
-    it "should accept an exception as a parameter" do
-      exception_presenter = InfluxDB::Rails::ExceptionPresenter.new(@exception)
-      expect(exception_presenter).not_to be_nil
-    end
-  end
-end

data/spec/unit/influxdb_rails_spec.rb

@@ -1,78 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails do
-  before do
-    InfluxDB::Rails.configure do |config|
-      config.application_name = "my-rails-app"
-      config.ignored_environments = []
-      config.time_precision = "ms"
-    end
-  end
-
-  describe ".current_timestamp" do
-    let(:timestamp) { 1_513_009_229_111 }
-
-    it "should return the current timestamp in the configured precision" do
-      expect(Process).to receive(:clock_gettime)
-        .with(Process::CLOCK_REALTIME, :millisecond)
-        .and_return(timestamp)
-
-      expect(InfluxDB::Rails.current_timestamp).to eq(timestamp)
-    end
-  end
-
-  describe ".ignorable_exception?" do
-    it "should be true for exception types specified in the configuration" do
-      class DummyException < RuntimeError; end
-      exception = DummyException.new
-
-      InfluxDB::Rails.configure do |config|
-        config.ignored_exceptions << "DummyException"
-      end
-
-      expect(InfluxDB::Rails.ignorable_exception?(exception)).to be_truthy
-    end
-
-    it "should be true for exception types specified in the configuration" do
-      exception = ActionController::RoutingError.new("foo")
-      expect(InfluxDB::Rails.ignorable_exception?(exception)).to be_truthy
-    end
-
-    it "should be false for valid exceptions" do
-      exception = ZeroDivisionError.new
-      expect(InfluxDB::Rails.ignorable_exception?(exception)).to be_falsey
-    end
-  end
-
-  describe "rescue" do
-    it "should transmit an exception when passed" do
-      expect(InfluxDB::Rails.client).to receive(:write_point)
-
-      InfluxDB::Rails.rescue do
-        raise ArgumentError, "wrong"
-      end
-    end
-
-    it "should also raise the exception when in an ignored environment" do
-      InfluxDB::Rails.configure do |config|
-        config.ignored_environments = %w[development test]
-      end
-
-      expect do
-        InfluxDB::Rails.rescue do
-          raise ArgumentError, "wrong"
-        end
-      end.to raise_error(ArgumentError)
-    end
-  end
-
-  describe "rescue_and_reraise" do
-    it "should transmit an exception when passed" do
-      expect(InfluxDB::Rails.client).to receive(:write_point)
-
-      expect do
-        InfluxDB::Rails.rescue_and_reraise { raise ArgumentError, "wrong" }
-      end.to raise_error(ArgumentError)
-    end
-  end
-end

data/spec/unit/middleware/render_subscriber_spec.rb

@@ -1,90 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails::Middleware::RenderSubscriber do
-  let(:config) { InfluxDB::Rails::Configuration.new }
-  let(:logger) { double(:logger) }
-
-  before do
-    allow(config).to receive(:application_name).and_return("my-rails-app")
-    allow(config).to receive(:ignored_environments).and_return([])
-    allow(config).to receive(:time_precision).and_return("ms")
-  end
-
-  describe ".call" do
-    let(:start_time)   { Time.at(1_517_567_368) }
-    let(:finish_time)  { Time.at(1_517_567_370) }
-    let(:series_name) { "series_name" }
-    let(:payload) { { identifier: "index.html", count: 43, cache_hits: 42 } }
-    let(:result) do
-      {
-        values:    {
-          value: 2000
-        },
-        tags:      {
-          filename:   "index.html",
-          location:   "Foo#bar",
-          count:      43,
-          cache_hits: 42
-        },
-        timestamp: 1_517_567_370_000
-      }
-    end
-
-    subject { described_class.new(config, series_name) }
-
-    before do
-      Thread.current[:_influxdb_rails_controller] = "Foo"
-      Thread.current[:_influxdb_rails_action]     = "bar"
-    end
-
-    after do
-      Thread.current[:_influxdb_rails_action]     = nil
-      Thread.current[:_influxdb_rails_controller] = nil
-    end
-
-    context "successfully" do
-      it "writes to InfluxDB" do
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with(
-          series_name, result
-        )
-        subject.call("name", start_time, finish_time, "id", payload)
-      end
-
-      context "with empty tags" do
-        before do
-          payload[:count] = nil
-          result[:tags].delete(:count)
-        end
-
-        it "does not write empty tags" do
-          expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with(
-            series_name, result
-          )
-          subject.call("name", start_time, finish_time, "id", payload)
-        end
-      end
-
-      context "disabled" do
-        subject { described_class.new(config, nil) }
-
-        it "does not write a data point" do
-          expect_any_instance_of(InfluxDB::Client).not_to receive(:write_point)
-          subject.call("name", start_time, finish_time, "id", payload)
-        end
-      end
-    end
-
-    context "unsuccessfully" do
-      before do
-        allow(config).to receive(:logger).and_return(logger)
-        InfluxDB::Rails.configuration = config
-      end
-
-      it "does log exceptions" do
-        allow_any_instance_of(InfluxDB::Client).to receive(:write_point).and_raise("boom")
-        expect(logger).to receive(:error).with(/boom/)
-        subject.call("name", start_time, finish_time, "id", payload)
-      end
-    end
-  end
-end

data/spec/unit/middleware/request_subscriber_spec.rb

@@ -1,106 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails::Middleware::RequestSubscriber do
-  let(:config) { InfluxDB::Rails::Configuration.new }
-
-  before do
-    allow(config).to receive(:time_precision).and_return("ms")
-  end
-
-  subject { described_class.new(config) }
-
-  describe "#call" do
-    let(:start)   { Time.at(1_517_567_368) }
-    let(:finish)  { Time.at(1_517_567_370) }
-    let(:payload) { { view_runtime: 2, db_runtime: 2, controller: "MyController", action: "show", method: "GET", format: "*/*", status: 200 } }
-    let(:data)    do
-      {
-        values:    {
-          value: 2
-        },
-        tags:      {
-          method:      "MyController#show",
-          status:      200,
-          format:      "*/*",
-          http_method: "GET",
-          server:      Socket.gethostname,
-          app_name:    "my-rails-app",
-        },
-        timestamp: 1_517_567_370_000
-      }
-    end
-
-    context "application_name is set" do
-      before do
-        allow(config).to receive(:application_name).and_return("my-rails-app")
-      end
-
-      it "sends metrics with taggings and timestamps" do
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with(
-          "rails.controller", data.merge(values: { value: 2000 })
-        )
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.view", data)
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.db", data)
-
-        subject.call("unused", start, finish, "unused", payload)
-      end
-
-      context "when tags_middleware is overwritten" do
-        before do
-          allow(config).to receive(:tags_middleware).and_return(tags_middleware)
-        end
-
-        let(:tags_middleware) { ->(tags) { tags.merge(static: "value") } }
-
-        it "processes tags throught the middleware" do
-          tags = data[:tags].merge(static: "value")
-
-          expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.controller", include(tags: tags))
-          expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.view", include(tags: tags))
-          expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.db", include(tags: tags))
-
-          subject.call("unused", start, finish, "unused", payload)
-        end
-      end
-    end
-
-    context "application_name is nil" do
-      before do
-        allow(config).to receive(:application_name).and_return(nil)
-      end
-
-      it "does not add the app_name tag to metrics" do
-        tags = {
-          method:      "MyController#show",
-          status:      200,
-          format:      "*/*",
-          http_method: "GET",
-          server:      Socket.gethostname,
-        }
-
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with(
-          "rails.controller", data.merge(values: { value: 2000 }, tags: tags)
-        )
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.view", data.merge(tags: tags))
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with("rails.db", data.merge(tags: tags))
-
-        subject.call("unused", start, finish, "unused", payload)
-      end
-    end
-
-    context "not successfull" do
-      let(:logger) { double(:logger) }
-
-      before do
-        allow(config).to receive(:logger).and_return(logger)
-        InfluxDB::Rails.configuration = config
-      end
-
-      it "does log an error" do
-        allow_any_instance_of(InfluxDB::Client).to receive(:write_point).and_raise("boom")
-        expect(logger).to receive(:error).with(/boom/)
-        subject.call("name", start, finish, "id", payload)
-      end
-    end
-  end
-end

data/spec/unit/middleware/sql_subscriber_spec.rb

@@ -1,79 +0,0 @@
-require "spec_helper"
-
-RSpec.describe InfluxDB::Rails::Middleware::SqlSubscriber do
-  let(:config) { InfluxDB::Rails::Configuration.new }
-  let(:logger) { double(:logger) }
-
-  before do
-    allow(config).to receive(:application_name).and_return("my-rails-app")
-    allow(config).to receive(:ignored_environments).and_return([])
-    allow(config).to receive(:time_precision).and_return("ms")
-  end
-
-  describe ".call" do
-    let(:start_time)   { Time.at(1_517_567_368) }
-    let(:finish_time)  { Time.at(1_517_567_370) }
-    let(:series_name) { "series_name" }
-    let(:payload) { { sql: "SELECT * FROM POSTS WHERE id = 1", name: "Post Load", binds: %w[1 2 3] } }
-    let(:result) do
-      {
-        values:    {
-          value: 2000,
-          sql:   "SELECT * FROM POSTS WHERE id = xxx"
-        },
-        tags:      {
-          location:   "Foo#bar",
-          operation:  "SELECT",
-          class_name: "Post",
-          name:       "Post Load",
-        },
-        timestamp: 1_517_567_370_000
-      }
-    end
-
-    subject { described_class.new(config, series_name) }
-
-    before do
-      Thread.current[:_influxdb_rails_controller] = "Foo"
-      Thread.current[:_influxdb_rails_action]     = "bar"
-    end
-
-    after do
-      Thread.current[:_influxdb_rails_action]     = nil
-      Thread.current[:_influxdb_rails_controller] = nil
-    end
-
-    context "successfully" do
-      it "writes to InfluxDB" do
-        expect_any_instance_of(InfluxDB::Client).to receive(:write_point).with(
-          series_name, result
-        )
-        subject.call("name", start_time, finish_time, "id", payload)
-      end
-
-      context "with not relevant queries" do
-        before do
-          payload[:sql] = "SHOW FULL FIELDS FROM `users`"
-        end
-
-        it "does not write to InfluxDB" do
-          expect_any_instance_of(InfluxDB::Client).not_to receive(:write_point)
-          subject.call("name", start_time, finish_time, "id", payload)
-        end
-      end
-    end
-
-    context "unsuccessfully" do
-      before do
-        allow(config).to receive(:logger).and_return(logger)
-        InfluxDB::Rails.configuration = config
-      end
-
-      it "does log exceptions" do
-        allow_any_instance_of(InfluxDB::Client).to receive(:write_point).and_raise("boom")
-        expect(logger).to receive(:error).with(/boom/)
-        subject.call("name", start_time, finish_time, "id", payload)
-      end
-    end
-  end
-end