Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

Enable coverage report in unit tests #1175

Merged
merged 4 commits into from
Aug 25, 2020
Merged

Enable coverage report in unit tests #1175

merged 4 commits into from
Aug 25, 2020

Conversation

tfujiwar
Copy link
Contributor

No description provided.

@blueoil-butler blueoil-butler bot added the CI: auto-run Run CI automatically label Aug 24, 2020
@bo-code-review-bot
Copy link

This PR needs Approvals as follows.

  • Ownership Approval for / from iizukak, tkng, ruimashita

Please choose reviewers and requet reviews!

Click to see how to approve each reviews

You can approve this PR by triggered comments as follows.

  • Approve all reviews requested to you (readability and ownership) and LGTM review
    Approval, LGTM

  • Approve all ownership reviews
    Ownership Approval or OA

  • Approve all readability reviews
    Readability Approval or RA

  • Approve specified review targets

    • Example of Ownership Reviewer of /: Ownership Approval for / or OA for /
    • Example of Readability Reviewer of Python: Readability Approval for Python or RA for Python
  • Approve LGTM review
    LGTM

See all trigger comments

Please replace [Target] to review target

  • Ownership Approval
    • Ownership Approval for [Target]
    • OA for [Target]
    • Ownership Approval
    • OA
    • Approval
  • Readability Approval
    • Readability Approval for [Target]
    • RA for [Target]
    • [Target] Readability Approval
    • [Target] RA
    • Readability Approval
    • RA
    • Approval
  • LGTM
    • LGTM
    • lgtm

@tfujiwar
Copy link
Contributor Author

Coverage report will be output after unit tests like this.

Name                                               Stmts   Miss  Cover   Missing
--------------------------------------------------------------------------------
blueoil/converter/__init__.py                          2      0   100%
blueoil/converter/code_generator.py                   85      0   100%
blueoil/converter/core/__init__.py                     0      0   100%
blueoil/converter/core/config.py                      29      3    90%   49, 53, 57
blueoil/converter/core/data_types.py                 136     43    68%   22, 28, 32, 36, 40, 44, 53, 59, 65, 69, 75, 81, 85, 91, 95, 101, 105, 111, 115, 135, 151, 155, 161, 165, 171, 175, 181, 185, 201, 205, 211, 217, 221, 227, 233, 237, 243, 249, 255, 261, 265, 281, 285
...

@tfujiwar tfujiwar requested a review from iizukak August 24, 2020 09:47
Copy link
Member

@iizukak iizukak left a comment

Choose a reason for hiding this comment

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

OA

@tfujiwar
Copy link
Contributor Author

/ready

@bo-mergebot
Copy link
Contributor

bo-mergebot bot commented Aug 25, 2020

⏳Merge job is queued...

@bo-mergebot bo-mergebot bot merged commit 58fc5ad into blue-oil:master Aug 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI: auto-run Run CI automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants