Skip to content

Add test to demonstrate that AVET vs EAVT inconsistency has been solved#782

Open
jonasseglare wants to merge 2 commits intomainfrom
validate-avet-vs-eavt2
Open

Add test to demonstrate that AVET vs EAVT inconsistency has been solved#782
jonasseglare wants to merge 2 commits intomainfrom
validate-avet-vs-eavt2

Conversation

@jonasseglare
Copy link
Collaborator

This MR adds an additional unit test to verify that the PR

#781

solved the EAVT vs AVET issue. If this unit test would be rebased on the commit before that PR, it will fail as follows:

Fail in test-load-taxonomy-data0

expected: (= eavt-sample avet-sample)
  actual: (not          
           (=
            #{[52 :relation/concept-1 48 536870921 true]}
            #{[49 :relation/concept-1 48 536870917 true]
              [52 :relation/concept-1 48 536870921 true]}))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant