iugusdk 1.0.0.alpha.1 → 1.0.0.alpha.2

data/spec/fabricators/account_domain_fabricator.rb

@@ -1,4 +1,8 @@
 Fabricator(:account_domain) do
-  account_id 1
+  account
+  url "www.testing.net"
+end
+
+Fabricator(:account_domain_without_account, :class_name => :account_domain) do
   url "www.testing.net"
 end

data/spec/models/account_domain_spec.rb

@@ -10,15 +10,24 @@ describe AccountDomain do
   it { should validate_presence_of(:account_id) }
 
   it 'should accept url with correct pattern' do
-    @account_domain = Fabricate.build(:account_domain) do
-      url 'www.t3st.net'
-    end.should be_valid
+    @account = Fabricate(:account)
+    @domain = AccountDomain.create(:url => 'valid.url.test', :account => @account)
+    @domain.valid?.should be_true
   end
 
   it 'should not accept url with incorrect pattern' do
-    Fabricate.build(:account_domain) do
+    @account = Fabricate(:account)
+    @account.account_domains << @domain = Fabricate.build(:account_domain) do
       url 'http://www.t3st.net'
-    end.should_not be_valid
+    end
+    @domain.valid?.should be_false
+  end
+
+  it 'should not accept repeated domain' do
+    @account = Fabricate(:account)
+    @account.account_domains << @domain1 = AccountDomain.create(:url => 'valid.url.test')
+    @account.account_domains << @domain2 = AccountDomain.create(:url => 'valid.url.test')
+    @domain2.new_record?.should be_true
   end
 
   it 'should not accept url in the blacklist' do
@@ -37,6 +46,16 @@ describe AccountDomain do
     end
   end
 
+  it 'should set the first verified domain as primary if the primary domain is destroyed' do
+    @account = Fabricate(:account)
+    @account.account_domains << @domain1 = AccountDomain.create(:url => "url1.test.test", :verified => true)
+    @account.account_domains << @domain2 = AccountDomain.create(:url => "url2.test.test", :verified => true)
+    @domain1.set_primary
+    @domain1.destroy
+    @domain2.reload
+    @domain2.primary.should be_true
+  end
+
   context "calculate_token" do
     it 'should return url token' do
       @account_domain = Fabricate(:account_domain) do
@@ -77,7 +96,15 @@ describe AccountDomain do
       @account_domain1.reload
       @account_domain1.verified.should be_false
     end
-  
+    
+    it 'should set primary as true if its the first domain of the account' do
+      @account = Fabricate(:account)
+      @account.account_domains << @domain = Fabricate(:account_domain) { url 'www.test.net' }
+      @domain.verify
+      @domain.reload
+      @domain.primary.should be_true
+    end
+
   end
 
   context "set_primary" do
@@ -86,23 +113,33 @@ describe AccountDomain do
       @account.account_domains <<  @domain1 = Fabricate(:account_domain) do
         url "www.url1.net"
         verified true
-        primary false
+        primary true
       end
       @account.account_domains << @domain2 = Fabricate(:account_domain) do
         url "www.url2.net"
         verified true
-        primary true
+        primary false
       end
+      @domain1.account = @account
+      @domain2.account = @account
+      @domain1.save
+      @domain2.save
     end
 
     it 'should make domain primary' do
-      @domain1.set_primary
-      @domain1.primary.should be_true
+      @domain2.set_primary
+      @domain2.primary.should be_true
     end
 
     it 'should make other account domains not primary' do
-      @domain1.set_primary
-      @domain2.reload
+      @domain2.set_primary
+      @domain1.reload
+      @domain1.primary.should be_false
+    end
+
+    it 'should not make domain primary if its not verified' do
+      @domain2.update_attribute(:verified, false)
+      @domain2.set_primary
       @domain2.primary.should be_false
     end
   

data/spec/models/account_spec.rb

@@ -9,6 +9,7 @@ describe Account do
   it { should have_many(:account_users) }
   it { should have_many(:users).through(:account_users) }
   it { should validate_uniqueness_of(:subdomain) }
+  it { should validate_uniqueness_of(:api_token) }
 
   it "should return true for a valid user of account" do
     @user = Fabricate(:user, :email => "me@me.com")
@@ -22,6 +23,11 @@ describe Account do
     @account = Fabricate.build(:account) { subdomain "subdominio" }.should_not be_valid
   end
 
+  it 'should set api_token before create' do
+    @account = Fabricate(:account)
+    @account.api_token.should_not be_nil
+  end
+
   context "destruction_job method" do
     before(:each) do
       @account = Fabricate(:account)
@@ -132,10 +138,7 @@ describe Account do
       @account = Fabricate(:account) do
         subdomain "subdomainfind"
       end
-      @account.account_domains << Fabricate(:account_domain) do 
-        url "getfromdomain.account.test"
-        verified true
-      end
+      @account.account_domains << AccountDomain.create(:url => "getfromdomain.account.test", :verified => true)
     end
 
     it 'should return the account which owns the domain' do
@@ -150,6 +153,31 @@ describe Account do
       Account.get_from_domain("notused.domain.test").should be_nil
     end
   end
+
+  context "generate_api_token" do
+    before(:each) do
+      @account = Fabricate(:account)
+    end
+    
+    it 'should return a new token' do
+      old_token = @account.api_token
+      @account.send(:generate_api_token).should_not == old_token
+    end
+
+  end
+
+  context "update_api_token" do
+    before(:each) do
+      @account = Fabricate(:account)
+    end
+
+    it 'should replace api_token' do
+      old_token = @account.api_token
+      @account.update_api_token
+      @account.api_token.should_not == old_token
+    end
+  
+  end
   
 
 end

data/spec/models/user_invitation_spec.rb

@@ -4,7 +4,15 @@ describe UserInvitation do
 
   it { should belong_to :account }
 
-  context "before save" do
+  it 'should not accept an email already used by an account user' do
+    @account = Fabricate(:account)
+    @user = Fabricate(:user, :email => "invited@test.test")
+    @account.users << @user
+    @user_invitation = UserInvitation.new(:email => "invited@test.test", :account_id => @account.id)
+    @user_invitation.should_not be_valid
+  end
+
+  context "before save" do 
 
     context "if has no token" do
       it 'should generate one' do
@@ -81,6 +89,20 @@ describe UserInvitation do
       @user_invitation.accept(@user)
       AccountUser.last.is?('user').should be_true
     end
+
+    it 'should return true if successfull' do
+      @user_invitation.account = @account
+      @user_invitation.save
+      @user_invitation.accept(@user).should be_true
+    end
+
+    it 'should return false if user is already member of the account' do
+      @user_invitation.account = @account
+      @user_invitation.save
+      @account.users << @user
+      @user_invitation.accept(@user).should be_false
+    end
+
   end
 end
   

data/spec/models/user_spec.rb

@@ -7,6 +7,63 @@ describe User do
   it { should have_many(:account_users) }
   it { should have_many(:accounts).through(:account_users) }
   it { should have_many(:social_accounts) }
+  it { should validate_presence_of(:email) }
+
+  it 'should not require email if guest == true' do
+    user = Fabricate.build(:user) do
+      email nil
+      guest true
+    end.should be_valid
+  end
+
+  context "when enable_user_confirmation == false" do
+    before(:each) do
+      IuguSDK::enable_user_confirmation = false
+    end
+
+    it 'should skip confirmation' do
+      @user = User.create(:email => "confirmation@skip.test", :password => "testtest", :password_confirmation => "testtest")
+      @user.confirmed?.should be_true
+    end
+  end
+
+  context "when enable_user_confirmation == true" do
+    before(:each) do
+      IuguSDK::enable_user_confirmation = true
+    end
+
+    it 'should not skip confirmation' do
+      @user = User.create(:email => "confirmation@needed.test", :password => "testtest", :password_confirmation => "testtest")
+      @user.confirmed?.should be_false
+    end
+
+  end
+
+  context "when enable_email_reconfirmation == true" do
+    before(:each) do
+      IuguSDK::enable_email_reconfirmation = true
+    end
+
+    it 'should not skip email reconfirmation' do
+      @user = User.create(:email => "reconfirmation@needed.test", :password => "testtest", :password_confirmation => "testtest")
+      @user.email = "new@email.test"
+      @user.save
+      @user.email.should == "reconfirmation@needed.test"
+    end
+  end
+
+  context "when enable_email_reconfirmation == false" do
+    before(:each) do
+      IuguSDK::enable_email_reconfirmation = false
+    end
+
+    it 'should skip email reconfirmation' do
+      @user = User.create(:email => "reconfirmation@needed.test", :password => "testtest", :password_confirmation => "testtest")
+      @user.email = "new@email.test"
+      @user.save
+      @user.email.should == "new@email.test"
+    end
+  end
 
   context "create_social" do
     before do
@@ -183,5 +240,24 @@ describe User do
   
   end
 
+  context "create_guest method" do
+    before(:each) do
+      @user = User.create_guest("pt-BR")
+    end
+
+    it { @user.class.should == User }
+
+    it { @user.email.should be_blank }
+  
+    it { @user.guest.should be_true }
+
+    it { @user.name.should == "Guest" }
+  
+    it { @user.locale.should == "pt-BR" }
+
+    it { @user.confirmed?.should be_true }
+    
+  end
+
 
 end

data/spec/requests/account_domain_spec.rb

@@ -3,72 +3,106 @@ require 'spec_helper'
 describe "Account Domains requests" do
   context "index view" do
     before(:each) do
+      IuguSDK::enable_social_login = true
       visit "/account/auth/facebook"
       @user = User.last
       @account = Account.last
-      @account.account_domains << @domain = Fabricate(:account_domain) { url "index.test.net" }
+      @account.account_domains << @domain = AccountDomain.create( url: "index.test.net" )
     end
 
-    context "when domain is primary" do
+    context "when enable_subdomain == true" do
       before(:each) do
-        @domain.update_attribute(:primary, true)
+        IuguSDK::enable_subdomain = true
         visit account_domains_index_path(@account.id)
       end
 
-      it { page.should have_content I18n.t("iugu.primary") }
-
+      it { page.should have_field 'account[subdomain]' }
+    
     end
-  
-    context "when domain is not primary" do
+
+    context "when enable_subdomain == false" do
       before(:each) do
-        @domain.update_attribute(:primary, false)
+        IuguSDK::enable_custom_domain = true
+        IuguSDK::enable_subdomain = false
         visit account_domains_index_path(@account.id)
       end
 
-      it { page.should have_link I18n.t("iugu.set_primary") }
-
-    end
+      it { page.should_not have_field 'account[subdomain]' }
     
-    context "when domain is verified" do
+    end
+
+    context "when enable_custom_domain == true" do
       before(:each) do
-        @domain.update_attribute(:verified, true)
-        visit account_domains_index_path(@account.id)
+        IuguSDK::enable_custom_domain = true
       end
-      
-      it { page.should have_content I18n.t("iugu.verified") }
 
-    end
+      context "and domain is primary" do
+        before(:each) do
+          @domain.update_attribute(:primary, true)
+          visit account_domains_index_path(@account.id)
+        end
+
+        it { page.should have_content I18n.t("iugu.primary") }
   
-    context "when domain is not verified" do
-      before(:each) do
-        @domain.update_attribute(:verified, false)
-        visit account_domains_index_path(@account.id)
       end
+  
+      context "and domain is not primary" do
+        before(:each) do
+          @domain.update_attribute(:primary, false)
+        end
+
+        context "and domain is verified" do
+          before(:each) do
+            @domain.update_attribute(:verified, true)
+            visit account_domains_index_path(@account.id)
+          end
+
+          it { page.should have_link I18n.t("iugu.set_primary") }
+      
+        end
 
-      it { page.should have_link I18n.t("iugu.not_verified") }
+        context "and domain is not verified" do
+          before(:each) do
+            @domain.update_attribute(:verified, false)
+            visit account_domains_index_path(@account.id)
+          end
 
-    end
+          it { page.should_not have_link I18n.t("iugu.set_primary") }
+      
+        end
 
-    context "when current_user is the owner of the account" do
-      before(:each) do
-        @account_user = AccountUser.find_by_user_id_and_account_id(@user.id, @account.id)
-        @account_user.set_roles ["owner"]
-        visit account_domains_index_path(@account.id)
       end
+    
+      context "and domain is verified" do
+        before(:each) do
+          @domain.update_attribute(:verified, true)
+          visit account_domains_index_path(@account.id)
+        end
+      
+        it { page.should have_content I18n.t("iugu.verified") }
 
-      it { page.should have_link I18n.t("iugu.remove") }
+      end
+  
+      context "and domain is not verified" do
+        before(:each) do
+          @domain.update_attribute(:verified, false)
+          visit account_domains_index_path(@account.id)
+        end
 
-    end
+        it { page.should have_link I18n.t("iugu.not_verified") }
 
-    context "when current_user is not the owner of the account" do
-      before(:each) do
-        @account.account_users << Fabricate(:account_user) { user Fabricate(:user) { email "notowner@account.test" } }
-        @account_user = AccountUser.find_by_user_id_and_account_id(@user.id, @account.id)
-        @account_user.set_roles ["user"]
-        visit account_domains_index_path(@account.id)
       end
 
