gooddata 0.6.11 → 0.6.12

data/spec/integration/rest_spec.rb

@@ -0,0 +1,209 @@
+# encoding: UTF-8
+require 'tempfile'
+
+require 'gooddata/core/rest'
+
+describe GoodData do
+  before(:each) do
+    @client = ConnectionHelper.create_default_connection
+    # @project = ProjectHelper.get_default_project(:client => @client)
+    @project = @client.create_project(title: 'Project for schedule testing', auth_token: ConnectionHelper::GD_PROJECT_TOKEN)
+  end
+
+  after(:each) do
+    @project && @project.delete
+    @client.disconnect
+  end
+
+  describe '#get_project_webdav_path' do
+    it 'Returns path' do
+      @client.get_project_webdav_path('test-file.csv', :project => @project)
+    end
+  end
+
+  def test_webdav_upload(params)
+    GoodData.with_project(@project, :client => @client) do
+
+      # use current timestamp as a directory name on webdav
+      dir = params[:no_dir] ? nil : Time.now.to_i.to_s
+      dir = "#{dir}/#{dir}" if params[:nested_dir]
+      dir = "#{dir}/" if params[:slash_in_dir]
+
+      # local file path
+      path = 'spec/data/test-ci-data.csv'
+
+      if params[:special_chars]
+        source_file = Tempfile.new('abc-16:55:29+ha#he.csv')
+        FileUtils.cp(path, source_file)
+        path = source_file.path
+      end
+
+      path = File.expand_path(path) if params[:absolute_path]
+
+      # upload it there
+      upload_method = GoodData.method(params[:upload_method])
+      upload_method.call(path, directory: dir)
+
+      # download it from there
+      temp_file = Tempfile.new('foo.csv')
+      expect(temp_file.size).to be == 0
+
+      download_method = GoodData.method(params[:download_method])
+
+      file_basename = File.basename(path)
+      file_basename = "NOTTHERE_#{file_basename}" if params[:unknown_file]
+
+      download_block = proc do
+        if params[:path_in_file]
+          # pass the dir directly in the first param
+          # e.g. GoodData.download_from_project_webdav('1234/test-ci-data.csv', '/tmp/myfile.csv')
+          download_method.call(File.join(dir, file_basename).to_s, temp_file)
+        else
+          # pass the dir in the :directory option
+          # e.g. GoodData.download_from_project_webdav('test-ci-data.csv', '/tmp/myfile.csv', :directory => '1234')
+          download_method.call(file_basename, temp_file, directory: dir)
+        end
+      end
+
+      # if it's unknown it should raise an error, otherwise it should download the right stuff
+      if params[:unknown_file]
+        expect{ download_block.call }.to raise_error(ArgumentError)
+      else
+        download_block.call
+
+        expect(temp_file.size).to be > 0
+
+        # expect the contents of the original file and the downloaded file are the same
+        expect(IO.read(temp_file)).to be == IO.read(path)
+      end
+    end
+  end
+
+  describe '#download_from_project_webdav' do
+    it "Works with directory directly in the 'file' param for project" do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav',
+        path_in_file: true
+      )
+    end
+    it "Works with directory directly in the 'file' param for user" do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav',
+        path_in_file: true
+      )
+    end
+    it "Works with directories with slash at the end for project" do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav',
+        slash_in_dir: true
+      )
+    end
+    it "Works with directories with slash at the end for user" do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav',
+        slash_in_dir: true
+      )
+    end
+    it "Fails for non-existing file for project" do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav',
+        unknown_file: true
+      )
+    end
+    it "Fails for non-existing file for user" do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav',
+        unknown_file: true
+      )
+    end
+    it 'Works with nested directories for project' do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav',
+        nested_dir: true
+      )
+    end
+    it 'Works with nested directories for user' do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav',
+        nested_dir: true
+      )
+    end
+    it 'Works with no directory for user' do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav',
+        no_dir: true
+      )
+    end
+    it 'Works with no directory for project' do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav',
+        no_dir: true
+      )
+    end
+  end
+  describe '#upload_to_project_webdav' do
+    it 'Uploads file with relative path to a dir' do
+      test_webdav_upload(
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav'
+      )
+    end
+
+    it 'Uploads file with absolute path to a dir' do
+      test_webdav_upload(
+        absolute_path: true,
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav'
+      )
+    end
+
+    it 'Uploads file with special chars in filename' do
+      test_webdav_upload(
+        special_chars: true,
+        upload_method: 'upload_to_project_webdav',
+        download_method: 'download_from_project_webdav'
+      )
+    end
+  end
+
+  describe '#upload_to_user_webdav' do
+    it 'Uploads file with relative path to a dir' do
+      test_webdav_upload(
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav'
+      )
+    end
+
+    it 'Uploads file with absolute path to a dir' do
+      test_webdav_upload(
+        absolute_path: true,
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav'
+      )
+    end
+    it 'Uploads file with special chars in filename' do
+      test_webdav_upload(
+        special_chars: true,
+        upload_method: 'upload_to_user_webdav',
+        download_method: 'download_from_user_webdav'
+      )
+    end
+  end
+
+
+  describe '#get_user_webdav_path' do
+    it 'Gets the path' do
+      @client.get_user_webdav_path('test.csv', :project => @project)
+    end
+  end
+end

data/spec/integration/user_filters_spec.rb

@@ -0,0 +1,193 @@
+require 'gooddata'
+
+describe "User filters implementation", :constraint => 'slow' do
+  before(:all) do
+    @spec = JSON.parse(File.read("./spec/data/test_project_model_spec.json"), :symbolize_names => true)
+    @client = ConnectionHelper::create_default_connection
+    @project = @client.create_project_from_blueprint(@spec, :auth_token => ConnectionHelper::GD_PROJECT_TOKEN)
+    @domain = @client.domain(ConnectionHelper::DEFAULT_DOMAIN)
+
+    GoodData.with_project(@project) do |p|
+      @label = GoodData::Attribute.find_first_by_title('Dev', client: @client, project: @project).label_by_name('email')
+      
+      blueprint = GoodData::Model::ProjectBlueprint.new(@spec)
+      commits_data = [
+        ["lines_changed","committed_on","dev_id","repo_id"],
+        [1,"01/01/2014",1,1],
+        [3,"01/02/2014",2,2],
+        [5,"05/02/2014",3,1]]
+      GoodData::Model.upload_data(commits_data, blueprint, 'commits', :client => @client, :project => @project)
+      # blueprint.find_dataset('commits').upload(commits_data)
+
+      devs_data = [
+        ["dev_id", "email"],
+        [1, "tomas@gooddata.com"],
+        [2, "petr@gooddata.com"],
+        [3, "jirka@gooddata.com"]]
+      GoodData::Model.upload_data(devs_data, blueprint, 'devs', :client => @client, :project => @project)
+      # blueprint.find_dataset('devs').upload(devs_data)
+    end
+  end
+
+  after(:all) do
+    @project.delete if @project
+  end
+
+  after(:each) do
+    @project.data_permissions.pmap &:delete
+  end
+
+  it "should create a mandatory user filter" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com', 'jirka@gooddata.com']]
+
+    metric = @project.create_metric("SELECT SUM(#\"Lines Changed\")", :title => 'x')
+    # [jirka@gooddata.com | petr@gooddata.com | tomas@gooddata.com]
+    # [5.0                | 3.0               | 1.0               ]
+
+    metric.execute.should == 9
+    @project.add_data_permissions(filters)
+    metric.execute.should == 6
+    r = @project.compute_report(left: [metric], top: [@label.attribute])
+    r.include_column?(['tomas@gooddata.com', 1]).should == true
+
+    r.include_column?(['tomas@gooddata.com', 1]).should == true
+    r.include_column?(['jirka@gooddata.com', 5]).should == true
+    r.include_column?(['petr@gooddata.com', 3]).should == false
+  end
+
+  it "should fail when asked to set a user not in project. No filters should be set up." do
+    filters = [
+      ['nonexistent_user@gooddata.com', @label.uri, "tomas@gooddata.com"],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]
+    ]
+    expect do
+      @project.add_data_permissions(filters)
+    end.to raise_error
+    expect(@project.data_permissions.count).to eq 0
+  end
+
+  it "should pass and set users that are in the projects" do
+    filters = [
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_data_permissions(filters)
+    expect(@project.data_permissions.count).to eq 1
+  end
+
+  it "should pass and set only users that are in the projects if asked" do
+    filters = [
+      ['nonexistent_user@gooddata.com', @label.uri, 'tomas@gooddata.com'],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com']
+    ]
+    @project.add_data_permissions(filters, users_must_exist: false)
+    expect(@project.data_permissions.count).to eq 1
+    expect(@project.data_permissions.first.pretty_expression).to eq "[Dev] IN ([tomas@gooddata.com])"
+  end
+
+  it "should fail when asked to set a value not in the proejct" do
+    filters = [
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, '%^&*( nonexistent value', 'tomas@gooddata.com'],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com']]
+    expect do
+      @project.add_data_permissions(filters)
+    end.to raise_error
+    expect(@project.data_permissions.count).to eq 0
+  end
+
+  it "should add a filter with nonexistent values when asked" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, '%^&*( nonexistent value', 'jirka@gooddata.com']]
+    @project.add_data_permissions(filters, ignore_missing_values: true)
+
+    expect(@project.data_permissions.pmap {|m| m.pretty_expression}).to eq ["[Dev] IN ([jirka@gooddata.com])"]
+    expect(@project.data_permissions.count).to eq 1
+  end
+
+  it "should be able to add mandatory filter to a user not in the project if domain is provided" do
+    domain = @client.domain(ConnectionHelper::DEFAULT_DOMAIN)
+    u = domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+
+    filters = [[u.login, @label.uri, "tomas@gooddata.com"]]
+    expect do
+      @project.add_data_permissions(filters)
+    end.to raise_error
+    @project.add_data_permissions(filters, :domain => domain)
+    filters = @project.data_permissions
+    expect(filters.first.related.login).to eq u.login
+    expect(filters.count).to eq 1
+    expect(filters.first.pretty_expression).to eq "[Dev] IN ([tomas@gooddata.com])"
+  end
+
+  it "should be able to print data permissions in a human readable form" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]]
+    @project.add_data_permissions(filters)
+    perms = @project.data_permissions
+    pretty = perms.pmap {|f| [f.related.login, f.pretty_expression]}
+    expect(perms.first.related).to eq @client.user
+    expect(pretty).to eq [["svarovsky+gem_tester@gooddata.com", "[Dev] IN ([tomas@gooddata.com])"]]
+  end
+
+  it "sets up mandatory users based on the state given as an end state by default." do
+    # first let's prepare some user filters
+    user_with_already_set_up_filter = @project.get_user(ConnectionHelper::DEFAULT_USERNAME)
+
+    filters = [
+      [user_with_already_set_up_filter.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_data_permissions(filters)
+    expect(@project.data_permissions.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"]]
+
+    # Now let's add user filter to a different user. If we do not explicitely state that
+    # user_with_already_set_up_filter should keep his filter it will be removed
+    another_user = @domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+    @project.add_user(another_user, 'Admin', domain: @domain)
+    new_filters = [
+      [another_user.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_data_permissions(new_filters)
+    expect(@project.data_permissions.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[another_user.login, "[Dev] IN ([tomas@gooddata.com])"]]
+  end
+
+  it "should set up false if all values are nonexistent" do
+    metric = GoodData::Fact.find_first_by_title('Lines Changed', client: @client, project: @project).create_metric
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "NONEXISTENT1", "NONEXISTENT2", "NONEXISTENT3"]]
+    @project.add_data_permissions(filters, ignore_missing_values: true)
+    expect(metric.execute).to eq 9
+    @project.add_data_permissions(filters, ignore_missing_values: true, restrict_if_missing_all_values: true)
+    expect(metric.execute).to eq nil
+  end
+
+  it "you can switch the updates. Whatever is not mentioned will not be touched" do
+    # first let's prepare some user filters
+    user_with_already_set_up_filter = @project.get_user(ConnectionHelper::DEFAULT_USERNAME)
+
+    filters = [
+      [user_with_already_set_up_filter.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_data_permissions(filters)
+    expect(@project.data_permissions.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"]]
+
+    # Now let's add user filter to a different user. If we do not explicitely state that
+    # user_with_already_set_up_filter should keep his filter it will be removed
+    another_user = @domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+    @project.add_user(another_user, 'Admin', domain: @domain)
+    new_filters = [
+      [another_user.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_data_permissions(new_filters, do_not_touch_filters_that_are_not_mentioned: true)
+    expect(@project.data_permissions.map {|f| [f.related.login, f.pretty_expression] })
+      .to include([ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"], [another_user.login, "[Dev] IN ([tomas@gooddata.com])"])
+  end
+
+  it "should be able to update the filter value" do
+    pending('We cannot swap filters yet')
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com", "jirka@gooddata.com"]]
+    @project.add_data_permissions(filters)
+    perm = @project.data_permissions.first
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]]
+    @project.add_data_permissions(filters)
+    expect(perm.reload!.pretty_expression).to eq '[Dev] IN ([tomas@gooddata.com, jirka@gooddata.com])'
+  end
+end

data/spec/integration/variables_spec.rb

@@ -0,0 +1,196 @@
+require 'gooddata'
+
+describe "Variables implementation", :constraint => 'slow' do
+  before(:all) do
+    @spec = JSON.parse(File.read("./spec/data/test_project_model_spec.json"), :symbolize_names => true)
+    @client = ConnectionHelper::create_default_connection
+    @project = @client.create_project_from_blueprint(@spec, :auth_token => ConnectionHelper::GD_PROJECT_TOKEN)
+    @domain = @client.domain(ConnectionHelper::DEFAULT_DOMAIN)
+
+    @label = GoodData::Attribute.find_first_by_title('Dev', client: @client, project: @project).label_by_name('email')
+    
+    blueprint = GoodData::Model::ProjectBlueprint.new(@spec)
+    commits_data = [
+      ["lines_changed","committed_on","dev_id","repo_id"],
+      [1,"01/01/2014",1,1],
+      [3,"01/02/2014",2,2],
+      [5,"05/02/2014",3,1]]
+    GoodData::Model.upload_data(commits_data, blueprint, 'commits', :client => @client, :project => @project)
+    # blueprint.find_dataset('commits').upload(commits_data)
+    
+    devs_data = [
+      ["dev_id", "email"],
+      [1, "tomas@gooddata.com"],
+      [2, "petr@gooddata.com"],
+      [3, "jirka@gooddata.com"]]
+    GoodData::Model.upload_data(devs_data, blueprint, 'devs', :client => @client, :project => @project)
+    # blueprint.find_dataset('devs').upload(devs_data)
+    
+    @variable = @project.create_variable(title: 'uaaa', attribute: @label.attribute).save
+
+  end
+
+  after(:all) do
+    @project.delete if @project
+  end
+
+  after(:each) do
+    @variable.user_values.select {|x| x.level == :user}.pmap &:delete
+  end
+
+  it "should create a variable filter" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com', 'jirka@gooddata.com']]
+
+    metric = @project.create_metric("SELECT SUM(#\"Lines Changed\")", :title => 'x')
+
+    # [jirka@gooddata.com | petr@gooddata.com | tomas@gooddata.com]
+    # [5.0                | 3.0               | 1.0               ]
+    @project.add_variable_permissions(filters, @variable)
+    r = @project.compute_report(left: [metric], top: [@label.attribute], filters: [@variable])
+    r.include_column?(['tomas@gooddata.com', 1]).should == true
+    r.include_column?(['jirka@gooddata.com', 5]).should == true
+    r.include_column?(['petr@gooddata.com', 3]).should == false
+
+    r = @project.compute_report(left: [metric], top: [@label.attribute])
+    r.include_column?(['tomas@gooddata.com', 1]).should == true
+    r.include_column?(['jirka@gooddata.com', 5]).should == true
+    r.include_column?(['petr@gooddata.com', 3]).should == true
+  end
+
+  it "should fail when asked to set a user not in project. No filters should be set up." do
+    filters = [
+      ['nonexistent_user@gooddata.com', @label.uri, "tomas@gooddata.com"],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]
+    ]
+    expect do
+      @project.add_variable_permissions(filters, @variable)
+    end.to raise_error
+    expect(@variable.user_values.count).to eq 0
+  end
+
+  it "should pass and set users that are in the projects" do
+    filters = [
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_variable_permissions(filters, @variable)
+    expect(@variable.user_values.count).to eq 1
+  end
+  
+  it "should pass and set only users that are in the projects if asked" do
+    filters = [
+      ['nonexistent_user@gooddata.com', @label.uri, 'tomas@gooddata.com'],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com']
+    ]
+    @project.add_variable_permissions(filters, @variable, users_must_exist: false)
+    expect(@variable.user_values.count).to eq 1
+  end
+  
+  it "should fail when asked to set a value not in the proejct" do
+    filters = [
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, '%^&*( nonexistent value', 'tomas@gooddata.com'],
+      [ConnectionHelper::DEFAULT_USERNAME, @label.uri, 'tomas@gooddata.com']]
+    expect do
+      @project.add_variable_permissions(filters, @variable)
+    end.to raise_error
+    expect(@variable.user_values.count).to eq 0
+  end
+  
+  it "should add a filter with nonexistent values when asked" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, '%^&*( nonexistent value', 'jirka@gooddata.com']]
+    @project.add_variable_permissions(filters, @variable, ignore_missing_values: true)
+  
+    expect(@variable.user_values.pmap {|m| m.pretty_expression}).to eq ["[Dev] IN ([jirka@gooddata.com])"]
+    expect(@variable.user_values.count).to eq 1
+  end
+  
+  it "should be able to add mandatory filter to a user not in the project if domain is provided" do
+    domain = @client.domain(ConnectionHelper::DEFAULT_DOMAIN)
+    u = domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+    filters = [[u.login, @label.uri, "tomas@gooddata.com"]]
+    expect do
+      @project.add_variable_permissions(filters, @variable)
+    end.to raise_error
+    @project.add_variable_permissions(filters, @variable, :domain => domain)
+    filters = @variable.user_values
+    expect(filters.first.related.login).to eq u.login
+    expect(filters.count).to eq 1
+  end
+  
+  it "should be able to print data permissions in a human readable form" do
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]]
+    @project.add_variable_permissions(filters, @variable)
+    perms = @variable.user_values
+    pretty = perms.pmap {|f| [f.related.login, f.pretty_expression]}
+    expect(perms.first.related).to eq @client.user
+    expect(pretty).to eq [["svarovsky+gem_tester@gooddata.com", "[Dev] IN ([tomas@gooddata.com])"]]
+  end
+
+  it "sets up mandatory users based on the state given as an end state by default." do
+    # first let's prepare some user filters
+    user_with_already_set_up_filter = @project.get_user(ConnectionHelper::DEFAULT_USERNAME)
+
+    filters = [
+      [user_with_already_set_up_filter.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_variable_permissions(filters, @variable)
+    expect(@variable.user_values.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"]]
+
+    # Now let's add user filter to a different user. If we do not explicitely state that
+    # user_with_already_set_up_filter should keep his filter it will be removed
+    another_user = @domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+    @project.add_user(another_user, 'Admin', domain: @domain)
+    new_filters = [
+      [another_user.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_variable_permissions(new_filters, @variable)
+    expect(@variable.user_values.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[another_user.login, "[Dev] IN ([tomas@gooddata.com])"]]
+  end
+
+  it "should set up false if all values are nonexistent" do
+    metric = GoodData::Fact.find_first_by_title('Lines Changed', client: @client, project: @project).create_metric
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "NONEXISTENT1", "NONEXISTENT2", "NONEXISTENT3"]]
+    @project.add_variable_permissions(filters, @variable, ignore_missing_values: true)
+    # expect(metric.execute).to eq 9
+    expect(@variable.user_values.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[ConnectionHelper::DEFAULT_USERNAME, "TRUE"]]
+
+    @project.add_variable_permissions(filters, @variable, ignore_missing_values: true, restrict_if_missing_all_values: true)
+    # expect(metric.execute).to eq nil
+    expect(@variable.user_values.count).to eq 0
+  end
+
+  it "you can switch the updates. Whatever is not mentioned will not be touched" do
+    # first let's prepare some user filters
+    user_with_already_set_up_filter = @project.get_user(ConnectionHelper::DEFAULT_USERNAME)
+
+    filters = [
+      [user_with_already_set_up_filter.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_variable_permissions(filters, @variable)
+    expect(@variable.user_values.map {|f| [f.related.login, f.pretty_expression] })
+      .to eq [[ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"]]
+    # Now let's add user filter to a different user. If we do not explicitely state that
+    # user_with_already_set_up_filter should keep his filter it will be removed
+    another_user = @domain.users.find { |u| u.login != ConnectionHelper::DEFAULT_USERNAME }
+    @project.add_user(another_user, 'Admin', domain: @domain)
+    new_filters = [
+      [another_user.login, @label.uri, "tomas@gooddata.com"]
+    ]
+    @project.add_variable_permissions(new_filters, @variable, do_not_touch_filters_that_are_not_mentioned: true)
+    expect(@variable.user_values.map {|f| [f.related.login, f.pretty_expression] })
+      .to include([ConnectionHelper::DEFAULT_USERNAME, "[Dev] IN ([tomas@gooddata.com])"],
+                  [another_user.login, "[Dev] IN ([tomas@gooddata.com])"])
+  end
+
+  it "should be able to update the filter value in place" do
+    pending('We cannot swap filters yet')
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com", "jirka@gooddata.com"]]
+    @project.add_variable_permissions(filters, @variable)
+    perm = @variable.user_values.first
+    filters = [[ConnectionHelper::DEFAULT_USERNAME, @label.uri, "tomas@gooddata.com"]]
+    @project.add_variable_permissions(filters, @variable)
+    expect(perm.reload!.pretty_expression).to eq '[Dev] IN ([tomas@gooddata.com, jirka@gooddata.com])'
+  end
+end