Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUN-2196: Update API tests harness project import #8888

Merged
merged 8 commits into from Feb 16, 2024

Conversation

gschueler
Copy link
Member

Is this a bugfix, or an enhancement? Please describe.
Updates API/selenium tests to allow importing test project content as a directory instead of an opaque jar

Describe the solution you've implemented

  • replace opaque jars in git with the expanded versions
  • update test classes to import the dirs

Describe alternatives you've considered

Additional context
Opaque jars make it hard to create or modify test content

@gschueler gschueler changed the title [wip] Update API tests harness project import Update API tests harness project import Feb 6, 2024
@gschueler gschueler marked this pull request as ready for review February 6, 2024 22:35
@gschueler gschueler changed the title Update API tests harness project import RUN-2196: Update API tests harness project import Feb 6, 2024
@ronaveva ronaveva self-requested a review February 9, 2024 17:52
Copy link
Contributor

@ronaveva ronaveva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gschueler gschueler force-pushed the exp/api-test-import-project-dir branch from a76ba03 to 081534f Compare February 16, 2024 17:28
@gschueler gschueler added this to the 5.2.0 milestone Feb 16, 2024
@gschueler gschueler force-pushed the exp/api-test-import-project-dir branch from 081534f to a3827c8 Compare February 16, 2024 17:31
@gschueler gschueler merged commit c033c54 into main Feb 16, 2024
3 checks passed
@gschueler gschueler deleted the exp/api-test-import-project-dir branch February 16, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants