You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jmeter.apache.org by codecov-io <gi...@git.apache.org> on 2017/12/12 00:42:23 UTC

[GitHub] jmeter issue #356: Sonar fixes

Github user codecov-io commented on the issue:

    https://github.com/apache/jmeter/pull/356
  
    # [Codecov](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=h1) Report
    > Merging [#356](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=desc) into [trunk](https://codecov.io/gh/apache/jmeter/commit/427907f9d2ca45f8077acf9398a140834d75e3bd?src=pr&el=desc) will **decrease** coverage by `0.01%`.
    > The diff coverage is `37.5%`.
    
    [![Impacted file tree graph](https://codecov.io/gh/apache/jmeter/pull/356/graphs/tree.svg?token=6Q7CI1wFSh&src=pr&height=150&width=650)](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=tree)
    
    ```diff
    @@             Coverage Diff              @@
    ##              trunk     #356      +/-   ##
    ============================================
    - Coverage     58.19%   58.18%   -0.02%     
    + Complexity    10210    10194      -16     
    ============================================
      Files          1161     1161              
      Lines         74189    74151      -38     
      Branches       7322     7319       -3     
    ============================================
    - Hits          43174    43143      -31     
    + Misses        28517    28511       -6     
    + Partials       2498     2497       -1
    ```
    
    
    | [Impacted Files](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
    |---|---|---|---|
    | [...nts/org/apache/jmeter/visualizers/BSFListener.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvdmlzdWFsaXplcnMvQlNGTGlzdGVuZXIuamF2YQ==) | `9.52% <ø> (-4.12%)` | `2 <0> (-1)` | |
    | [...apache/jmeter/modifiers/BeanShellPreProcessor.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvbW9kaWZpZXJzL0JlYW5TaGVsbFByZVByb2Nlc3Nvci5qYXZh) | `18.75% <ø> (-4.78%)` | `3 <0> (-1)` | |
    | [...re/org/apache/jmeter/testbeans/gui/FileEditor.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvcmUvb3JnL2FwYWNoZS9qbWV0ZXIvdGVzdGJlYW5zL2d1aS9GaWxlRWRpdG9yLmphdmE=) | `60.86% <ø> (ø)` | `7 <0> (ø)` | :arrow_down: |
    | [.../org/apache/jmeter/visualizers/JSR223Listener.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvdmlzdWFsaXplcnMvSlNSMjIzTGlzdGVuZXIuamF2YQ==) | `13.33% <ø> (-5.42%)` | `2 <0> (-1)` | |
    | [...org/apache/jmeter/gui/action/SearchTreeDialog.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvcmUvb3JnL2FwYWNoZS9qbWV0ZXIvZ3VpL2FjdGlvbi9TZWFyY2hUcmVlRGlhbG9nLmphdmE=) | `1.86% <ø> (ø)` | `1 <0> (ø)` | :arrow_down: |
    | [...g/apache/jmeter/extractor/JSR223PostProcessor.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvZXh0cmFjdG9yL0pTUjIyM1Bvc3RQcm9jZXNzb3IuamF2YQ==) | `75% <ø> (-2.78%)` | `3 <0> (-1)` | |
    | [...pache/jmeter/extractor/BeanShellPostProcessor.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvZXh0cmFjdG9yL0JlYW5TaGVsbFBvc3RQcm9jZXNzb3IuamF2YQ==) | `16.66% <ø> (-4.39%)` | `3 <0> (-1)` | |
    | [...nents/org/apache/jmeter/timers/BeanShellTimer.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvbXBvbmVudHMvb3JnL2FwYWNoZS9qbWV0ZXIvdGltZXJzL0JlYW5TaGVsbFRpbWVyLmphdmE=) | `47.36% <ø> (-2.64%)` | `4 <0> (-1)` | |
    | [...rc/core/org/apache/jmeter/services/FileServer.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvcmUvb3JnL2FwYWNoZS9qbWV0ZXIvc2VydmljZXMvRmlsZVNlcnZlci5qYXZh) | `71.13% <ø> (ø)` | `56 <0> (ø)` | :arrow_down: |
    | [...g/apache/jmeter/samplers/StandardSampleSender.java](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree#diff-c3JjL2NvcmUvb3JnL2FwYWNoZS9qbWV0ZXIvc2FtcGxlcnMvU3RhbmRhcmRTYW1wbGVTZW5kZXIuamF2YQ==) | `75% <ø> (ø)` | `6 <0> (ø)` | :arrow_down: |
    | ... and [41 more](https://codecov.io/gh/apache/jmeter/pull/356/diff?src=pr&el=tree-more) | |
    
    ------
    
    [Continue to review full report at Codecov](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=continue).
    > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
    > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
    > Powered by [Codecov](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=footer). Last update [427907f...5378f28](https://codecov.io/gh/apache/jmeter/pull/356?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---