-      it { page.should_not have_link I18n.t("iugu.remove") }
+      context "and current_user is the owner of the account" do
+        before(:each) do
+          @account_user = AccountUser.find_by_user_id_and_account_id(@user.id, @account.id)
+          @account_user.set_roles ["owner"]
+          visit account_domains_index_path(@account.id)
+        end
+  
+        it { page.should have_link I18n.t("iugu.remove") }
+  
+      end
 
     end
 

data/spec/requests/account_roles_spec.rb

@@ -3,6 +3,7 @@ require 'spec_helper'
 describe 'Account Roles requests' do
   context "edit view" do
     before(:each) do
+      IuguSDK::enable_social_login = true
       visit '/account/auth/facebook'
       @user = User.last
       @account = @user.accounts.first

data/spec/requests/account_spec.rb

@@ -2,6 +2,7 @@ require 'spec_helper'
 
 describe 'accounts settings view' do
   before(:each) do
+    IuguSDK::enable_social_login = true
     visit '/account/auth/facebook'
     visit account_settings_path
     @user = User.last
@@ -10,11 +11,24 @@ describe 'accounts settings view' do
 
   it { page.should have_link I18n.t("iugu.settings") }
 
-  context "when allow_create_account == false" do
+  context "when enable_multiple_accounts == true" do
     before(:each) do
-      IuguSDK::allow_create_account = false
+      IuguSDK::enable_multiple_accounts = true
+      visit account_settings_path
     end
 
+    it { page.should have_link I18n.t("iugu.create_account") }
+  
+  end
+
+  context "when enable_multiple_accounts == false" do
+    before(:each) do
+      IuguSDK::enable_multiple_accounts = false
+      visit account_settings_path
+    end
+
+    it { page.should_not have_link I18n.t("iugu.create_account") }
+
     context "and user has only one account" do
       before(:each) do
         @user = User.last
@@ -47,16 +61,35 @@ describe 'accounts settings view' do
       @user = User.last
       @user.accounts << @target_account = Fabricate(:account)
       @account_user = AccountUser.last
+      IuguSDK::enable_custom_domain = true
+      IuguSDK::enable_subdomain = true
+    end
+
+    context "when current_user admin the account" do
+      before(:each) do
+        @account_user.set_roles ["admin"]
+        visit account_view_path(@target_account.id)
+      end
+
+      it { page.should have_link I18n.t("iugu.manage") }
+
     end
 
     context "when current_user owns the account" do
       before(:each) do
-        AccountUser.last.set_roles ["owner"]
+        @account_user.set_roles ["owner"]
         visit account_view_path(@target_account.id)
       end
+      it { page.should have_link I18n.t("iugu.manage") }
 
       it { page.should have_link I18n.t("iugu.cancel_account") }
+
+      it { page.should have_content @target_account.api_token }
       
+      it { page.should have_link I18n.t("iugu.generate_new_token") }
+
+      it { page.should have_field 'account[name]' }
+
       context "when account is being canceled" do
         context "if delay_account_exclusion == 0" do
           before(:each) do
@@ -88,8 +121,37 @@ describe 'accounts settings view' do
         visit account_view_path(@target_account.id)
       end
 
+      it { page.should_not have_field 'account[name]' }
+
+      it { page.should_not have_link I18n.t("iugu.manage") }
+
       it { page.should_not have_link I18n.t("iugu.cancel_account") }
 
+      it { page.should_not have_content @target_account.api_token }
+
+      it { page.should_not have_link I18n.t("iugu.generate_new_token") }
+
+    end
+
+    context "when enable_custom_domain == false && enable_subdomain == false" do
+      before(:each) do
+        IuguSDK::enable_custom_domain = false
+        IuguSDK::enable_subdomain = false
+        visit account_view_path(@target_account.id)
+      end
+    
+      it { page.should_not have_link I18n.t("iugu.manage") }
+      
+    end
+
+    context "when enable_account_api_token == false" do
+      before(:each) do
+        IuguSDK::enable_account_api_token = false
+        visit account_view_path(@target_account.id)
+      end
+      
+      it { page.should_not have_link I18n.t("iugu.generate_new_token") }
+    
     end
     
   

data/spec/requests/account_users_spec.rb

@@ -2,6 +2,7 @@ require 'spec_helper'
 
 describe "Account Users Requests" do
   before(:each) do
+    IuguSDK::enable_social_login = true
     @user = Fabricate(:user)
     visit '/account/auth/facebook'
     @current_user = User.last