Hey Mark, of course. This is not easy to create a unit test for it I think, as it need some guvnor (or some url with auth). Are there some tests which are more like integration tests where I can add it? In the ticket I added a main class where I could reproduce it directly using the same java.net.* classes.

https://issues.jboss.org/secure/attachment/12364295/UrlTest.java

Thanks,

Demian




On Mon, Sep 9, 2013 at 1:39 AM, Mark Proctor <mproctor@codehaus.org> wrote:
We don't  normally merge fixes, without unit tests to catch regressions - can you add a test?

Mark
On 25 Aug 2013, at 17:25, Demian Calcaprina <calcacuervo@gmail.com> wrote:

Hey Guys.Yesterday I found an issue in drools 5.5. Basically, the ResourceChangeScanner is removing my URL resource from the KnowledgeAgent.

I created a ticket with the description, the cause and a pull request.


It seems to be quite critical, so you may want to merge to 5.6 branch.

Thanks!

Demian
_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev


_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev