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

[plugin-mobile-app] Add step to download the file from the device and… #3749

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

IBrun
Copy link
Contributor

@IBrun IBrun commented Mar 23, 2023

… save its content to variable

@codecov
Copy link

codecov bot commented Mar 23, 2023

Codecov Report

Merging #3749 (b5d13a5) into master (ace0fc0) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3749      +/-   ##
============================================
- Coverage     96.99%   96.98%   -0.01%     
- Complexity     6082     6085       +3     
============================================
  Files           854      854              
  Lines         17425    17434       +9     
  Branches       1136     1136              
============================================
+ Hits          16901    16909       +8     
- Misses          418      419       +1     
  Partials        106      106              
Impacted Files Coverage Δ
...va/org/vividus/mobileapp/action/DeviceActions.java 100.00% <100.00%> (ø)
.../java/org/vividus/mobileapp/steps/DeviceSteps.java 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@IBrun IBrun marked this pull request as ready for review March 23, 2023 13:47
@IBrun IBrun requested a review from a team as a code owner March 23, 2023 13:47
When I download file `$filePath` from device and save its content to $scopes variable `$variableName`
----

include::partial$mobile-remote-file-path.adoc[]
Copy link
Collaborator

Choose a reason for hiding this comment

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

the description for $scopes and $variableName is missing

@@ -78,4 +79,19 @@ public void rotate(ScreenOrientation orientation)
webDriverSessionInfo.reset(WebDriverSessionAttributes.SCREEN_SIZE);
}
}

/**
* Pull a file by <b>deviceFilePath</b> from the device
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Pull a file by <b>deviceFilePath</b> from the device
* Pull a file at <b>deviceFilePath</b> from the device


/**
* Pull a file by <b>deviceFilePath</b> from the device
* @param deviceFilePath The path to an existing remote file on the device. This variable can be predefined
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* @param deviceFilePath The path to an existing remote file on the device. This variable can be predefined
* @param deviceFilePath The path to an existing remote file on the device. This parameter can be predefined

@@ -88,6 +93,37 @@ public void deleteFileFromDevice(String filePath)
deviceActions.deleteFile(filePath);
}

/**
* Download a file by <b>filePath</b> from the device and save its content to <b>scopes</b> variables with name
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* Download a file by <b>filePath</b> from the device and save its content to <b>scopes</b> variables with name
* Downloads a file at <b>filePath</b> from the device and save its content to <b>scopes</b> variables with name

* When I download file `$filePath` from device and save its content to $scopes variable `$variableName`
* </code>
*
* @param filePath The path to an existing remote file on the device. This variable can be predefined
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* @param filePath The path to an existing remote file on the device. This variable can be predefined
* @param filePath The path to an existing remote file on the device. This parameter can be predefined

@IBrun IBrun force-pushed the download_file_from_device branch from 012018e to b5d13a5 Compare March 27, 2023 08:24
@valfirst valfirst merged commit c717fb5 into vividus-framework:master Mar 28, 2023
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.

5 participants