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.

Sign up for GitHub

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

Adaptive learning: Add endpoint to retrieve student course metrics #8508

Merged
merged 39 commits into from
May 14, 2024

Update MetricsService.java

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

Adaptive learning: Add endpoint to retrieve student course metrics #8508

Update MetricsService.java
53b8bc0
Select commit
Loading
Failed to load commit list.
GitHub Actions / Java Architecture Tests succeeded May 14, 2024 in 0s

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✅ 5s
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.EnforceRoleInResourceArchitectureTest.xml 2✅ 138ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.FeedbackArchitectureTest.xml 2✅ 368ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.RepositoryArchitectureTest.xml 9✅ 254ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ResourceArchitectureTest.xml 4✅ 89ms
build/test-results/test/TEST-de.tum.in.www1.artemis.architecture.ServiceArchitectureTest.xml 4✅ 140ms
build/test-results/test/TEST-de.tum.in.www1.artemis.authorization.AuthorizationArchitectureTest.xml 8✅ 52ms
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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

✅ 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()
View more details on GitHub Actions