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

Basic 2385/caleb #175

Open
wants to merge 36 commits into
base: 5.x
Choose a base branch
from
Open

Basic 2385/caleb #175

wants to merge 36 commits into from

Conversation

caleb-jones
Copy link

@caleb-jones caleb-jones commented Aug 7, 2017

READY FOR REVIEW

@jbickar I ran and fixed, if necessary, all the tests I could. It's ready for your review.

Summary

  • Caleb:
    Refactored all of the Jumpstart product features to not use Scenario Outlines. All of the Scenario Outline pieces are written out as individual steps.

A few of the commit messages were labelled as the incorrect product, namely:

  • stanford_person_layout_views.feature labelled as jsa is actually in jse
  • The second homepage.feature labelled as jsa is actually in jsl

Needed By (Date)

  • When possible

Criticality

  • Fixes slow scenarios - Will make Behat tests run faster

Steps to Test

1. Switch to this branch

2. Test stanford_jumpstart_home

For JSA, start from linky_clicky/products/jsa
../../bin/behat -p sites-sws-build-jsa -s dev features/stanford_jumpstart_home

For JSE start from: linky_clicky/products/jse
./../bin/behat -p sites-sws-build-jse -s dev features/stanford_jumpstart_home

3. Test stanford_person

  • Edit Haley's profile and change Affiliation from Student to Students. See bug report
  1. Login to the site
  2. Go to people/all/grid/grouped
  3. Click on edit for Haley
  4. Change Affiliation from Student to Students

For JSA, From linky_clicky/products/jse
../../bin/behat -p sites-sws-build-jse -s dev features/stanford_person

For JSE From linky_clicky/products/jsa
../../bin/behat -p sites-sws-build-jsa -s dev features/stanford_person

4. Test stanford_sites_helper/stanford_helper/stanford_sites_helper.feature

You'll need a brand new site.... I did not test this.

5. Test jsa/homepage.feature

From linky_clicky/products/jsa
../../bin/behat -p sites-sws-build-jsa -s dev features/homepage.feature

6. Test jsl/features/homepage.feature

I don't know of a JSL site. I did not test this.

7. Test jslbre/features/homepage.feature

I don't know of a JSLBRE site. I did not test this.

8. Test jsplus/features/homepage.feature

From: linky_clicky/products/jsplus
../../bin/behat -p sites-sws-build-jsplus -s dev features/homepage.feature

9. Test jsv/features/homepage.feature

From: linky_clicky/products/jsv
../../bin/behat -p sites-sws-build-jsv -s dev features/homepage.feature

10. Test vpsa/features/homepage.feature

From: linky_clicky/products/jsvpsa
../../bin/behat -p sites-sws-build-jsvpsa -s dev features/homepage.feature

Affects

  • Most products especially JSE and JSA

Associated Issues and/or People

Related JIRA ticket(s)

https://stanfordits.atlassian.net/browse/BASIC-2385

Related PRs

More Information

The student view is broken. See https://stanfordits.atlassian.net/browse/BASIC-2415

Example JSA profile (in behat.local.yml)

sites-sws-build-jsa:
  extensions:
    Behat\MinkExtension:
      base_url: https://sws-build-jsa.stanford.edu
    Drupal\DrupalExtension:
      drush:
        alias: sse.sws-build-jsa

Example JSE profile (in behat.local.yml)

sites-sws-build-jse:
  extensions:
    Behat\MinkExtension:
      base_url: https://sites.stanford.edu/sws-build-jse
    Drupal\DrupalExtension:
      drush:
        alias: sse.sws-build-jse

Folks to notify

See Also

@nickmanning nickmanning requested a review from jbickar August 7, 2017 21:05
@nickmanning
Copy link
Contributor

John, I did a quick glance at this. Hopefully, this is what you were looking for. I am sure Caleb would be open to feedback and best practices. Maybe, even more, understanding of what he has been working on.

@jbickar
Copy link
Contributor

jbickar commented Aug 8, 2017

@caleb-jones these will have to be modified to read Scenario instead of Scenario Outline, as well. I can show you when you are in the office next.

@nickmanning
Copy link
Contributor

Oh, I didn't catch that. Thanks, @jbickar!

@caleb-jones
Copy link
Author

@jbickar Let me know if I missed anything. I think I got all of the instances of Scenario Outline.

@jbickar jbickar mentioned this pull request Oct 13, 2017
@cjwest cjwest mentioned this pull request Feb 8, 2018
@cjwest
Copy link
Member

cjwest commented Apr 23, 2018

@jbickar
Ping

@jbickar jbickar removed their request for review August 11, 2018 14:51
@jbickar jbickar removed their assignment Aug 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants