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

IAR: Moving test source files to fix error introduced with 02513e80353… #1097

Merged
merged 1 commit into from Jun 5, 2017

Conversation

dangu
Copy link

@dangu dangu commented May 31, 2017

Pull request #1059 broke the IAR build as the test source files were moved. Now the IAR project reflect these changes.

Also increasing the virtual flash size to make room for more tests in the IAR environment.

…ll request cpputest#1059)

There is a problem with dependencies: "AllTests.o" will be different between CppUTestTest and CppUTestExtTest, which is not handled correctly in IAR Embedded Workbench 7.70.1. A full rebuild is needed when trying to run the tests, as the last built AllTests.o is used when linking regardless of which project it was built for.

Interestingly enough, this problem only occurs within Embedded Workbench. When running the scripted tests from within platforms\iar\settings there are no problems. (If the scripts are missing, try to open each test project from within Embedded Workbench, then they should be created)
@dangu dangu changed the title Moving test source files to fix error introduced with 02513e80353… IAR: Moving test source files to fix error introduced with 02513e80353… May 31, 2017
@dangu dangu mentioned this pull request May 31, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.876% when pulling 0fbbe4a on dangu:dev/move_test_files into d0ce532 on cpputest:master.

@basvodde basvodde merged commit 69d1b24 into cpputest:master Jun 5, 2017
@basvodde
Copy link
Member

basvodde commented Jun 5, 2017

Thanks!

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