Problem/Motivation

All of our non-kernel tests are SimpleTest. We should move to BrowserTest. We also have a lot of duplicate testing helper methods in our base classes that could be traits (for ourselves and contrib.) This is the planning issue for simplifying and moving our tests to BrowserTest.

Proposed resolution

The following items:

  • Move common testing helper methods into traits and update tests to use them
  • Move tests based on SimpleTest to BrowserTest
  • See if we can move traits to \Drupal\Tests\commerce_*

Any issues relating will be added as child items.

Remaining tasks

See child issues.

Comments

mglaman created an issue. See original summary.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Issue summary: View changes

Discovered that SimpleTest cannot autoload classes in \Drupal\Tests\ namespace. Our test traits will need to be in src/ and documented that they are for tests.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented

We now have PhantomJS in our TravisCI build with a Javascript test!

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented

Once all child tickets are done, we will be 100% off of Simpletest!

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Title: Pave the way for BrowserTest » Pave the way for BrowserTest; remove Simpletest tests

Updating title to reflect fact that some child issues involve porting to Kernel test so we no longer have simpletest tests.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Issue tags: +Needs issue summary update

I need to update this issue with the few remaining tasks now that we have 0 simpletest tests.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Active » Fixed

Long overdue closure of this :) Thanks to everyone who helped in the effort.

Status: Fixed » Closed (fixed)

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