Skip to content

Restructure `supertest` tests that are running against the app

Tomas Vik (OOO back on 2024-11-19) requested to merge move-integration-tests into develop

General folder-wide renames

Renaming the supertest tests to have the same prefix request- so they are displayed together, but removing api from the web test folder name. Putting supertest tests separately from other tests is important, because they use a different timeout from the rest of our suite (https://gitlab.com/gitlab-org/gitter/webapp/merge_requests/1454#note_186174074)

  • test/api-tests -> test/request-api-tests
  • test/api-web-tests -> test/request-web-tests

2 special renames

The new position within handlers folder better reflects the production code folder structure. The test's name matches the name of the handler it's testing.

  • test/api-web-tests/integration-settings-tests.js -> test/request-web-tests/handlers/settings-tests.js
  • test/api-web-tests/archive-handler-tests.js -> test/request-web-tests/handlers/app/archive-tests.js

related to !1454 (closed)

Edited by Tomas Vik (OOO back on 2024-11-19)

Merge request reports