ICRAR / ijson

100%

master: 100%

LAST BUILD ON BRANCH python-parse-value-clarification

branch: python-parse-value-clarification

CHANGE BRANCH

x

Reset

• python-parse-value-clarification
• async-fixes
• async-string-stream
• async_c
• bytes-prefix
• check-3.1.2.post0
• common_functions
• coros
• coverage-c
• coverage-increase
• decimal-c
• decoding-errors
• default_backend
• design_notes
• develop
• document-event-interception
• dump
• empty-members
• fixes-for-known-problems
• gh-actions
• ijson-dumps-multiple
• improved-readme
• issue-41
• issue-42
• issue-66
• items-and-kvitems-dont-join-path
• items_kwargs
• kvitems
• map_type
• master
• memory-leak
• more_tests
• new-devels
• no-tests-in-binary
• python-3.10
• python_multiple_values
• reader_conversion
• release-3.2.0
• rtobar-patch-1
• test
• test-comments
• test-module-rename
• test-wheels-on-travis
• toplevel_array
• travis-c-ext
• travis-more-python
• travis-xenial
• update-travis
• upload-check
• use_float
• v2.5
• v2.5.1
• v2.6.0
• v2.6.1
• v3.0
• v3.0.1
• v3.0.2
• v3.0.3
• v3.0.4
• v3.0rc1
• v3.0rc2
• v3.0rc3
• v3.1
• v3.1.1
• v3.1.2
• v3.1.2.post0
• v3.1.3
• v3.1.4
• v3.1.post0
• windows-wheels
• yajl109_test
• yajl2_c_parse_memleak
• zeros-python

Build # 230

Build Type

push

travis-ci-com

Committed by rtobar

Commit Message

Clarify parse_value's state_stack's state

The previous version of this code had 3 lines missing from the coverage.
These lines were missing because they were after an "if state_stack"
condition, which I initially thought could be true or false depending on
the program execution. Extensive testing has showed us that these lines
are never touched, and closer code inspection also reveals that the
state_stack list is never empty before entering into the main "if/elif"
control block, as this is ensured in the previous try/catch block.

This commit therefore removes the conditions resulting in missing lines,
and instead adds an assert expression after the try/catch block to show
more explicitly the state of the execution.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

Run Details

1517 of 1522 relevant lines covered (99.67%)

367.88 hits per line