[ARVADOS] updated: 538caa064785b645a2b8f815bf77a30192b20665

git at public.curoverse.com git at public.curoverse.com
Mon Feb 9 21:52:44 EST 2015


Summary of changes:
 apps/workbench/Gemfile                             |  2 +-
 apps/workbench/Gemfile.lock                        |  4 +-
 .../controllers/collections_controller_test.rb     | 19 +++++++
 .../integration/filterable_infinite_scroll_test.rb | 10 ++--
 sdk/ruby/lib/arvados/keep.rb                       |  5 +-
 sdk/ruby/test/test_keep_manifest.rb                | 66 ++++++++++++++++++++++
 services/api/Gemfile                               |  2 +-
 services/api/Gemfile.lock                          | 16 +++---
 services/api/test/fixtures/collections.yml         | 29 +++++++++-
 .../arvados/v1/collections_controller_test.rb      |  7 ++-
 .../api/test/integration/collections_api_test.rb   | 21 +++++++
 11 files changed, 158 insertions(+), 23 deletions(-)

       via  538caa064785b645a2b8f815bf77a30192b20665 (commit)
       via  9f1fafa8c7c7f3750d6769d863b82cb826d7ed6e (commit)
       via  330a46e91b4ceaefba2bcfc383931eb59c77d461 (commit)
       via  fefce5e8e133a8fa064bbcdf31d85d41dc4a6729 (commit)
       via  204f433a870e2bf1cf7af1fbe076e91f427ef05e (commit)
       via  9b61792d905324a98b24224d45347082efbe5205 (commit)
       via  16b720950262eb559358cf357f5098a142901665 (commit)
       via  b6a7a62f4f38710f50d08a91a6a9b210700bb011 (commit)
       via  aaffcb23198b4223c48092ccd30ef7152b434187 (commit)
       via  20f5b178a850b029ecd501ed49e4ed0a537c1fad (commit)
       via  e2da84c1fd9052791ed2b684741469570e09ea35 (commit)
       via  2527b9cd7958d89a5ae0dd84856027908c48ae53 (commit)
       via  dd645c9e973b9b725f310513ce309fa1e1a82421 (commit)
      from  e73af668c24cd259800c344c3efe8b7d769903da (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 538caa064785b645a2b8f815bf77a30192b20665
Author: Radhika Chippada <radhika at curoverse.com>
Date:   Mon Feb 9 21:50:41 2015 -0500

    5110: full text search seems to treat '_' characters as spaces. Thus, when searched for 'pipeline_1', objects matching 'pipeline'
    are also fetched since '1' is a common word and ignored. Hence, update the test to use more specific search filters.

diff --git a/apps/workbench/test/integration/filterable_infinite_scroll_test.rb b/apps/workbench/test/integration/filterable_infinite_scroll_test.rb
index b4dadcd..777696a 100644
--- a/apps/workbench/test/integration/filterable_infinite_scroll_test.rb
+++ b/apps/workbench/test/integration/filterable_infinite_scroll_test.rb
@@ -10,18 +10,18 @@ class FilterableInfiniteScrollTest < ActionDispatch::IntegrationTest
   # unused ?search=foo param to pre-populate the search field.
   test 'no double-load if text input has a value at page load time' do
     visit page_with_token('admin', '/pipeline_instances')
-    assert_text 'pipeline_2'
-    visit page_with_token('admin', '/pipeline_instances?search=pipeline_1')
+    assert_text 'pipeline_with_job'
+    visit page_with_token('admin', '/pipeline_instances?search=pipeline_with_tagged')
     # Horrible hack to ensure the search results can't load correctly
     # on the second attempt.
     assert_selector '#recent-pipeline-instances'
     assert page.evaluate_script('$("#recent-pipeline-instances[data-infinite-content-href0]").attr("data-infinite-content-href0","/give-me-an-error").length == 1')
     # Wait for the first page of results to appear.
-    assert_text 'pipeline_1'
+    assert_text 'pipeline_with_tagged_collection_input'
     # Make sure the results are filtered.
-    assert_no_text 'pipeline_2'
+    assert_no_text 'pipeline_with_job'
     # Make sure pipeline_2 didn't disappear merely because the results
     # were replaced with an error message.
-    assert_text 'pipeline_1'
+    assert_text 'pipeline_with_tagged_collection_input'
   end
 end

commit 9f1fafa8c7c7f3750d6769d863b82cb826d7ed6e
Merge: e73af66 330a46e
Author: Radhika Chippada <radhika at curoverse.com>
Date:   Mon Feb 9 21:36:07 2015 -0500

    Merge branch '4523-full-text-search' into 5110-workbench-full-text-search


-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list