Problem/Motivation

In #3587416: Update ddev plugins and gitignore, we added some DDEV tooling to make it easier to set up Elasticsearch Connector for contributing.

In a similar issue for another module, @boromino and @mparker17 came up with a script to further simplify setting up the module for development: a custom ddev reset-test-env command to install dependencies, re-install Drupal and the module, and display a link that contributors can use to log into the test site.

This seems like a useful addition to Elasticsearch Connector, so lets port that custom command to this module.

Proposed resolution

Add a .ddev/commands/host/reset-test-env file that looks like...

#!/bin/bash
ddev poser
ddev exec "cd web/core && yarn install"
ddev symlink-project
ddev drush -y site:install standard
ddev drush -y pm:install elasticsearch_connector_test facets
ddev drush -y elasticsearch_connector_test:add-test-content
ddev drush -y search-api:index test_elasticsearch_index
ddev drush -y cr
ddev drush -y uli

Once this issue is merged, update the documentation accordingly.

Remaining tasks

  1. Merge request for 9.0.x - merge request !221 created by @mparker17 in #3
  2. Merge request for 8.0.x - merge request !220 created by @mparker17 in #2
  3. Community/maintainer review - skipped by @mparker17 in #4
  4. Commit to 9.0.x - done by @mparker17 in #5
  5. Commit to 8.0.x - done by @mparker17 in #6
  6. Update the docs - done by @mparker17 in #7
  7. Release 9.0.x
  8. Release 8.0.x

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork elasticsearch_connector-3588301

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

mparker17 created an issue. See original summary.

mparker17 opened merge request !220

mparker17 opened merge request !221

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes
Status: Active » Reviewed & tested by the community

Merge requests created; testbot is happy with them. I'm going to skip community/maintainer review since the patches were written by a maintainer. Merging shortly.

  • mparker17 committed 42eaf1c2 on 9.0.x 
    feat: #3588301 Add a reset-test-env ddev host command for easier...

  • mparker17 committed 5c3157c1 on 8.0.x 
    feat: #3588301 Add a reset-test-env ddev host command for easier...
mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Merged to both branches, and I've updated the documentation. I will update this issue when this change is released.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes

(fix typo in issue summary)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.