Follow-up to #2773099: Add tests for base fields of revisonable entity

We only have upgrade path tests basefield for non-revisonable entities right now. Add upgrade path test for basefield for revisonable entities with and without data table. Also see #2772611-19: target_id_int column doesn't exist for field revision table

CommentFileSizeAuthor
#5 add_upgrade_path_tests-2795247-5.patch125.77 KBjibran
#5 interdiff.txt19.31 KBjibran
#3 add_upgrade_path_tests-2795247-3.patch106.71 KBjibran

Comments

jibran created an issue. See original summary.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Version: 8.x-1.x-dev » 8.x-2.x-dev
jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Active » Needs review
StatusFileSize
new add_upgrade_path_tests-2795247-3.patch106.71 KB

Here is the patch and DB with the sample file.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented

I'll write tests in the morning.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
StatusFileSize
new interdiff.txt19.31 KB
new add_upgrade_path_tests-2795247-5.patch125.77 KB

Here is the test and added some tweaks to the script and added some assert to an existing test as well.

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

  • jibran committed 3c4c8ff on 8.x-2.x 
    Issue #2795247 by jibran: Add upgrade path tests for base fields of...
jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Reviewed & tested by the community » Fixed

Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.