<div class="gmail_quote">2012/4/30 Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de" target="_blank">mail@milianw.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Sunday 29 April 2012 20:36:44 Miha Čančula wrote:<br>
> Alright, we have school holidays this week, so that means some extra<br>
> kdevelop time for me. I removed the CTestFinder plugin and merge the<br>
> test-finding code to CMakeManager. This means no manual reloading is<br>
> needed, and tests are reported in the GUI as soon as they are parsed. I'll<br>
> try to do the same for PHP tomorrow.<br>
<br>
</div>Sounds good, thanks. But sadly I cannot try it out, at least one file is<br>
missing in kdevplatform (testcontroller.cpp) - can you please make sure you've<br>
committed all files?<br>
<br>
Btw have you managed to find a solution to the "lets remove/update changed<br>
tests" problem yet?<br></blockquote><div>Oh, hi. I'm sorry for not replying earlier, I didn't see the mail due to a misconfigured Gmail. I found some extra kdevelop time since then, and I think I fixed that missing file as well. <br>
<br>About updating tests: They are updated automatically, whenever the relevant files are reparsed. I have not yet testing this, so I'm not sure if it works or not. If it doesn't yet, I don't think it takes much effort to make it work. I modified the TestController to overwrite the tests by default when a new one with the same name and project is registered. There is also a difference between changing the CMake file and the source where the test class is declared/defined. There is no way yet of knowing when the list of test cases is changed. <br>
<br>Automatic removal is harder. I paid no attention to it yet, as I think it's a lower priority. Note that this includes renamed test suites. The one way I imagine this would work is storing a file with every suite, and removing all suites from each file as it is reparsed. Personally I think this is too much work for the computer, and might be bad for usability if tests temporarily disappear because of a missing semicolon in some file. But if you think it's beneficial I'll do my best to add it. <br>
<br>Another thing I have planned is an option to enable or disable test output. This would allow things like Commit Review to run tests and receive reports without other toolviews popping up. <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div class="im"><br>
> I'm still having trouble with a custom Job subclass, especially because of<br>
> the OutputJob dependency. So I'm thinking it would be better to keep the<br>
> current interface (KJob* launchCase() etc. ) and leave result reporting to<br>
> the ITestController. Except that instead of having a result() method on the<br>
> TestSuite itself, a TestResult argument to the testRunFinished() signal<br>
> could be added.<br>
<br>
</div>I can try to have a look at that.<br>
<br>
Cheers<br>
<div class="HOEnZb"><div class="h5">--<br>
Milian Wolff<br>
<a href="mailto:mail@milianw.de">mail@milianw.de</a><br>
<a href="http://milianw.de" target="_blank">http://milianw.de</a></div></div><br>--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
<br></blockquote></div><br>