You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
The TCK does not include metadata from declared dependencies. See this PR comment for context.
To Reproduce
Remove the configurationFileDirectories.from(tck.metadataRoot.dir... lines from the graphql-java-extended-validation TCK build.gradle and run:
./gradlew test -Pcoordinates=com.graphql-java:graphql-java-extended-validation:19.1
The test suite should fail on missing metadata related to jboss-logging, hibernate-validator and graphql-java - all libraries with actual metadata in this repository.
Expected behavior
The test should pass without manually adding the metadata for known dependencies listed in the "requires" attribute of the index.json document.
The text was updated successfully, but these errors were encountered:
This commit fixes the metadata repository not being used in the TCK.
Before, it was assumed that each test was working in isolation, but
in practice, oracle#211 showed that it is not the case.
So instead of the TCK figuring out by itself which metadata config
to include, and completely ignoring the "requires" clauses, we
will now simply configure the tests to use the metadata from the
current state of the repository. It makes things easier, at the
cost of possible side effects: it is possible that the metadata
from a transitive dependency is actually what makes a build pass.
However this drawback should be covered by regular code reviews,
as it is unexpected that metadata for a transitive "talks" about
the current module under test.
Fixesoracle#211
This commit fixes the metadata repository not being used in the TCK.
Before, it was assumed that each test was working in isolation, but
in practice, #211 showed that it is not the case.
So instead of the TCK figuring out by itself which metadata config
to include, and completely ignoring the "requires" clauses, we
will now simply configure the tests to use the metadata from the
current state of the repository. It makes things easier, at the
cost of possible side effects: it is possible that the metadata
from a transitive dependency is actually what makes a build pass.
However this drawback should be covered by regular code reviews,
as it is unexpected that metadata for a transitive "talks" about
the current module under test.
Fixes#211
Describe the bug
The TCK does not include metadata from declared dependencies. See this PR comment for context.
To Reproduce
Remove the
configurationFileDirectories.from(tck.metadataRoot.dir...
lines from the graphql-java-extended-validation TCKbuild.gradle
and run:The test suite should fail on missing metadata related to jboss-logging, hibernate-validator and graphql-java - all libraries with actual metadata in this repository.
Expected behavior
The test should pass without manually adding the metadata for known dependencies listed in the
"requires"
attribute of theindex.json
document.The text was updated successfully, but these errors were encountered: