Skip to content

Commit

Permalink
[ML] Functional API tests - fix mml request bodies (#62116) (#62152)
Browse files Browse the repository at this point in the history
This PR adjusts the request bodies in the model memory estimation endpoint tests.
  • Loading branch information
pheyos authored Apr 1, 2020
1 parent 397fd15 commit b59f7fe
Showing 1 changed file with 8 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export default ({ getService }: FtrProviderContext) => {
},
},
{
testTitleSuffix: 'with 1 metrics and 1 influencers same as split field',
testTitleSuffix: 'with 1 metric and 1 influencer same as split field',
user: USER.ML_POWERUSER,
requestBody: {
indexPattern: 'ecommerce',
Expand All @@ -57,7 +57,7 @@ export default ({ getService }: FtrProviderContext) => {
detectors: [
{
function: 'avg',
field_name: 'geoip.city_name',
field_name: 'taxless_total_price',
by_field_name: 'geoip.city_name',
},
],
Expand All @@ -84,7 +84,7 @@ export default ({ getService }: FtrProviderContext) => {
{
function: 'mean',
by_field_name: 'geoip.city_name',
field_name: 'geoip.city_name',
field_name: 'taxless_total_price',
},
],
influencers: ['geoip.city_name', 'customer_gender', 'customer_full_name.keyword'],
Expand All @@ -100,7 +100,8 @@ export default ({ getService }: FtrProviderContext) => {
},
},
{
testTitleSuffix: '4 influencers, split by customer_id and filtering by country code',
testTitleSuffix:
'2 detectors split by city and manufacturer, 4 influencers, filtering by country code',
user: USER.ML_POWERUSER,
requestBody: {
indexPattern: 'ecommerce',
Expand All @@ -109,13 +110,13 @@ export default ({ getService }: FtrProviderContext) => {
detectors: [
{
function: 'mean',
by_field_name: 'customer_id.city_name',
field_name: 'customer_id.city_name',
by_field_name: 'geoip.city_name',
field_name: 'taxless_total_price',
},
{
function: 'avg',
by_field_name: 'manufacturer.keyword',
field_name: 'manufacturer.keyword',
field_name: 'taxless_total_price',
},
],
influencers: [
Expand Down

0 comments on commit b59f7fe

Please sign in to comment.