support for x-enum-varnames in chaotic #712
Conversation
| @@ -262,13 +263,35 @@ def _gen_integer( | |||
| assert schema.format is None | |||
| assert user_cpp_type is None | |||
|
|
|||
| enum_names = [] | |||
|
|
|||
| if hasattr(schema, 'x_properties') and 'x-enum-varnames' in schema.x_properties: | |||
There was a problem hiding this comment.
x_properties должны быть всегда
| result.extend([part.capitalize() for part in SPLIT_WORDS_RE.findall(word)]) | ||
| return ''.join(result) | ||
|
|
||
| for i in range(len(schema.enum)): |
|
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/ You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
| schemas: | ||
| Enum: | ||
| type: object | ||
| additionalProperties: false |
There was a problem hiding this comment.
это должен быть не голден тест, а тест на то, что static_cast(kInner) == 0 и т.п.
There was a problem hiding this comment.
я про тест совсем не понял. Где это должно проверяться и почему именно static_cast(kInner)
There was a problem hiding this comment.
где - где-то в chaotic/tests/back/cpp
почему - ну ты написал некую функциональность, которая генерит другие имена констант; нужно проверить, что эти константы равны нужным значениям, а не абы каким
голден тесты - это не тесты, а лишь витрина для стороннего наблюдателя
|
fixes #710 |
support for x-enum-varnames in chaotic