Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurable adapter #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Configurable adapter #18

wants to merge 2 commits into from

Conversation

dantleech
Copy link
Contributor

Fixes issue with double indexing and adds search adapter functional tests.

require_once __DIR__ . '/Transport/Fs/Filesystem/Adapter/AdapterTestCase.php';
require_once __DIR__ . '/Transport/Fs/Search/Adapter/AdapterTestCase.php';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm. We should really upgrade the autoloading in the Jackalope tests.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@dantleech dantleech force-pushed the fixed_double_indexing branch from 5f40f87 to 7dfb444 Compare November 16, 2014 10:43
@dantleech
Copy link
Contributor Author

  • ArrayAdapter ls does not work (does not return correct data format).
  • Globs not supported by array adapter.
  • Add /improve adapter test
  • Test is timing out for unknown reason.

@dantleech dantleech self-assigned this Feb 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants