test: update integration tests to check non-public schemas #403

vjeeva · 2024-02-27T00:42:38Z

This includes a big refactor on the integration test, to make it modular for multiplexing the workflow against various test cases.

This PR is missing the following, which may be just filed as separate issues:

Parallelization of the integration tests - this needs some thought (especially on how to collect all the output in a legible way)
Finishing up the Exodus tests - they won't pass in the current state because we do a total dump comparison between the source and destination. Exodus implies that only a subset of data gets migrated, so the test needs to support that.

autodesk-chorus · 2024-02-27T00:42:55Z

Chorus detected one or more security issues with this pull request. See the Checks tab for more details.

As a reminder, please follow the secure code review process as part of the Secure Coding Non-Negotiable requirement.

vjeeva · 2024-02-27T00:43:43Z

Related to #398 and #397

vjeeva · 2024-02-27T17:10:15Z

#405 #404 were filed to address later.

vjeeva force-pushed the vjeeva/non-public-schemas-integration branch from cbd121f to 2850890 Compare February 27, 2024 00:43

vjeeva force-pushed the vjeeva/non-public-schemas-integration branch from 2850890 to 6623ece Compare February 27, 2024 15:36

jdcallet approved these changes Feb 27, 2024

View reviewed changes

vjeeva force-pushed the vjeeva/non-public-schemas-integration branch from 6623ece to 58d441a Compare February 27, 2024 16:59

test: update integration tests to check non-public schemas

ddf3351

vjeeva force-pushed the vjeeva/non-public-schemas-integration branch from 58d441a to ddf3351 Compare February 27, 2024 17:05

vjeeva merged commit 3ec6499 into main Feb 27, 2024
11 checks passed

vjeeva deleted the vjeeva/non-public-schemas-integration branch February 27, 2024 17:10

github-actions bot locked and limited conversation to collaborators Feb 27, 2024

Provide feedback