Skip to content

Cellular: Fix Greentea cellular power test #8236

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

Closed
wants to merge 1 commit into from

Conversation

AriParkkila
Copy link

Description

Fixed Greentea cellular power test, that was missing some include headers.

Arm internal ref IOTCELL-1417.

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

@AriParkkila
Copy link
Author

@jarvte please review

@AnttiKauppila
Copy link

@0xc0170 Can you trigger a build for this?

@NirSonnenschein
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 7, 2018

Build : FAILURE

Build number : 3269
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8236/

@NirSonnenschein
Copy link
Contributor

@AriParkkila there seems to be a build issue (probably just a configuration issue):
[Error] main.cpp@27,0: #35: #error directive: [NOT_SUPPORTED] CELLULAR_DEVICE must be defined
[DEBUG] Return: 1
[DEBUG] Output: "./features/cellular/TESTS/api/cellular_information/main.cpp", line 27: Error: #35: #error directive: [NOT_SUPPORTED] CELLULAR_DEVICE must be defined
[DEBUG] Output: ./features/cellular/TESTS/api/cellular_information/main.cpp: 0 warnings, 1 error
Building project udp (K64F, ARM)
Please take a look

@AriParkkila
Copy link
Author

We have excluded tests from Greentea by #error [NOT_SUPPORTED] Something more must be defined to run this test case. I guess that's not correct way anymore?

@AriParkkila
Copy link
Author

Include headers added already in #7304

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.

8 participants