camunda / zeebe

Distributed Workflow Engine for Microservices Orchestration
https://zeebe.io
3.05k stars 549 forks source link

refactor: modularize operate-data-generator module #18188

Closed romansmirnov closed 1 week ago

romansmirnov commented 2 weeks ago

Description

This PR removes the need to have a "do nothing" data generator bean by modularizing the data-generator module in the same way as done for other modules.

Related issues

related #17579

github-actions[bot] commented 2 weeks ago

Operate Test Results

778 tests   - 12   763 :white_check_mark:  - 12   16m 30s :stopwatch: + 2m 42s 130 suites  -  3    15 :zzz: ± 0  130 files    -  3     0 :x: ± 0 

Results for commit 9eeaa28b. ± Comparison against base commit f807b15f.

This pull request removes 12 tests. ``` io.camunda.operate.webapp.security.CsrfRequireMatcherTest ‑ shouldMatchForInternalAPIAccess io.camunda.operate.webapp.security.CsrfRequireMatcherTest ‑ shouldNotMatchForPublicAPIAccessWithBearerToken io.camunda.operate.webapp.security.CsrfRequireMatcherTest ‑ shouldNotMatchForSwagger io.camunda.operate.webapp.security.CsrfRequireMatcherTest ‑ shouldNotMatchHttpMethods io.camunda.operate.webapp.security.CsrfRequireMatcherTest ‑ shouldNotMatchPaths io.camunda.operate.webapp.security.es.CsrfTokenIT ‑ requestToInternalAPIShouldFailWithoutCSRF io.camunda.operate.webapp.security.es.CsrfTokenIT ‑ requestToInternalAPIShouldSucceedWithCSRF io.camunda.operate.webapp.security.es.CsrfTokenIT ‑ requestToPublicAPIShouldSucceedWithoutCSRF io.camunda.operate.webapp.security.sso.CsrfTokenIT ‑ requestToPublicAPIShouldSucceedWithoutCSRF io.camunda.operate.webapp.security.sso.CsrfTokenIT ‑ testAccessNoPermission … ```
Zelldon commented 2 weeks ago

Happy to go with you together over this pr as I don't know any of these modules 🙃😆