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
build: BAZEL to SBT migration (DEV-508) #2002
Conversation
webapi/src/test/scala/org/knora/webapi/e2e/http/ServerVersionE2ESpec.scala
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.
I don't feel like I have much to say but it looks interesting :) Added mostly minor deletion suggestions for uncommented code.
"v1/test", | ||
"v2/test" |
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.
Don't we use this anymore?
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.
It is not necessary anymore. Inside the test that tests this functionality, the config contains these entries.
@@ -112,7 +113,8 @@ class ITKnoraLiveSpec(_system: ActorSystem) | |||
|
|||
override def afterAll(): Unit = | |||
/* Stop the server when everything else has finished */ | |||
appActor ! AppStop() | |||
// appActor ! AppStop() |
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.
Do we still need this line of code?
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.
no, removed
webapi/src/test/scala/org/knora/webapi/e2e/ClientTestDataCollector.scala
Outdated
Show resolved
Hide resolved
webapi/src/test/scala/org/knora/webapi/e2e/admin/lists/OldListsRouteADMFeatureE2ESpec.scala
Show resolved
Hide resolved
webapi/src/test/scala/org/knora/webapi/e2e/http/ServerVersionE2ESpec.scala
Outdated
Show resolved
Hide resolved
webapi/src/test/scala/org/knora/webapi/http/version/BuildInfoSpec.scala
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.
certainly the license and URLs need to be adjusted in the build file. Apart from that just minor comments and questions.
(Also, sorry that I didn't bundle most comments with the review!)
Overall I like SBT much better than Bazel :)
clientTestDataCollector.addFile( | ||
TestDataFileContent( | ||
filePath = TestDataFilePath( | ||
directoryPath = clientTestDataPath, | ||
filename = "create-int-value-with-custom-UUID-request", | ||
fileExtension = "json" | ||
), | ||
text = jsonLDEntity | ||
) | ||
) | ||
|
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.
why is this deleted?
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.
Interesting finding, I can't even spot it in the git blame who did it 😄
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.
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.
because it was in the "do not create an integer value if the custom UUID is not part of the custom IRI" test. the "create an integer value with a custom UUID" was the previous test (same file name).
Kudos, SonarCloud Quality Gate passed!
|
resolves DEV-508