fix: filter null from enum values (orval-labs#928)

Co-authored-by: Victor <[email protected]>
voiceflow · Sep 11, 2023 · 0860340 · 0860340
1 parent 663cafa
commit 0860340
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/packages/core/src/getters/scalar.ts b/packages/core/src/getters/scalar.ts
@@ -20,6 +20,7 @@ export const getScalar = ({
  context: ContextSpecs;
 }): ScalarValue => {
  const nullable = item.nullable ? ' | null' : '';
+ const enumItems = item.enum?.filter((enumItem) => enumItem !== null);
 
  if (!item.type && item.items) {
  item.type = 'array';
@@ -34,8 +35,8 @@ export const getScalar = ({
  : 'number';
  let isEnum = false;
 
- if (item.enum) {
- value = item.enum.map((enumItem: string) => `${enumItem}`).join(' | ');
+ if (enumItems) {
+ value = enumItems.map((enumItem: string) => `${enumItem}`).join(' | ');
  isEnum = true;
  }
 
@@ -77,8 +78,8 @@ export const getScalar = ({
  let value = 'string';
  let isEnum = false;
 
- if (item.enum) {
- value = `'${item.enum
+ if (enumItems) {
+ value = `'${enumItems
  .map((enumItem: string) =>
  isString(enumItem) ? escape(enumItem) : `${enumItem}`,
  )
@@ -122,8 +123,8 @@ export const getScalar = ({
 
  case 'object':
  default: {
- if (item.enum) {
- const value = `'${item.enum
+ if (enumItems) {
+ const value = `'${enumItems
  .map((enumItem: string) =>
  isString(enumItem) ? escape(enumItem) : `${enumItem}`,
  )

diff --git a/tests/specifications/null-type.yaml b/tests/specifications/null-type.yaml
@@ -55,7 +55,9 @@ components:
  - type: 'string'
  - type: 'null'
  NullEnum:
+ nullable: true
  enum:
+ - foo
  - null
  BlankEnum:
  enum: