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

LPD-25046 Move ExpectedLogMethodTestRule to test/rule #284

Open
wants to merge 83 commits into
base: master
Choose a base branch
from

Conversation

MarinhoFeliphe
Copy link
Owner

No description provided.

@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@MarinhoFeliphe
Copy link
Owner Author

ci:forward

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:relevant
  •     ci:test:sf
  •     ci:test:stable

The pull request will automatically be forwarded to the user brianchandotcom If the following test suites pass:

  •     ci:test:relevant
  •     ci:test:sf
  •     ci:test:stable

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 23 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 5f99245b97d0d9a7faebf0f4cba9e1ae31d48c85

Sender Branch:

Branch Name: LPD-25046_20240507-1
Branch GIT ID: 46f018d32a53025da7e54153e8ce8cad4bd7dac6

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

Jenkins Report:jenkins-report.html
Jenkins Suite:sf
Pull Request:MarinhoFeliphe#284
Testray Routine:EE Pull Request
Testray Build ID:2566172289

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:stable - 29 out of 30 jobs passed

❌ ci:test:relevant - 42 out of 52 jobs passed in 3 hours 10 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 5f99245b97d0d9a7faebf0f4cba9e1ae31d48c85

Upstream Comparison:

Branch GIT ID: 5387a0b00f21b6ca109e75b2c3f22a9291988fa7
Jenkins Build URL: EE Development Acceptance (master) - 528 - 2024-05-06[08:38:35]

ci:test:stable - 29 out of 30 jobs PASSED

1 Failed Jobs:

29 Successful Jobs:
    ci:test:relevant - 42 out of 52 jobs PASSED

    10 Failed Jobs:

    42 Successful Jobs:
      For more details click here.

      Failures unique to this pull:

      1. modules-compile-jdk8/0/0
        Build was aborted
      2. modules-unit-jdk8/0/4
        1. com.liferay.portal.search.opensearch2.internal.cluster.ReplicasClusterListenerTest.testAHappyDay - UNTESTED
        2. com.liferay.portal.search.opensearch2.internal.cluster.ReplicasClusterListenerTest.testLiferayClusterReportsEmpty - UNTESTED
        3. com.liferay.portal.search.opensearch2.internal.cluster.ReplicasClusterListenerTest.testMasterTokenAcquired - UNTESTED
        4. ...
      3. modules-unit-jdk8/0/5
        1. com.liferay.portal.search.opensearch2.internal.logging.OpenSearchExceptionHandlerTest.testDeleteIndexNotFoundLogExceptionsOnlyFalse - UNTESTED
        2. com.liferay.portal.search.opensearch2.internal.logging.OpenSearchExceptionHandlerTest.testDeleteIndexNotFoundLogExceptionsOnlyTrue - UNTESTED
        3. com.liferay.portal.search.opensearch2.internal.logging.OpenSearchExceptionHandlerTest.testDeleteLogExceptionsOnlyFalse - UNTESTED
        4. ...
      4. modules-unit-jdk8/0/6
        1. com.liferay.portal.search.opensearch2.internal.search.engine.adapter.OpenSearchSearchEngineAdapterSnapshotRequestTest.testCreateSnapshot - UNTESTED
        2. com.liferay.portal.search.opensearch2.internal.search.engine.adapter.OpenSearchSearchEngineAdapterSnapshotRequestTest.testCreateSnapshotRepository - UNTESTED
        3. com.liferay.portal.search.opensearch2.internal.search.engine.adapter.OpenSearchSearchEngineAdapterSnapshotRequestTest.testDeleteSnapshot - UNTESTED
        4. ...
      5. ...

      @liferay-continuous-integration
      Copy link
      Collaborator

      shuyangzhou and others added 14 commits May 6, 2024 19:21
      …segments experiences
      
      # breaking
      
      ## What modules/apps/segments/segments-service/src/main/java/com/liferay/segments/service/impl/SegmentsExperienceLocalServiceImpl.java
      
      Adds new ExternalReferenceCode parameter to addSegmentsExperience and addDefaultSegmentsExperience method.
      
      ## Why
      
      Adds supports for ExternalReferenceCode in SegmentsExperience entity.
      
      ----
      
      # breaking
      
      ## What modules/apps/segments/segments-service/src/main/java/com/liferay/segments/service/impl/SegmentsExperienceServiceImpl.java
      
      Adds new ExternalReferenceCode parameter to addSegmentsExperience method.
      
      ## Why
      
      Adds supports for ExternalReferenceCode in SegmentsExperience entity.
      
      ----
      sandrodw3 and others added 29 commits May 7, 2024 08:06
      … they share the same number format to avoid a known issue on JDK8 with some Portuguese locales, see https://bugs.openjdk.org/browse/JDK-8187711 and https://bugs.openjdk.org/browse/JDK-8178872. With that JDK bug and with CLDR configured as the default locale provider, It requires that the decimal separator for the Locale(pt, PT) be  instead of , as described in the ticket mentioned early
      … robots, hidden, imageIconId, themeId, colorSchemeId, styleBookEntryId, css, faviconFileEntryId, lastPublishDatem status, statusByUserId, statusByUserName) to merge for CT publish
      …ationDate, lastPublishDate) to merge during CT publish
      …usByUserId, statusByUserName, statusDate) to merge during CT publish
      …tusByUserId, statusByUserName, statusDate) to merge during CT publish
      @MarinhoFeliphe MarinhoFeliphe force-pushed the LPD-25046_20240507-1 branch from 46f018d to c346c34 Compare May 7, 2024 14:23
      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.