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

Script to check issue 2428 #2097

Closed
wants to merge 4 commits into from

Conversation

mkorniichuk
Copy link
Contributor

@mkorniichuk mkorniichuk commented Oct 5, 2018

ATF Test Scripts to check #2428 #1829

This PR is not ready for review.
This is draft version of the script.

Summary

A test script to verify that issue is fixed

Notes:

Script succeeded only at SDL which is built with CMAKE_BUILD_TYPE=Release

CLA

@IGapchuk
Copy link
Contributor

The PR is rebased on the current develop branch (273b79d).
Script is applicable for all policy flows and transport types.
The script has been moved to 6_1 folder.
Test set for the script is added.

@dboltovskyi
Copy link
Contributor

dboltovskyi commented Oct 26, 2020

Script is updated to be included into 7.0 Release. However successful result can be received only at Release build type.
So the merge should be postponed for now.

@GetmanetsIrina
Copy link
Contributor

The successful result can be still received only at Release build type. Checked on https://github.com/smartdevicelink/sdl_core/tree/release/7.1.0-RC1 (c163c2d)

@GetmanetsIrina
Copy link
Contributor

PR is closed because the test is passed on SDL build in the Release mode. There is no need to support the current SDL behaviour for the Debug mode.

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