Merge pull request #1208 from travis-ci/sc-get-cc-last-digits

Return cc_last_digits attribute for invoice

Author: travis-architect

lib/travis/api/v3/models/invoice.rb

@@ -1,13 +1,14 @@
 module Travis::API::V3
   class Models::Invoice
-    attr_reader :id, :created_at, :status, :url, :amount_due
+    attr_reader :id, :created_at, :status, :url, :amount_due, :cc_last_digits
 
     def initialize(attributes = {})
       @id = attributes.fetch('id')
       @created_at = attributes.fetch('created_at') && DateTime.parse(attributes.fetch('created_at'))
       @status = attributes.fetch('status')
       @url = attributes.fetch('url')
       @amount_due = attributes.fetch('amount_due')
+      @cc_last_digits = attributes.fetch('cc_last_digits')
     end
   end
 end

lib/travis/api/v3/renderer/invoice.rb

@@ -1,5 +1,5 @@
 module Travis::API::V3
   class Renderer::Invoice < ModelRenderer
-    representation(:standard, :id, :created_at, :status, :url, :amount_due)
+    representation(:standard, :id, :created_at, :status, :url, :amount_due, :cc_last_digits)
   end
 end

spec/v3/billing_client_spec.rb

@@ -52,10 +52,11 @@
 
     it 'returns a list of invoices' do
       stub_billing_request(:get, "/subscriptions/#{subscription_id}/invoices", auth_key: auth_key, user_id: user_id)
-        .to_return(body: JSON.dump([{'id' => invoice_id, 'created_at' => Time.now, 'url' => 'https://billing-test.travis-ci.com/invoices/111.pdf', amount_due: 999, status: 'paid' }]))
+        .to_return(body: JSON.dump([{'id' => invoice_id, 'created_at' => Time.now, 'url' => 'https://billing-test.travis-ci.com/invoices/111.pdf', amount_due: 999, status: 'paid', cc_last_digits: '4242' }]))
       expect(subject.first).to be_a(Travis::API::V3::Models::Invoice)
       expect(subject.first.id).to eq(invoice_id)
       expect(subject.first.amount_due).to eq(999)
+      expect(subject.first.cc_last_digits).to eq('4242')
     end
 
     it 'returns an empty list if there are no invoices' do
@@ -71,10 +72,11 @@
 
     it 'returns a list of invoices' do
       stub_billing_request(:get, "/v2/subscriptions/#{subscription_id}/invoices", auth_key: auth_key, user_id: user_id)
-        .to_return(body: JSON.dump([{'id' => invoice_id, 'created_at' => Time.now, 'url' => 'https://billing-test.travis-ci.com/invoices/111.pdf', amount_due: 999, status: 'paid' }]))
+        .to_return(body: JSON.dump([{'id' => invoice_id, 'created_at' => Time.now, 'url' => 'https://billing-test.travis-ci.com/invoices/111.pdf', amount_due: 999, status: 'paid', cc_last_digits: '4242' }]))
       expect(subject.first).to be_a(Travis::API::V3::Models::Invoice)
       expect(subject.first.id).to eq(invoice_id)
       expect(subject.first.amount_due).to eq(999)
+      expect(subject.first.cc_last_digits).to eq('4242')
     end
 
     it 'returns an empty list if there are no invoices' do

spec/v3/services/subscription/invoices_spec.rb

@@ -27,9 +27,10 @@
     let(:amount_due) { 100 }
     let(:status) { 'paid' }
     let(:subscription_id) { rand(999) }
+    let(:cc_last_digits) { '4242' }
     before do
       stub_billing_request(:get, "/subscriptions/#{subscription_id}/invoices", auth_key: billing_auth_key, user_id: user.id)
-        .to_return(status: 200, body: JSON.dump([{'id' => invoice_id, 'created_at' => created_at, 'url' => url, 'amount_due' => amount_due, 'status' => status }]))
+        .to_return(status: 200, body: JSON.dump([{'id' => invoice_id, 'created_at' => created_at, 'url' => url, 'amount_due' => amount_due, 'status' => status, 'cc_last_digits' => cc_last_digits }]))
     end
 
     it 'responds with list of subscriptions' do
@@ -47,7 +48,8 @@
           'created_at' => created_at,
           'url' => url,
           'amount_due' => amount_due,
-          'status' => status
+          'status' => status,
+          'cc_last_digits' => cc_last_digits
         }]
       })
     end

spec/v3/services/v2_subscription/invoices_spec.rb

@@ -27,9 +27,11 @@
     let(:amount_due) { 100 }
     let(:status) { 'paid' }
     let(:subscription_id) { rand(999) }
+    let(:cc_last_digits) { '4242' }
+
     before do
       stub_billing_request(:get, "/v2/subscriptions/#{subscription_id}/invoices", auth_key: billing_auth_key, user_id: user.id)
-        .to_return(status: 200, body: JSON.dump([{'id' => invoice_id, 'created_at' => created_at, 'url' => url, 'amount_due' => amount_due, 'status' => status }]))
+        .to_return(status: 200, body: JSON.dump([{'id' => invoice_id, 'created_at' => created_at, 'url' => url, 'amount_due' => amount_due, 'status' => status, 'cc_last_digits' => cc_last_digits }]))
     end
 
     it 'responds with list of subscriptions' do
@@ -47,7 +49,8 @@
           'created_at' => created_at,
           'url' => url,
           'amount_due' => amount_due,
-          'status' => status
+          'status' => status,
+          'cc_last_digits' => cc_last_digits
         }]
       })
     end