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 <mproc...@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 <calcacue...@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. > > https://issues.jboss.org/browse/DROOLS-240 > > 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 >
_______________________________________________ rules-dev mailing list rules-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-dev
