Skip to content

Commit

Permalink
make artemis concurrency errors in tests less likely by not submittin…
Browse files Browse the repository at this point in the history
…g the cleared assessment
  • Loading branch information
Feuermagier committed Jul 23, 2024
1 parent 4b46e60 commit 7362007
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions src/test/java/edu/kit/kastel/sdq/artemis4j/End2EndTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -67,17 +67,11 @@ public void setup() throws ArtemisClientException, IOException {

// ensure that the submission is locked
this.assessment = this.programmingSubmission.tryLock(this.gradingConfig).orElseThrow();
this.cleanupFeedback();
this.assessment = this.programmingSubmission.tryLock(this.gradingConfig).orElseThrow();
this.assessment.clearAnnotations();

Assertions.assertTrue(this.assessment.getAnnotations().isEmpty());
}

private void cleanupFeedback() throws ArtemisClientException {
this.assessment.clearAnnotations();
this.assessment.submit();
}

@Test
void testCreationOfSimpleAnnotations() throws ArtemisClientException {
MistakeType mistakeType = this.gradingConfig.getMistakeTypes().get(1);
Expand Down

0 comments on commit 7362007

Please sign in to comment.