Skip to content

Commit

Permalink
Merge pull request #705 from stripe/remi-add-radar-review
Browse files Browse the repository at this point in the history
Add support for the Review resource
  • Loading branch information
remi-stripe authored Nov 28, 2018
2 parents 3c7995f + 0383de9 commit ed10fc8
Show file tree
Hide file tree
Showing 6 changed files with 45 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ sudo: false
env:
global:
# If changing this number, please also change it in `test/test_helper.rb`.
- STRIPE_MOCK_VERSION=0.37.0
- STRIPE_MOCK_VERSION=0.38.0

cache:
directories:
Expand Down
1 change: 1 addition & 0 deletions lib/stripe.rb
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@
require "stripe/reporting/report_run"
require "stripe/reporting/report_type"
require "stripe/reversal"
require "stripe/review"
require "stripe/sigma/scheduled_query_run"
require "stripe/sku"
require "stripe/source"
Expand Down
14 changes: 14 additions & 0 deletions lib/stripe/review.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# frozen_string_literal: true

module Stripe
class Review < APIResource
extend Stripe::APIOperations::List

OBJECT_NAME = "review".freeze

def approve(params = {}, opts = {})
resp, opts = request(:post, resource_url + "/approve", params, opts)
initialize_from(resp.data, opts)
end
end
end
1 change: 1 addition & 0 deletions lib/stripe/util.rb
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ def self.object_classes # rubocop:disable Metrics/MethodLength
Reporting::ReportRun::OBJECT_NAME => Reporting::ReportRun,
Reporting::ReportType::OBJECT_NAME => Reporting::ReportType,
Reversal::OBJECT_NAME => Reversal,
Review::OBJECT_NAME => Review,
SKU::OBJECT_NAME => SKU,
Sigma::ScheduledQueryRun::OBJECT_NAME => Sigma::ScheduledQueryRun,
Source::OBJECT_NAME => Source,
Expand Down
27 changes: 27 additions & 0 deletions test/stripe/review_test.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# frozen_string_literal: true

require ::File.expand_path("../../test_helper", __FILE__)

module Stripe
class ReviewTest < Test::Unit::TestCase
should "be listable" do
reviews = Stripe::Review.list
assert_requested :get, "#{Stripe.api_base}/v1/reviews"
assert reviews.data.is_a?(Array)
assert reviews.first.is_a?(Stripe::Review)
end

should "be retrievable" do
review = Stripe::Review.retrieve("prv_123")
assert_requested :get, "#{Stripe.api_base}/v1/reviews/prv_123"
assert review.is_a?(Stripe::Review)
end

should "be approvable" do
review = Stripe::Review.retrieve("prv_123")
review.approve
assert_requested :post, "#{Stripe.api_base}/v1/reviews/prv_123/approve"
assert review.is_a?(Stripe::Review)
end
end
end
2 changes: 1 addition & 1 deletion test/test_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
require ::File.expand_path("../test_data", __FILE__)

# If changing this number, please also change it in `.travis.yml`.
MOCK_MINIMUM_VERSION = "0.37.0".freeze
MOCK_MINIMUM_VERSION = "0.38.0".freeze
MOCK_PORT = ENV["STRIPE_MOCK_PORT"] || 12_111

# Disable all real network connections except those that are outgoing to
Expand Down

0 comments on commit ed10fc8

Please sign in to comment.