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

Revamp tests #350

Merged
merged 20 commits into from
Jan 22, 2025
Merged

Revamp tests #350

merged 20 commits into from
Jan 22, 2025

Conversation

uglide
Copy link
Contributor

@uglide uglide commented Nov 29, 2024

  • Use endpoints.json instead of hardcoded configs
  • Use client-libs-test image
  • Add run-tests action
  • Use [Theory] based approach to run tests against multiple endpoints
  • Remove old config and env files

@uglide uglide force-pushed the revamp_tests branch 2 times, most recently from a60ad3e to 0f6c630 Compare November 29, 2024 15:19
@uglide uglide force-pushed the revamp_tests branch 2 times, most recently from f14b0a3 to 78f1d22 Compare January 13, 2025 15:36
- Use endpoints.json instead of hardcoded configs
- Use client-libs-test image
- Add run-tests action
- Use [Theory] based approach to run tests against multiple endpoints
- Remove old config and env files
- Use stack-based images
- Use xunit.skippablefact
@uglide uglide marked this pull request as ready for review January 22, 2025 07:00
@uglide uglide requested a review from atakavci January 22, 2025 07:00
.github/workflows/integration.yml Outdated Show resolved Hide resolved
.github/workflows/integration.yml Show resolved Hide resolved
Copy link
Collaborator

@atakavci atakavci left a comment

Choose a reason for hiding this comment

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

LGTM

@atakavci atakavci merged commit 26a1d0e into master Jan 22, 2025
15 checks passed
@atakavci atakavci deleted the revamp_tests branch January 22, 2025 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants