#56 ✓resolved
baphled (at boodah)

Cleanup step definitions

Reported by baphled (at boodah) | June 27th, 2010 @ 08:45 PM | in 0.8.2

A lot of our step definitions over lap and/or could be implemented better.

We need to review each step and refactor them accordingly.

The first set of steps will be the basic and incrementally move on from there.

Comments and changes to this ticket

  • baphled (at boodah)

    baphled (at boodah) July 16th, 2010 @ 09:01 PM

    • State changed from “new” to “open”
    • Milestone order changed from “1” to “0”

    Once we have cleaned up all our visit based steps we'll go over our other steps to make them more flexible, removing all duplicate and unused steps

  • baphled (at boodah)

    baphled (at boodah) July 16th, 2010 @ 11:37 PM

    • Milestone order changed from “11” to “0”

    Removed all visit based steps, as they are no longer used.

  • baphled (at boodah)

    baphled (at boodah) July 18th, 2010 @ 09:39 PM

    • Milestone order changed from “2” to “0”

    Removed a bunch of unused steps by using the ueage format and cross referencing the unused with all our features.

    Next step is to go over all of the remaining steps and refactor the ones that either are not explicit or could be centralised.

  • baphled (at boodah)

    baphled (at boodah) July 25th, 2010 @ 07:09 PM

    • Milestone order changed from “1” to “0”

    Centralised our 'there are' and 'there is a' steps so that they are generic and moved them to the general_steps file.

    Have also gone through all the steps that needed refactoring or improved and refactored all but two of them. These will be done at a later stage when we have removed the remaining unused steps.

  • baphled (at boodah)

    baphled (at boodah) July 25th, 2010 @ 10:32 PM

    Centralised most of the general steps into one helper. Also removed the rest of our unused steps.

    We'll give our steps a once over after we have cleaned up our specs

  • baphled (at boodah)

    baphled (at boodah) July 27th, 2010 @ 08:33 PM

    • State changed from “open” to “resolved”

    Pending specs have been implemented, still need to review them and make them more readable.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Bridging the gap between clients and developers

Shared Ticket Bins

People watching this ticket

Pages