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

Add regression tests for config processing #139

Merged

Conversation

rwalton-arm
Copy link
Contributor

@rwalton-arm rwalton-arm commented Dec 1, 2020

Description

Add regression tests at the public interface level so we can rework the
config processing code and have some confidence we haven't broken anything.

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 Dec 1, 2020

Codecov Report

Merging #139 (eb96b2a) into master (e1c5268) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files          95       95           
  Lines        2668     2668           
=======================================
  Hits         2577     2577           
  Misses         91       91           

@rwalton-arm rwalton-arm force-pushed the dev/rwalton-arm/config-tests branch from 08ebba7 to f8b2fe6 Compare December 2, 2020 16:26
@rwalton-arm rwalton-arm force-pushed the dev/rwalton-arm/config-tests branch 9 times, most recently from f81f85f to 53962ab Compare December 9, 2020 15:49
@rwalton-arm rwalton-arm marked this pull request as ready for review December 9, 2020 15:50
@Patater Patater self-requested a review December 10, 2020 12:15
@rwalton-arm rwalton-arm force-pushed the dev/rwalton-arm/config-tests branch from 53962ab to f155626 Compare December 10, 2020 13:20
tests/build/test_generate_config.py Show resolved Hide resolved
tests/build/test_generate_config.py Show resolved Hide resolved
tests/build/test_generate_config.py Show resolved Hide resolved
tests/build/test_generate_config.py Show resolved Hide resolved
@rwalton-arm rwalton-arm force-pushed the dev/rwalton-arm/config-tests branch 3 times, most recently from 6f878d1 to 82572f7 Compare December 11, 2020 13:26
Add regression tests at the public interface level so we can rework the
config processing code and have some confidence we haven't broken anything.
@rwalton-arm rwalton-arm force-pushed the dev/rwalton-arm/config-tests branch from 82572f7 to eb96b2a Compare December 14, 2020 09:46
@rwalton-arm rwalton-arm merged commit b2d8784 into ARMmbed:master Dec 14, 2020
@rwalton-arm rwalton-arm deleted the dev/rwalton-arm/config-tests branch December 14, 2020 10:05
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