airbrake 3.1.17 → 3.2.0

data/CHANGELOG

@@ -1,3 +1,12 @@
+Version 3.2.0 - 2014-05-23 17:37:35 -0700
+===============================================================================
+
+Ali Faiz (4):
+      Remove JS notifier from templates and libraries
+      Cleanup references to JS notifier in tests
+      Merge pull request #297 from airbrake/remove-js-notifier
+
+
 Version 3.1.17 - 2014-05-20 21:00:25 -0700
 ===============================================================================
 
@@ -260,7 +269,7 @@ Hrvoje Šimić (6):
       thread safety fix
       add the secure option to the generator
       fix the broken generator method call
-      be more careful with @template in javascript notifier 
+      be more careful with @template in javascript notifier
       update tested rails versions
 
 Marko Šiftar (2):
@@ -1358,5 +1367,6 @@ Nick Quaranto (3):
 
 
 
+
 
 

data/Rakefile

@@ -158,7 +158,7 @@ def cucumber_opts
 
   case ENV["BUNDLE_GEMFILE"]
   when /rails/
-    opts << "features/rails.feature features/rails_with_js_notifier.feature features/metal.feature features/user_informer.feature"
+    opts << "features/rails.feature features/metal.feature features/user_informer.feature"
   when /rack/
     opts << "features/rack.feature"
   when /sinatra/

data/features/step_definitions/rails_application_steps.rb

@@ -192,32 +192,6 @@ When /^I configure the application to filter parameter "([^\"]*)"$/ do |paramete
   end
 end
 
-Then /^I should see the notifier JavaScript for the following:$/ do |table|
-  hash = table.hashes.first
-  host        = hash['host']        || 'api.airbrake.io'
-  secure      = hash['secure']      || false
-  api_key     = hash['api_key']
-  environment = hash['environment'] || 'production'
-
-  steps %{
-    Then the output should contain "#{host}/javascripts/notifier.js"
-    And the output should contain "Airbrake.setKey('#{api_key}');"
-    And the output should contain "Airbrake.setHost('#{host}');"
-    And the output should contain "Airbrake.setEnvironment('#{environment}');"
-  }
-end
-
-Then /^the notifier JavaScript should provide the following errorDefaults:$/ do |table|
-  hash = table.hashes.first
-  hash.each do |key, value|
-    assert_matching_output("Airbrake\.setErrorDefaults.*#{key}: \"#{value}\"",all_output)
-  end
-end
-
-Then /^I should not see notifier JavaScript$/ do
-  step %{the output should not contain "script[type='text/javascript'][src$='/javascripts/notifier.js']"}
-end
-
 When /^I have set up authentication system in my app that uses "([^\"]*)"$/ do |current_user|
   application_controller = File.join(rails_root, 'app', 'controllers', "application_controller.rb")
   definition =

data/lib/airbrake/configuration.rb

@@ -2,7 +2,7 @@ module Airbrake
   # Used to set up and modify settings for the notifier.
   class Configuration
 
-    OPTIONS = [:api_key, :js_api_key, :backtrace_filters, :development_environments,
+    OPTIONS = [:api_key, :backtrace_filters, :development_environments,
         :development_lookup, :environment_name, :host,
         :http_open_timeout, :http_read_timeout, :ignore, :ignore_by_filters,
         :ignore_user_agent, :notifier_name, :notifier_url, :notifier_version,
@@ -14,12 +14,6 @@ module Airbrake
     # The API key for your project, found on the project edit form.
     attr_accessor :api_key
 
-    # If you're using the Javascript notifier and would want to separate
-    # Javascript notifications into another Airbrake project, specify
-    # its APi key here.
-    # Defaults to #api_key (of the base project)
-    attr_writer :js_api_key
-
     # The host to connect to (defaults to airbrake.io).
     attr_accessor :host
 
@@ -158,7 +152,6 @@ module Airbrake
     alias_method :use_system_ssl_cert_chain?, :use_system_ssl_cert_chain
 
     def initialize
-      @js_api_key               = nil
       @secure                   = false
       @use_system_ssl_cert_chain= false
       @host                     = 'api.airbrake.io'
@@ -311,14 +304,6 @@ module Airbrake
       @rescue_rake_exceptions = val
     end
 
-    def js_api_key
-      @js_api_key || self.api_key
-    end
-
-    def js_notifier=(*args)
-      warn '[AIRBRAKE] config.js_notifier has been deprecated and has no effect.  You should use <%= airbrake_javascript_notifier %> directly at the top of your layouts.  Be sure to place it before all other javascript.'
-    end
-
     def ca_bundle_path
       if use_system_ssl_cert_chain? && File.exist?(OpenSSL::X509::DEFAULT_CERT_FILE)
         OpenSSL::X509::DEFAULT_CERT_FILE

data/lib/airbrake/rails.rb

@@ -2,7 +2,6 @@ require 'airbrake'
 require 'airbrake/rails/controller_methods'
 require 'airbrake/rails/action_controller_catcher'
 require 'airbrake/rails/error_lookup'
-require 'airbrake/rails/javascript_notifier'
 
 module Airbrake
   module Rails
@@ -11,7 +10,6 @@ module Airbrake
         ActionController::Base.send(:include, Airbrake::Rails::ActionControllerCatcher)
         ActionController::Base.send(:include, Airbrake::Rails::ErrorLookup)
         ActionController::Base.send(:include, Airbrake::Rails::ControllerMethods)
-        ActionController::Base.send(:include, Airbrake::Rails::JavascriptNotifier)
       end
 
       rails_logger = if defined?(::Rails.logger)

data/lib/airbrake/railtie.rb

@@ -41,12 +41,6 @@ module Airbrake
 
         include Airbrake::Rails::ControllerMethods
       end
-
-      if defined?(::ActionController::Base)
-        require 'airbrake/rails/javascript_notifier'
-
-        ::ActionController::Base.send(:include, Airbrake::Rails::JavascriptNotifier)
-      end
     end
   end
 end

data/lib/airbrake/version.rb

@@ -1,3 +1,3 @@
 module Airbrake
-  VERSION = "3.1.17".freeze
+  VERSION = "3.2.0".freeze
 end

data/test/configuration_test.rb

@@ -127,14 +127,6 @@ class ConfigurationTest < Test::Unit::TestCase
     assert_appends_value :rake_environment_filters
   end
 
-  should "warn when attempting to write js_notifier" do
-    config = Airbrake::Configuration.new
-    config.
-      expects(:warn).
-      with(regexp_matches(/deprecated/i))
-    config.js_notifier = true
-  end
-
   should "allow ignored user agents to be appended" do
     assert_appends_value :ignore_user_agent
   end

data/test/integration.rb

@@ -11,5 +11,4 @@ silence_warnings do
   require "action_view"
 end
 
-require File.expand_path "../integration/javascript_notifier_test", __FILE__
 require File.expand_path "../integration/catcher_test", __FILE__

metadata

@@ -1,46 +1,52 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 3.1.17
+  version: 3.2.0
+  prerelease: 
 platform: ruby
 authors:
 - Airbrake
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-21 00:00:00.000000000 Z
+date: 2014-05-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bourne
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -48,6 +54,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -55,6 +62,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber-rails
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -62,6 +70,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -69,6 +78,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: fakeweb
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -76,6 +86,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -83,6 +94,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -90,6 +102,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -97,6 +110,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -104,6 +118,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -111,6 +126,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: sham_rack
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -118,6 +134,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -125,6 +142,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: json-schema
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -132,6 +150,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -139,6 +158,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: capistrano
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -146,6 +166,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -153,62 +174,71 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: aruba
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: girl_friday
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sucker_punch
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - '='
       - !ruby/object:Gem::Version
@@ -216,6 +246,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - '='
       - !ruby/object:Gem::Version
@@ -223,62 +254,71 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: shoulda-matchers
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: shoulda-context
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: coveralls
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -286,6 +326,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -297,42 +338,12 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- CHANGELOG
-- Gemfile
-- Guardfile
-- INSTALL
-- LICENSE
-- README.md
-- README_FOR_HEROKU_ADDON.md
-- Rakefile
-- TESTED_AGAINST
-- airbrake.gemspec
-- bin/airbrake
-- features/metal.feature
-- features/rack.feature
-- features/rails.feature
-- features/rails_with_js_notifier.feature
-- features/rake.feature
-- features/sinatra.feature
-- features/step_definitions/file_steps.rb
-- features/step_definitions/rack_steps.rb
-- features/step_definitions/rails_application_steps.rb
-- features/step_definitions/rake_steps.rb
-- features/support/airbrake_shim.rb.template
-- features/support/aruba.rb
-- features/support/env.rb
-- features/support/matchers.rb
-- features/support/rails.rb
-- features/support/rake/Rakefile
-- features/user_informer.feature
 - generators/airbrake/airbrake_generator.rb
 - generators/airbrake/lib/insert_commands.rb
 - generators/airbrake/lib/rake_commands.rb
 - generators/airbrake/templates/airbrake_tasks.rake
 - generators/airbrake/templates/capistrano_hook.rb
 - generators/airbrake/templates/initializer.rb
-- install.rb
-- lib/airbrake.rb
 - lib/airbrake/backtrace.rb
 - lib/airbrake/capistrano.rb
 - lib/airbrake/capistrano3.rb
@@ -347,12 +358,11 @@ files:
 - lib/airbrake/jobs/send_job.rb
 - lib/airbrake/notice.rb
 - lib/airbrake/rack.rb
-- lib/airbrake/rails.rb
 - lib/airbrake/rails/action_controller_catcher.rb
 - lib/airbrake/rails/controller_methods.rb
 - lib/airbrake/rails/error_lookup.rb
-- lib/airbrake/rails/javascript_notifier.rb
 - lib/airbrake/rails/middleware.rb
+- lib/airbrake/rails.rb
 - lib/airbrake/rails3_tasks.rb
 - lib/airbrake/railtie.rb
 - lib/airbrake/rake_handler.rb
@@ -360,32 +370,41 @@ files:
 - lib/airbrake/sender.rb
 - lib/airbrake/shared_tasks.rb
 - lib/airbrake/sinatra.rb
-- lib/airbrake/tasks.rb
 - lib/airbrake/tasks/airbrake.cap
+- lib/airbrake/tasks.rb
 - lib/airbrake/user_informer.rb
 - lib/airbrake/utils/params_cleaner.rb
 - lib/airbrake/utils/rack_filters.rb
 - lib/airbrake/version.rb
+- lib/airbrake.rb
 - lib/airbrake_tasks.rb
 - lib/rails/generators/airbrake/airbrake_generator.rb
-- lib/templates/javascript_notifier_configuration
-- lib/templates/javascript_notifier_loader
 - lib/templates/rescue.erb
 - rails/init.rb
-- resources/README.md
 - resources/airbrake_3_0.json
 - resources/ca-bundle.crt
 - resources/notice.xml
+- resources/README.md
 - script/integration_test.rb
+- airbrake.gemspec
+- CHANGELOG
+- Gemfile
+- Guardfile
+- INSTALL
+- LICENSE
+- Rakefile
+- README_FOR_HEROKU_ADDON.md
+- README.md
+- TESTED_AGAINST
+- install.rb
 - test/airbrake_tasks_test.rb
 - test/backtrace_test.rb
 - test/capistrano_test.rb
 - test/configuration_test.rb
 - test/controller_methods_test.rb
 - test/helper.rb
-- test/integration.rb
 - test/integration/catcher_test.rb
-- test/integration/javascript_notifier_test.rb
+- test/integration.rb
 - test/logger_test.rb
 - test/notice_test.rb
 - test/notifier_test.rb
@@ -397,28 +416,46 @@ files:
 - test/sender_test.rb
 - test/support/response_shim.xml
 - test/user_informer_test.rb
+- features/metal.feature
+- features/rack.feature
+- features/rails.feature
+- features/rake.feature
+- features/sinatra.feature
+- features/step_definitions/file_steps.rb
+- features/step_definitions/rack_steps.rb
+- features/step_definitions/rails_application_steps.rb
+- features/step_definitions/rake_steps.rb
+- features/support/airbrake_shim.rb.template
+- features/support/aruba.rb
+- features/support/env.rb
+- features/support/matchers.rb
+- features/support/rails.rb
+- features/support/rake/Rakefile
+- features/user_informer.feature
+- bin/airbrake
 homepage: http://www.airbrake.io
 licenses: []
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 1.8.24
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Send your application errors to our hosted service and reclaim your inbox.
 test_files:
 - test/airbrake_tasks_test.rb
@@ -428,7 +465,6 @@ test_files:
 - test/controller_methods_test.rb
 - test/helper.rb
 - test/integration/catcher_test.rb
-- test/integration/javascript_notifier_test.rb
 - test/integration.rb
 - test/logger_test.rb
 - test/notice_test.rb
@@ -444,7 +480,6 @@ test_files:
 - features/metal.feature
 - features/rack.feature
 - features/rails.feature
-- features/rails_with_js_notifier.feature
 - features/rake.feature
 - features/sinatra.feature
 - features/step_definitions/file_steps.rb
@@ -458,3 +493,4 @@ test_files:
 - features/support/rails.rb
 - features/support/rake/Rakefile
 - features/user_informer.feature
+has_rdoc: 

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: dfc2885576a30927690a4535d69e4ca351010c80
-  data.tar.gz: d46ef484b70f08951c33fff46079c8a08317c2be
-SHA512:
-  metadata.gz: 2194af5fb0d2c3610728f6ddcacc260aa366d6b30e4311c452ff8f54815f43b71aa05c632bef75d3ff447c40a2d45399393d40a4b21c09d30234db6ad3423def
-  data.tar.gz: 736d7e758269212bbdbb0440c1aa8d436489f16de8813aa8b3778efad5ad0dbe9be69494514e9b18b20d1c07fa2a149af58630f24b22efb5ce229db3ad6d053d

data/features/rails_with_js_notifier.feature

@@ -1,104 +0,0 @@
-Feature: Install the Gem in a Rails application and enable the JavaScript notifier
-
-  Background:
-    Given I successfully run `rails new rails_root -O --skip-gemfile`
-    And I cd to "rails_root"
-    And I configure the Airbrake shim
-
-  Scenario: Include the Javascript notifier when enabled
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.api_key     = "myapikey"
-      """
-    And I define a response for "TestController#index":
-      """
-        render :inline => '<html><head profile="http://example.com"><%= airbrake_javascript_notifier %></head><body></body></html>'
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index" in the "production" environment
-    Then I should see the notifier JavaScript for the following:
-      | api_key  | environment | host           |
-      | myapikey | production  | api.airbrake.io |
-    And the notifier JavaScript should provide the following errorDefaults:
-      | url                           | component | action |
-      | http://example.com:123/test/index | test      | index  |
-
-  Scenario: Include the Javascript notifier when enabled using custom configuration settings
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.development_environments = []
-      config.api_key     = "myapikey!"
-      config.host        = "myairbrake.com"
-      config.port        = 3001
-      """
-    And I define a response for "TestController#index":
-      """
-        render :inline => '<html><head><%= airbrake_javascript_notifier %></head><body></body></html>'
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index"
-    Then I should see the notifier JavaScript for the following:
-      | api_key   | environment | host               |
-      | myapikey! | test  | myairbrake.com:3001 |
-
-  Scenario: Don't include the Javascript notifier by default
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.api_key = "myapikey!"
-      """
-    And I define a response for "TestController#index":
-      """
-        render :inline => "<html><head></head><body></body></html>"
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index"
-    Then I should not see notifier JavaScript
-
-  Scenario: Don't include the Javascript notifier when enabled in non-public environments
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.api_key          = "myapikey!"
-      config.environment_name = 'test'
-      """
-    And I define a response for "TestController#index":
-      """
-        render :inline => '<html><head><%= airbrake_javascript_notifier %></head><body></body></html>'
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index" in the "test" environment
-    Then I should not see notifier JavaScript
-
-  Scenario: Use the js_api_key if present
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.api_key     = "myapikey!"
-      config.js_api_key     = "myjsapikey!"
-      """
-    And I define a response for "TestController#index":
-      """
-        render :inline => '<html><head><%= airbrake_javascript_notifier %></head><body></body></html>'
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index" in the "production" environment
-    Then I should see the notifier JavaScript for the following:
-      | api_key     | environment | host        |
-      | myjsapikey! | production  | api.airbrake.io |
-
-  Scenario: Being careful with user's instance variables
-    When I configure the notifier to use the following configuration lines:
-      """
-      config.api_key     = "myapikey"
-      """
-    And I define a response for "TestController#index":
-      """
-        @template = "this is some random instance variable"
-        render :inline => '<html><head><%= airbrake_javascript_notifier %></head><body></body></html>'
-      """
-    And I route "/test/index" to "test#index"
-    And I perform a request to "http://example.com:123/test/index" in the "production" environment
-    Then I should see the notifier JavaScript for the following:
-      | api_key  | environment | host           |
-      | myapikey | production  | api.airbrake.io |
-    And the notifier JavaScript should provide the following errorDefaults:
-      | url                           | component | action |
-      | http://example.com:123/test/index | test      | index  |

data/lib/airbrake/rails/javascript_notifier.rb

@@ -1,83 +0,0 @@
-module Airbrake
-  module Rails
-    module JavascriptNotifier
-      def self.included(base) #:nodoc:
-        base.send :helper_method, :airbrake_javascript_notifier
-        base.send :helper_method, :airbrake_javascript_loader
-        base.send :helper_method, :airbrake_javascript_configuration
-      end
-
-      private
-
-        def airbrake_javascript_notifier
-          if Airbrake.configuration.public?
-            airbrake_javascript_loader + airbrake_javascript_configuration
-          end
-        end
-
-        def airbrake_javascript_loader
-          if Airbrake.configuration.public?
-            path = File.join File.dirname(__FILE__), '..', '..', 'templates', 'javascript_notifier_loader'
-
-            _airbrake_render_part path
-          end
-        end
-
-        def airbrake_javascript_configuration
-          if Airbrake.configuration.public?
-            path = File.join File.dirname(__FILE__), '..', '..', 'templates', 'javascript_notifier_configuration'
-
-            options              = {
-              :api_key         => Airbrake.configuration.js_api_key,
-              :environment     => Airbrake.configuration.environment_name,
-              :action_name     => action_name,
-              :controller_name => controller_name,
-              :url             => request.url
-            }
-
-            _airbrake_render_part path, options
-          end
-        end
-
-      protected
-        attr_reader :template
-
-        def _airbrake_render_part(path, locals={})
-          locals[:host] = _airbrake_host
-
-          options              = {
-            :file              => path,
-            :layout            => false,
-            :use_full_path     => false,
-            :handlers          => [:erb],
-            :locals            => locals
-          }
-
-          result = _airbrake_render_template options
-
-          if result.respond_to?(:html_safe)
-            result.html_safe
-          else
-            result
-          end
-        end
-
-        def _airbrake_render_template(options)
-          case template
-          when ActionView::Template
-            template.render options
-          else
-            render_to_string options
-          end
-        end
-
-        def _airbrake_host
-          host = Airbrake.configuration.host.dup
-          port = Airbrake.configuration.port
-          host << ":#{port}" unless [80, 443].include?(port)
-
-          host
-        end
-    end
-  end
-end

data/lib/templates/javascript_notifier_configuration

@@ -1,12 +0,0 @@
-<%= javascript_tag %Q{
-    try {
-      window.Airbrake = (typeof(Airbrake) == 'undefined' && typeof(Hoptoad) != 'undefined') ? Hoptoad : Airbrake
-      Airbrake.setKey('#{api_key}');
-      Airbrake.setHost('#{host}');
-      Airbrake.setEnvironment('#{environment}');
-      Airbrake.setErrorDefaults({ url: "#{escape_javascript url}", component: "#{controller_name}", action: "#{action_name}" });
-    } catch (e) {
-      window.onerror = null;
-    }
-  }
-%>

data/lib/templates/javascript_notifier_loader

@@ -1,6 +0,0 @@
-<%= javascript_tag %Q{
-  (function(){
-    var notifierJsScheme = (("https:" == document.location.protocol) ? "https://" : "http://");
-    document.write(unescape("%3Cscript src='" + notifierJsScheme + "#{host}/javascripts/notifier.js' type='text/javascript'%3E%3C/script%3E"));
-  })();
-}%>

data/test/integration/javascript_notifier_test.rb

@@ -1,56 +0,0 @@
-require 'airbrake/rails/javascript_notifier'
-require 'ostruct'
-
-class JavascriptNotifierTest < Test::Unit::TestCase
-  module FakeRenderer
-    def javascript_tag(text)
-      "<script>#{text}</script>"
-    end
-    def escape_javascript(text)
-      "ESC#{text}ESC"
-    end
-  end
-
-  class FakeController
-    def self.helper_method(*args)
-    end
-
-    include Airbrake::Rails::JavascriptNotifier
-
-    def action_name
-      "action"
-    end
-
-    def controller_name
-      "controller"
-    end
-
-    def request
-      @request ||= OpenStruct.new
-    end
-
-    def render_to_string(options)
-      context = OpenStruct.new(options[:locals])
-      context.extend(FakeRenderer)
-      context.instance_eval do
-        erb = ERB.new(IO.read(options[:file]))
-        erb.result(binding)
-      end
-    end
-  end
-
-  should "make sure escape_javacript is called on the request.url" do
-    Airbrake.configure do
-    end
-    controller = FakeController.new
-    controller.request.url = "bad_javascript"
-    assert controller.send(:airbrake_javascript_notifier)['"ESCbad_javascriptESC"']
-    assert ! controller.send(:airbrake_javascript_notifier)['"bad_javascript"']
-  end
-
-  should "not raise exceptions for the non-public requests" do
-    Airbrake::Configuration.any_instance.stubs(:public? => false)
-    controller = FakeController.new
-    assert_nothing_raised { controller.send(:airbrake_javascript_notifier) }
-  end
-end