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

FSFAT_SDCARD_INSTALLED - Accepted from mbed_app.json only #67

Merged
merged 2 commits into from
Nov 14, 2017
Merged

FSFAT_SDCARD_INSTALLED - Accepted from mbed_app.json only #67

merged 2 commits into from
Nov 14, 2017

Conversation

deepikabhavnani
Copy link

@deepikabhavnani deepikabhavnani commented Nov 14, 2017

FSFAT_SDCARD_INSTALLED if present in mbed_lib.json which is default, is not accepted by tests.
Allowing FSFAT_SDCARD_INSTALLED define from mbed_lib.json and mbed_app.json files.

Changing erase command to trim in block device test

Issue was seen in #65

Deepika added 2 commits November 14, 2017 09:47
FSFAT_SDCARD_INSTALLED if present in SD library no test was actually tested.
Using MBED_CONF_APP_FSFAT_SDCARD_INSTALLED or MBED_CONF_SD_FSFAT_SDCARD_INSTALLED,
to allow this define to be used from app and lib json files.
@deepikabhavnani deepikabhavnani requested a review from geky November 14, 2017 15:58
Copy link
Contributor

@geky geky left a comment

Choose a reason for hiding this comment

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

You could also use the "macro_name" attribute in the lib file to set the config option to whatever name you want.

Although this makes a lot more sense 👍

@deepikabhavnani deepikabhavnani merged commit a601d85 into PelionIoT:master Nov 14, 2017
@deepikabhavnani deepikabhavnani deleted the test_checks branch November 14, 2017 22:53
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.

2 participants