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

Test false "false positive" for ProvisioningStateMustBeReadOnly #561

Conversation

AkhilaIlla
Copy link
Collaborator

@AkhilaIlla AkhilaIlla commented Jul 25, 2023

@AkhilaIlla AkhilaIlla changed the title add test case to test readOnly property added to envelope property Test false "false positive" for ProvisioningStateMustBeReadOnly Jul 25, 2023
@@ -212,6 +212,106 @@ test("ProvisioningStateMustBeReadOnly should find errors when multiple verbs spe
})
})

test("ProvisioningStateSpecified should find errors when readOnly property is present in the envelope property", () => {
const oasDoc = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

really nice to see this test being added!

Copy link
Member

@rkmanda rkmanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@AkhilaIlla AkhilaIlla merged commit f28c372 into Azure:main Jul 27, 2023
4 checks passed
@cataggar
Copy link
Member

I'm still receiving many false positives for the ProvisioningStateMustBeReadOnly check #637.

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

Successfully merging this pull request may close these issues.

4 participants