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

test(cat-gateway): Setup Scylla testing environment #1406

Merged
merged 19 commits into from
Dec 23, 2024

Conversation

apskhem
Copy link
Collaborator

@apskhem apskhem commented Dec 17, 2024

Description

Create a testing evironment to add integration test to test with EventDB.

Related Issue(s)

Closes #1405

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

@apskhem apskhem self-assigned this Dec 17, 2024
@apskhem apskhem linked an issue Dec 17, 2024 that may be closed by this pull request
@apskhem apskhem added the draft Draft label Dec 17, 2024
@apskhem apskhem marked this pull request as draft December 17, 2024 09:43
Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@apskhem apskhem force-pushed the test/setup-event-db-environment branch from 9f603f3 to 2bb6b4d Compare December 19, 2024 08:55
@apskhem apskhem force-pushed the test/setup-event-db-environment branch from 9f7d5bd to e3754b2 Compare December 19, 2024 13:50
Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@apskhem apskhem force-pushed the test/setup-event-db-environment branch from 324f2b5 to e3754b2 Compare December 19, 2024 15:06
Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@apskhem apskhem changed the base branch from main to test/indexdb-queries December 20, 2024 14:38
@apskhem apskhem added review me PR is ready for review and removed draft Draft labels Dec 20, 2024
@apskhem apskhem marked this pull request as ready for review December 20, 2024 14:39
Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

catalyst-gateway/Earthfile Outdated Show resolved Hide resolved
catalyst-gateway/Earthfile Outdated Show resolved Hide resolved
Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

@apskhem apskhem requested a review from kukkok3 December 23, 2024 13:31
Copy link
Contributor

@kukkok3 kukkok3 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

Test Report | ${\color{lightgreen}Pass: 334/334}$ | ${\color{red}Fail: 0/334}$ |

Copy link
Contributor

@saibatizoku saibatizoku left a comment

Choose a reason for hiding this comment

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

Looks good!

@apskhem apskhem merged commit e22210e into test/indexdb-queries Dec 23, 2024
41 checks passed
@apskhem apskhem deleted the test/setup-event-db-environment branch December 23, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review me PR is ready for review
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

🛠️ [TASK] : Setup Scylla testing environment
4 participants