Skip to content

[SELC-6472] feat: add integration tests with github actions#80

Merged
duffizipierluigi merged 8 commits intomainfrom
feature/SELC-6472
May 26, 2025
Merged

[SELC-6472] feat: add integration tests with github actions#80
duffizipierluigi merged 8 commits intomainfrom
feature/SELC-6472

Conversation

@duffizipierluigi
Copy link
Collaborator

@duffizipierluigi duffizipierluigi commented May 21, 2025

List of Changes

  • Added github workflow integration_tests_ms
  • Added mock products.json (removing access to private repository)
  • Integrated testcontainers to load automatically the docker-compose.yml
  • Cleanup of properties

Motivation and Context

Automate integration tests

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@duffizipierluigi duffizipierluigi requested review from a team and giulia-tremolada as code owners May 21, 2025 12:42
@github-actions github-actions bot added the ops label May 21, 2025
@duffizipierluigi duffizipierluigi marked this pull request as draft May 22, 2025 08:18
@duffizipierluigi duffizipierluigi marked this pull request as ready for review May 26, 2025 09:56
@sonarqubecloud
Copy link

Copy link
Contributor

@gaetano-miglionico gaetano-miglionico left a comment

Choose a reason for hiding this comment

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

lgtm

@duffizipierluigi duffizipierluigi merged commit 2f57789 into main May 26, 2025
11 of 13 checks passed
@duffizipierluigi duffizipierluigi deleted the feature/SELC-6472 branch May 26, 2025 14: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.

2 participants