Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function 'patchPropertiesCorrespondToPutProperties' threw an exception: Cannot read property 'length' of undefined #610

Open
abatishchev opened this issue Oct 11, 2023 · 6 comments

Comments

@abatishchev
Copy link

Describe the bug

I discovered the following errors in the logs of ~[NotRequired_Staging] Swagger LintDiff failed for a PR for my RP:

 - FATAL
spectralPluginFunc: Failed validating: 'file:///mnt/vss/_work/1/azure-rest-api-specs-pr/specification/throttling/resource-manager/Microsoft.Throttling/preview/2022-06-01-preview/throttling.json'. Errors encountered:
Error: Error running Nimma
Error 1: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 2: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 3: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 4: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 5: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 6: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")
Error 7: $[paths,'x-ms-paths'].* threw: ErrorWithCause("Function \"patchPropertiesCorrespondToPutProperties\" threw an exception: Cannot read property 'length' of undefined")

See https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3154242&view=logs&j=688669d0-441c-57c3-cf6d-f89a22ccfa5d&t=b91b1e88-b042-5e18-36d8-34e4fb3a9b3b&l=619

To Reproduce

N/A

Expected behavior

The function should handle incorrect input gracefully.

Screenshots

N/A

Desktop (please complete the following information):

N/A

Additional context

N/A

@abatishchev abatishchev changed the title 'Function 'patchPropertiesCorrespondToPutProperties' threw an exception: Cannot read property 'length' of undefined' Function 'patchPropertiesCorrespondToPutProperties' threw an exception: Cannot read property 'length' of undefined Oct 11, 2023
@abhishek-msft
Copy link

abhishek-msft commented Oct 11, 2023

Yes, got the same issue in our PR build as well.
https://github.com/Azure/azure-rest-api-specs-pr/pull/15304/checks?check_run_id=17595166341

@alexandrami22
Copy link

@wchigit
Copy link

wchigit commented Oct 24, 2023

@jacote-msft
Copy link

@abhishek-msft
Copy link

[NotRequired_Staging] check can be ignored and it does not block the merging of the PR. It didn't block our PR from merging.

@abhishek-msft
Copy link

I think, what I also learned was Swagger ApiView check failure also doesn't block the merge apart from the [NotRequired_Staging] check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants