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

adding resource for azure snapshots #671

Merged
merged 3 commits into from
Nov 22, 2022

Conversation

Jiaming1826
Copy link
Contributor

Signed-off-by: Jiaming Wang jiaming.wang@sap.com

Description

Added new files to support retrieving Azure Snapshot Resources

Issues Resolved

List any existing issues this PR resolves, or any Discourse or StackOverflow discussion that's relevant

Check List

@Jiaming1826 Jiaming1826 requested a review from a team as a code owner September 5, 2022 09:17
@netlify
Copy link

netlify bot commented Sep 5, 2022

Deploy Preview for inspec-azure canceled.

Name Link
🔨 Latest commit 112f835
🔍 Latest deploy log https://app.netlify.com/sites/inspec-azure/deploys/637c9c88e7e47a0008bb2bac

@sathish-progress
Copy link
Contributor

@Jiaming1826 , could you please write some tests to verify this is working?

@Jiaming1826
Copy link
Contributor Author

@Jiaming1826 , could you please write some tests to verify this is working?

@sathish-progress , please check the screenshots below for tests in inspec shell
Plural resource:
image
Singular resource:
image

@Jiaming1826 Jiaming1826 force-pushed the add_resources_snapshots branch from 5efd49d to aca5192 Compare September 16, 2022 08:44
@sathish-progress
Copy link
Contributor

@Jiaming1826 please refer this https://github.com/inspec/inspec-azure/tree/main/test/unit/resources, we need some unit tests for the resources

@Jiaming1826 Jiaming1826 force-pushed the add_resources_snapshots branch from aca5192 to d3528e6 Compare October 21, 2022 04:19
Signed-off-by: Jiaming Wang <jiaming.wang@sap.com>
@soumyo13 soumyo13 added the azure label Nov 17, 2022
@soumyo13 soumyo13 merged commit a7da29c into inspec:main Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants