bugsnag-api 1.0.2 → 2.0.3

data/spec/bugsnag/api/client/organizations_spec.rb

@@ -0,0 +1,53 @@
+require "spec_helper"
+
+describe Bugsnag::Api::Client::Organizations do
+  before do
+    @client = basic_auth_client
+    Bugsnag::Api.reset!
+  end
+
+  describe ".create_organization", :vcr do
+    it "creates a new organization" do
+      organization = @client.create_organization "testOrg"
+      expect(organization).to be_kind_of(Object)
+      expect(organization.name).to eq("testOrg")
+
+      assert_requested :post, bugsnag_url("/organizations")
+    end
+  end
+
+  context "with organization", :vcr do
+    before do
+      @organization = @client.create_organization "testOrg"
+    end
+
+    describe ".organization", :vcr do
+      it "returns the requested organization" do
+        organization = @client.organization @organization.id
+        expect(organization.id).to eq(@organization.id)
+
+        assert_requested :get, bugsnag_url("/organizations/#{@organization.id}")
+      end
+    end
+
+    describe ".update_organization", :vcr do
+      it "updates and returns the organization" do
+        updatedOrg = @client.update_organization @organization.id, "updated_name", false, {:invoice_address => "test_string"}
+        expect(updatedOrg.id).to eq(@organization.id)
+        expect(updatedOrg.name).to eq("updated_name")
+        expect(updatedOrg.auto_upgrade).to be false
+
+        assert_requested :patch, bugsnag_url("/organizations/#{@organization.id}")
+      end
+    end
+
+    describe ".delete_organization", :vcr do
+      it "deletes the organization" do
+        deletedOrg = @client.delete_organization @organization.id
+        expect(deletedOrg).to be true
+      
+        assert_requested :delete, bugsnag_url("/organizations/#{@organization.id}")
+      end
+    end
+  end
+end

data/spec/bugsnag/api/client/pivots_spec.rb

@@ -0,0 +1,49 @@
+require "spec_helper"
+
+describe Bugsnag::Api::Client::Pivots do
+  before do
+    skip "event_fields API has issues currently"
+    @client = auth_token_client
+    @project_id = test_bugsnag_project_id
+    @error_id = test_bugsnag_error_id
+    @event_field_id = test_bugsnag_eventfield_id
+    Bugsnag::Api.reset!
+  end
+
+  describe ".pivots", :vcr do
+    it "returns a list of pivots on an error" do
+      pivots = @client.pivots @project_id, @error_id
+      expect(pivots).to be_a_kind_of(Array)
+      expect(pivots.length).to be > 0
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/errors/#{@error_id}/pivots")
+    end
+
+    it "returns a list of pivots on a project" do
+      pivots = @client.pivots @project_id
+      expect(pivots).to be_a_kind_of(Array)
+      expect(pivots.length).to be > 0
+      
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/pivots")
+    end
+  end
+
+  describe ".pivot_values", :vcr do
+    it "returns values of pivots on an error" do
+      values = @client.pivot_values @project_id, @event_field_id, @error_id
+      expect(values).to be_a_kind_of(Array)
+      expect(values.length).to be > 0
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/errors/#{@error_id}/pivots/#{@event_field_id}/values")
+    end
+
+    it "returns values of pivots on a project" do
+      values = @client.pivot_values @project_id, @event_field_id
+      expect(values).to be_a_kind_of(Array)
+      expect(values.length).to be > 0
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/pivots/#{@event_field_id}/values")
+    end
+  end
+end
+

data/spec/bugsnag/api/client/projects_spec.rb

@@ -2,85 +2,61 @@ require "spec_helper"
 
 describe Bugsnag::Api::Client::Projects do
   before do
+    @client = auth_token_client
+    @org_id = test_bugsnag_org_id
     Bugsnag::Api.reset!
-    @client = basic_auth_client
-  end
-
-  describe ".account_projects", :vcr do
-    context "when using account credentials" do
-      it "returns all projects" do
-        client = auth_token_client
-        projects = client.account_projects
-        expect(projects).to be_kind_of(Array)
-        expect(projects.first.name).not_to be_nil
-
-        assert_requested :get, bugsnag_url("/account/projects")
-      end
-    end
-
-    context "when using user credentials" do
-      it "raises an error" do
-        expect { @client.account_projects }.to raise_error Bugsnag::Api::AccountCredentialsRequired
-
-        assert_not_requested :get, basic_bugsnag_url("/account/projects")
-      end
-    end
-
-    it "returns all projects on an account" do
-      projects = @client.account_projects(test_bugsnag_account)
-      expect(projects).to be_kind_of(Array)
-      expect(projects.first.name).not_to be_nil
-
-      assert_requested :get, basic_bugsnag_url("/accounts/#{test_bugsnag_account}/projects")
-    end
-  end
-
-  describe ".user_projects", :vcr do
-    it "returns all projects for a user" do
-      projects = @client.user_projects(test_bugsnag_user)
-      expect(projects).to be_kind_of(Array)
-      expect(projects.first.name).not_to be_nil
-
-      assert_requested :get, basic_bugsnag_url("/users/#{test_bugsnag_user}/projects")
-    end
   end
 
   describe ".create_project", :vcr do
