Trouble with generating an array of number enums

[morbalint]

The generator removes an array from the swagger.json schema when the array contains a number enum.
simplified example:

{ 
  "definitions": {
    "MyModel": {
      "type":"object",
      "properties": {
          "foo": {
          "uniqueItems": true,
          "type": "array",
          "items": {
            "format": "int32",
            "enum": [
              0,
              1,
              2,
              4,
              8
            ],
            "type": "integer"
          }
        },
      }
    }
  }
}

the generated class will look something like this:

export class MyModel {
  foo: 0 | 1 | 2 | 4 | 8;
  constructor(data?: any) {
    if (data) {
      this['foo'] = data['foo'];
    }
  }
}

Sorry for the low quality example :(

I believe, I have found the error, and I will create a PR later.

Can you confirm its still an issue in your latest development branch ?

[Manweill]

Sure, I will confirm it

[Manweill]

I fix it ,it will generate like this:

type INumberArrayEnumModelFoo = 0 | 1 | 2 | 4 | 8;

export class NumberArrayEnumModel {
  /**  */
  foo: INumberArrayEnumModelFoo[]

  constructor(data?: any) {
    if (data) {
      this['foo'] = data['foo'];
    }
  }
}

[morbalint]

Thank you!

[Manweill]

You are welcome!
Thank you for your interest in this library and application to the actual project.

[morbalint]

Hi!

I tried, your fix and found an error with it. When you have more than one number enum, the generator does not separate the types and creates file with wrong syntax.

Created pull request with example to show, and a fix.

Should, this be a separate issue, or can we track it in this one?

Also, what is your release process, should I increase the version number and add the issue to the changelog ?

[Manweill]

I reopen this issue.
When i publish, I will test the example with the generator.
If the test passes, add the issue to the changelog and increase the version

[Manweill]

@morbalint
The PR #11 need me publish now?