gooddata 0.6.3 → 0.6.4

data/spec/unit/models/project_blueprint_spec.rb

@@ -1,5 +1,4 @@
 # encoding: UTF-8
-
 require 'gooddata'
 
 describe GoodData::Model::ProjectBlueprint do
@@ -20,19 +19,34 @@ describe GoodData::Model::ProjectBlueprint do
   end
 
   it 'valid blueprint should be marked as valid' do
-    @blueprint.model_valid?.should == true
+    @blueprint.valid?.should == true
   end
 
   it 'valid blueprint should give you empty array of errors' do
-    expect(@blueprint.validate_model).to be_empty
+    expect(@blueprint.validate).to be_empty
+  end
+
+  it 'model should be invalid if it contains more than one anchor' do
+    builder = GoodData::Model::ProjectBuilder.create("my_bp") do |p|
+      p.add_dataset("repos") do |d|
+        d.add_anchor("repo_id")
+        d.add_anchor("repo_id2")
+        d.add_attribute("name")
+      end
+    end
+    bp = GoodData::Model::ProjectBlueprint.new(builder)
+    bp.valid?.should == false
+    errors = bp.validate
+    errors.first.should == {:anchor => 2}
+    errors.count.should == 1
   end
 
   it 'invalid blueprint should be marked as invalid' do
-    @invalid_blueprint.model_valid?.should == false
+    @invalid_blueprint.valid?.should == false
   end
   
   it 'invalid blueprint should give you list of violating references' do
-    errors = @invalid_blueprint.validate_model
+    errors = @invalid_blueprint.validate
     errors.size.should == 1
     errors.first.should == {
         type: 'reference',
@@ -47,6 +61,19 @@ describe GoodData::Model::ProjectBlueprint do
     expect(refs).to be_empty
   end
 
+  it "should be possible to create from ProjectBlueprint from ProjectBuilder in several ways" do
+    builder = GoodData::Model::SchemaBuilder.new("stuff") do |d|
+      d.add_attribute("id", :title => "My Id")
+      d.add_fact("amount", :title => "Amount")
+    end
+
+    bp1 = GoodData::Model::ProjectBlueprint.new(builder)
+    bp1.valid?.should == true
+
+    bp2 = builder.to_blueprint
+    bp2.valid?.should == true
+  end
+
   it 'should be able to get dataset by name' do
     ds = @blueprint.find_dataset('devs')
     ds.name.should == 'devs'
@@ -107,9 +134,9 @@ describe GoodData::Model::ProjectBlueprint do
         d.add_label("name", :reference => "invalid_ref")
       end
     end
-    bp = GoodData::Model::ProjectBlueprint.from_json(builder.to_hash)
-    bp.model_valid?.should == false
-    errors = bp.validate_model
+    bp = GoodData::Model::ProjectBlueprint.new(builder)
+    bp.valid?.should == false
+    errors = bp.validate
     errors.count.should == 1
   end
 

data/spec/unit/models/project_role_spec.rb

@@ -16,16 +16,16 @@ describe GoodData::ProjectRole do
   end
 
   describe '#author' do
-    it 'Returns author as GoodData::AccountSettings' do
+    it 'Returns author as GoodData::Profile' do
       res = @role.author
-      expect(res).to be_an_instance_of(GoodData::AccountSettings)
+      expect(res).to be_an_instance_of(GoodData::Profile)
     end
   end
 
   describe '#contributor' do
-    it 'Returns contributor as GoodData::AccountSettings' do
+    it 'Returns contributor as GoodData::Profile' do
       res = @role.contributor
-      expect(res).to be_an_instance_of(GoodData::AccountSettings)
+      expect(res).to be_an_instance_of(GoodData::Profile)
     end
   end
 
@@ -79,11 +79,11 @@ describe GoodData::ProjectRole do
   end
 
   describe '#users' do
-    it 'Returns users as Array<GoodData::AccountSettings>' do
+    it 'Returns users as Array<GoodData::Profile>' do
       res = @role.users
       expect(res).to be_an_instance_of(Array)
       res.each do |user|
-        expect(user).to be_an_instance_of(GoodData::AccountSettings)
+        expect(user).to be_an_instance_of(GoodData::Profile)
       end
     end
   end

data/spec/unit/models/project_spec.rb

@@ -11,13 +11,38 @@ describe GoodData::Project do
     GoodData.disconnect
   end
 
+  def load_users_from_csv
+    GoodData::Helpers::Csv.read(:path => CsvHelper::CSV_PATH_IMPORT, :header => true) do |row|
+      json = {
+        'user' => {
+          'content' => {
+            'email' => row[2],
+            'login' => row[2],
+            'firstname' => row[0],
+            'lastname' => row[1],
+
+            # Following lines are ugly hack
+            'role' => row[6],
+            'password' => row[3],
+            'domain' => row[9],
+
+            # And following lines are even much more ugly hack
+            # 'sso_provider' => '',
+            # 'authentication_modes' => ['sso', 'password']
+          },
+          'meta' => {}
+        }
+      }
+
+      GoodData::Membership.new(json)
+    end
+  end
+
   describe '#[]' do
     it 'Accepts :all parameter' do
-      pending 'Investigate which credentials use'
-      
-      project = GoodData::Project[:all]
-      project.should_not be_nil
-      project.should be_a_kind_of(Array)
+      projects = GoodData::Project[:all]
+      projects.should_not be_nil
+      projects.should be_a_kind_of(Array)
     end
 
     it 'Returns project if ID passed' do
@@ -40,9 +65,9 @@ describe GoodData::Project do
 
   describe '#all' do
     it 'Returns all projects' do
-      pending 'Investigate which credentials use'
-
-      GoodData::Project.all
+      projects = GoodData::Project.all
+      projects.should_not be_nil
+      projects.should be_a_kind_of(Array)
     end
   end
 
@@ -73,14 +98,81 @@ describe GoodData::Project do
     end
   end
 
+  describe "#member" do
+    it 'Returns GoodData::Membership when looking for existing user using email' do
+      project = ProjectHelper.get_default_project
+      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
+      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
+      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
+      res = project.member('jan.kokotko@gooddata.com')
+      res.should be_nil
+    end
+  end
+
+  describe "#member?" do
+    it 'Returns true when looking for existing user using email' do
+      project = ProjectHelper.get_default_project
+      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
+      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
+      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
+      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
+      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
+      list = []
+      res = project.member?('svarovsky+gem_tester@gooddata.com', list)
+      res.should be_false
+    end
+  end
+
   describe '#processes' do
     it 'Returns the processes' do
-      pending 'Investigate which credentials to use'
 
       GoodData.project = ProjectHelper::PROJECT_ID
-
       proj = GoodData.project
-      procs = proj.processes
+      processes = proj.processes
+      expect(processes).to be_a_kind_of(Array)
     end
   end
 
@@ -98,7 +190,6 @@ describe GoodData::Project do
 
   describe '#users' do
     it 'Returns array of GoodData::Users' do
-      pending 'Investigate which credentials to use'
 
       project = GoodData::Project[ProjectHelper::PROJECT_ID]
 
@@ -110,7 +201,7 @@ describe GoodData::Project do
       expect(users).to be_instance_of(Array)
 
       users.each do |user|
-        expect(user).to be_instance_of(GoodData::User)
+        expect(user).to be_instance_of(GoodData::Membership)
 
         roles = user.roles
         roles.should_not be_nil
@@ -140,4 +231,126 @@ describe GoodData::Project do
       end
     end
   end
+
+  describe '#users_create' do
+    it 'Creates new users' do
+      project = ProjectHelper.get_default_project
+
+      users = (0...10).map do |i|
+        num = rand(1e6)
+        login = "gemtest#{num}@gooddata.com"
+
+        json = {
+          'user' => {
+            'content' => {
+              'email' => login,
+              'login' => login,
+              'firstname' => 'the',
+              'lastname' => num.to_s,
+
+              # Following lines are ugly hack
+              'role' => 'admin',
+              'password' => 'password',
+              'domain' => ConnectionHelper::DEFAULT_DOMAIN,
+
+              # And following lines are even much more ugly hack
+              # 'sso_provider' => '',
+              # 'authentication_modes' => ['sso', 'password']
+            },
+            'meta' => {}
+          }
+        }
+
+        GoodData::Membership.new(json)
+      end
+
+      res = GoodData::Domain.users_create(users)
+
+      project.users_create(users)
+
+      expect(res).to be_an_instance_of(Array)
+      res.each do |r|
+        expect(r).to be_an_instance_of(GoodData::Profile)
+        r.delete
+      end
+    end
+  end
+
+  describe '#users_import' do
+    it 'Import users from CSV' do
+
+      project = GoodData::Project[ProjectHelper::PROJECT_ID]
+
+      new_users = load_users_from_csv
+
+      project.users_import(new_users)
+    end
+  end
+
+  describe '#set_user_roles' do
+    it 'Properly updates user roles as needed' do
+      project = ProjectHelper.get_default_project
+
+      project.set_user_roles(ConnectionHelper::DEFAULT_USERNAME, 'admin')
+    end
+  end
+
+  describe '#set_users_roles' do
+    it 'Properly updates user roles as needed for bunch of users' do
+      project = ProjectHelper.get_default_project
+
+      list = load_users_from_csv
+
+      # Create domain users
+      domain_users = GoodData::Domain.users_create(list, ConnectionHelper::DEFAULT_DOMAIN)
+      expect(domain_users.length).to equal(list.length)
+
+      # Create list with user, desired_roles hashes
+      domain_users.each_with_index do |user, index|
+        list[index] = {
+          :user => user,
+          :roles => list[index].json['user']['content']['role'].split(' ').map { |r| r.downcase }.sort
+        }
+      end
+
+      res = project.set_users_roles(list)
+      expect(res.length).to equal(list.length)
+      res.each do |update_result|
+        expect(update_result[:result]['projectUsersUpdateResult']['successful'][0]).to include(update_result[:user].uri)
+      end
+
+      domain_users.each do |user|
+         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
+
+      list = [
+        {
+          :user => ConnectionHelper::DEFAULT_USERNAME,
+          :roles => ['admin']
+        }
+      ]
+
+      res = project.set_users_roles(list)
+      expect(res.length).to equal(list.length)
+    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
+
+      list = [
+        {
+          :user => ConnectionHelper::DEFAULT_USERNAME,
+          :roles => 'admin'
+        }
+      ]
+
+      res = project.set_users_roles(list)
+      expect(res.length).to equal(list.length)
+    end
+
+  end
 end

data/spec/unit/models/schedule_spec.rb

@@ -9,6 +9,13 @@ describe GoodData::Schedule, :broken => true do
     :cron => '2 2 2 2 *'
   }
 
+  TEST_DATA_WITH_OPTIONAL_PARAM = {
+      :timezone => 'UTC',
+      :cron => '2 2 2 2 *',
+      :reschedule => 15
+  }
+
+
   TEST_PROCESS_ID = 'f12975d2-5958-4248-9c3d-4c8f2e1f067d'
 
   SCHEDULE_ID = '53642303e4b0ae3190e464a8'
@@ -72,6 +79,16 @@ describe GoodData::Schedule, :broken => true do
       sched.delete
     end
 
+    it 'Creates new schedule if mandatory params passed and optional params are present' do
+      sched = nil
+      expect {
+        sched = GoodData::Schedule.create(TEST_PROCESS_ID, TEST_CRON, @project_executable, TEST_DATA_WITH_OPTIONAL_PARAM)
+      }.not_to raise_error
+
+      sched.should_not be_nil
+      sched.delete
+    end
+
     it 'Throws exception when no process ID specified' do
       expect {
         sched = GoodData::Schedule.create(nil, TEST_CRON, @project_executable, TEST_DATA)
@@ -489,4 +506,45 @@ describe GoodData::Schedule, :broken => true do
       expect(@schedule.dirty).to eq(true)
     end
   end
+
+  describe '#reschedule' do
+    before(:each) do
+      @schedule = GoodData::Schedule.create(TEST_PROCESS_ID, TEST_CRON, @project_executable, TEST_DATA_WITH_OPTIONAL_PARAM)
+    end
+
+    after(:each) do
+      @schedule.delete
+    end
+
+    it 'Should return reschedule as integer' do
+      res = @schedule.reschedule
+      res.should_not be_nil
+      res.should_not be_empty
+      res.should be_a_kind_of(Integer)
+    end
+
+
+
+  end
+
+  describe '#reschedule=' do
+    before(:each) do
+      @schedule = GoodData::Schedule.create(TEST_PROCESS_ID, TEST_CRON, @project_executable, TEST_DATA_WITH_OPTIONAL_PARAM)
+    end
+
+    after(:each) do
+      @schedule.delete
+    end
+
+    it 'Assigns the reschedule and marks the object dirty' do
+      test_reschedule = 45
+
+      @schedule.reschedule = test_reschedule
+      expect(@schedule.reschedule).to eq(test_reschedule)
+      expect(@schedule.dirty).to eq(true)
+    end
+  end
+
+
+
 end