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

LPS-157505 Refactor KB portlet to use MVC action commands #3236

Closed
wants to merge 55 commits into from

Conversation

adolfopa
Copy link
Collaborator

No description provided.

@liferay-content-management
Copy link
Owner

ci:test:sf

@liferay-content-management
Copy link
Owner

ci:test:relevant

@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.

@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: 64e30a3d6793ecde6b0c64840870872978fbd877

Sender Branch:

Branch Name: LPS-157505
Branch GIT ID: 9b031f84186a1cec2a4f23da4c857e84761e2e86

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

@liferay-continuous-integration
Copy link
Collaborator

@adolfopa
Copy link
Collaborator Author

ci:stop

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 0 out of 1 jobs passed in 14 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 64e30a3d6793ecde6b0c64840870872978fbd877

Upstream Comparison:

Branch GIT ID: 64e30a3d6793ecde6b0c64840870872978fbd877
Jenkins Build URL: Acceptance Upstream DXP (master) #3531

ci:test:relevant - 0 out of 1 jobs PASSED
For more details click here.

Failures unique to this pull:

  1. test-portal-acceptance-pullrequest(master)
    Job Results:

    0 Jobs Passed.
    1 Job Failed.

    Build was aborted

For upstream results, click here.

@adolfopa
Copy link
Collaborator Author

ci:test:knowledge-base

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:knowledge-base - 35 out of 42 jobs passed in 1 hour 37 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 64e30a3d6793ecde6b0c64840870872978fbd877

Upstream Comparison:

Branch GIT ID: 64e30a3d6793ecde6b0c64840870872978fbd877
Jenkins Build URL: Acceptance Upstream DXP (master) #3531

ci:test:knowledge-base - 35 out of 42 jobs PASSED
35 Successful Jobs:
For more details click here.

Failures unique to this pull:


Failures in common with acceptance upstream results at 64e30a3:
  1. central-requirements-jdk8/0/0
    1. com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles
      junit.framework.AssertionFailedError: Incorrect configuration of dependency {compileOnly group: "com.liferay.portal", name: "com.liferay.portal.kernel", version: "default"} in modules/apps/cookies/cookies-test/build.gradle, use one of these instead: "testCompile", "testIntegrationCompile", "testIntegrationRuntime", "testRuntime"
        at com.liferay.portal.modules.ModulesStructureTest._testGradleFile(ModulesStructureTest.java:1506)
        at com.liferay.portal.modules.ModulesStructureTest.access$800(ModulesStructureTest.java:68)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:402)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:376)
        at java.nio.file.Files.walkFileTree(Files.java:2670)
        at java.nio.file.Files.walkFileTree(Files.java:2742)
        at com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles(ModulesStructureTest.java:374)
      
  2. modules-semantic-versioning-jdk8/0/4
    1. com.liferay.semantic.versioning.SemanticVersioningTest.testSemanticVersioning[/apps/notification/notification-rest-api]
      java.lang.AssertionError:   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
      = ================================================== ========== ========== ========== ========== ==========
      * com.liferay.notification.rest.dto.v1_0             MINOR      1.1.0      1.1.0      1.2.0      VERSION INCREASE REQUIRED
        +   class      com.liferay.notification.rest.dto.v1_0.NotificationQueueEntry
          +   implements java.io.Serializable
          +   field      actions
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_ONLY'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.util.Map
          +   field      bcc
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_WRITE'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.lang.String
          +   field      body
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_WRITE'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.lang.Strin...
  3. ...

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: fc48ef9852786b74b8c9d4a6c55faf831429fb34

Sender Branch:

Branch Name: LPS-157505
Branch GIT ID: 4b7cdb307c832516677ff610b2303edce65d66df

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 - 26 out of 26 jobs passed

✔️ ci:test:relevant - 48 out of 49 jobs passed in 2 hours 25 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: fc48ef9852786b74b8c9d4a6c55faf831429fb34

Upstream Comparison:

Branch GIT ID: fc48ef9852786b74b8c9d4a6c55faf831429fb34
Jenkins Build URL: Acceptance Upstream DXP (master) #3538

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

This pull contains no unique failures.


