Skip to content

Fix broken master: Update karma for new expansion cell markup

André Luís requested to merge 13439-fix-broken-karma-test-on-ee-side into master

What does this MR do?

After this MR was merged https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/30927 we started getting failures on the EE side: https://gitlab.com/gitlab-org/gitlab-ee/issues/13439 (details on broken master here)

Weirdly, this should fail too on CE.

This MR fixes the test and keeps its robustness by checking for the new markup.

tl;dr: we no longer have .line_content for the match/context lines. I added an explicit class to keep that check in the test.

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/13439

Merge request reports

Loading