-    it "creates a project on an account" do
-      project = @client.create_project(test_bugsnag_account, :name => "Example")
-      expect(project.name).to eq("Example")
+    it "creates a new project" do
+    new_project = @client.create_project @org_id, "testProject", "other"
+    expect(new_project.name).to eq("testProject")
 
-      assert_requested :post, basic_bugsnag_url("/accounts/#{test_bugsnag_account}/projects")
+    assert_requested :post, bugsnag_url("/organizations/#{@org_id}/projects")
     end
   end
 
-  context "with project", :vcr do
+  context "given a project", :vcr do
     before do
-      @project = @client.create_project(test_bugsnag_account, :name => "Example")
+      @project = @client.create_project @org_id, "testProject", "other"
     end
 
-    describe ".project" do
-      it "returns a project" do
-        project = @client.project(@project.id)
+    describe ".project", :vcr do
+      it "returns the requested project" do
+        project = @client.project @project.id
         expect(project.id).to eq(@project.id)
+        expect(project.name).to eq("testProject")
+
+        assert_requested :get, bugsnag_url("/projects/#{@project.id}")
+      end
+    end
+
+    describe ".update_project", :vcr do
+      it "updates and returns the project" do
+        updated_project = @client.update_project @project.id, {:name => "otherName"}
+        expect(updated_project.id).to eq(@project.id)
+        expect(updated_project.name).to eq("otherName")
 
-        assert_requested :get, basic_bugsnag_url("/projects/#{@project.id}")
+        assert_requested :patch, bugsnag_url("/projects/#{@project.id}")
       end
     end
 
-    describe ".update_project" do
-      it "updates an existing project" do
-        updated_project = @client.update_project(@project.id, :name => "New project name")
+    describe ".regenerate_api_key", :vcr do
+      it "removes the current api key and replaces it with a new api key" do
+        updated_project = @client.regenerate_api_key @project.id
         expect(updated_project.id).to eq(@project.id)
-        assert_requested :patch, basic_bugsnag_url("/projects/#{@project.id}")
+        expect(updated_project.api_key).not_to eq(@project.api_key)
+        
+        assert_requested :delete, bugsnag_url("/projects/#{@project.id}/api_key")
       end
     end
+    
+    describe ".delete_project", :vcr do
+      it "deletes the project" do
+        deleted_project = @client.delete_project @project.id
+        expect(deleted_project).to be true
 
-    describe ".delete_project" do
-      it "deletes an existing project" do
-        response = @client.delete_project(@project.id)
-        expect(response).to be true
-        assert_requested :delete, basic_bugsnag_url("/projects/#{@project.id}")
+        assert_requested :delete, bugsnag_url("/projects/#{@project.id}")
       end
     end
   end

data/spec/bugsnag/api/client/trends_spec.rb

@@ -0,0 +1,44 @@
+require "spec_helper"
+
+describe Bugsnag::Api::Client::Trends do
+  before do
+    @project_id = test_bugsnag_project_id
+    @error_id = test_bugsnag_error_id
+    @client = auth_token_client
+    Bugsnag::Api.reset!
+  end
+
+  describe ".trends_buckets", :vcr do
+    it "returns a list of error trends in bucket form" do
+      trends = @client.trends_buckets @project_id, 5, @error_id
+      expect(trends).to be_a_kind_of(Array)
+      expect(trends.length).to eq(5)
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/errors/#{@error_id}/trend?buckets_count=5")
+    end
+
+    it "returns a list of project trends in bucket form" do
+      trends = @client.trends_buckets @project_id, 5
+      expect(trends).to be_a_kind_of(Array)
+      expect(trends.length).to eq(5)
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/trend?buckets_count=5")
+    end
+  end
+
+  describe ".trends_resolution", :vcr do
+    it "returns a list of trends in resolution form" do
+      trends = @client.trends_resolution @project_id, "12h", @error_id
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/errors/#{@error_id}/trend?resolution=12h")
+    end
+
+    it "returns a list of project trends in resolution form" do
+      trends = @client.trends_resolution @project_id, "12h"
+      expect(trends).to be_a_kind_of(Array)
+      expect(trends.length).to be > 0
+
+      assert_requested :get, bugsnag_url("/projects/#{@project_id}/trend?resolution=12h")
+    end
+  end
+end

data/spec/bugsnag/api/client_spec.rb

@@ -1,3 +1,4 @@
+require "spec_helper"
 require "json"
 
 describe Bugsnag::Api::Client do
@@ -44,14 +45,14 @@ describe Bugsnag::Api::Client do
     end
 
     it "handles query params", :vcr do
-      Bugsnag::Api.get "/", :foo => "bar"
-      assert_requested :get, "https://api.bugsnag.com?foo=bar"
+      Bugsnag::Api.get bugsnag_url("/"), :foo => "bar"
+      assert_requested :get, bugsnag_url("?foo=bar")
     end
 
     it "handles headers" do
       request = stub_get("/zen").
         with(:query => {:foo => "bar"}, :headers => {:accept => "text/plain"})
-      Bugsnag::Api.get "/zen", :foo => "bar", :accept => "text/plain"
+      Bugsnag::Api.get bugsnag_url("/zen"), :foo => "bar", :accept => "text/plain"
       assert_requested request
     end
   end
@@ -62,7 +63,7 @@ describe Bugsnag::Api::Client do
       client = Bugsnag::Api.client
       expect(client.last_response).to be_nil
 
-      client.get "/"
+      client.get bugsnag_url("/")
       expect(client.last_response.status).to eq(200)
     end
   end
@@ -81,15 +82,81 @@ describe Bugsnag::Api::Client do
     end
   end
 
+  describe ".deep_merge" do
+    it "returns a merged hash" do
+      client = Bugsnag::Api::Client.new(:auth_token => "example")
+      lhs = {
+        :foo => "foo"
+      }
+      rhs = {
+        :bar => "bar"
+      }
+      merged = client.deep_merge(lhs, rhs)
+      expect(merged).to_not eq(lhs)
+      expect(merged).to_not eq(rhs)
+      expect(merged).to eq({
+        :foo => "foo",
+        :bar => "bar"
+      })
+    end
+
+    it "favors rhs over lhs" do
+      client = Bugsnag::Api::Client.new(:auth_token => "example")
+      lhs = {
+        :foo => "foo"
+      }
+      rhs = {
+        :foo => "bar"
+      }
+      merged = client.deep_merge(lhs, rhs)
+      expect(merged).to eq({:foo => "bar"})
+    end
+
+    it "recursively merges hashes" do
+      client = Bugsnag::Api::Client.new(:auth_token => "example")
+      lhs = {
+        :foo => {
+          :bar => "bar"
+        }
+      }
+      rhs = {
+        :foo => {
+          :foobar => "foobar"
+        }
+      }
+      merged = client.deep_merge(lhs, rhs)
+      expect(merged).to eq(
+        {:foo => {
+          :bar => "bar",
+          :foobar => "foobar"
+        }
+      })
+    end
+  end
+
   context "error handling" do
+
+    before do
+      VCR.turn_off!
+    end
+
+    after do
+      VCR.turn_on!
+    end
+
     it "raises on 404" do
       stub_get('/booya').to_return(:status => 404)
-      expect { Bugsnag::Api.get('/booya') }.to raise_error(Bugsnag::Api::NotFound)
+      expect { Bugsnag::Api.get(bugsnag_url('/booya')) }.to raise_error(Bugsnag::Api::NotFound)
+    end
+
+    it "raises on 429" do
+      stub_get('/test').to_return(:status => 429)
+      expect { Bugsnag::Api.get(bugsnag_url('/test')) }.to raise_error(Bugsnag::Api::RateLimitExceeded)
     end
 
     it "raises on 500" do
       stub_get('/boom').to_return(:status => 500)
-      expect { Bugsnag::Api.get('/boom') }.to raise_error(Bugsnag::Api::InternalServerError)
+      expect { Bugsnag::Api.get(bugsnag_url('/boom')) }.to raise_error(Bugsnag::Api::InternalServerError)
     end
 
     it "includes an error" do
@@ -101,7 +168,7 @@ describe Bugsnag::Api::Client do
         },
         :body => {:error => "Comments must contain a message"}.to_json
       begin
-        Bugsnag::Api.get('/boom')
+        Bugsnag::Api.get(bugsnag_url('/boom'))
       rescue Bugsnag::Api::UnprocessableEntity => e
         expect(e.message).to include("Error: Comments must contain a message")
       end
@@ -114,7 +181,7 @@ describe Bugsnag::Api::Client do
           :content_type => "application/json",
         },
         :body => {:message => "I'm a teapot"}.to_json
-      expect { Bugsnag::Api.get('/user') }.to raise_error(Bugsnag::Api::ClientError)
+      expect { Bugsnag::Api.get(bugsnag_url('/user')) }.to raise_error(Bugsnag::Api::ClientError)
     end
 
     it "raises on unknown server errors" do
@@ -124,7 +191,7 @@ describe Bugsnag::Api::Client do
           :content_type => "application/json",
         },
         :body => {:message => "Bandwidth exceeded"}.to_json
-      expect { Bugsnag::Api.get('/user') }.to raise_error(Bugsnag::Api::ServerError)
+      expect { Bugsnag::Api.get(bugsnag_url('/user')) }.to raise_error(Bugsnag::Api::ServerError)
     end
   end
 end

data/spec/bugsnag/api_spec.rb

@@ -1,3 +1,5 @@
+require "spec_helper"
+
 describe Bugsnag::Api do
   before do
     Bugsnag::Api.reset!

data/spec/cassettes/Bugsnag_Api_Client/_get/handles_query_params.yml

@@ -8,49 +8,59 @@ http_interactions:
       string: ''
     headers:
       User-Agent:
-      - Bugsnag API Ruby Gem 1.0.0
+      - Bugsnag API Ruby Gem 2.0.1
       Content-Type:
       - application/json
+      X-Version:
+      - '2'
+      X-Bugsnag-Api:
+      - 'true'
       Accept-Encoding:
       - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
       Accept:
-      - '*/*'
+      - "*/*"
   response:
     status:
       code: 200
       message: OK
     headers:
-      Cache-Control:
-      - max-age=0, private, must-revalidate
       Content-Type:
       - application/json; charset=utf-8
-      Date:
-      - Tue, 02 Sep 2014 02:49:29 GMT
-      Server:
-      - nginx + Phusion Passenger
-      Status:
-      - 200 OK
-      Strict-Transport-Security:
-      - max-age=31536000
+      Etag:
+      - W/"ee46b5c82cf98ff0d1c89f4cf77ab144"
+      Cache-Control:
+      - max-age=0, private, must-revalidate
+      X-Request-Id:
+      - 0044fd8d-8cb8-43c6-95ed-c291bd55fdb2
+      X-Runtime:
+      - '0.286559'
+      Vary:
+      - Origin
+      X-Rack-Cors:
+      - miss; no-origin
+      Content-Security-Policy:
+      - 'default-src ''self''; child-src https://*.stripe.com headway-widget.net;
+        connect-src ''self'' https://api.bugsnag.com https://notify.bugsnag.com https://*.stripe.com
+        https://bugsnag.zendesk.com https://api.lever.co; font-src ''self'' https://maxcdn.bootstrapcdn.com;
+        img-src ''self'' data: https://notify.bugsnag.com https://*.stripe.com https://maxcdn.bootstrapcdn.com;
+        script-src ''self'' https://d2wy8f7a9ursnm.cloudfront.net https://*.stripe.com
+        https://maxcdn.bootstrapcdn.com https://code.jquery.com https://assets.zendesk.com
+        https://cdn.headwayapp.co; style-src ''self'' ''unsafe-inline'' https://maxcdn.bootstrapcdn.com'
       X-Content-Type-Options:
       - nosniff
+      X-Download-Options:
+      - noopen
       X-Frame-Options:
-      - SAMEORIGIN
-      X-Powered-By:
-      - Phusion Passenger
-      X-Request-Id:
-      - d5805f6d-7f5f-4513-af4d-3a4a259a1707
-      X-Runtime:
-      - '0.004698'
+      - DENY
+      X-Permitted-Cross-Domain-Policies:
+      - none
       X-Xss-Protection:
       - 1; mode=block
-      Content-Length:
-      - '96'
-      Connection:
-      - keep-alive
+      Transfer-Encoding:
+      - chunked
     body:
       encoding: UTF-8
-      string: '{"projects_url":"https://api.bugsnag.com/account/projects","users_url":"https://api.bugsnag.com/account/users"}'
+      string: '{"organizations_url":"https://api.bugsnag.com/user/organizations"}'
     http_version: 
-  recorded_at: Tue, 02 Sep 2014 02:49:29 GMT
-recorded_with: VCR 2.9.2
+  recorded_at: Tue, 12 Dec 2017 12:00:16 GMT
+recorded_with: VCR 2.9.3