JBehave
  1. JBehave
  2. JBEHAVE-902

Improve extensibility of core elements

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.9
    • Component/s: Core
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Some classes as Embedder, Configuration and AnnotationBuilder have its attribute's default values being initialized in its declaration statement. To complicate a bit more those attributes are private.

      To be able to extend those classes and be able to set default values properly we need to remove that initialization from the attribute declaration and put in some method with a protected visibility.

        Activity

        Cristiano Gavião made changes -
        Field Original Value New Value
        Assignee Cristiano Gavião [ cvgaviao ]
        Cristiano Gavião made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Mauro Talevi made changes -
        Summary Improve extendability Improve extendability of core elements
        Fix Version/s 3.8.1 [ 19035 ]
        Fix Version/s 4.0 [ 18486 ]
        Affects Version/s 4.x [ 18279 ]
        Show
        Mauro Talevi added a comment - https://github.com/jbehave/jbehave-core/pull/48
        Mauro Talevi made changes -
        Summary Improve extendability of core elements Improve extensibility of core elements
        Mauro Talevi made changes -
        Resolution Fixed [ 1 ]
        Status In Progress [ 3 ] Closed [ 6 ]
        Hide
        Mauro Talevi added a comment -

        Issues should not be closed, only resolved.

        Show
        Mauro Talevi added a comment - Issues should not be closed, only resolved.
        Mauro Talevi made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Mauro Talevi made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Cristiano Gavião
            Reporter:
            Cristiano Gavião
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: