fixes a bug where a null schema would make the generation process derail

[baywet]

partial #135

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

[PureKrome]

Thanks @baywet - as in the screen shot above, I gave it a quick try but couldn't stay awake 💤 So you beat me to it :)

[PureKrome]

@baywet - i was just playing with this PR on localhost and the "petstore" still fails with an exception.

> System.InvalidOperationException: adding an item namespace at the root is not supported

I'm assuming that Kiota should work against the "petstore" json file?

(looks like this section is failing?? )

EDIT:

this was the command I used, which can be used to repo this:

launchSettings.json

{
  "profiles": {
    "kiota": {
      "commandName": "Project",
      "commandLineArgs": "--openapi https://petstore.swagger.io/v2/swagger.json -o OpenApiClient\\Generated -c PetShopGraphClient --loglevel Information"
    }
  }
}

[darrelmiller]

Interestingly, when I try processing Petstore I'm getting a different issue:

            else throw new InvalidOperationException("un handled case, might be object type or array type");
            // object type array of object are technically already handled in properties but if we have a root with those we might be missing some cases here

this is caused by

    Pets:
      type: array
      items:
        $ref: "#/components/schemas/Pet"

We haven't run into this case with Graph OpenAPI descriptions because OData never allows an array at the root of a response.

[PureKrome]

Should it be handled? Should the "petshop" be a valid sample that should generate?

[darrelmiller]

@PureKrome Sorry if that wasn't clear. Absolutely it should be handled. We should handle any valid OpenAPI doc.

[zengin]

LGTM