Report Portal tests review#164
Conversation
| @@ -0,0 +1,14 @@ | |||
| [pytest] | |||
| rp_endpoint = https://tests.aparcar.org | |||
| rp_api_key = ${{ secrets.API_KEY }} | |||
There was a problem hiding this comment.
This value is only set within a CI run and can't stay in a file as is. Please check the documentation, it should be something like -o "rp_launch_attributes=Smoke Tests". https://reportportal.io/docs/log-data-in-reportportal/test-framework-integration/Python/pytest#configuration
There was a problem hiding this comment.
I think is working good this time, I runed at the github actions and is working fine.
| rp_project = openwrt | ||
|
|
||
| rp_launch_description = "OpenWrt One Snapshot" | ||
| rp_launch_attributes = 'device:openwrt_one' 'revision:r30292-77b9393d2f' 'version:SNAPSHOT' |
There was a problem hiding this comment.
These values should be set dynamically in the CI
There was a problem hiding this comment.
I am still not very sure how that works but in this case I just coment it
|
These tests are mostly relevant for the CI, please check the daily workflow CI file and integrate those tests there. |
| --firmware ../../openwrt/bin/targets/malta/be/openwrt-malta-be-vmlinux-initramfs.elf | ||
| --firmware ../../openwrt/bin/targets/malta/be/openwrt-malta-be-vmlinux-initramfs.elf \ | ||
| --reportportal \ | ||
| -rp-launch="name_of_test" \ |
There was a problem hiding this comment.
Please remove the last backslash, it would attach an empty line to command. Please read this to get an idea https://unix.stackexchange.com/questions/368753/what-does-this-command-with-a-backslash-at-the-end-do
bryansancheZ199
force-pushed
the
main
branch
from
95303e7 to
9d22fd1
Compare
2 participants
the review of the tests are showing perfect by report portal, but the system does not have a guest user without a password