-
Notifications
You must be signed in to change notification settings - Fork 5
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
Support Spring #123
Support Spring #123
Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is src/main/resources/META-INF/resources/swagger
for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If mysql connector 8.x is used, docker compose should be updated to use 8.0 too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3e132dc
to
c14287f
Compare
src/test/groovy/com/paiondata/astraios/config/SpringbootJarITSpec.groovy
Outdated
Show resolved
Hide resolved
src/test/groovy/com/paiondata/astraios/config/DockerComposeITSpec.groovy
Outdated
Show resolved
Hide resolved
src/test/groovy/com/paiondata/astraios/config/AbstractITSpec.groovy
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/ApplicationConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/keyConfig/ApplicationConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/config/SecurityConfigProperties.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better 👍🏻 ...and can be even much better
ee000be
to
b3080d4
Compare
src/main/java/com/paiondata/astraios/settings/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/com/paiondata/astraios/settings/JpaDatastoreConfig.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost done. Just minor tweaks 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
Description
Change
elide.version
from 7.0.0-pr6 to 7.0.0 becauseElideSettings
conflicts within the last method undersrc/main/java/com/paiondata/astraios/config/ElideConfiguration.java
.The version 7.0.0 is also officially used in Elide Springboot Example.I turn off checking the annotation requirements for the main method by changing
UncommentedMain
value form error to warning incheckstyle.xml
beacause even though I annotated it,it still reported an error.I don't think turning off the check will cause too many problems and it maybe the bug.Fix bug, In
src/test/groovy/com/paiondata/astraios/config/DockerComposeITSpec.groovy
, I set the wait time to 10 minutes because it did not wait for docker to start before sending the request.I overide
stop()
method insrc/test/java/com/paiondata/astraios/config/KeepAliveMySQLContainer.java
to keep the mysql container running for later test insrc/test/groovy/com/paiondata/astraios/config/AbstractITSpec.groovy
Create a static holder class
src/test/java/com/paiondata/astraios/config/TestContextHolder.java
to save and retrieve the random port. Solved the problem that multiple tests preempt each other's ports.