-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Some code start tests test the version which is not compatible with releases #15142
Comments
/cc @maxandersen, @quarkusio/devtools |
Ok I will look for all snapshots containing versions and fix it.. |
Yeah, I don't think we can test this easily so I would rather drop the test on the version. Otherwise, you need a reliable way to get the current version, not sure if we have that. |
void generatePicocliProject(TestInfo testInfo) throws Throwable {
final QuarkusPlatformDescriptor platformDescriptor = getPlatformDescriptor();
final QuarkusJBangCodestartProjectInput input = QuarkusJBangCodestartProjectInput.builder()
.addCodestart("jbang-picocli-code")
>>> .putData(QUARKUS_BOM_GROUP_ID, platformDescriptor.getBomGroupId())
>>> .putData(QUARKUS_BOM_ARTIFACT_ID, platformDescriptor.getBomArtifactId())
>>> .putData(QUARKUS_BOM_VERSION, platformDescriptor.getBomVersion())
.build();
final Path projectDir = testDirPath.resolve("picocli");
getCatalog().createProject(input).generate(projectDir);
assertThatDirectoryTreeMatchSnapshots(testInfo, projectDir);
assertThatMatchSnapshot(testInfo, projectDir, "src/GreetingCommand.java");
} @gsmet the problem is that we use snapshot testing to test that the code generation is valid. Having dynamic version is not relevant here and should be mocked for the test to run properly everywhere. |
Got these errors while releasing:
We can't count on the version to always be
999-SNAPSHOT
, we shouldn't test that.The text was updated successfully, but these errors were encountered: