Skip to content

Verify if custom_attribute_names whether it should be used in tests

Verify if custom_attribute_names whether it should be used in tests

The following test are using custom_attribute_names attribute

https://git.eliatra.com/eliatra-suite/eliatra-opensearch-integration-tests/-/blob/rename-basics-fix-configuration/test/_testsuite_enterprise_1/test_internal_users.sh#L13

https://git.eliatra.com/eliatra-suite/eliatra-opensearch-integration-tests/-/blob/rename-basics-fix-configuration/test/_testsuite_enterprise_1/test_jwt.sh#L64

https://git.eliatra.com/eliatra-suite/eliatra-opensearch-integration-tests/-/blob/rename-basics-fix-configuration/test/_testsuite_enterprise_2/test_ldap_filtered_attributes.sh#L68

https://git.eliatra.com/eliatra-suite/eliatra-opensearch-integration-tests/-/blob/rename-basics-fix-configuration/test/_testsuite_enterprise_2/test_ldap.sh#L68

The custom_attribute_names seems to return an empty value e.g for the following test

https://git.eliatra.com/eliatra-suite/eliatra-opensearch-integration-tests/-/blob/rename-basics-fix-configuration/test/_testsuite_enterprise_3/test_env_var_substitution.bats#L25

curl -Ss --insecure -u ldap_hr_employee:ldap_hr_employee -XGET "https://sgssl-0.example.com:9200/_eliatra/security/authinfo?pretty"
{
  "user" : "User ldap_hr_employee <basic/ldap> [backend_roles=[hr, kibanauser]]",
  "user_name" : "ldap_hr_employee",
  "user_requested_tenant" : null,
  "remote_address" : "172.16.0.230:48968",
  "backend_roles" : [
    "hr",
    "kibanauser"
  ],
  "custom_attribute_names" : [ ],
  "attribute_names" : [ ],
  "effective_roles" : [
    "sp_human_resources",
    "DASHBOARDS_USER",
    "OWN_INDEX"
  ],
  "effective_tenants" : {
    "ldap_hr_employee" : true,
    "human_resources" : true,
    "performance_data" : true,
    "GLOBAL_TENANT" : true,
    "management" : true,
    "business_intelligence" : false
  },
  "principal" : null,
  "peer_certificates" : "0",
  "cluster_name" : "sptest_docker"
}

After disabling this request the sample test was successful. Should we remove the tests that use this value or should they be modified to use a different value?