gooddata 0.6.7 → 0.6.8

data/spec/unit/models/profile_spec.rb

@@ -7,11 +7,11 @@ require 'gooddata/models/project'
 
 describe GoodData::Profile do
   before(:all) do
-    ConnectionHelper.create_default_connection
-    @user = GoodData::Domain.find_user_by_login(ConnectionHelper::DEFAULT_DOMAIN, ConnectionHelper::DEFAULT_USERNAME)
+    @client = ConnectionHelper.create_default_connection
+    @user = GoodData::Domain.find_user_by_login(ConnectionHelper::DEFAULT_DOMAIN, ConnectionHelper::DEFAULT_USERNAME, :client => @client)
 
     @users = [
-      GoodData::Profile.new(
+      @client.create(GoodData::Profile,
         {
           'accountSetting' => {
             'email' => 'petr.cvengros@gooddata.com',
@@ -21,7 +21,7 @@ describe GoodData::Profile do
         }
       ),
 
-      GoodData::Profile.new(
+      @client.create(GoodData::Profile,
         {
           'accountSetting' => {
             'email' => 'tomas.korcak@gooddata.com',
@@ -31,7 +31,7 @@ describe GoodData::Profile do
         }
       ),
 
-      GoodData::Profile.new(
+      @client.create(GoodData::Profile,
         {
           'accountSetting' => {
             'email' => 'patrick.mcconlogue@gooddata.com',
@@ -42,7 +42,7 @@ describe GoodData::Profile do
         }
       ),
 
-      GoodData::Profile.new(
+      @client.create(GoodData::Profile,
         {
           'accountSetting' => {
             'email' => 'tomas.svarovsky@gooddata.com',
@@ -55,20 +55,24 @@ describe GoodData::Profile do
   end
 
   after(:all) do
-    GoodData.disconnect
+    @client.disconnect
+  end
+
+  def deep_dup(obj)
+    Marshal.load(Marshal.dump(obj))
   end
 
   describe '#==' do
     it 'Returns true for same objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
       res = user1 == user2
       res.should be_true
     end
 
     it 'Returns false for different objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
 
       # Do some little modification
       user2.first_name = 'kokos'
@@ -80,15 +84,15 @@ describe GoodData::Profile do
 
   describe '#!=' do
     it 'Returns false for same objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
       res = user1 != user2
       res.should be_false
     end
 
     it 'Returns true for different objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
 
       # Do some little modification
       user2.first_name = 'kokos'
@@ -100,8 +104,8 @@ describe GoodData::Profile do
 
   describe '#apply' do
     it 'When diff of two objects applied to first result should be same as second object' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
 
       # Do some little modification
       user2.first_name = 'kokos'
@@ -120,16 +124,16 @@ describe GoodData::Profile do
 
   describe '#diff' do
     it 'Returns empty hash for same objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
       res = user1.diff(user2)
       expect(res).to be_instance_of(Hash)
       res.length.should eql(0)
     end
 
     it 'Returns non empty hash for different objects' do
-      user1 = GoodData.user.dup
-      user2 = GoodData.user.dup
+      user1 = deep_dup(@user)
+      user2 = deep_dup(@user)
 
       # Do some little modification
       user2.first_name = 'kokos'

data/spec/unit/models/project_blueprint_spec.rb

@@ -102,33 +102,33 @@ describe GoodData::Model::ProjectBlueprint do
   end
 
   it 'should be able to grab attribute' do
-    pending('UAAA')
+    pending('Wrap into object')
     @repos.labels.size.should == 1
     @repos.labels.first.attribute.name.should == 'repo_id'
   end
 
   it 'anchor should have labels' do
-    pending('UAAA')
+    pending('Wrap into object')
     @repos.anchor.labels.first.identifier.should == 'label.repos.repo_id'
   end
 
   it 'attribute should have labels' do
-    pending('UAAA')
+    pending('Wrap into object')
     @repos.attributes.first.labels.first.identifier.should == 'label.repos.department'
   end
 
   it 'commits should have one fact' do
-    pending('UAAA')
+    pending('Wrap into object')
     @commits.facts.size.should == 1
   end
 
   it 'Anchor on repos should have a label' do
-    pending('UAAA')
+    pending('Wrap into object')
     @repos.anchor.labels.size.should == 2
   end
 
   it 'should not have a label for a dataset without anchor with label' do
-    pending('UAAA')
+    pending('Wrap into object')
     @commits.anchor.should == nil
     # @commits.to_schema.anchor.labels.empty?.should == true
   end

data/spec/unit/models/project_role_spec.rb

@@ -5,14 +5,14 @@ require 'gooddata/models/project_role'
 
 describe GoodData::ProjectRole do
   before(:all) do
-    ConnectionHelper.create_default_connection
-    @project = ProjectHelper.get_default_project
+    @client = ConnectionHelper.create_default_connection
+    @project = ProjectHelper.get_default_project(:client => @client)
     @roles = @project.roles
     @role = @roles.first
   end
 
   after(:all) do
-    GoodData.disconnect
+    @client.disconnect
   end
 
   describe '#author' do
@@ -80,8 +80,6 @@ describe GoodData::ProjectRole do
 
   describe '#users' do
     it 'Returns users as Array<GoodData::Profile>' do
-      pending  'Disable as it is TOOOO SLOOOW'
-      
       res = @role.users
       expect(res).to be_an_instance_of(Array)
       res.each do |user|

data/spec/unit/models/project_spec.rb

@@ -1,14 +1,16 @@
 # encoding: UTF-8
 
+require 'pmap'
+
 require 'gooddata'
 
 describe GoodData::Project do
   before(:each) do
-    ConnectionHelper::create_default_connection
+    @client = ConnectionHelper::create_default_connection
   end
 
   after(:each) do
-    GoodData.disconnect
+    @client.disconnect
   end
 
   def load_users_from_csv
@@ -40,19 +42,22 @@ describe GoodData::Project do
 
   describe '#[]' do
     it 'Accepts :all parameter' do
-      projects = GoodData::Project[:all]
+      projects = GoodData::Project[:all, :client => @client]
       projects.should_not be_nil
       projects.should be_a_kind_of(Array)
+      projects.pmap do |project|
+        expect(project).to be_an_instance_of(GoodData::Project)
+      end
     end
 
     it 'Returns project if ID passed' do
-      project = ProjectHelper.get_default_project
+      project = GoodData::Project[ProjectHelper::PROJECT_ID, :client => @client]
       project.should_not be_nil
       project.should be_a_kind_of(GoodData::Project)
     end
 
     it 'Returns project if URL passed' do
-      project = ProjectHelper.get_default_project
+      project = GoodData::Project[ProjectHelper::PROJECT_URL, :client => @client]
       project.should_not be_nil
       project.should be_a_kind_of(GoodData::Project)
     end
@@ -65,7 +70,7 @@ describe GoodData::Project do
 
   describe '#all' do
     it 'Returns all projects' do
-      projects = GoodData::Project.all
+      projects = GoodData::Project.all(:client => @client)
       projects.should_not be_nil
       projects.should be_a_kind_of(Array)
     end
@@ -73,7 +78,7 @@ describe GoodData::Project do
 
   describe '#get_role_by_identifier' do
     it 'Looks up for role by identifier' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       role = project.get_role_by_identifier('readOnlyUserRole')
       role.should_not be_nil
       role.should be_a_kind_of(GoodData::ProjectRole)
@@ -82,7 +87,7 @@ describe GoodData::Project do
 
   describe '#get_role_by_summary' do
     it 'Looks up for role by summary' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       role = project.get_role_by_summary('read only user role')
       role.should_not be_nil
       role.should be_a_kind_of(GoodData::ProjectRole)
@@ -91,7 +96,7 @@ describe GoodData::Project do
 
   describe '#get_role_by_title' do
     it 'Looks up for role by title' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       role = project.get_role_by_title('Viewer')
       role.should_not be_nil
       role.should be_a_kind_of(GoodData::ProjectRole)
@@ -100,26 +105,26 @@ describe GoodData::Project do
 
   describe "#member" do
     it 'Returns GoodData::Membership when looking for existing user using email' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member('svarovsky+gem_tester@gooddata.com')
       expect(res).to be_instance_of(GoodData::Membership)
     end
 
     it 'Returns GoodData::Membership when looking for existing user using URL' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member(ConnectionHelper::DEFAULT_USER_URL)
       expect(res).to be_instance_of(GoodData::Membership)
     end
 
     it 'Returns GoodData::Membership when looking for existing user using GoodData::Profile' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       user = project.members.first
       res = project.member(user)
       expect(res).to be_instance_of(GoodData::Membership)
     end
 
     it 'Returns null for non-existing user' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member('jan.kokotko@gooddata.com')
       res.should be_nil
     end
@@ -127,39 +132,39 @@ describe GoodData::Project do
 
   describe "#member?" do
     it 'Returns true when looking for existing user using email' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member?('svarovsky+gem_tester@gooddata.com')
       res.should be_true
     end
 
     it 'Returns true when looking for existing user using URL' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member?(ConnectionHelper::DEFAULT_USER_URL)
       res.should be_true
     end
 
     it 'Returns true when looking for existing user using GoodData::Profile' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       user = project.members.first
       res = project.member?(user)
       res.should be_true
     end
 
     it 'Returns false for non-existing user' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       res = project.member?('jan.kokotko@gooddata.com')
       res.should be_false
     end
 
     it 'Returns true for existing user when using optional list' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       list = project.members
       res = project.member?('svarovsky+gem_tester@gooddata.com', list)
       res.should be_true
     end
 
     it 'Returns false for non-existing user when using optional list' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
       list = []
       res = project.member?('svarovsky+gem_tester@gooddata.com', list)
       res.should be_false
@@ -169,16 +174,16 @@ describe GoodData::Project do
   describe '#processes' do
     it 'Returns the processes' do
 
-      GoodData.project = ProjectHelper::PROJECT_ID
-      proj = GoodData.project
-      processes = proj.processes
+      project = GoodData::Project[ProjectHelper::PROJECT_ID, {:client => @client}]
+      processes = project.processes
       expect(processes).to be_a_kind_of(Array)
     end
   end
 
   describe '#roles' do
     it 'Returns array of GoodData::ProjectRole' do
-      project = ProjectHelper.get_default_project
+      pending 'Investigate why is this soo slooow'
+      project = ProjectHelper.get_default_project(:client => @client)
       roles = project.roles
       expect(roles).to be_instance_of(Array)
 
@@ -190,9 +195,9 @@ describe GoodData::Project do
 
   describe '#users' do
     it 'Returns array of GoodData::Users' do
-      pending 'Disable as it is TOOOO SLOOOW'
+      pending 'Investigate why is this soo slooow'
 
-      project = GoodData::Project[ProjectHelper::PROJECT_ID]
+      project = GoodData::Project[ProjectHelper::PROJECT_ID, {:client => @client}]
 
       invitations = project.invitations
       invitations.should_not be_nil
@@ -220,7 +225,7 @@ describe GoodData::Project do
         # invitations = user.invitations
         # invitations.should_not be_nil
 
-        if(user.email == 'tomas.korcak@gooddata.com')
+        if (user.email == 'tomas.korcak@gooddata.com')
           projects = user.projects
           projects.should_not be_nil
           expect(projects).to be_instance_of(Array)
@@ -235,7 +240,7 @@ describe GoodData::Project do
 
   describe '#users_create' do
     it 'Creates new users' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       users = (0...10).map do |i|
         num = rand(1e6)
@@ -265,7 +270,8 @@ describe GoodData::Project do
         GoodData::Membership.new(json)
       end
 
-      res = GoodData::Domain.users_create(users)
+      project = GoodData::Project[ProjectHelper::PROJECT_ID, {:client => @client}]
+      res = GoodData::Domain.users_create(users, nil, {:client => @client, :project => project})
 
       project.users_create(users)
 
@@ -280,7 +286,7 @@ describe GoodData::Project do
   describe '#users_import' do
     it 'Import users from CSV' do
 
-      project = GoodData::Project[ProjectHelper::PROJECT_ID]
+      project = GoodData::Project[ProjectHelper::PROJECT_ID, {:client => @client}]
 
       new_users = load_users_from_csv
 
@@ -290,7 +296,7 @@ describe GoodData::Project do
 
   describe '#set_user_roles' do
     it 'Properly updates user roles as needed' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       project.set_user_roles(ConnectionHelper::DEFAULT_USERNAME, 'admin')
     end
@@ -298,12 +304,12 @@ describe GoodData::Project do
 
   describe '#set_users_roles' do
     it 'Properly updates user roles as needed for bunch of users' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       list = load_users_from_csv
 
       # Create domain users
-      domain_users = GoodData::Domain.users_create(list, ConnectionHelper::DEFAULT_DOMAIN)
+      domain_users = GoodData::Domain.users_create(list, ConnectionHelper::DEFAULT_DOMAIN, :client => @client, :project => project)
       expect(domain_users.length).to equal(list.length)
 
       # Create list with user, desired_roles hashes
@@ -326,12 +332,12 @@ describe GoodData::Project do
       end
 
       domain_users.each do |user|
-         user.delete if user.email != ConnectionHelper::DEFAULT_USERNAME
+        user.delete if user.email != ConnectionHelper::DEFAULT_USERNAME
       end
     end
 
     it 'Properly updates user roles when user specified by email and :roles specified as array of string with role names' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       list = [
         {
@@ -345,7 +351,7 @@ describe GoodData::Project do
     end
 
     it 'Properly updates user roles when user specified by email and :roles specified as string with role name' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       list = [
         {
@@ -361,7 +367,7 @@ describe GoodData::Project do
 
   describe '#summary' do
     it 'Properly gets title of project' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       res = project.summary
       expect(res).to include(ProjectHelper::PROJECT_SUMMARY)
@@ -370,7 +376,7 @@ describe GoodData::Project do
 
   describe '#title' do
     it 'Properly gets title of project' do
-      project = ProjectHelper.get_default_project
+      project = ProjectHelper.get_default_project(:client => @client)
 
       res = project.title
       expect(res).to include(ProjectHelper::PROJECT_TITLE)