This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 254120 - rerun file method not working, it defaults to whole file
Summary: rerun file method not working, it defaults to whole file
Status: VERIFIED FIXED
Alias: None
Product: php
Classification: Unclassified
Component: Codeception (show other bugs)
Version: 8.1
Hardware: All All
: P3 normal (vote)
Assignee: Tomas Mysik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-04 10:27 UTC by dynasource
Modified: 2015-08-18 11:16 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
IDE log (339.58 KB, text/plain)
2015-08-04 10:27 UTC, dynasource 		Details
patch (1.33 KB, patch)
2015-08-14 02:55 UTC, junichi11 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description dynasource 2015-08-04 10:27:39 UTC 
Product Version = NetBeans IDE Dev (Build nbms-and-javadoc-179-on-20150730)
Operating System = Windows 7 version 6.1 running on amd64
Java; VM; Vendor = 1.8.0_45
Runtime = Java HotSpot(TM) 64-Bit Server VM 25.45-b02

When running a focussed test method, everyting works fine.
However, when rerunning this test, it runs the whole file and loosed the focus on one method.
Comment 1 dynasource 2015-08-04 10:27:44 UTC 
Created attachment 155161 [details]
IDE log
Comment 2 dynasource 2015-08-04 10:28:53 UTC 
important to mention: this is about the codeception plugin
Comment 3 junichi11 2015-08-13 11:35:59 UTC 
Thanks for catching it. I'll fix it.
Comment 4 junichi11 2015-08-14 02:55:05 UTC 
Created attachment 155333 [details]
patch

Tomas, I've created a patch. Could you please review it?

Thanks.
Comment 5 Tomas Mysik 2015-08-17 08:04:40 UTC 
Patch applied, thanks a lot for it!

(In reply to junichi11 from comment #4)
> Tomas, I've created a patch. Could you please review it?

I did it but not know Codeception well so cannot say, sorry :) Junichi, please verify that the patch did not break any other Codeception functionality.

Thanks!

http://hg.netbeans.org/web-main/rev/0a9b05e92c3a
Comment 6 junichi11 2015-08-17 11:06:44 UTC 
(In reply to Tomas Mysik from comment #5)
> Patch applied, thanks a lot for it!

Thanks a lot for applying!

> I did it but not know Codeception well so cannot say, sorry :) Junichi,
> please verify that the patch did not break any other Codeception
> functionality.

Of course, It's OK if there are no problems(style, format, e.t.c.) in my code :)
I'll verify that.

Thanks!
Comment 7 junichi11 2015-08-17 11:29:28 UTC 
@dynasource
Could you try to test that if possible? Perhaps, it is available in next nightly build[1] (tomorrow or later).

Thanks.

[1] http://bits.netbeans.org/download/trunk/nightly/latest/
Comment 8 Quality Engineering 2015-08-18 01:17:31 UTC 
Integrated into 'main-silver', will be available in build *201508180002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/0a9b05e92c3a
User: Tomas Mysik <tmysik@netbeans.org>
Log: #254120 - rerun file method not working, it defaults to whole file

Patch contributed by junichi11@netbeans.org.
Comment 9 dynasource 2015-08-18 07:14:37 UTC 
patch is working.
Thank you very much junichi11. Great work.
Comment 10 Tomas Mysik 2015-08-18 08:08:19 UTC 
(In reply to dynasource from comment #9)
> patch is working.

Marking as VERIFIED. Thanks.
Comment 11 junichi11 2015-08-18 11:15:22 UTC 
I also have verified that.

Then, tested the following in a CalculatorSample project:
- Run test
- Debug
- Coverage
- Rerun failed 
- Rerun (all, single and forcused test method)

It works fine.

Thanks!
Comment 12 Tomas Mysik 2015-08-18 11:16:11 UTC 
Thanks a lot, Junichi!