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

Exam mode: Change test case default visibility to after results release date #8451

Merged
merged 98 commits into from
May 18, 2024

Merge branch 'refs/heads/develop' into enhancement/exam/change-test-c…

8a56815
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

Exam mode: Change test case default visibility to after results release date #8451

Merge branch 'refs/heads/develop' into enhancement/exam/change-test-c…
8a56815
Select commit
Loading
Failed to load commit list.
GitHub Actions / Java Architecture Tests succeeded May 17, 2024 in 1s

50 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ArchitectureTest.xml 19✅ 4s
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest.xml 2✅ 295ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest.xml 2✅ 293ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest.xml 9✅ 258ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ResourceArchitectureTest.xml 4✅ 80ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ServiceArchitectureTest.xml 4✅ 153ms
build/test-results/test/TEST-de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest.xml 8✅ 57ms
build/test-results/test/TEST-de.tum.in.www1.artemis.domain.notification.NotificationPlaceholderArchitectureTest.xml 2✅ 25ms

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ArchitectureTest.xml

19 tests were completed in 4s with 19 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.ArchitectureTest 19✅ 4s

✅ de.tum.in.www1.artemis.architecture.ArchitectureTest

✅ ensureSpringComponentsAreProfileAnnotated()
✅ testNoJunitJupiterAssertions()
✅ hasMatchingAuthorizationTestClassBeCorrectlyImplemented()
✅ testFileWriteUsage()
✅ testCorrectStringUtils()
✅ testValidSimpMessageSendingOperationsUsage()
✅ testLogging()
✅ testNoHazelcastUsageInConstructors()
✅ testJSONImplementations()
✅ testNoRestControllersImported()
✅ testNoDirectGitCommitCalls()
✅ testNoCollectorsToList()
✅ testClassNameAndVisibility()
✅ testGsonExclusion()
✅ testNoJUnit4()
✅ shouldNotUserAutowiredAnnotation()
✅ testCorrectLoggerFields()
✅ testJsonIncludeNonEmpty()
✅ testNullnessAnnotations()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest.xml

2 tests were completed in 295ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest 2✅ 295ms

✅ de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest

✅ testEnforceRoleInExerciseEndpointHasExerciseIdParameter()
✅ testEnforceRoleInCourseEndpointHasCourseIdParameter()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest.xml

2 tests were completed in 293ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest 2✅ 293ms

✅ de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest

✅ testSetLongFeedbackTextNotUsed()
✅ testGetLongFeedbackTextNotUsed()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest.xml

9 tests were completed in 258ms with 9 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest 9✅ 258ms

✅ de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest

✅ testNoEntityGraphsOnQueries()
✅ testNoUnusedRepositoryMethods()
✅ testTransactional()
✅ shouldBeInRepositoryPackage()
✅ testRepositoryParamAnnotation()
✅ persistenceShouldNotAccessServices()
✅ testJPQLStyle()
✅ shouldBeNamedRepository()
✅ testOnlySpringTransactionalAnnotation()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ResourceArchitectureTest.xml

4 tests were completed in 80ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.ResourceArchitectureTest 4✅ 80ms

✅ de.tum.in.www1.artemis.architecture.ResourceArchitectureTest

✅ allPublicMethodsShouldReturnResponseEntity()
✅ shouldBeNamedResource()
✅ shouldBeInResourcePackage()
✅ shouldCorrectlyUseRequestMappingAnnotations()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ServiceArchitectureTest.xml

4 tests were completed in 153ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.architecture.ServiceArchitectureTest 4✅ 153ms

✅ de.tum.in.www1.artemis.architecture.ServiceArchitectureTest

✅ shouldBeNamedService()
✅ testCorrectServiceAnnotation()
✅ shouldBeInServicePackage()
✅ testNoWrongServiceImports()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest.xml

8 tests were completed in 57ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest 8✅ 57ms

✅ de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest

✅ testEnforceAdminAnnotations()
✅ testEnforceAtLeastEditorAnnotations()
✅ testNoPreAuthorizeOnRestEndpoints()
✅ testEnforceAtLeastTutorAnnotations()
✅ testEnforceAtLeastInstructorAnnotations()
✅ testEnforceAtLeastStudentAnnotations()
✅ testEnforceNothingAnnotations()
✅ testNoPreAuthorizeOnRestControllers()

✅ build/test-results/test/TEST-de.tum.in.www1.artemis.domain.notification.NotificationPlaceholderArchitectureTest.xml

2 tests were completed in 25ms with 2 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
de.tum.in.www1.artemis.domain.notification.NotificationPlaceholderArchitectureTest 2✅ 25ms

✅ de.tum.in.www1.artemis.domain.notification.NotificationPlaceholderArchitectureTest

✅ testNotificationPlaceholderCreatorStringArrayCreation()
✅ testPlaceholderCreatorMethodSignature()