You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't understand why the self_factor.contradicts(other_factor) call didn't have a context param. And I don't exactly understand how the all_generic_factors_match method is needed. Adding the missing context param fixed the broken test test_consistent_factor_groups_with_context, but I don't understand why the all_generic_factors_match doesn't make the context param redundant.
The text was updated successfully, but these errors were encountered:
mscarey
changed the title
add test coverage for Procedure.consistent_with
add test coverage for ComparableGroup.consistent_with
Aug 27, 2020
I don't understand why the
self_factor.contradicts(other_factor)
call didn't have a context param. And I don't exactly understand how theall_generic_factors_match
method is needed. Adding the missing context param fixed the broken testtest_consistent_factor_groups_with_context
, but I don't understand why theall_generic_factors_match
doesn't make the context param redundant.The text was updated successfully, but these errors were encountered: