Terra-core: Replace Nightwatch Tests with Webdriver.io Tests

Created on 28 Feb 2018  路  36Comments  路  Source: cerner/terra-core

Issue Description

Remove the existing nightwatch tests and replace them with webdriver.io tests. This has been done for some components but these need to happen for the remaining components:

Checked off means done or claimed

  • [x] terra-content-container
  • [x] terra-demographics-banner
  • [x] terra-divider
  • [x] terra-dynamic-grid
  • [x] terra-form-checkbox
  • [x] terra-form-field
  • [x] terra-form-textarea
  • [x] terra-form-select
  • [x] terra-heading
  • ~terra-hookshot~ Migrated to terra-framework https://github.com/cerner/terra-framework/pull/128
  • [x] terra-icon
  • [x] terra-image
  • [x] terra-profile-image
  • [x] terra-progress-bar
  • [x] terra-props-table
  • ~terra-slide-panel~ Migrated to terra-framework https://github.com/cerner/terra-framework/pull/131
  • [x] terra-spacer
  • [x] terra-status
  • [x] terra-text
  • [x] terra-toggle
  • [x] terra-grid
  • [x] terra-list
  • [x] terra-responsive-element
  • [x] terra-signature
  • [x] terra-slide-group
  • [x] terra-table
  • [x] terra-toggle-button
  • [x] terra-abstract-modal
  • [x] terra-date-picker
  • [x] terra-menu
  • [x] terra-search-field

Remaining Components

  • [x] terra-data-time-picker
  • [x] terra-embedded-content-consumer

Issue Type

  • [ ] New Feature
  • [ ] Enhancement
  • [ ] Bug
  • [x] Other

Most helpful comment

This has now been resolved. Thanks to everyone who helped contribute on this issue!

All 36 comments

picking up terra-divider for replacing nightwatch with wdio tests.

Be sure to specify terra-defined viewports in the test setup: see https://github.com/cerner/terra-core/issues/1444

picking up terra-status for replacing nightwatch with wdio tests.

picking up content container tests

picking up the demographics banner tests

picking up the dynamic grid

picking up the form field

I'll grab content container

@Blackop778 There is a PR out for content container: https://github.com/cerner/terra-core/pull/1468
terra-heading or terra-text would be another good one to pick up instead.

I'll grab props table. terra-heading and terra-text don't actually test anything with nightwatch.

@Blackop778 The nightwatch tests fir terra-heading and terra-text aren't great, but that is because these components only apply css, which we don't want to test for as these values change with browsers and browser versions. So, the urls are hit to ensure the page loaded with the content, however we do want screenshots of these examples to ensure they render as expected visually.

picking up spacer.

picking up progress bar.

picking up terra-profile-image.

picking up form-field

Picking up time-input

Picking up icon.

terra-form-select was uplifted with: https://github.com/cerner/terra-core/pull/1501

Picking up toggle button

I can take terra-table

picking up terra-grid (not dynamic grid)

picking up list

Picking up terra-signature

Picking up slide group

@StephenEsser Has updated responsive element to wdio tests in his PR #1547

picking up terra-search-field

picking up terra-abstract-modal

Picked up terra-menu

Note: terra-search-field is going to be used in future in another new component (filter-view) based on feedback from UX. This blocks on how to go about search-field's web driver accessibility issues.

picking up terra-date-picker

Picking up terra-search-field

Picking up textarea

I'll go ahead and remove the markdown tests as part of textarea since it's small

I'll grab terra-embedded-content-consumer

I'll take date-time-picker

This has now been resolved. Thanks to everyone who helped contribute on this issue!

Was this page helpful?
0 / 5 - 0 ratings