Skip to content

Commit 91f94ce

Browse files
kosarkoclaude
andcommitted
Fix test compilation: update createClarinLicense call sites for new label arg
The backport added a `label` String parameter to the createClarinLicense test helper but left six 4-arg call sites unchanged, breaking testCompile in dspace-server-webapp. Pass a label at each remaining call site ("lbl"; "lbl1"/"lbl2" for the paired-license test) to match the helper's new signature, preserving the previously hard-coded "lbl" behaviour. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 0cf2651 commit 91f94ce

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinWorkspaceItemRestRepositoryIT.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -833,7 +833,7 @@ public void addClarinLicenseViaSectionPatch() throws Exception {
833833

834834
String clarinLicenseName = "Test Section Clarin License";
835835
ClarinLicense clarinLicense = createClarinLicense(clarinLicenseName, "Test Def", "Test R Info",
836-
Confirmation.NOT_REQUIRED);
836+
"lbl", Confirmation.NOT_REQUIRED);
837837
context.restoreAuthSystemState();
838838

839839
List<Operation> replaceOperations = new ArrayList<Operation>();
@@ -871,7 +871,7 @@ public void getWorkspaceItemReturnsDistinctLicenseSections() throws Exception {
871871

872872
String clarinLicenseName = "Distinct Sections Clarin License";
873873
createClarinLicense(clarinLicenseName, "Test Def", "Test R Info",
874-
Confirmation.NOT_REQUIRED);
874+
"lbl", Confirmation.NOT_REQUIRED);
875875
context.restoreAuthSystemState();
876876

877877
// Apply the CLARIN license through the section-scoped path
@@ -913,7 +913,7 @@ public void patchSelectWithEmptyValueClearsLicense() throws Exception {
913913

914914
String clarinLicenseName = "Empty Value Clarin License";
915915
ClarinLicense clarinLicense = createClarinLicense(clarinLicenseName, "Test Def", "Test R Info",
916-
Confirmation.NOT_REQUIRED);
916+
"lbl", Confirmation.NOT_REQUIRED);
917917
context.restoreAuthSystemState();
918918

919919
String tokenAdmin = getAuthToken(admin.getEmail(), password);
@@ -959,8 +959,8 @@ public void patchSelectReplacesPreviousLicense() throws Exception {
959959

960960
String firstName = "First Clarin License";
961961
String secondName = "Second Clarin License";
962-
ClarinLicense first = createClarinLicense(firstName, "Def1", "Info1", Confirmation.NOT_REQUIRED);
963-
ClarinLicense second = createClarinLicense(secondName, "Def2", "Info2", Confirmation.NOT_REQUIRED);
962+
ClarinLicense first = createClarinLicense(firstName, "Def1", "Info1", "lbl1", Confirmation.NOT_REQUIRED);
963+
ClarinLicense second = createClarinLicense(secondName, "Def2", "Info2", "lbl2", Confirmation.NOT_REQUIRED);
964964
context.restoreAuthSystemState();
965965

966966
String tokenAdmin = getAuthToken(admin.getEmail(), password);
@@ -1042,7 +1042,7 @@ public void patchSelectAsAnonymousIsUnauthorized() throws Exception {
10421042
context.turnOffAuthorisationSystem();
10431043
WorkspaceItem witem = createWorkspaceItemWithFile();
10441044
String clarinLicenseName = "Anon Clarin License";
1045-
createClarinLicense(clarinLicenseName, "Def", "Info", Confirmation.NOT_REQUIRED);
1045+
createClarinLicense(clarinLicenseName, "Def", "Info", "lbl", Confirmation.NOT_REQUIRED);
10461046
context.restoreAuthSystemState();
10471047

10481048
List<Operation> ops = new ArrayList<Operation>();

0 commit comments

Comments
 (0)