You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jmeter.apache.org by ham1 <gi...@git.apache.org> on 2017/11/21 02:04:20 UTC

[GitHub] jmeter pull request #334: Enable running of JUnit tests from within IntelliJ...

GitHub user ham1 opened a pull request:

    https://github.com/apache/jmeter/pull/334

    Enable running of JUnit tests from within IntelliJ with default config

    ## Description
    With the default IntelliJ config tests are not run in the `bin` or project root folder which meant tests didn't run without a custom run setting.
    
    ## Motivation and Context
    Makes running tests far easier in IntelliJ.
    
    ## How Has This Been Tested?
    Running a sample of unit tests in IntelliJ
    
    ## Screenshots (if appropriate):
    
    ## Types of changes
    - Dev enhancement
     
    ## Checklist:
    <!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
    <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
    - [x] My code follows the [code style][style-guide] of this project.
    - [x] I have updated the documentation accordingly.
    
    [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ham1/jmeter run-test-intellij

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jmeter/pull/334.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #334
    
----
commit 57c8c3871d9eb7158a1c06d3e81c1917dd85357d
Author: Graham Russell <gr...@ham1.co.uk>
Date:   2017-11-21T01:56:56Z

    enabled running of JUnit tests from within IntelliJ with default config

----


---

[GitHub] jmeter pull request #334: Enable running of JUnit tests from within IntelliJ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/jmeter/pull/334


---

[GitHub] jmeter issue #334: Enable running of JUnit tests from within IntelliJ with d...

Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:

    https://github.com/apache/jmeter/pull/334
  
    # [Codecov](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=h1) Report
    > Merging [#334](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=desc) into [trunk](https://codecov.io/gh/apache/jmeter/commit/3a60f2b80633727093c59ce788912819aeca3c81?src=pr&el=desc) will **decrease** coverage by `<.01%`.
    > The diff coverage is `0%`.
    
    [![Impacted file tree graph](https://codecov.io/gh/apache/jmeter/pull/334/graphs/tree.svg?width=650&token=6Q7CI1wFSh&height=150&src=pr)](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree)
    
    ```diff
    @@             Coverage Diff              @@
    ##              trunk     #334      +/-   ##
    ============================================
    - Coverage     57.93%   57.93%   -0.01%     
    - Complexity    10025    10026       +1     
    ============================================
      Files          1148     1148              
      Lines         73748    73754       +6     
      Branches       7321     7323       +2     
    ============================================
      Hits          42728    42728              
    - Misses        28539    28546       +7     
    + Partials       2481     2480       -1
    ```
    
    
    | [Impacted Files](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
    |---|---|---|---|
    | [...st/src/org/apache/jmeter/junit/JMeterTestCase.java](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree#diff-dGVzdC9zcmMvb3JnL2FwYWNoZS9qbWV0ZXIvanVuaXQvSk1ldGVyVGVzdENhc2UuamF2YQ==) | `30.86% <0%> (-1.19%)` | `10 <0> (ø)` | |
    | [...c/org/apache/jmeter/junit/JMeterTestCaseJUnit.java](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree#diff-dGVzdC9zcmMvb3JnL2FwYWNoZS9qbWV0ZXIvanVuaXQvSk1ldGVyVGVzdENhc2VKVW5pdC5qYXZh) | `9.67% <0%> (-0.5%)` | `3 <0> (ø)` | |
    | [...ocol/jms/org/apache/jmeter/protocol/jms/Utils.java](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree#diff-c3JjL3Byb3RvY29sL2ptcy9vcmcvYXBhY2hlL2ptZXRlci9wcm90b2NvbC9qbXMvVXRpbHMuamF2YQ==) | `54.83% <0%> (-4.31%)` | `15% <0%> (ø)` | |
    | [...apache/jmeter/extractor/TestBoundaryExtractor.java](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree#diff-dGVzdC9zcmMvb3JnL2FwYWNoZS9qbWV0ZXIvZXh0cmFjdG9yL1Rlc3RCb3VuZGFyeUV4dHJhY3Rvci5qYXZh) | `97.88% <0%> (ø)` | `17% <0%> (ø)` | :arrow_down: |
    | [...mpler/hc/JMeterPoolingClientConnectionManager.java](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=tree#diff-c3JjL3Byb3RvY29sL2h0dHAvb3JnL2FwYWNoZS9qbWV0ZXIvcHJvdG9jb2wvaHR0cC9zYW1wbGVyL2hjL0pNZXRlclBvb2xpbmdDbGllbnRDb25uZWN0aW9uTWFuYWdlci5qYXZh) | `30.95% <0%> (+3.17%)` | `11% <0%> (+1%)` | :arrow_up: |
    
    ------
    
    [Continue to review full report at Codecov](https://codecov.io/gh/apache/jmeter/pull/334?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/334?src=pr&el=footer). Last update [3a60f2b...e625f95](https://codecov.io/gh/apache/jmeter/pull/334?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---