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

Migrate ReasoningIT to BDD #81

Merged
merged 25 commits into from
Jul 28, 2020
Merged

Conversation

alexjpwalker
Copy link
Member

@alexjpwalker alexjpwalker commented Jul 20, 2020

What is the goal of this PR?

To identify and de-duplicate integration tests that test the fundamentals of Reasoner, categorise them appropriately and migrate them to BDD scenarios.

This PR focuses on ReasoningIT in grakn.

This PR forms part of the work towards typedb/typedb#5721

What are the changes implemented in this PR?

Added tests that were migrated from the relevant IT files in grakn. Also moved some tests around, so that everything is in the right place.

@alexjpwalker alexjpwalker changed the title Migrate basic reasoner tests to BDD Migrate ReasoningIT to BDD Jul 21, 2020
@alexjpwalker alexjpwalker marked this pull request as ready for review July 24, 2020 12:21
behaviour/graql/reasoner/relation-inference.feature Outdated Show resolved Hide resolved
behaviour/graql/reasoner/BUILD Outdated Show resolved Hide resolved
behaviour/graql/reasoner/value-predicate.feature Outdated Show resolved Hide resolved
@alexjpwalker alexjpwalker merged commit 39f044e into typedb:master Jul 28, 2020
@alexjpwalker alexjpwalker deleted the basic-reasoner branch July 28, 2020 10:56
alexjpwalker pushed a commit to typedb/typedb that referenced this pull request Jul 28, 2020
## What is the goal of this PR?

To identify and de-duplicate integration tests that test the fundamentals of Reasoner, categorise them appropriately and migrate them to BDD scenarios.

This PR will focus exclusively on ReasoningIT.

This PR forms part of the work towards #5721

## What are the changes implemented in this PR?

Migrate tests away from IT files as they get moved to BDD in typedb/typedb-behaviour#81
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.

3 participants