Skip to content

[TTINC2] Added main logic to allow the remote code execution#62

Merged
IrminDev merged 33 commits into
mainfrom
feature/rabbitMQ
May 25, 2026
Merged

[TTINC2] Added main logic to allow the remote code execution#62
IrminDev merged 33 commits into
mainfrom
feature/rabbitMQ

Conversation

@IrminDev

@IrminDev IrminDev commented May 24, 2026

Copy link
Copy Markdown
Owner

Title

Code execution sandbox and code editor

Description

This PR merges the features that were developed for the second increment of this project. It add completely functional the sandbox to execute code safely and adds the code editor in the frontend using monaco.

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Style/UI update (formatting, styling, etc.)
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • Unit tests
  • Integration tests
  • Manual testing

Test Configuration:

  • Java version: 21
  • PostgreSQL version: 18.1
  • OS: Arch Linux 7.0.7

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Test Coverage

  • All new code is covered by tests
  • Overall coverage is at least 70%
  • Coverage for changed files is at least 80%

Screenshots (if applicable)

Additional Notes

Authors: @JohannTF and @rodolfo-rgb

IrminDev and others added 30 commits April 6, 2026 22:30
…ter understanding in the workflow and fixed business logic
JohannTF and others added 2 commits May 23, 2026 18:37
…Executor to standardize container configuration across executors
Copilot AI review requested due to automatic review settings May 24, 2026 23:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown

Test Results

 31 files  + 6   31 suites  +6   14s ⏱️ +4s
154 tests +18  154 ✅ +18  0 💤 ±0  0 ❌ ±0 
155 runs  +18  155 ✅ +18  0 💤 ±0  0 ❌ ±0 

Results for commit 6113f06. ± Comparison against base commit 0c27bcb.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown

Worker Test Results

24 tests   24 ✅  1m 4s ⏱️
 9 suites   0 💤
 9 files     0 ❌

Results for commit 6113f06.

♻️ This comment has been updated with latest results.

@github-actions

Copy link
Copy Markdown

Code Coverage Report

Overall Project 59.93% -22.71%
Files changed 54.54%

File Coverage
SignUpRequest.java 100% 🍏
PasswordResetToken.java 100% 🍏
CreateAssignmentRequest.java 100% 🍏
ExecutionRequest.java 100% 🍏
PageResponse.java 100% 🍏
CheckExecutionController.java 100% 🍏
SecurityConfig.java 100% 🍏
MinioConfig.java 100% 🍏
RabbitConfig.java 100% 🍏
JwtUtil.java 100% 🍏
ExecutionType.java 100% 🍏
Language.java 100% 🍏
ComparatorType.java 100% 🍏
ExecutionStatus.java 100% 🍏
ExecutionDTO.java 100% 🍏
UserDTO.java 100% 🍏
AssignmentController.java 96.43% -3.57% 🍏
AssignmentDTO.java 96.3% -3.7% 🍏
AdminInitializer.java 91.43% 🍏
User.java 90.43% 🍏
AuthService.java 88.11% -7.34%
ExecutionReport.java 87.64% -12.36% 🍏
AuthController.java 80.77% -5.77%
Assignment.java 80.49% -19.51% 🍏
AssignmentService.java 78.39% -21.61%
ExecutionRequestService.java 75% -25%
ObjectKeyBuilder.java 70.16% -29.84%
TestCase.java 59.57% -40.43%
ReferenceSolution.java 54.55% -45.45%
Execution.java 50% -50%
AssignmentMapper.java 43.54% -56.46%
ExecutionMapper.java 43.24% -56.76%
FileExtensionUtil.java 39.13% -60.87%
TestCaseInfo.java 33.33% -66.67%
TestGenerationJob.java 31.82% -68.18%
ExecutionJob.java 30.88% -69.12%
CsvProgressWebSocketHandler.java 25.64% 🍏
TestGenerationRequestProducer.java 15.15% -84.85%
Submission.java 13.95% -86.05%
ExecutionResultListener.java 13.33% -86.67%
TestGenerationResultListener.java 12.82% -87.18%
ExecutionRequestProducer.java 10.64% -89.36%
ExecutionResultService.java 9.8% -90.2%
CsvRegistrationService.java 9.34% -2.73%
ObjectStorageService.java 8.22% -91.78%
TestCaseMapper.java 0%
SubmissionMapper.java 0%
ReferenceSolutionMapper.java 0%
TestGenerationResult.java 0%
TestCaseResult.java 0%
SubmissionDTO.java 0%
TestCaseDTO.java 0%
SampleTestCaseDTO.java 0%
ReferenceSolutionDTO.java 0%

@IrminDev IrminDev merged commit 6288cf8 into main May 25, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants