any23-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lewismc <>
Subject [GitHub] any23 pull request: Initial move towards addressing ANY23-280 Refa...
Date Wed, 06 Apr 2016 19:50:17 GMT
GitHub user lewismc opened a pull request:

    Initial move towards addressing ANY23-280 Refactor ContentExtractor to improve extraction

    Hi Folks,
    This is an initial crack at addressing
    Essentially, the main API difference is the complete removal of ```public interface ContentExtractor
extends Extractor<InputStream>``` from the Extractor interface in the api module.
    This patch has a long way to go with numerous failing tests however I wanted to post it
for feedback.
    Although Any23 still builds with -DskipTests, without that flag the failing tests are
as follows
    Results :
    Failed tests:
      RoverTest.testRunMultiURLs:104->runWithMultiSourcesAndVerify:134 Unexpected number
of statements.
    Tests in error:
      Any23Test.testProgrammaticExtraction:279 » NullPointer
» Runtime
» Runtime
» Runtime
» Runtime
» Runtime
» NullPointer
» NullPointer
    Tests run: 403, Failures: 5, Errors: 8, Skipped: 11
    You will see that some of the tests concern
as well.

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

    $ git pull ANY23-280

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

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

    This closes #24
commit 801f2f93967bfd1295700223085eef3f54181517
Author: Lewis John McGibbney <>
Date:   2016-04-06T19:44:35Z

    Initial move towards addressing ANY23-280 Refactor ContentExtractor to improve extraction


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message