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

UnitTests and Coverage reporting #444

Merged
merged 5 commits into from
May 17, 2024
Merged

UnitTests and Coverage reporting #444

merged 5 commits into from
May 17, 2024

Conversation

joenpera
Copy link
Collaborator

Jira-Id: SECO-4966

@joenpera joenpera marked this pull request as draft April 25, 2024 08:36
@joenpera joenpera self-assigned this Apr 25, 2024
@joenpera joenpera force-pushed the unittest_impl branch 5 times, most recently from 9e568c2 to cd61f0a Compare April 29, 2024 06:45
4966 - Comms Service Discovery
4967 - Comms Settings
4968 - Comms Status

Jira-Id: SECO-4966, SECO-4967, SECO-4968

Signed-off-by: Mika Joenpera <mika.joenpera@unikie.com>
joenpera and others added 2 commits May 3, 2024 11:56
Develop to working branch
Jira-Id: SECO-4967, SECO-4961

Signed-off-by: Mika Joenpera <mika.joenpera@unikie.com>
@joenpera joenpera marked this pull request as ready for review May 10, 2024 09:32
Copy link
Collaborator

@pentestiing pentestiing left a comment

Choose a reason for hiding this comment

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

Plz also check the mesh_status 📃

- Commented code delete (comms_command.py)
- try/except removal and [] index checks added (comms_settings.py)

Jira-Id: SECO-4966

Signed-off-by: Mika Joenpera <mika.joenpera@unikie.com>
@joenpera joenpera requested a review from pentestiing May 15, 2024 12:49
Jira-Id: SECO-4966

Signed-off-by: Mika Joenpera <mika.joenpera@unikie.com>
Copy link
Collaborator

@pentestiing pentestiing left a comment

Choose a reason for hiding this comment

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

Great party! 🎉

@joenpera joenpera merged commit f8340f6 into develop May 17, 2024
2 checks passed
@joenpera joenpera deleted the unittest_impl branch May 17, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants