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-88139 update unit test #66198

Closed
wants to merge 5 commits into from
Closed

Conversation

linolaoj
Copy link

No description provided.

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 1 minute 47 seconds 809 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 30a58299f1b061a77ab065580e4d93fc00905fcd

Sender Branch:

Branch Name: pr-172
Branch GIT ID: 89d1003670d45b704c48fc7e6c4d8e0b41da7c23

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

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 23 out of 25 jobs passed in 1 hour 10 minutes 2 seconds 128 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: b044872f3d40d00facf19bc46cd0586789cd5c1c

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 50bfef71122954d4156e2182dbd878a791d0dccf

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

This pull contains no unique failures.


Failures in common with acceptance upstream results at cfcc38e:

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 522e707...1c8b5a0

@brianchandotcom
Copy link
Owner

Strange I'm not able to see where "loadDDMFieldTypes" is used? I only see the key, but not its usage.

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.

3 participants