testomatio / reporter

Testomatio Reporter
125 stars 22 forks source link

Don't show Artifacts error message duplicates #261

Closed TetianaKhomenko closed 1 week ago

TetianaKhomenko commented 11 months ago

To Reproduce Steps to reproduce the behavior:

  1. open a project
  2. go to Settings > Artifacts
  3. delete any credential, for example ENDPOINT
  4. save
  5. launch automated tests
  6. see the issue

Expected behavior

[TESTOMATIO] To disable artifact uploads set: TESTOMATIO_DISABLE_ARTIFACTS=1
[TESTOMATIO] To enable PRIVATE uploads set: TESTOMATIO_PRIVATE_ARTIFACTS=1

should be shown one time

Screenshots

telegram-cloud-photo-size-2-5438431064033974479-y

mykhailiukVitalii commented 11 months ago

Available on the 1.1.0-beta.artifact-count-retry-msg-1 beta version for the testing @TetianaKhomenko

TetianaKhomenko commented 11 months ago

@mykhailiukVitalii the issue is still there

Image

mykhailiukVitalii commented 11 months ago

Good catch. I will check this error later today. Thx) @TetianaKhomenko

mykhailiukVitalii commented 10 months ago

PR was created -> https://github.com/testomatio/reporter/pull/293 after @DavertMik review it will be merge for testing

mykhailiukVitalii commented 10 months ago

already fixed with https://github.com/testomatio/reporter/issues/238 this case. Now the user see more additional information only in the DEBUG mode. We can check after merging - https://github.com/testomatio/reporter/pull/293 PR @TetianaKhomenko

TetianaKhomenko commented 8 months ago

@mykhailiukVitalii The issue is reproducing in our Playwright TS example project with the wrong storage credentials. Testomatio Reporter v1.2.2.

olexandr13 commented 2 months ago

should be retested on refactored artifact uploader (changed to class) and on version 2+