search

openedx / wg-build-test-release

Open edX Build / Test / Release Working Group
24 stars 14 forks source link

[Test failure] TC_AUTHOR_142: Certificate preview not showing signatures #364

Open alexjmpb opened 1 month ago

alexjmpb commented 1 month ago

Release

Redwood

Expected behavior

"1. Go to a course with a certificate already set up

  1. Preview certificate
  2. Select print option. The layout (signatures, course name, ID, org...) are in place"

Actual behavior

The printing layout is correct overall except for the signatures which are not showing up.

Steps to reproduce

"1. Go to a course with a certificate already set up

  1. Preview certificate
  2. Review that signatures are not showing.

Additional information

image

image

This test case is indicated that also failed in Quince.

mariajgrimaldi commented 1 month ago

Hello. Thank you for the report!

I reported this issue a while ago: https://github.com/openedx/wg-build-test-release/issues/177#issuecomment-1124246022. In Quince, it was marked as failing, but it actually wasn't: https://github.com/openedx/wg-build-test-release/issues/332#issuecomment-1849050582. Honor code certificates use the organization logo and signatory information, but do not include signature images, which are used only for verified certificates.

As you mentioned, I tested it out on the sandbox with the verified course mode, and this time it didn't work for honor or verified.

After trying a few things, I noticed you didn't have any signature images in your setup so I added them, and it worked: https://redwood.demo.edly.io/certificates/course/course-v1:Test+1+1?preview=verified, for both honor and verified. I'm not sure if this is the usual behavior though. I'll test in a quince environment to confirm.

mariajgrimaldi commented 1 month ago

The behavior I reported also happens in quince, but since signatures are optional, it's a bit misleading. I'll report this elsewhere.

mariajgrimaldi commented 1 month ago

I reported this in the #redwood_release_planning slack channel: https://openedx.slack.com/archives/C06AMDA6CAZ/p1716551180358129

jmakowski1123 commented 1 month ago

I would characterize this as a bug. I would expect the signatory info to display independently of the uploaded image. @cablaa77 - are you familiar with the intended certificate function?

mariajgrimaldi commented 1 month ago

I'll be taking a look into this. I'll make sure to report back once I know more. Thank you!