gooddata 0.6.3 → 0.6.4

data/spec/spec_helper.rb

@@ -18,12 +18,21 @@ RSpec.configure do |config|
   config.include BlueprintHelper
   config.include CliHelper
   config.include ConnectionHelper
+  config.include CsvHelper
   config.include ProjectHelper
   config.include SchemaHelper
 
   config.filter_run_excluding :broken => true
 
   config.before(:all) do
+    # TODO: Move this to some method.
+    # TODO Make more intelligent so two test suites can run at the same time.
+    # ConnectionHelper.create_default_connection
+    # users = GoodData::Domain.users_map(ConnectionHelper::DEFAULT_DOMAIN)
+    # users.each do |user|
+    #   user.delete if user.email != ConnectionHelper::DEFAULT_USERNAME
+    # end
+
     # TODO: Fully setup global environment
     GoodData.logging_off
   end

data/spec/unit/commands/command_user_spec.rb

@@ -9,7 +9,7 @@ describe GoodData::Command::User do
     GoodData.disconnect
   end
 
-  it "Is Possible to create GoodData::Command::User instance" do
+  it "Is Possible to create GoodData::Command::Membership instance" do
     cmd = GoodData::Command::User.new()
     cmd.should be_a(GoodData::Command::User)
   end

data/spec/unit/extensions/hash_spec.rb

@@ -0,0 +1,19 @@
+# encoding: UTF-8
+
+describe Hash do
+
+  describe '#deep_dup' do
+    it 'should crete a deep copy' do
+      x = {
+        :a => {
+          :b => :c
+        }
+      }
+      y = x.dup
+      deep_y = x.deep_dup
+
+      y[:a].object_id.should === x[:a].object_id
+      deep_y[:a].object_id.should_not === x[:a].object_id
+    end
+  end
+end

data/spec/unit/godzilla/goodzilla_spec.rb

@@ -56,4 +56,19 @@ describe GoodData::SmallGoodZilla do
     interpolated.should == "SELECT SUM([a]) WHERE [d]=[x] AND [snap]=1"
   end
 
+  it "should be able to parse several ids correctly." do
+    metric = "SELECT SUM(
+        CASE
+          WHEN ![attr.customer_profiles.profile_id] = [/gdc/md/pxa3aic06undadkc5s7t5lxpne11vgyt/obj/257/elements?id=65] THEN ![fact.perfomance_by_campaign_id.general_score_1]
+          WHEN ![attr.customer_profiles.profile_id] = [/gdc/md/pxa3aic06undadkc5s7t5lxpne11vgyt/obj/257/elements?id=50] THEN ![fact.perfomance_by_campaign_id.general_score_5]
+          WHEN ![attr.customer_profiles.profile_id] = [/gdc/md/pxa3aic06undadkc5s7t5lxpne11vgyt/obj/257/elements?id=20]  THEN ![fact.perfomance_by_campaign_id.general_score_12]
+        END)"
+    GoodData::SmallGoodZilla.get_ids(metric).should == [
+      "attr.customer_profiles.profile_id",
+      "fact.perfomance_by_campaign_id.general_score_1",
+      "fact.perfomance_by_campaign_id.general_score_5",
+      "fact.perfomance_by_campaign_id.general_score_12"
+    ]
+  end
+
 end

data/spec/unit/helpers/csv_helper_spec.rb

@@ -0,0 +1,18 @@
+# encoding: UTF-8
+
+require 'gooddata/helpers/csv_helper'
+
+describe GoodData::Helpers::Csv do
+  describe '#read' do
+    it 'Reads data from CSV file' do
+      data = GoodData::Helpers::Csv.read(:path => CsvHelper::CSV_PATH_IMPORT)
+    end
+  end
+
+  describe '#write' do
+    it 'Writes data to CSV' do
+      data = []
+      GoodData::Helpers::Csv.write(:path => CsvHelper::CSV_PATH_EXPORT, :data => data)
+    end
+  end
+end

data/spec/unit/models/domain_spec.rb

@@ -13,7 +13,9 @@ describe GoodData::Domain do
 
   describe '#add_user' do
     it 'Should add user' do
-      GoodData::Domain.add_user(ConnectionHelper::DEFAULT_DOMAIN, "gemtest#{rand(1e6)}@gooddata.com", 'password')
+      user = GoodData::Domain.add_user(:domain => ConnectionHelper::DEFAULT_DOMAIN, :login => "gemtest#{rand(1e6)}@gooddata.com", :password => 'password')
+      expect(user).to be_an_instance_of(GoodData::Profile)
+      user.delete
     end
   end
 
@@ -22,7 +24,7 @@ describe GoodData::Domain do
       users = GoodData::Domain.users(ConnectionHelper::DEFAULT_DOMAIN)
       expect(users).to be_instance_of(Array)
       users.each do |user|
-        expect(user).to be_an_instance_of(GoodData::AccountSettings)
+        expect(user).to be_an_instance_of(GoodData::Profile)
       end
     end
 
@@ -30,7 +32,7 @@ describe GoodData::Domain do
       users = GoodData::Domain.users(ConnectionHelper::DEFAULT_DOMAIN, {:limit =>1})
       expect(users).to be_instance_of(Array)
       users.each do |user|
-        expect(user).to be_an_instance_of(GoodData::AccountSettings)
+        expect(user).to be_an_instance_of(GoodData::Profile)
       end
     end
 
@@ -38,7 +40,48 @@ describe GoodData::Domain do
       users = GoodData::Domain.users(ConnectionHelper::DEFAULT_DOMAIN, {:offset => 1})
       expect(users).to be_instance_of(Array)
       users.each do |user|
-        expect(user).to be_an_instance_of(GoodData::AccountSettings)
+        expect(user).to be_an_instance_of(GoodData::Profile)
+      end
+    end
+  end
+
+  describe '#users_create' do
+    it 'Creates new users from list' do
+      list = []
+      (0...10).each 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' => {}
+          }
+        }
+        user = GoodData::Membership.new(json)
+        list << user
+      end
+
+      res = GoodData::Domain.users_create(list, ConnectionHelper::DEFAULT_DOMAIN)
+
+      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

data/spec/unit/models/md_object_spec.rb

@@ -44,4 +44,12 @@ describe GoodData::Model::MdObject do
       result.should == be_an_instance_of(String)
     end
   end
+
+  describe '#all' do
+    it 'Throws an error. This is implemented on subclasses' do
+      expect do
+        GoodData::MdObject.all
+      end.to raise_exception(NotImplementedError)
+    end
+  end
 end

data/spec/unit/models/membership_spec.rb

@@ -0,0 +1,128 @@
+# encoding: UTF-8
+
+require 'gooddata/models/domain'
+require 'gooddata/models/membership'
+require 'gooddata/models/project_role'
+
+describe GoodData::Membership do
+  before(:all) do
+    ConnectionHelper.create_default_connection
+
+    @users = [
+      GoodData::Membership.new(
+        {
+          'user' => {
+            'content' => {
+              'email' => 'petr.cvengros@gooddata.com',
+              'firstname' => 'Petr',
+              'lastname' => 'Cvengros'
+            },
+            'meta' => {}
+          }
+        }
+      ),
+
+      GoodData::Membership.new(
+        {
+          'user' => {
+            'content' => {
+              'email' => 'tomas.korcak@gooddata.com',
+              'firstname' => 'Tomas',
+              'lastname' => 'Korcak'
+            },
+            'meta' => {}
+          }
+        }
+      ),
+
+      GoodData::Membership.new(
+        {
+          'user' => {
+            'content' => {
+              'email' => 'patrick.mcconlogue@gooddata.com',
+              'firstname' => 'Patrick',
+              'lastname' => 'McConlogue'
+            },
+            'meta' => {}
+          }
+        }
+      ),
+
+      GoodData::Membership.new(
+        {
+          'user' => {
+            'content' => {
+              'email' => 'tomas.svarovsky@gooddata.com',
+              'firstname' => 'Tomas',
+              'lastname' => 'Svarovsky'
+            },
+            'meta' => {}
+          }
+        }
+      ),
+    ]
+  end
+
+  after(:all) do
+    GoodData.disconnect
+  end
+
+  describe '#diff_list' do
+    it 'Returns empty diff for same arrays' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = [
+        @users[0]
+      ]
+
+      diff = GoodData::Membership.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes added element' do
+      l1 = []
+
+      l2 = [
+        @users[0]
+      ]
+
+      diff = GoodData::Membership.diff_list(l1, l2)
+      diff[:added].length.should eql(1)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes changed element' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = [
+        GoodData::Membership.new(@users[0].json.deep_dup)
+      ]
+      l2[0].first_name = 'Peter'
+
+      diff = GoodData::Membership.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(1)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes removed element' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = []
+
+      diff = GoodData::Membership.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(1)
+    end
+  end
+end

data/spec/unit/models/metadata_spec.rb

@@ -0,0 +1,38 @@
+# encoding: UTF-8
+
+require 'gooddata'
+
+RAW_DATA = {'metric'=>
+  {'content'=>
+    {'format'=>'#,##0',
+     'expression'=> 'SELECT SUM([/gdc/md/ksjy0nr3goz6k8yrpklz97l0mych7nez/obj/700])'},
+   'meta' =>
+    {'author'=>'/gdc/account/profile/4e1e8cacc4989228e0ae531b30853248',
+     'uri'=>'/gdc/md/ksjy0nr3goz6k8yrpklz97l0mych7nez/obj/70',
+     'tags'=>'a b cg r t',
+     'created'=>'2014-04-30 22:47:57',
+     'identifier'=>'afo7bx1VakCz',
+     'deprecated'=>'0',
+     'summary'=>'',
+     'title'=>'sum of Lines changed',
+     'category'=>'metric',
+     'updated'=>'2014-05-05 20:00:42',
+     'contributor'=>'/gdc/account/profile/4e1e8cacc4989228e0ae531b30853248'}}}
+
+
+describe GoodData::MdObject do
+  before(:each) do
+    @instance = GoodData::MdObject.new(RAW_DATA)
+  end
+
+  describe '#identifier=' do
+    it 'Allows setting a new identifier' do
+      identifier = @instance.identifier
+      @instance.identifier = 'new_id'
+      new_identifier = @instance.identifier
+
+      new_identifier.should_not == identifier
+      new_identifier.should == 'new_id'
+    end
+  end
+end

data/spec/unit/models/profile_spec.rb

@@ -0,0 +1,212 @@
+# encoding: UTF-8
+
+
+require 'gooddata/models/profile'
+require 'gooddata/models/domain'
+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)
+
+    @users = [
+      GoodData::Profile.new(
+        {
+          'accountSetting' => {
+            'email' => 'petr.cvengros@gooddata.com',
+            'firstName' => 'Petr',
+            'lastName' => 'Cvengros'
+          }
+        }
+      ),
+
+      GoodData::Profile.new(
+        {
+          'accountSetting' => {
+            'email' => 'tomas.korcak@gooddata.com',
+            'firstName' => 'Tomas',
+            'lastName' => 'Korcak'
+          }
+        }
+      ),
+
+      GoodData::Profile.new(
+        {
+          'accountSetting' => {
+            'email' => 'patrick.mcconlogue@gooddata.com',
+            'firstName' => 'Patrick',
+            'lastName' => 'McConlogue'
+
+          }
+        }
+      ),
+
+      GoodData::Profile.new(
+        {
+          'accountSetting' => {
+            'email' => 'tomas.svarovsky@gooddata.com',
+            'firstName' => 'Tomas',
+            'lastName' => 'Svarovsky'
+          }
+        }
+      ),
+    ]
+  end
+
+  after(:all) do
+    GoodData.disconnect
+  end
+
+  describe '#==' do
+    it 'Returns true for same objects' do
+      user1 = GoodData.user.dup
+      user2 = GoodData.user.dup
+      res = user1 == user2
+      res.should be_true
+    end
+
+    it 'Returns false for different objects' do
+      user1 = GoodData.user.dup
+      user2 = GoodData.user.dup
+
+      # Do some little modification
+      user2.first_name = 'kokos'
+
+      res = user1 == user2
+      res.should be_false
+    end
+  end
+
+  describe '#!=' do
+    it 'Returns false for same objects' do
+      user1 = GoodData.user.dup
+      user2 = GoodData.user.dup
+      res = user1 != user2
+      res.should be_false
+    end
+
+    it 'Returns true for different objects' do
+      user1 = GoodData.user.dup
+      user2 = GoodData.user.dup
+
+      # Do some little modification
+      user2.first_name = 'kokos'
+
+      res = user1 != user2
+      res.should be_true
+    end
+  end
+
+  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
+
+      # Do some little modification
+      user2.first_name = 'kokos'
+
+      diff = user1.diff(user2)
+
+      expect(diff).to be_instance_of(Hash)
+      diff.length.should_not eql(0)
+
+      user1.apply(diff)
+
+      res = user1 == user2
+      res.should be_true
+    end
+  end
+
+  describe '#diff' do
+    it 'Returns empty hash for same objects' do
+      user1 = GoodData.user.dup
+      user2 = GoodData.user.dup
+      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
+
+      # Do some little modification
+      user2.first_name = 'kokos'
+
+      res = user1.diff(user2)
+      expect(res).to be_instance_of(Hash)
+      res.length.should_not eql(0)
+    end
+  end
+
+  describe '#diff_list' do
+    it 'Returns empty diff for same arrays' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = [
+        @users[0]
+      ]
+
+      diff = GoodData::Profile.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes added element' do
+      l1 = []
+
+      l2 = [
+        @users[0]
+      ]
+
+      diff = GoodData::Profile.diff_list(l1, l2)
+      diff[:added].length.should eql(1)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes changed element' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = [
+        GoodData::Profile.new(@users[0].json.deep_dup)
+      ]
+      l2[0].first_name = 'Peter'
+
+      diff = GoodData::Profile.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(1)
+      diff[:removed].length.should eql(0)
+    end
+
+    it 'Recognizes removed element' do
+      l1 = [
+        @users[0]
+      ]
+
+      l2 = []
+
+      diff = GoodData::Profile.diff_list(l1, l2)
+      diff[:added].length.should eql(0)
+      diff[:changed].length.should eql(0)
+      diff[:removed].length.should eql(1)
+    end
+  end
+
+  describe '#projects' do
+    it 'Returns user projects as array of GoodData::Project' do
+      projects = @user.projects
+      expect(projects).to be_an_instance_of(Array)
+
+      projects.each do |project|
+        expect(project).to be_an_instance_of(GoodData::Project)
+      end
+    end
+  end
+end