Skip to content

Commit

Permalink
fix(specs): fix missing params and types
Browse files Browse the repository at this point in the history
  • Loading branch information
shortcuts committed Jul 4, 2022
1 parent d358fdf commit 48cd2a1
Show file tree
Hide file tree
Showing 14 changed files with 974 additions and 387 deletions.
242 changes: 0 additions & 242 deletions clients/algoliasearch-client-php/lib/Model/Search/SchemasQuery.php

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ public JavaCTSManager(String client) {

@Override
public void addSupportingFiles(List<SupportingFile> supportingFiles) {
supportingFiles.add(new SupportingFile("build.mustache", ".", "build.gradle"));
supportingFiles.add(new SupportingFile("build.mustache", "", "build.gradle"));
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,13 @@ private void handleModel(
}
if (varSpec == null) {
throw new CTSException(
"Parameter " + entry.getKey() + " not found in " + paramName + ". You might have a type conflict in the spec for " + baseType
"Parameter '" +
entry.getKey() +
"' not found in '" +
paramName +
"'. You might have a type conflict in the spec for '" +
baseType +
"'"
);
}

Expand Down
6 changes: 5 additions & 1 deletion specs/abtesting/common/parameters.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,11 @@ endAt:

createdAt:
type: string
description: End date for the A/B test expressed as YYYY-MM-DDThh:mm:ssZ.
description: Create date for the A/B test expressed as YYYY-MM-DDThh:mm:ssZ.

updatedAt:
type: string
description: Update date for the A/B test expressed as YYYY-MM-DDThh:mm:ssZ.

name:
type: string
Expand Down
3 changes: 3 additions & 0 deletions specs/abtesting/common/schemas/ABTest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ ABTest:
description: A/B test significance based on conversion data. Should be > 0.95 to be considered significant (no matter which variant is winning).
endAt:
$ref: '../parameters.yml#/endAt'
updatedAt:
$ref: '../parameters.yml#/updatedAt'
createdAt:
$ref: '../parameters.yml#/createdAt'
name:
Expand All @@ -34,6 +36,7 @@ ABTest:
- name
- createdAt
- endAt
- updatedAt
- conversionSignificance
- clickSignificance
- abTestID
Expand Down
Loading

0 comments on commit 48cd2a1

Please sign in to comment.