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

One point zero #91

Merged
merged 57 commits into from
Feb 2, 2016
Merged

One point zero #91

merged 57 commits into from
Feb 2, 2016

Conversation

san650
Copy link
Owner

@san650 san650 commented Dec 20, 2015

This PR is work in progress!

To do

Required

  • Remove build method
  • Remove component property
  • Migrate code to use Ceibo
    • Allows us to use properties instead of functions (Breaks compatibility)
    • Allows to navigate to parent properties
  • Make scopes inheritable by default (Breaks compatibility)
  • Add resetScope option to all properties
  • Remove custom helper (Breaks compatibility)
    • Remove code
    • Write examples of how to write similar helpers using descriptors
  • Make properties to throw an error when multiple elements are matched (Breaks compatibility)
    • Except for count attribute 😄
  • Make collections zero-based (Breaks compatibility)
  • Update website documentation
  • Rename property option index to at
  • Allow to require properties directly import { create, text, count } from './page-object'
  • isVisible fails when element is not present (isVisible fails when element is not present #95)
  • Add an easy way to extend existing page objects

Nice to have

  • Review visitable method signature (is there a better alternative?)
  • Debug Mode (print in console everything similar to what ENV.APP.LOG_RESOLVER does)
  • Support component integration tests
  • Rethink how to interact with collection of values (Allow mapping of collections #54, textList helper #81)
    • Maybe we could support something like text('span', { multiple: true })

@san650 san650 added this to the v1.0 milestone Dec 29, 2015
@san650 san650 merged commit 9ec8712 into master Feb 2, 2016
@san650 san650 deleted the one-point-zero branch February 2, 2016 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants