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

Quiz exercises: Fix an issue with the multiple choice visual mode #9284

Merged
merged 9 commits into from
Sep 11, 2024

Merge branch 'develop' into bugfix/quiz-exercises/fix-visual-mode

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

Quiz exercises: Fix an issue with the multiple choice visual mode #9284

Merge branch 'develop' into bugfix/quiz-exercises/fix-visual-mode
b8c9175
Select commit
Loading
Failed to load commit list.
GitHub Actions / Java Architecture Tests succeeded Sep 10, 2024 in 0s

54 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 20✅ 5s
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest.xml 2✅ 268ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest.xml 2✅ 352ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest.xml 11✅ 336ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ResourceArchitectureTest.xml 4✅ 195ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ServiceArchitectureTest.xml 5✅ 180ms
build/test-results/test/TEST-de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest.xml 8✅ 69ms
build/test-results/test/TEST-de.tum.in.www1.artemis.domain.notification.NotificationPlaceholderArchitectureTest.xml 2✅ 26ms

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

20 tests were completed in 5s with 20 passed, 0 failed and 0 skipped.

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

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

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

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

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

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

✅ 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 352ms with 2 passed, 0 failed and 0 skipped.

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

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

✅ testSetLongFeedbackTextNotUsed()
✅ testGetLongFeedbackTextNotUsed()

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

11 tests were completed in 336ms with 11 passed, 0 failed and 0 skipped.

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

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

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

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

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

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

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

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

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

5 tests were completed in 180ms with 5 passed, 0 failed and 0 skipped.

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

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

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

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

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

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

✅ 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 26ms with 2 passed, 0 failed and 0 skipped.

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

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

✅ testNotificationPlaceholderCreatorStringArrayCreation()
✅ testPlaceholderCreatorMethodSignature()