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

Created script for Defect 2432 #2008

Merged
merged 7 commits into from
Apr 20, 2021

Conversation

TMyza
Copy link
Contributor

@TMyza TMyza commented Aug 10, 2018

ATF Test Scripts to check #2432

This PR is ready for review.

Summary

A set of test scripts to verify that issue is fixed

ATF version

develop

Changelog

CLA

common.getMobileSession():ExpectResponse(cid, { success = true, resultCode = "SUCCESS" })
:ValidIf(function(_, data)
if data.payload.info then
return false, "Unexpected parameter in SDL response"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please define that unexpected parameter in response, and in what response:
"Unexpected parameter 'info' in DialNumber response"

@GetmanetsIrina GetmanetsIrina force-pushed the Defect_2432_SDL_is_crashed branch from b645810 to 8378e82 Compare April 29, 2020 13:47
@GetmanetsIrina
Copy link
Contributor

GetmanetsIrina commented Apr 29, 2020

PR is rebased on actual develop(a7db5cc) branch

@GetmanetsIrina
Copy link
Contributor

Script is applicable for all policy flows and transport types.
Script has been moved to 6_1 folder.
Test set for script is added.

PTU with corresponding policies has been added.

@GetmanetsIrina
Copy link
Contributor

PR is still relevant.

@GetmanetsIrina
Copy link
Contributor

@jordynmackool, PR is ready for Livio review.
Please notice PR relates to the issue that was closed within the 6.1 Release.

@iCollin iCollin merged commit 33294a3 into smartdevicelink:develop Apr 20, 2021
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.

6 participants