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

[WIP, DNM] Adding test scripts #32

Open
wants to merge 67 commits into
base: master
Choose a base branch
from
Open

Conversation

mlaz
Copy link
Contributor

@mlaz mlaz commented Oct 4, 2019

Tests that should cover most of the functionality.
Depends on #27

vrahane and others added 30 commits September 27, 2019 17:50
- Make sure it builds and works
- Make stat_mgmt buildable
- Some fixes to img_mgmt
- Update mynewt port wrt features from mynewt-core
- For flash functions define 'impl' functions
- Make sure everything builds
- Read UICR to check for a configured address before advertising
- Remove the ANS service UUID so that we do not connect to devices
running blecent
- Add capability to talk to the omp_svr over BLE and shell
- Enabled GATT transport in OIC to talk to the device using BLE
mlaz and others added 23 commits September 27, 2019 17:50
- Earlier the entries were read one entry at a time and that is
different what we have currently in mynewt, this needs to change
so that we read logs hcunk by chunk until we reach the entry length
- The encode funtion needs to get fixed which will be in a subsequent
PR
- Log entries get read chunk by chunk and should get encoded at the end
- Earlier while reading logs were not chunked and now they are chunked,
  syscfgs have to match that functional approach
Closing CBOR containers on error situations before returning.
- entries were encoded together earlier which caused issues, encoding
them chunk by chunk fixes it.
- Move config headers to include directory and general clean up
  and address review comments
- mbuf APIs and structs containing mbuf ptrs need to be
  MYNEWT specific
cmd/fs fixing package dependency chain
@mlaz
Copy link
Contributor Author

mlaz commented Jan 30, 2020

This is being done here: #63

@mlaz mlaz closed this Jan 30, 2020
@utzig
Copy link
Member

utzig commented Jan 30, 2020

This is being done here: #63

I don't think those have the same purpose, I am just setting up a build of the smp_svr bundled on Zephyr to assure changes in mcumgr do not break it. No testing is being worked on at least for now.

@mlaz
Copy link
Contributor Author

mlaz commented Jan 30, 2020

This is being done here: #63

I don't think those have the same purpose, I am just setting up a build of the smp_svr bundled on Zephyr to assure changes in mcumgr do not break it. No testing is being worked on at least for now.

Ok reopening it. I'll have to look what will your PR comprise in order to see up to which point I'll add up tests to it or do things separately.

@mlaz
Copy link
Contributor Author

mlaz commented Jan 30, 2020

This is being done here: #63

I don't think those have the same purpose, I am just setting up a build of the smp_svr bundled on Zephyr to assure changes in mcumgr do not break it. No testing is being worked on at least for now.

Ok reopening it. I'll have to look what will your PR comprise in order to see up to which point I'll add up tests to it or do things separately.

@mlaz mlaz reopened this Jan 30, 2020
@utzig
Copy link
Member

utzig commented Jan 30, 2020

Also rebase on latest master!

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.

3 participants