Skip to content
This repository has been archived by the owner on Dec 5, 2021. It is now read-only.

Introduce BehaviorAssertion to implement BDD-style assertions #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Introduce BehaviorAssertion to implement BDD-style assertions #18

wants to merge 3 commits into from

Conversation

RoRoche
Copy link

@RoRoche RoRoche commented Feb 7, 2019

  • introduce BDD steps using an interface
  • define Assertion that aggregates Steps
  • cover by unit tests

@skapral
Copy link
Member

skapral commented Feb 8, 2019

@RoRoche thanks for contributing. Please create an issue for this change.

skapral
skapral reviewed Feb 8, 2019
View reviewed changes
tests-core/src/main/java/com/pragmaticobjects/oo/tests/bdd/BehaviorAssertion.java Outdated

import com.pragmaticobjects.oo.tests.Assertion;

public final class BehaviorAssertion<S, R> implements Assertion {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oo-atom prohibits final classes for certain reasons. Let the class being open and finalize its methods instead.

@RoRoche
Copy link
Author

RoRoche commented Feb 8, 2019

Refers to #20

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@skapral skapral skapral left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@RoRoche @skapral