Closed
Conversation
Contributor
Author
|
I’ve decided to close it for now, as I need more time to better understand the build process and CI configuration with GitHub Actions. I’ll revisit and update this contribution once I have a clearer understanding. Thanks for your feedback and patience! |
Owner
|
No worries, Let me know if i can help. It hasn't been updated in a while I appreciate the support . |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request: Fix PlatformIO Native Build and Improve Context Tests
Summary
This pull request addresses build issues with the PlatformIO native environment and improves the reliability and clarity of context reset tests in the ArduinoFake project.
Changes
PlatformIO Build Fix
[env:native]inplatformio.inito add-Wa,-mbig-objtobuild_flags, resolving "too many sections" and "file too big" errors on Windows with large test binaries.Test Main Improvements
tearDown()function stub totest/main.cppfor completeness.main()to properly return the result ofUNITY_END().Context Test Enhancements
test/test_context.h, improved the test for context reset:ArduinoFakeInstancesto ensure the reset is effective.Example (from context test)
Note:
Previous CI testing on Ubuntu did not reveal the problem in
test_context, as the issue with pointer reuse and large object files is specific to the Windows toolchain and memory allocator behavior. The new changes ensure the tests are robust and portable across both Ubuntu and Windows environments.