- Issue created by @a.dmitriiev
- πΊπΈUnited States phenaproxima Massachusetts
This is blocked until π Add recipe for search backend Active lands.
- π©πͺGermany a.dmitriiev
At the moment recipe adds the processor "Entity status" to the index, so unpublished entities are not indexed as well.
- π©πͺGermany a.dmitriiev
I have added the first draft of the test https://git.drupalcode.org/issue/drupal_cms-3468271/-/merge_requests/2 . It is in the same issue fork as https://www.drupal.org/project/drupal_cms/issues/3468271 π Add recipe for search backend Active .
Cypress installs standard profile, correct? I think there the
/search
page exists from search core module? Or why the screenshot of the failure is the following:And the url in the screenshot is
/web/search/node
, but in test the instruction is to visit/search
page - Merge request !267Issue #3488687 by a.dmitriiev: Add end to end tests (cypress) for search recipe β (Merged) created by a.dmitriiev
- πΊπΈUnited States phenaproxima Massachusetts
Well that just looks damn near perfect. A couple of suggestions for clean-up, and maybe a few more comments would be useful, but really I think that's quite clear!
- π©πͺGermany a.dmitriiev
I have added the suggestions to MR, but the problem with core search module is still there. It seems my changes in
beforeEach
do not work, as the core search module is still installed and that is why the test fails, as instead of Search API search page with views, the core search page is displayed.Is there already Cypress command to uninstall the module?
- πΊπΈUnited States phenaproxima Massachusetts
π€ I wonder why it is that your code in the UI to uninstall Search doesn't work. Probably some more looking into that is needed.
- First commit to issue fork.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
I fixed the existing test. Not sure if we need to add more or is this enough?
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Search e2e passed on CI. Failures look like random connection mysql failures in a different test.
-
phenaproxima β
committed 2686b39b on 1.x authored by
a.dmitriiev β
Issue #3488687 by a.dmitriiev, penyaskito, phenaproxima: Add end to end...
-
phenaproxima β
committed 2686b39b on 1.x authored by
a.dmitriiev β
- πΊπΈUnited States phenaproxima Massachusetts
Merged into 1.x and will cherry-pick to 1.0.x shortly. Thanks!
-
phenaproxima β
committed 8e83b916 on 1.0.x authored by
a.dmitriiev β
Issue #3488687 by a.dmitriiev, penyaskito, phenaproxima: Add end to end...
-
phenaproxima β
committed 8e83b916 on 1.0.x authored by
a.dmitriiev β
Automatically closed - issue fixed for 2 weeks with no activity.