Skip to content

Prefer `expect_next_instance_of` over `expect_any_instance_of`

Lin Jen-Shin requested to merge unify-delete_user_worker_spec.rb into master

What does this MR do?

Prefer expect_next_instance_of over expect_any_instance_of to eliminate CE/EE difference

Are there points in the code the reviewer needs to double check?

It might be nice to just allow us to rename it, but that might involve some black magic in expect_next_instance_of and it's probably not worth it.

EE at: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6265

Why was this MR needed?

Eliminate CE/EE difference

Does this MR meet the acceptance criteria?

  • Tests added for this feature/bug
  • Conform by the code review guidelines
    • Has been reviewed by a Backend maintainer

What are the relevant issue numbers?

&27 (closed)

Edited by Lin Jen-Shin

Merge request reports

Loading