Skip to content

Commit d5f0d55

Browse files
committed
"updating test cases"
1 parent 88be15e commit d5f0d55

File tree

3 files changed

+53
-5
lines changed

3 files changed

+53
-5
lines changed

services/src/main/java/org/jfrog/artifactory/client/impl/jackson/RepositorySettingsMixIn.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,8 @@
4646
@JsonSubTypes.Type(value = CargoRepositorySettingsImpl.class, name = "cargo"),
4747
@JsonSubTypes.Type(value = TerraformRepositorySettingsImpl.class, name = "terraform"),
4848
@JsonSubTypes.Type(value = OciRepositorySettingsImpl.class, name = "oci"),
49-
@JsonSubTypes.Type(value = HuggingFaceRepositorySettingsImpl.class, name = "huggingfaceml")
49+
@JsonSubTypes.Type(value = HuggingFaceRepositorySettingsImpl.class, name = "huggingfaceml"),
50+
@JsonSubTypes.Type(value = HuggingFaceRepositorySettingsImpl.class, name = "HuggingFaceML")
5051
})
5152

5253
public abstract class RepositorySettingsMixIn {

services/src/test/java/org/jfrog/artifactory/client/HuggingFacePackageTypeRepositoryTest.java renamed to services/src/test/groovy/org/jfrog/artifactory/client/HuggingFacePackageTypeRepositoryTest.java

Lines changed: 51 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import static org.testng.Assert.assertEquals;
1313
import static org.testng.Assert.assertNotNull;
1414

15-
public class HuggingFacePackageTypeRepositoryTest extends ArtifactoryTestsBase {
15+
public class HuggingFacePackageTypeRepositoryTest extends BaseRepositoryTests {
1616

1717
private final String localRepo = "huggingface-local-1";
1818
private final String federatedRepo = "huggingface-federated-1";
@@ -38,7 +38,10 @@ public void testHuggingFaceLocalRepo() {
3838
.build();
3939

4040
// Use position 0 for create, as required by the interface
41-
artifactory.repositories().create(0, localRepository);
41+
String localCreateResult = artifactory.repositories().create(0, localRepository);
42+
System.out.println("[DIAG] create(local) result: " + localCreateResult);
43+
printEnvironmentDiagnostics();
44+
printRepositoryJson(localRepo);
4245
Repository localRepoFromServer = artifactory.repository(localRepo).get();
4346
assertNotNull(localRepoFromServer);
4447
assertEquals(localRepoFromServer.getKey(), localRepo);
@@ -59,7 +62,10 @@ public void testHuggingFaceFederatedRepo() {
5962
.repositorySettings(settings)
6063
.build();
6164

62-
artifactory.repositories().create(0, federatedRepository);
65+
String federatedCreateResult = artifactory.repositories().create(0, federatedRepository);
66+
System.out.println("[DIAG] create(federated) result: " + federatedCreateResult);
67+
printEnvironmentDiagnostics();
68+
printRepositoryJson(federatedRepo);
6369
Repository federatedRepoFromServer = artifactory.repository(federatedRepo).get();
6470
assertNotNull(federatedRepoFromServer);
6571
assertEquals(federatedRepoFromServer.getKey(), federatedRepo);
@@ -71,18 +77,59 @@ public void testHuggingFaceFederatedRepo() {
7177
public void testHuggingFaceVirtualRepo() {
7278
HuggingFaceRepositorySettings settings = new HuggingFaceRepositorySettingsImpl();
7379
RepositoryBuilders repositoryBuilders = artifactory.repositories().builders();
80+
// Ensure we have at least one backing repo to include in the virtual
81+
Repository localForVirtual = repositoryBuilders.localRepositoryBuilder()
82+
.key(localRepo)
83+
.description("local for virtual huggingface repo")
84+
.notes("some notes")
85+
.repositorySettings(settings)
86+
.build();
87+
artifactory.repositories().create(0, localForVirtual);
7488
Repository virtualRepository = repositoryBuilders.virtualRepositoryBuilder()
7589
.key(virtualRepo)
7690
.description("new virtual huggingface repo")
7791
.notes("some notes")
7892
.repositorySettings(settings)
93+
.repositories(java.util.Collections.singletonList(localRepo))
7994
.build();
8095

81-
artifactory.repositories().create(0, virtualRepository);
96+
String virtualCreateResult = artifactory.repositories().create(0, virtualRepository);
97+
System.out.println("[DIAG] create(virtual) result: " + virtualCreateResult);
98+
printEnvironmentDiagnostics();
99+
printRepositoryJson(virtualRepo);
82100
Repository virtualRepoFromServer = artifactory.repository(virtualRepo).get();
83101
assertNotNull(virtualRepoFromServer);
84102
assertEquals(virtualRepoFromServer.getKey(), virtualRepo);
85103
assertEquals(virtualRepoFromServer.getDescription(), "new virtual huggingface repo");
86104
assertEquals(virtualRepoFromServer.getNotes(), "some notes");
87105
}
106+
107+
private void printEnvironmentDiagnostics() {
108+
try {
109+
String versionJson = curl("api/system/version");
110+
System.out.println("[DIAG] Artifactory version: " + versionJson);
111+
} catch (Exception e) {
112+
System.out.println("[DIAG] Failed to fetch version: " + e.getMessage());
113+
}
114+
try {
115+
String licensesJson = curl("api/system/licenses");
116+
System.out.println("[DIAG] Artifactory licenses: " + licensesJson);
117+
} catch (Exception e1) {
118+
try {
119+
String licenseJson = curl("api/system/license");
120+
System.out.println("[DIAG] Artifactory license: " + licenseJson);
121+
} catch (Exception e2) {
122+
System.out.println("[DIAG] Failed to fetch license info: " + e2.getMessage());
123+
}
124+
}
125+
}
126+
127+
private void printRepositoryJson(String repoKey) {
128+
try {
129+
String repoJson = curl("api/repositories/" + repoKey);
130+
System.out.println("[DIAG] Repository JSON (" + repoKey + "): " + repoJson);
131+
} catch (Exception e) {
132+
System.out.println("[DIAG] Failed to fetch repo JSON for '" + repoKey + "': " + e.getMessage());
133+
}
134+
}
88135
}

services/src/test/java/org/jfrog/artifactory/client/HuggingFaceRepositorySettingsTest.java renamed to services/src/test/groovy/org/jfrog/artifactory/client/HuggingFaceRepositorySettingsTest.java

File renamed without changes.

0 commit comments

Comments
 (0)