Failures in common with acceptance upstream results at fc48ef9:
  1. central-requirements-jdk8/0/0
    1. com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles
      junit.framework.AssertionFailedError: Incorrect configuration of dependency {compileOnly group: "com.liferay.portal", name: "com.liferay.portal.kernel", version: "default"} in modules/apps/cookies/cookies-test/build.gradle, use one of these instead: "testCompile", "testIntegrationCompile", "testIntegrationRuntime", "testRuntime"
        at com.liferay.portal.modules.ModulesStructureTest._testGradleFile(ModulesStructureTest.java:1506)
        at com.liferay.portal.modules.ModulesStructureTest.access$800(ModulesStructureTest.java:68)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:402)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:376)
        at java.nio.file.Files.walkFileTree(Files.java:2670)
        at java.nio.file.Files.walkFileTree(Files.java:2742)
        at com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles(ModulesStructureTest.java:374)
      

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:knowledge-base - 38 out of 42 jobs passed in 2 hours 53 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: fc48ef9852786b74b8c9d4a6c55faf831429fb34

Upstream Comparison:

Branch GIT ID: fc48ef9852786b74b8c9d4a6c55faf831429fb34
Jenkins Build URL: Acceptance Upstream DXP (master) #3538

ci:test:knowledge-base - 38 out of 42 jobs PASSED
38 Successful Jobs:
For more details click here.

This pull contains no unique failures.


Failures in common with acceptance upstream results at fc48ef9:
  1. central-requirements-jdk8/0/0
    1. com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles
      junit.framework.AssertionFailedError: Incorrect configuration of dependency {compileOnly group: "com.liferay.portal", name: "com.liferay.portal.kernel", version: "default"} in modules/apps/cookies/cookies-test/build.gradle, use one of these instead: "testCompile", "testIntegrationCompile", "testIntegrationRuntime", "testRuntime"
        at com.liferay.portal.modules.ModulesStructureTest._testGradleFile(ModulesStructureTest.java:1506)
        at com.liferay.portal.modules.ModulesStructureTest.access$800(ModulesStructureTest.java:68)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:402)
        at com.liferay.portal.modules.ModulesStructureTest$4.visitFile(ModulesStructureTest.java:376)
        at java.nio.file.Files.walkFileTree(Files.java:2670)
        at java.nio.file.Files.walkFileTree(Files.java:2742)
        at com.liferay.portal.modules.ModulesStructureTest.testScanGradleFiles(ModulesStructureTest.java:374)
      
  2. modules-semantic-versioning-jdk8/0/4
    1. com.liferay.semantic.versioning.SemanticVersioningTest.testSemanticVersioning[/apps/notification/notification-rest-api]
      java.lang.AssertionError:   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
      = ================================================== ========== ========== ========== ========== ==========
      * com.liferay.notification.rest.dto.v1_0             MINOR      1.1.0      1.1.0      1.2.0      VERSION INCREASE REQUIRED
        +   class      com.liferay.notification.rest.dto.v1_0.NotificationQueueEntry
          +   implements java.io.Serializable
          +   field      actions
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_ONLY'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.util.Map
          +   field      bcc
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_WRITE'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.lang.String
          +   field      body
            +   access     protected
            +   annotated  com.fasterxml.jackson.annotation.JsonProperty
              +   property   access='READ_WRITE'
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   annotated  com.liferay.portal.vulcan.graphql.annotation.GraphQLField
            +   return     java.lang.Strin...
  3. modules-semantic-versioning-jdk8/0/5
    1. com.liferay.semantic.versioning.SemanticVersioningTest.testSemanticVersioning[/apps/object/object-api]
      java.lang.AssertionError:   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
      = ================================================== ========== ========== ========== ========== ==========
      * com.liferay.object.field.business.type             MAJOR      2.1.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
        -   interface  com.liferay.object.field.business.type.ObjectFieldBusinessTypeServicesTracker
          -   access     abstract
          -   method     getObjectFieldBusinessType(java.lang.String)
            -   access     abstract
            -   return     com.liferay.object.field.business.type.ObjectFieldBusinessType
          -   method     getObjectFieldBusinessTypes()
            -   access     abstract
            -   return     java.util.List
          -   method     getObjectFieldDBTypes()
            -   access     abstract
            -   return     java.util.Set
        +   interface  com.liferay.object.field.business.type.ObjectFieldBusinessTypeTracker
          +   access     abstract
          +   method     getObjectFieldBusinessType(java.lang.String)
            +   access     abstract
            +   return     com.liferay.object.field.business.type.ObjectFieldBusinessType
          +   method     getObjectFieldBusinessTypes()
            +   access     abstract
            +   return     java.util.List
          +   method     getObjectFieldDBTypes()
            +   access     abstract
            +   return     java.util.Set
      * com.liferay.object.field.filter.parser             MAJOR      1.0.0      1.0.0      2.0.0      VERSION INCREASE REQUIRED
        -   interface  com.lif...

@liferay-continuous-integration
Copy link
Collaborator

@AliciaGarciaGarcia
Copy link
Collaborator

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#120241
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.

8 participants