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

COMMERCE-8305 #2191

Closed
wants to merge 1,483 commits into from
Closed

COMMERCE-8305 #2191

wants to merge 1,483 commits into from

Conversation

brianikim
Copy link

@brianikim brianikim commented Feb 1, 2022

https://issues.liferay.com/browse/COMMERCE-8305

@ivicac , this one is another edge case similar to COMMERCE-7998.

Luizcarlosqueiroz and others added 30 commits January 31, 2022 22:59
…name of the zip file it's the name of the classname
…unt as different models so extract the method to know if are multiple or not this must change when exporting several models and several experiences.
…the fragmentEntryLink is updated.

Updating the current fragmentEntryLink module will keep the url of the resource, meaning that if it was already cached, the new version won't be show. Registering a new module taking into account the modifiedDate of the fragment will generate a new url, bypassing the possible caches
The default log level reports *a lot* of noise on a vanilla system that's not connected to AC - e.g. one message "DEBUG [BaseAnalyticsClientImpl:49] Analytics configuration tracker not active" _per http request_.
@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.

@brianikim
Copy link
Author

ci:test:sf

@brianikim
Copy link
Author

ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: b9b0dee253287fb23e10736f989d56872ef29f14

Sender Branch:

Branch Name: COMMERCE-8305
Branch GIT ID: 1f598fbc1e6cacdbf765dd5e9b75459f6baf138e

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

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:stable - 21 out of 26 jobs passed

✔️ ci:test:relevant - 22 out of 27 jobs passed in 2 hours 9 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: b9b0dee253287fb23e10736f989d56872ef29f14

Upstream Comparison:

Branch GIT ID: b9b0dee253287fb23e10736f989d56872ef29f14
Jenkins Build URL: Acceptance Upstream DXP (master) #2877

ci:test:stable - 21 out of 26 jobs PASSED
21 Successful Jobs:
ci:test:relevant - 21 out of 27 jobs PASSED
21 Successful Jobs:
For more details click here.

This pull contains no unique failures.


Failures in common with acceptance upstream results at b9b0dee:
  1. test-portal-acceptance-pullrequest-batch(master)/modules-compile-jdk8/0
    Job Results:

    15 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=14 #27763
           [exec] > Task :third-party:org-springframework-orm:compileJava FAILED
           [exec] 
           [exec] > Task :third-party:org-springframework-orm:copyLibs SKIPPED
           [exec] > Task :third-party:org-springframework-orm:copyPatchOriginalLibClasses
           [exec] > Task :third-party:org-springframework-orm:copyCSS NO-SOURCE
           [exec] > Task :third-party:org-springframework-orm:buildCSS NO-SOURCE
           [exec] > Task :third-party:org-springframework-orm:processResources
           [exec] > Task :third-party:org-springframework-orm:jarPatched
           [exec] > Task :third-party:org-springframework-orm:zipZippableResources UP-TO-DATE
           [exec] > Task :third-party:org-springframework-orm:jarSources
           [exec] Note: Some input files use unchecked or unsafe operations.
           [exec] Note: Recompile with -Xlint:unchecked for details.
           [exec] 
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:buildWSDLCompileCheckVatService
           [exec] 
           [exec] > Task :third-party:org-springframework-orm:jarSourcesCommercial
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:buildWSDLCheckVatService
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:buildWSDL
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:compileJava NO-SOURCE
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:copyLibs SKIPPED
           [exec] > Task :util:deployment-helper:compileJava
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:copyCSS NO-SOURCE
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:buildCSS NO-SOURCE
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:processResources
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:classes
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:zipZippableResources UP-TO-DATE
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:jar
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:autoUpdateXml SKIPPED
           [exec] > Task :third-party:vies-taxud-eu-europa-ec:assemble
           [exec] > Task :util:deployment-helper:copyLibs SKIPPED
           [exec] > Task :util:deployment-helper:copyCSS NO-SOURCE
           [exec] > Task :util:deployment-helper:buildCSS NO-SOURCE
           [exec] > Task :util:deployment-helper:processResources
           [exec] > Task :util:deployment-helper:classes
           [exec] > Task :util:deployment-helper:zipZippableResources UP-TO-DATE
           [exec] 
           [exec] > Task :util:deployment-helper:jar
           [exec] warning: Export com.liferay.deployment.helper,  has 1,  private references [org.zeroturnaround.zip]
           [exec] 
  2. test-portal-acceptance-pullrequest-batch(master)/rest-builder-jdk8/0
    Job Results:

    0 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=0 #349705

      Please fix semantic versioning on brianikim/COMMERCE-8305

           [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
           [exec] = ================================================== ========== ========== ========== ========== ==========
           [exec] * com.liferay.portal.util                            MAJOR      17.0.0     17.0.0     18.0.0     VERSION INCREASE REQUIRED
           [exec]   >   class      com.liferay.portal.util.PropsValues
           [exec]     -   field      PORTLET_URL_GENERATE_BY_PATH_ENABLED
           [exec]       -   access     final
           [exec]       -   access     static
           [exec]       -   return     boolean
           [exec] [Baseline Warning] Bundle Version Change Recommended: 22.0.0
           [exec] [Baseline Report] Mode: diff (persisted)
           [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-impl/portal-impl.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.impl/21.0.1/738b94219ff9ecbdac36ec1a817066bfb7086e42/com.liferay.portal.impl-21.0.1.jar

  3. test-portal-acceptance-pullrequest-batch(master)/semantic-versioning-jdk8/0
    Job Results:

    0 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=0 #339596

      Please fix semantic versioning on brianikim/COMMERCE-8305

           [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
           [exec] = ================================================== ========== ========== ========== ========== ==========
           [exec] * com.liferay.portal.kernel.util                     MAJOR      23.0.0     23.0.0     24.0.0     VERSION INCREASE REQUIRED
           [exec]   >   interface  com.liferay.portal.kernel.util.PropsKeys
           [exec]     -   field      PORTLET_URL_GENERATE_BY_PATH_ENABLED
           [exec]       -   access     final
           [exec]       -   access     static
           [exec]       -   constant   portlet.url.generate.by.path.enabled
           [exec]       -   return     java.lang.String
           [exec] [Baseline Warning] Bundle Version Change Recommended: 34.0.0
           [exec] [Baseline Report] Mode: diff (persisted)
           [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/33.0.0/a67dd6264c43c6276987fcefdfc7290fe126de7a/com.liferay.portal.kernel-33.0.0.jar

  4. test-portal-acceptance-pullrequest-batch(master)/modules-integration-mysql57-jdk8/0
    Job Results:

    3295 Tests Passed.
    5 Tests Failed.

    1. AXIS_VARIABLE=5 #29565
      1. com.liferay.headless.commerce.admin.channel.resource.v1_0.test.ChannelResourceTest.testDeleteChannelByExternalReferenceCode
        java.lang.UnsupportedOperationException: This method needs to be implemented
          at com.liferay.headless.commerce.admin.channel.resource.v1_0.test.BaseChannelResourceTestCase.testDeleteChannelByExternalReferenceCode_addChannel(BaseChannelResourceTestCase.java:519)
          at com.liferay.headless.commerce.admin.channel.resource.v1_0.test.BaseChannelResourceTestCase.testDeleteChannelByExternalReferenceCode(BaseChannelResourceTestCase.java:498)
          at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
          at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
          at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
          at java.lang.reflect.Method.invoke(Method.java:498)
          at com.liferay.arquillian.extension.junit.bridge.server.TestExecutorRunnable$3.evaluate(TestExecutorRunnable.java:353)
          at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
          at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.port...
      2. com.liferay.headless.commerce.admin.channel.resource.v1_0.test.ChannelResourceTest.testGetChannelByExternalReferenceCode
        java.lang.UnsupportedOperationException: This method needs to be implemented
          at com.liferay.headless.commerce.admin.channel.resource.v1_0.test.BaseChannelResourceTestCase.testGetChannelByExternalReferenceCode_addChannel(BaseChannelResourceTestCase.java:538)
          at com.liferay.headless.commerce.admin.channel.resource.v1_0.test.BaseChannelResourceTestCase.testGetChannelByExternalReferenceCode(BaseChannelResourceTestCase.java:526)
          at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
          at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
          at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
          at java.lang.reflect.Method.invoke(Method.java:498)
          at com.liferay.arquillian.extension.junit.bridge.server.TestExecutorRunnable$3.evaluate(TestExecutorRunnable.java:353)
          at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
          at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.ker...
      3. com.liferay.headless.commerce.admin.channel.resource.v1_0.test.ChannelResourceTest.testGraphQLGetChannelByExternalReferenceCode
        java.lang.IllegalArgumentException: Expected starts with '{', but found 'n' in 'null'
          at com.liferay.headless.commerce.admin.channel.client.json.BaseJSONParser._assertStartsWithAndEndsWith(BaseJSONParser.java:259)
          at com.liferay.headless.commerce.admin.channel.client.json.BaseJSONParser.parseToDTO(BaseJSONParser.java:50)
          at com.liferay.headless.commerce.admin.channel.client.serdes.v1_0.ChannelSerDes.toDTO(ChannelSerDes.java:38)
          at com.liferay.headless.commerce.admin.channel.resource.v1_0.test.BaseChannelResourceTestCase.testGraphQLGetChannelByExternalReferenceCode(BaseChannelResourceTestCase.java:551)
          at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
          at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
          at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
          at java.lang.reflect.Method.invoke(Method.java:498)
          at com.liferay.arquillian.extension.junit.bridge.server.TestExecutorRunnable$3.evaluate(TestExecutorRunnable.java:353)
          at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
          at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
          at com.liferay.portal.kernel.t...
      4. ...

@liferay-continuous-integration
Copy link
Collaborator

@ivicac
Copy link

ivicac commented Feb 2, 2022

ci:forward

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

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

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

Skipping previously passed test suites:
ci:test:relevant
ci:test:sf

@liferay-continuous-integration
Copy link
Collaborator

All required test suite(s) passed.
Forwarding pull request to brianchandotcom.
Console

@liferay-continuous-integration
Copy link
Collaborator

Pull request has been successfully forwarded to brianchandotcom#112962
Console

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.