Skip to content

Change specs for label inference processor

Important

If we want this merged, please merge this MR before !1340 (closed).

Context

This is the follow-up MR of !1340 (closed). In !1340 (closed), we introduce a new version for the label inference processor, without changing the specs.

The specs depend quite heavily on a few data structures which are not used in the proposed implementation (e.g. LABELS_TAXONOMY).

What does this MR do?

  • Removes the dependency on data structures from the original processor implementation.
  • Removes the data structures from the new implementation.

How to know if it works

I did a diff between the output of the following commands:

rspec spec/processor/label_inference_spec.rb --seed 341

Once I ran it from master, and the other one from this branch.

The diff

Screenshot_2022-04-28_at_14.43.52

Edited by David Dieulivol

Merge request reports