Add ECSql tests for ECDb (#7323) #7583
Conversation
Co-authored-by: soham-bentley <177123878+soham-bentley@users.noreply.github.com> Co-authored-by: RohitPtnkr1996 <111407262+RohitPtnkr1996@users.noreply.github.com> Co-authored-by: naronchen <naronchen@users.noreply.github.com> Co-authored-by: akshay05n <44081595+akshay05n@users.noreply.github.com> Co-authored-by: Paul Connelly <22944042+pmconne@users.noreply.github.com> Co-authored-by: imodeljs-admin <38288322+imodeljs-admin@users.noreply.github.com> (cherry picked from commit 46b679a) # Conflicts: # common/config/rush/pnpm-lock.yaml # core/backend/package.json # test-apps/display-test-app/android/imodeljs-test-app/app/build.gradle # test-apps/display-test-app/ios/imodeljs-test-app/imodeljs-test-app.xcodeproj/project.pbxproj # tools/internal/ios/core-test-runner/core-test-runner.xcodeproj/project.pbxproj
…release/4.11.x/pr-7323 # Conflicts: # core/backend/package.json # test-apps/display-test-app/android/imodeljs-test-app/app/build.gradle # test-apps/display-test-app/ios/imodeljs-test-app/imodeljs-test-app.xcodeproj/project.pbxproj # tools/internal/ios/core-test-runner/core-test-runner.xcodeproj/project.pbxproj
soham-bentley
requested review from
a team,
aruniverse,
calebmshafer,
ColinKerr and
wgoehrig
as code owners
|
Did we decide to backport this? I was under the impression we don’t want to do this. It’s a rather big PR which affects dependencies and builds. Just a week back I told Arun we’d not backport this |
Colin told me to backport it |
We will be back porting future ECDb changes to 4.x so ECDb changes propagate to all supported versions faster. Just like we did with 3.x. I want as much testing on 4.x as possible and new changes in 5.x will likely have tests using this framework |
…release/4.11.x/pr-7323
…release/4.11.x/pr-7323
This is a manual backport of PR #7323 into release/4.11.x
A part of the #7323 PR is backported with another PR #7540
The remaining PR is backported with this PR