Skip to content

Refactor inline markdown snapshot helper spec

Chad Woolley requested to merge caw-inline-markdown-snapshot-helper into master

What does this MR do and why?

Related: Tracking Issue: Implement GLFM scripts per the ... (#361241 - closed)

Completes task in #361241 (closed):

  • Inline spec/frontend/content_editor/markdown_snapshot_spec_helper.js into spec/frontend/content_editor/markdown_snapshot_spec.js, because there no longer needs to be a CE vs. EE split on the frontend.

The only way we would ever need an EE-specific version of the spec like we do on the backend is if we kept the deserialization which relies on the backend. But that is not the long-term plan. So, lets collapse this and make it simpler for now. We can always reintroduce this helper if we change our mind in the future.

References

See the GLFM Specification Guide development documentation for much more context on all of this work: https://docs.gitlab.com/ee/development/gitlab_flavored_markdown/specification_guide

How to set up and validate locally

  1. Run spec/frontend/content_editor/markdown_snapshot_spec.js and ensure it still works.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports