Skip to content
Pull Request Opened

Development: Add check for REST endpoints to be in kebab case #5930

Sign in to view logs

Development: Add check for REST endpoints to be in kebab case

Development: Add check for REST endpoints to be in kebab case #5930

Triggered via pull request August 12, 2024 17:41
@SimonEntholzerSimonEntholzer
opened #9210
Status Success
Total duration 11s
Artifacts –

pullrequest-opened.yml

on: pull_request_target
assign
2s
assign
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
de.tum.in.www1.artemis.architecture.ArchitectureTest ► testUseKebabCaseForRestEndpoints(): src/test/java/de/tum/in/www1/artemis/architecture/ArchitectureTest.java#L149
Failed test found in: build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ArchitectureTest.xml Error: java.lang.AssertionError: Architecture Violation [Priority: MEDIUM] - Rule 'methods should use kebab case for rest mapping annotations' was violated (1 times):
test#L0
Junit Results ran 7663 in 1090.897 seconds 0 Errored, 4 Failed, 2 Skipped
testUseKebabCaseForRestEndpoints(): /home/runner/work/Artemis/Artemis/src/test/java/de/tum/in/www1/artemis/architecture/ArchitectureTest.java#L139
Junit test testUseKebabCaseForRestEndpoints() failed java.lang.AssertionError: Architecture Violation [Priority: MEDIUM] - Rule 'methods should use kebab case for rest mapping annotations' was violated (4 times): de.tum.in.www1.artemis.web.rest.ExerciseGroupResource.createExerciseGroup(java.lang.Long, java.lang.Long, de.tum.in.www1.artemis.domain.exam.ExerciseGroup) violates rule to use kebab case for rest call annotations de.tum.in.www1.artemis.web.rest.LtiResource.lti13DeepLinking(java.lang.Long, java.util.Set, java.lang.String, java.lang.String) violates rule to use kebab case for rest call annotations de.tum.in.www1.artemis.web.rest.admin.AdminLtiConfigurationResource.lti13DynamicRegistration(java.lang.String, java.lang.String) violates rule to use kebab case for rest call annotations de.tum.in.www1.artemis.web.rest.open.PublicLtiResource.lti13LaunchRedirect(jakarta.servlet.http.HttpServletRequest, jakarta.servlet.http.HttpServletResponse) violates rule to use kebab case for rest call annotations
testCreateProgrammingExercise(): /home/runner/work/Artemis/Artemis/src/test/java/de/tum/in/www1/artemis/exercise/programming/ProgrammingExerciseLocalVCLocalCIIntegrationTest.java#L149
Junit test testCreateProgrammingExercise() failed org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in de.tum.in.www1.artemis.localvcci.LocalVCLocalCITestService was not fulfilled within 10 seconds.
testFetchPush_repositoryDoesNotExist(): /home/runner/work/Artemis/Artemis/src/test/java/de/tum/in/www1/artemis/localvcci/LocalVCIntegrationTest.java#L76
Junit test testFetchPush_repositoryDoesNotExist() failed org.apache.commons.io.IOExceptionList: 1 exception(s): [org.apache.commons.io.IOIndexedException: FileNotFoundException #6: File does not exist: /tmp/SHORTB42038AD0740004D8608FD0049C862C9AF23TSTEXC/some-repository-slug.git/gc.log.lock]
testPriorityRunningExam(): /home/runner/work/Artemis/Artemis/src/test/java/de/tum/in/www1/artemis/localvcci/LocalVCLocalCIIntegrationTest.java#L945
Junit test testPriorityRunningExam() failed org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in de.tum.in.www1.artemis.localvcci.LocalVCLocalCIIntegrationTest was not fulfilled within 10 seconds.
de.tum.in.www1.artemis.architecture.ArchitectureTest ► testUseKebabCaseForRestEndpoints(): src/test/java/de/tum/in/www1/artemis/architecture/ArchitectureTest.java#L149
Failed test found in: build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ArchitectureTest.xml Error: java.lang.AssertionError: Architecture Violation [Priority: MEDIUM] - Rule 'methods should use kebab case for rest mapping annotations' was violated (4 times):
de.tum.in.www1.artemis.exercise.programming.ProgrammingExerciseLocalVCLocalCIIntegrationTest ► testCreateProgrammingExercise(): src/test/java/de/tum/in/www1/artemis/localvcci/LocalVCLocalCITestService.java#L556
Failed test found in: build/test-results/test/TEST-de.tum.in.www1.artemis.exercise.programming.ProgrammingExerciseLocalVCLocalCIIntegrationTest.xml Error: org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in de.tum.in.www1.artemis.localvcci.LocalVCLocalCITestService was not fulfilled within 10 seconds.
de.tum.in.www1.artemis.localvcci.LocalVCIntegrationTest ► testFetchPush_repositoryDoesNotExist(): src/test/java/de/tum/in/www1/artemis/localvcci/LocalVCIntegrationTest.java#L84
Failed test found in: build/test-results/test/TEST-de.tum.in.www1.artemis.localvcci.LocalVCIntegrationTest.xml Error: org.apache.commons.io.IOExceptionList: 1 exception(s): [org.apache.commons.io.IOIndexedException: FileNotFoundException #6: File does not exist: /tmp/SHORTB42038AD0740004D8608FD0049C862C9AF23TSTEXC/some-repository-slug.git/gc.log.lock]
de.tum.in.www1.artemis.localvcci.LocalVCLocalCIIntegrationTest ► testPriorityRunningExam(): src/test/java/de/tum/in/www1/artemis/localvcci/LocalVCLocalCIIntegrationTest.java#L979
Failed test found in: build/test-results/test/TEST-de.tum.in.www1.artemis.localvcci.LocalVCLocalCIIntegrationTest.xml Error: org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in de.tum.in.www1.artemis.localvcci.LocalVCLocalCIIntegrationTest was not fulfilled within 10 seconds.
assign
The following actions use a deprecated Node.js version and will be forced to run on node20: technote-space/assign-author@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/