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.

Sign up for GitHub

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

Jump to bottom

Add getSystemCapability test to smoke test set #2200

Merged
merged 4 commits into from
Jun 19, 2019
Merged

Add getSystemCapability test to smoke test set #2200

merged 4 commits into from
Jun 19, 2019

Conversation

Jack-Byrne
Copy link
Collaborator

This PR is ready for review.

Summary

  • Add getSystemCapability test to smoke test set.
  • Add video streaming capabilities to the getSystemCapability test.

ATF version

Develop/Master

CLA

@dboltovskyi
Copy link
Contributor

dboltovskyi commented Jun 19, 2019 •
edited
Loading

@JackLivio
We already have a list of scripts to check RPCs in Smoke/API folder. They follow new Scenario approach and easy to support.
Unfortunately GetSystemCapability is missing.
Could you create a new one for this RPC based on existing ones?
Or I can help to create such one.

@Jack-Byrne
Copy link
Collaborator Author

@dboltovskyi Thats ok I can make the change.

@Jack-Byrne
Copy link
Collaborator Author

@dboltovskyi New test created under correct format.

@dboltovskyi
Copy link
Contributor

@JackLivio Thank you for update

jacobkeeler
jacobkeeler requested changes Jun 19, 2019
View reviewed changes
test_sets/smoke_tests.txt Outdated Show resolved Hide resolved
test_scripts/Smoke/API/048_GetSystemCapabilityRequest_SUCCESS.lua Outdated Show resolved Hide resolved
test_scripts/API/ATF_GetSystemCapability.lua Outdated Show resolved Hide resolved
test_scripts/Smoke/API/048_GetSystemCapabilityRequest_SUCCESS.lua Outdated Show resolved Hide resolved
@Jack-Byrne Jack-Byrne merged commit 3bd957c into develop Jun 19, 2019
@jacobkeeler jacobkeeler deleted the feature/update_get_system_capabilities branch February 2, 2021 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacobkeeler jacobkeeler jacobkeeler approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Jack-Byrne @dboltovskyi @jacobkeeler