freee 0.3.1 → 1.0.0

data/spec/setenv_spec.rb

@@ -10,8 +10,4 @@ describe Freee::Base do
   it 'should be able to get client' do
     expect(freee.client).not_to be_nil
   end
-
-  it 'should be call of get by Freee' do
-    expect(freee.client.get('/api/1/users/me?companies=true')).to include("user")
-  end
 end

data/spec/spec_helper.rb

@@ -1,8 +1,47 @@
+$:.unshift(File.realpath(File.dirname(__FILE__) + '/../lib'))
 require 'set'
+require 'webmock/rspec'
+require 'freee'
+require 'pry'
 
-$:.unshift(File.realpath(File.dirname(__FILE__) + '/../lib'))
+include WebMock::API
+WebMock.disable_net_connect!(allow_localhost: true)
 
-require 'freee'
+def fixture_path
+  File.expand_path("../fixtures", __FILE__)
+end
+
+def fixture(file)
+  File.new(fixture_path + '/' + file).read
+end
+
+def a_get(path)
+  a_request(:get, Freee::OPTIONS[:site] + path)
+end
+
+def a_post(path)
+  a_request(:post, Freee::OPTIONS[:site] + path)
+end
+
+def a_put(path)
+  a_request(:put, Freee::OPTIONS[:site] + path)
+end
+
+def a_delete(path)
+  a_request(:delete, Freee::OPTIONS[:site] + path)
+end
+
+def stub_get(path)
+  stub_request(:get, Freee::OPTIONS[:site] + path)
+end
+
+def stub_post(path)
+  stub_request(:post, Freee::OPTIONS[:site] + path)
+end
+
+def stub_put(path)
+  stub_request(:put, Freee::OPTIONS[:site] + path)
+end
 
 def get_client_id
   ENV["FREEE_CLIENT_ID"]
@@ -33,7 +72,6 @@ def get_walletable_id
 end
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
   config.filter_run :focus
   config.order = 'random'

data/spec/tag_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+describe Freee::Tag do
+
+  let(:company_id) { -1 }
+  let(:params) { fixture('create_tag.json') }
+  let(:json) { { params: params } }
+
+  before do
+    Freee::Base.config(get_client_id, get_secret_key, get_token)
+    @client = Freee::Tag
+  end
+  describe 'tag' do
+    context '#list' do
+      before do
+        stub_get('/api/1/tags?company_id=-1').to_return(body: fixture('tags.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.list(company_id)
+      end
+      after { WebMock.reset! }
+
+      describe 'should can be able to create instance' do
+        subject { @responses }
+
+        it { is_expected.not_to be_nil }
+        it { is_expected.to be_instance_of(Freee::Response::Tag) }
+      end
+
+      describe 'should get tags of first item for the company' do
+        subject { @responses['tags'].first }
+
+        it { is_expected.not_to be_nil }
+        it { is_expected.to include('id') }
+        it { is_expected.to include('company_id') }
+        it { is_expected.to include('name') }
+        it { is_expected.to include('shortcut1') }
+        it { is_expected.to include('shortcut2') }
+      end
+    end
+
+    context '#create' do
+      before do
+        stub_post('/api/1/tags').
+          with(body: json).
+          to_return(body: fixture('tag.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.create(params)
+      end
+      after { WebMock.reset! }
+
+      describe 'should can be able to create instance' do
+        subject { @responses }
+
+        it { is_expected.not_to be_nil }
+        it { is_expected.to be_instance_of(Freee::Response::Tag) }
+      end
+
+      describe 'should get tags of first item for the company' do
+        subject { @responses['tag'] }
+
+        it { is_expected.not_to be_nil }
+        it { is_expected.to include('id') }
+        it { is_expected.to include('company_id') }
+        it { is_expected.to include('name') }
+        it { is_expected.to include('shortcut1') }
+        it { is_expected.to include('shortcut2') }
+      end
+    end
+  end
+end

data/spec/tax_spec.rb

@@ -1,33 +1,74 @@
 require 'spec_helper'
 
 describe Freee::Tax do
-  let(:client_id) { get_client_id }
-  let(:secret_key) { get_secret_key }
-  let(:token) { get_token }
-  let(:company_id) { get_company_id }
-  let(:tax) { Freee::Tax }
-
-  before(:each) do
-    Freee::Base.config(client_id, secret_key, token)
-  end
+  let(:company_id) { -1 }
 
-  describe 'should can be able to create instance' do
-    subject { tax.list(company_id) }
-    it { is_expected.not_to be_nil }
-    it { is_expected.to be_instance_of(Freee::Response::Tax) }
+  before do
+    Freee::Base.config(get_client_id, get_secret_key, get_token)
+    @client = Freee::Tax
   end
 
-  describe 'should get item of tax for the company' do
-    subject { tax.list(company_id) }
+  describe 'tax' do
+    context '#list' do
+      before do
+        stub_get('/api/1/taxes?company_id=-1').to_return(body: fixture('taxes.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.list(company_id)
+      end
 
-    it { is_expected.to include('taxes') } 
-  end
+      it 'requests the correct resource' do
+        assert_request_requested a_get('/api/1/taxes?company_id=-1')
+      end
 
-  describe 'should get item of first tax for the company' do
-    subject { tax.list(company_id)['taxes'].first }
+      describe 'should can be able to create instance' do
+        subject { @responses }
+        it { is_expected.not_to be_nil }
+        it { is_expected.to be_instance_of(Freee::Response::Tax) }
+      end
 
-    it { is_expected.to include('id') }
-    it { is_expected.to include('name') }
-  end
+      describe 'should get item of tax for the company' do
+        subject { @responses }
+
+        it { is_expected.to include('taxes') }
+      end
+
+      describe 'should get item of first tax for the company' do
+        subject { @responses['taxes'].first }
+
+        it { is_expected.to include('id') }
+        it { is_expected.to include('name') }
+      end
+    end
+
+    context '#list_of_code' do
+      before do
+        stub_get('/api/1/taxes/codes').to_return(body: fixture('taxes_with_code.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.list_of_code
+      end
 
+      it 'requests the correct resource' do
+        assert_request_requested a_get('/api/1/taxes/codes')
+      end
+
+      describe 'should can be able to create instance' do
+        subject { @responses }
+        it { is_expected.not_to be_nil }
+        it { is_expected.to be_instance_of(Freee::Response::Tax) }
+      end
+
+      describe 'should get item of tax for the company' do
+        subject { @responses }
+
+        it { is_expected.to include('taxes') }
+      end
+
+      describe 'should get item of first tax for the company' do
+        subject { @responses['taxes'].first }
+
+        it { is_expected.to include('code') }
+        it { is_expected.to include('name') }
+        it { is_expected.to include('name_ja') }
+      end
+    end
+
+  end
 end

data/spec/transfer_spec.rb

@@ -1,45 +1,100 @@
 require 'spec_helper'
 
 describe Freee::Transfer do
-  let(:client_id) { get_client_id }
-  let(:secret_key) { get_secret_key }
-  let(:token) { get_token }
-  let(:company_id) { get_company_id }
-  let(:transfer) { Freee::Transfer }
-
-  before(:each) do
-    Freee::Base.config(client_id, secret_key, token)
-  end
 
-  describe 'should be able to get client' do
-    subject { transfer.list(company_id) }
-    it { is_expected.not_to be_nil }
-    it { is_expected.to be_instance_of(Freee::Response::Transfer) }
-  end
+  let(:company_id) { -1 }
+  let(:params) { fixture('create_transfer.json') }
+  let(:json) { { params: params } }
 
-  it 'should be get information of transfers for the company' do
-    expect(transfer.list(company_id)).to include('transfers')
+  before do
+    Freee::Base.config(get_client_id, get_secret_key, get_token)
+    @client = Freee::Transfer
   end
 
-  describe 'should be get information of first transfers for the company' do
-    subject { transfer.list(company_id)['transfers'].first }
-
-    it { is_expected.not_to be_nil }
-    it { is_expected.to include('id') }
-    it { is_expected.to include('company_id') }
-    it { is_expected.to include('date') }
-    it { is_expected.to include('amount') }
-    it { is_expected.to include('from_walletable_id') }
-    it { is_expected.to include('from_walletable_type') }
-    it { is_expected.to include('to_walletable_type') }
-    it { is_expected.to include('to_walletable_id') }
-    it { is_expected.to include('description') }
-  end
+  describe 'transfers' do
+    context '#list' do
+      before do
+        stub_get('/api/1/transfers?company_id=-1').to_return(body: fixture('transfers.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.list(company_id)
+      end
+      after { WebMock.reset! }
+
+      it 'requests the correct resource' do
+        assert_request_requested a_get('/api/1/transfers?company_id=-1')
+      end
+
+      it 'should can be able to create instance' do
+        expect(@responses).to include('transfers')
+      end
+
+      describe 'should be get information of account' do
+        subject { @responses }
+        it { is_expected.to be_instance_of(Freee::Response::Transfer) }
+      end
+
+      describe 'should be get information of first transfers for the company' do
+        subject { @responses['transfers'].first }
+
+        it { is_expected.not_to be_nil }
+        it { is_expected.to include('id') }
+        it { is_expected.to include('company_id') }
+        it { is_expected.to include('date') }
+        it { is_expected.to include('amount') }
+        it { is_expected.to include('from_walletable_id') }
+        it { is_expected.to include('from_walletable_type') }
+        it { is_expected.to include('to_walletable_type') }
+        it { is_expected.to include('to_walletable_id') }
+        it { is_expected.to include('description') }
+
+        context "with options" do
+          before do
+            stub_get('/api/1/transfers?company_id=-1&limit=3').to_return(body: fixture('transfers.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+            @responses = @client.list(company_id, limit: 3)['transfers'].length
+          end
+
+          it { assert_request_requested a_get('/api/1/transfers?company_id=-1&limit=3') }
+          it { expect(@responses).not_to be_nil }
+          it { expect(@responses).to eq 3 }
+        end
+      end
+    end
+
+    context '#create' do
+      before do
+        stub_post('/api/1/transfers').
+          with(body: json).
+          to_return(body: fixture('transfer.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.create(params)
+      end
+      after { WebMock.reset! }
+
+      it 'requests the correct resource' do
+        assert_request_requested a_post('/api/1/transfers').with(body: json)
+      end
+
+      it 'should can be able to create instance' do
+        expect(@responses).to include('transfer')
+      end
+
+      describe 'should be get information of account' do
+        subject { @responses }
+        it { is_expected.to be_instance_of(Freee::Response::Transfer) }
+      end
 
-  describe "should be get one's information of first transfers for the company" do
-    subject { transfer.list(company_id, limit: 3)['transfers'].length }
+      describe 'should be get information of first transfers for the company' do
+        subject { @responses['transfer'] }
 
-    it { is_expected.not_to be_nil }
-    it { is_expected.to eq 3 }
+        it { is_expected.not_to be_nil }
+        it { is_expected.to include('id') }
+        it { is_expected.to include('company_id') }
+        it { is_expected.to include('date') }
+        it { is_expected.to include('amount') }
+        it { is_expected.to include('from_walletable_id') }
+        it { is_expected.to include('from_walletable_type') }
+        it { is_expected.to include('to_walletable_type') }
+        it { is_expected.to include('to_walletable_id') }
+        it { is_expected.to include('description') }
+      end
+    end
   end
 end

data/spec/user_spec.rb

@@ -1,30 +1,57 @@
 require 'spec_helper'
 
 describe Freee::User do
-  let(:client_id) { get_client_id }
-  let(:secret_key) { get_secret_key }
-  let(:token) { get_token }
-  let(:user) { Freee::User }
 
-  before(:each) do
-    Freee::Base.config(client_id, secret_key, token)
+  before do
+    Freee::Base.config(get_client_id, get_secret_key, get_token)
+    @client = Freee::User
   end
 
-  describe 'should be get information of user' do
-    subject { user.me }
-    it { is_expected.not_to be_nil }
-    it { is_expected.to be_instance_of(Freee::Response::User) }
-    it { is_expected.to include('user') }
-  end
+  describe 'user' do
+    context '#me' do
+      before do
+        stub_get('/api/1/users/me').to_return(body: fixture('user.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.me
+      end
+      after { WebMock.reset! }
+
+      it 'requests the correct resource' do
+        assert_request_requested a_get('/api/1/users/me')
+      end
+
+      it 'should can be able to create instance' do
+        expect(@responses).to include('user')
+      end
+
+      describe 'should be get information of user' do
+        subject { @responses}
+        it { is_expected.to be_instance_of(Freee::Response::User) }
+      end
+    end
+
+    context '#me_all' do
+      before do
+        stub_get('/api/1/users/me?companies=true').to_return(body: fixture('user_with_companies.json'), headers: {content_type: 'application/json; charset=utf-8'} )
+        @responses = @client.me_all
+      end
+      after { WebMock.reset! }
+
+      it 'requests the correct resource' do
+        assert_request_requested a_get('/api/1/users/me?companies=true')
+      end
+
+      it 'should can be able to create instance' do
+        expect(@responses).to include('user')
+      end
 
-  it 'should be get information of user at all' do
-    result = user.me_all
-    expect(result).to include('user')
-    expect(result['user']).to include('companies')
-    user_company_info_of_first = result['user']['companies'].first
+      it 'should be get information of user at all' do
+        expect(@responses['user']).to include('companies')
+        user_company_info_of_first = @responses['user']['companies'].first
 
-    expect(user_company_info_of_first).to include('id')
-    expect(user_company_info_of_first).to include('display_name')
-    expect(user_company_info_of_first).to include('role')
+        expect(user_company_info_of_first).to include('id')
+        expect(user_company_info_of_first).to include('display_name')
+        expect(user_company_info_of_first).to include('role')
+      end
+    end
   end
 end

data/spec/util_spec.rb

@@ -1,20 +1,29 @@
 require 'spec_helper'
 
 describe Freee::Util do
-  let(:client_id) { get_client_id }
-  let(:secret_key) { get_secret_key }
-  let(:redirect_url) { get_redirect_url }
-  let(:authorization_code) { get_authorization_code }
-  let(:token) { get_token }
+
+  let(:authorization_code) { 'authorization_code' }
+  let(:redirect_uri) { get_redirect_url }
+  let(:access_token) { OpenStruct.new({token: 'a'*64 }) }
+
+  before do
+    stub_post("/oauth/token").
+      with(:body => "{\"grant_type\":\"authorization_code\",\"code\":\"#{authorization_code}\",\"redirect_uri\":#{redirect_uri}}").
+      to_return(:status => 200, :body => "", :headers => {})
+    allow_any_instance_of(OAuth2::Client).to receive(:get_token).
+      and_return(access_token)
+  end
 
   it 'should be get access token' do
     result = Freee::Util.create_token(
-      client_id: client_id,
-      secret_key: secret_key,
-      redirect_url: redirect_url,
+      client_id: get_client_id,
+      secret_key: get_secret_key,
+      redirect_url: redirect_uri,
       authorization_code: authorization_code
     )
-    expect(result).to be_kind_of(String)
-    expect(result).to match(/[a-z0-9]{64}/)
+
+    token = result[:token]
+    expect(token).to be_kind_of(String)
+    expect(token).to match(/[a-z0-9]{64}/)
   end
 end