Skip to content

Conversation

@tony-josi-aws
Copy link
Member

@tony-josi-aws tony-josi-aws commented Apr 2, 2025

Description

Update scripts to run unit tests with respect to Ubuntu 24.04

Test Steps

Tested on Ubuntu 24.04

Checklist:

  • I have tested my changes. No regression in existing tests.
  • [ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@tony-josi-aws tony-josi-aws requested a review from a team as a code owner April 2, 2025 07:32
@tony-josi-aws tony-josi-aws changed the title Update scripts to run unit tests with respect to Ubunut 24.04 Update scripts to run unit tests with respect to Ubuntu 24.04 Apr 2, 2025
amazonKamath
amazonKamath previously approved these changes Apr 2, 2025
aggarg
aggarg previously approved these changes Apr 2, 2025
FreeRTOS_UDP_IPv4.c | 100% 128| 100% 2| 100% 84
FreeRTOS_UDP_IPv6.c | 100% 143| 100% 4| 100% 84
=====================================================================
Total:| 100% 9388| 100% 525| 100% 4920
Copy link
Member

Choose a reason for hiding this comment

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

Did line coverage really increase that much? If so why?

Copy link
Member Author

Choose a reason for hiding this comment

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

The line, branch coverages are 100% from V3.0.0 released on Aug 10, 2022.
It's just that the sample output examples for the unit test readme scripts were not updated, as those are just for reference.

kstribrnAmzn
kstribrnAmzn previously approved these changes Apr 3, 2025
ActoryOu
ActoryOu previously approved these changes Apr 7, 2025
@tony-josi-aws tony-josi-aws merged commit 5cd9e83 into FreeRTOS:main Apr 7, 2025
10 checks passed
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.

7 participants