Skip to content

Conversation

@wernerlewis
Copy link
Contributor

Description

When connecting with J-Link, some files we rely on for identifying the board may not be present. This adds parsers for Segger.html and Board.html files, and extracts the type of J-Link in use and the slug from the manufacturer's board URL. This slug will not always correspond to the Mbed slug, so an additional function is added to search for the matching board. This attempts to find a board in the database which matches any of slug, board_name or board_type with the extracted slug.

Test Coverage

  • This change is covered by existing or additional automated tests.
  • Manual testing has been performed (and evidence provided) as automated testing was not feasible.
  • Additional tests are not required for this change (e.g. documentation update).
@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #247 (21221a4) into master (95ca591) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #247 +/- ## ========================================== + Coverage 97.03% 97.07% +0.04%  ========================================== Files 92 92 Lines 2729 2768 +39 ========================================== + Hits 2648 2687 +39  Misses 81 81 
Impacted Files Coverage Δ
src/mbed_tools/targets/__init__.py 100.00% <ø> (ø)
src/mbed_tools/devices/_internal/file_parser.py 100.00% <100.00%> (ø)
src/mbed_tools/devices/_internal/resolve_board.py 100.00% <100.00%> (ø)
src/mbed_tools/targets/get_board.py 100.00% <100.00%> (ø)
Copy link
Contributor

@rwalton-arm rwalton-arm left a comment

Choose a reason for hiding this comment

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

LGTM

When using J-Link, the only available board data may be the product page URL. The URL's slug may indicate the board, but not match the Mbed database slug, so this is searched against Mbed slug, board_type and board_name in the database to try to find a match.
@Patater Patater merged commit 6d92773 into ARMmbed:master Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants