-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Dev tools: tests for adding extensions from different platform members #18535
Dev tools: tests for adding extensions from different platform members #18535
Conversation
b62bf6c
to
6f894e1
Compare
This workflow status is outdated as a new workflow run has been triggered. 🚫 This workflow run has been cancelled. Failing Jobs - Building b62bf6c
Full information is available in the Build summary check run. Test Failures⚙️ Devtools Tests - JDK 11 #📦 integration-tests/devtools✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ⚙️ Devtools Tests - JDK 11 Windows #📦 integration-tests/devtools✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ ✖ |
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building 6f894e1
Full information is available in the Build summary check run. Test Failures⚙️ Devtools Tests - JDK 11 #📦 integration-tests/devtools✖ ✖ ✖ ⚙️ Devtools Tests - JDK 11 Windows #📦 integration-tests/devtools✖ ✖ ✖ |
6f894e1
to
ad9409a
Compare
ad9409a
to
e03ad3e
Compare
Failing Jobs - Building e03ad3e
Full information is available in the Build summary check run. Test Failures⚙️ MicroProfile TCKs Tests #📦 tcks/microprofile-fault-tolerance✖ |
sawFirstPlatform = true; | ||
} | ||
platformBoms.add(c.getBom()); | ||
} | ||
} else if (ElementCatalogBuilder.hasElementCatalog(mainCatalog)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
method? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it's checking whether there is an "element" catalog attached to the mainCatalog
. A bit of a hack but that's how it is for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant, it would be better to move this whole block to a specific method.
But I know you love those long methods 🤪
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -41,7 +41,7 @@ public QuarkusCommandOutcome execute(QuarkusCommandInvocation invocation) throws | |||
final MessageWriter log = invocation.log(); | |||
final boolean all = invocation.getValue(ListExtensions.ALL, true); | |||
final boolean installedOnly = invocation.getValue(ListExtensions.INSTALLED, false); | |||
final boolean cli = invocation.getValue(ListExtensions.FROM_CLI, false); | |||
//final boolean cli = invocation.getValue(ListExtensions.FROM_CLI, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh, too late
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do.
Basic tests for adding extensions to existing projects that belong to platform members whose BOMs haven't been imported yet.
The testsuite is still building up and will need to be properly organized.