Skip to content

Add FE integration tests for "User opens Web IDE"

Himanshu Kapoor requested to merge 270081-user-opens-webide into master

What does this MR do?

Issue: #270081 (closed)

Adds frontend integration tests for "User opens Web IDE"

Why integration test?

Recently, it was found out that feature specs for the Web IDE are really slow and sometimes take over 10 minutes per file. See #282436 (closed). Since we plan to convert those tests to integration tests anyway, I thought it would be best to write integration tests here too.

I was running into many problems with running feature specs for the Web IDE, most notably: slow tests and SQL query limits. It was quite a chore to write a test and wait 10 minutes for the test to pass or fail (even on local system).

And async components loaded with dynamic import statements somehow don't seem to work in local runs. (They work fine in production.) Which means that the test mentioned in #282436 (closed) never passes locally. And that is an unknown blocker I couldn't really get past, and FE integration tests were the only way out.

TL;DR: Feature specs are a chore to work with in the Web IDE. FE integration tests are fast and a good way to test the Web IDE.

Does this MR meet the acceptance criteria?

Conformity

Availability 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
Edited by Himanshu Kapoor

Merge request reports