[Snapshots] Blank PNG/JPG image generated for snapshots

[nikhilmandlik]

Resolves:

resolves: #4526

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Is this change backwards compatible? For example, developers won't need to change how they are calling the API or how they've extended core plugins such as Tables or Plots.

Author Checklist

• Changes address original issue?
• Unit tests included and/or updated with changes?
• Command line build passes?
• Has this been smoke tested?
• Testing instructions included in associated issue?

[codecov]

Codecov Report

Merging #4612 (9f3913c) into master (722e2e2) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4612      +/-   ##
==========================================
- Coverage   57.27%   57.22%   -0.05%     
==========================================
  Files         718      718              
  Lines       22771    22771              
  Branches     1653     1653              
==========================================
- Hits        13041    13031      -10     
- Misses       9314     9323       +9     
- Partials      416      417       +1     

Impacted Files	Coverage Δ
src/plugins/notebook/components/NotebookEmbed.vue	0.00% <ø> (ø)
src/tools/url.js	69.23% <0.00%> (-30.77%)	⬇️
src/api/composition/DefaultCompositionProvider.js	76.19% <0.00%> (-4.77%)	⬇️
src/ui/inspector/details/Properties.vue	63.63% <0.00%> (-1.82%)	⬇️
src/ui/router/Browse.js	86.48% <0.00%> (-1.36%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 722e2e2...9f3913c. Read the comment docs.

[unlikelyzero]

@nikhilmandlik can we add a test for this considering this has regressed at least once?

[nikhilmandlik]

@nikhilmandlik can we add a test for this considering this has regressed at least once?

I think I can add e2e tests for notebook

[michaelrogers]

This change looks good to me.

[shefalijoshi]

Reviewer Checklist

• Changes appear to address issue?
• Changes appear not to be breaking changes?
• Appropriate unit tests included?
• Code style and in-line documentation are appropriate?
• Commit messages meet standards?
• Has associated issue been labelled unverified? (only applicable if this PR closes the issue)
• Has associated issue been labelled bug? (only applicable if this PR is for a bug fix)
• List of Acceptance Tests Performed.