jakartaee / platform-tck

Jakartaee-tck
Other
125 stars 104 forks source link

Remove jakarta.annotation.managedbean tests from Platform TCK #1184

Open scottmarlow opened 1 year ago

scottmarlow commented 1 year ago

In support of https://github.com/jakartaee/common-annotations-api/pull/115 remove jakarta.annotation.managedbean Platform TCK tests.

Work is to be done on https://github.com/jakartaee/platform-tck/tree/tckrefactor

git status
On branch tckrefactor

grep -rli jakarta.annotation.ManagedBean --include=*.java
src/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecyclecdijsf/OverrideWithPostConstructBean.java
src/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecyclecdijsf/OverrideBean.java
appclient/src/main/java/com/sun/ts/tests/appclient/deploy/metadatacomplete/testapp/TestBean.java
jaxrs/src/main/java/com/sun/ts/tests/jaxrs/platform/managedbean/TSAppConfig.java
jaxrs/src/main/java/com/sun/ts/tests/jaxrs/platform/managedbean/ManagedBeanRootResource.java
jaxrs/src/main/java/com/sun/ts/tests/jaxrs/platform/managedbean/StringBuilderProvider.java
signaturetest/src/main/java/com/sun/ts/tests/signaturetest/caj/CAJSigTest.java
jms/src/main/java/com/sun/ts/tests/jms/ee20/cditests/ejbweb/MyManagedBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/misc/moduleName/appclientejb/ModuleMBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/misc/moduleName/twowars/ModuleMBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/misc/moduleName/twojars/ModuleMBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/invocationcontext/InvocationContextInterceptorBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/invocationcontext/InvocationContextBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/business/InterceptorOverride34Bean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/business/InterceptorOverrideBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/business/InterceptorBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecyclecdi/OverrideWithPostConstructBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecyclecdi/OverrideBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecycle/InterceptorOverride34Bean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecycle/AManagedBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecycle/InterceptorOverrideBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/lite/packaging/war/mbean/interceptor/lifecycle/InterceptorBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/assembly/appres/common/AppResManagedBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/assembly/mbean/appclient/TwoManagedBean.java
ejb30/src/main/java/com/sun/ts/tests/ejb30/assembly/mbean/appclient/OneManagedBean.java

The above tests need to be removed or updated to not reference jakarta.annotation.ManagedBean.

edburns commented 1 year ago

Hello @scottmarlow , can you please create a label "jea-linked" with color #0099cc? I want to attach this label to this issue because I have linked it to Azure Boards AB#118.

scottmarlow commented 1 year ago

Hello @scottmarlow , can you please create a label "jea-linked" with color #0099cc? I want to attach this label to this issue because I have linked it to Azure Boards AB#118.

@edburns done via https://github.com/jakartaee/platform-tck/labels/jea-linked

edburns commented 1 month ago

Please confirm this sketch for how a new helper could complete the task.

  1. Learn how to run the Jakarta 10 TCK against GlassFish 7.
  2. Ensure it runs successfully, including the tests to be removed under this issue.
  3. Perform the removal action.
  4. Ensure it still runs successfully, and the tests are removed.

Also, please confirm the changes for this issue should be made to the master branch of https://github.com/jakartaee/platform-tck .