mirror of
https://github.com/willmiao/ComfyUI-Lora-Manager.git
synced 2026-09-26 13:34:08 -03:00
A repository is not a model identity: collection repos on Hugging Face and ModelScope host many unrelated models, which were wrongly shown as versions of each other. - Hugging Face models no longer auto-group (the Hub exposes no site-native model id) - ModelScope models group by the site's native published-model id (MuseInfo modelVersion.modelId), extracted during enrichment and persisted on the sidecar as source_model_id/source_version_id; unenriched models stay standalone instead of collapsing a whole repo into one group - TensorArt grouping unchanged (its URL id is already model-level) - Frontend group-key derivation mirrors the new backend semantics
215 lines
7.5 KiB
JavaScript
215 lines
7.5 KiB
JavaScript
import { describe, it, expect, vi } from 'vitest';
|
|
|
|
const { I18N_MODULE } = vi.hoisted(() => ({
|
|
I18N_MODULE: new URL('../../../static/js/utils/i18nHelpers.js', import.meta.url).pathname,
|
|
}));
|
|
|
|
vi.mock(I18N_MODULE, () => ({
|
|
translate: vi.fn((key, params, fallback) => (typeof fallback === 'string' ? fallback : key)),
|
|
}));
|
|
|
|
const {
|
|
MODEL_SOURCES,
|
|
parseModelSourceUrl,
|
|
getModelSource,
|
|
getModelSourceInfo,
|
|
getModelSourceUrl,
|
|
getModelSourceGroupKey,
|
|
canEnrichModelSource,
|
|
getModelSourceViewTitle,
|
|
parseModelSourceGroupKey,
|
|
openModelSource,
|
|
} = await import('../../../static/js/utils/modelSourceHelpers.js');
|
|
|
|
describe('modelSourceHelpers', () => {
|
|
it('exposes one descriptor per supported platform', () => {
|
|
expect(MODEL_SOURCES.map((s) => s.platform)).toEqual([
|
|
'huggingface',
|
|
'modelscope',
|
|
'modelscope-ai',
|
|
'tensorart',
|
|
]);
|
|
});
|
|
|
|
describe('parseModelSourceUrl', () => {
|
|
it('recognises Hugging Face URLs', () => {
|
|
const info = parseModelSourceUrl('https://huggingface.co/user/repo');
|
|
expect(info.platform).toBe('huggingface');
|
|
expect(info.sourceId).toBe('user/repo');
|
|
});
|
|
|
|
it('recognises ModelScope URLs with view sub-paths', () => {
|
|
const info = parseModelSourceUrl('https://modelscope.cn/models/user/repo/summary');
|
|
expect(info.platform).toBe('modelscope');
|
|
expect(info.sourceId).toBe('user/repo');
|
|
expect(info.url).toBe('https://modelscope.cn/models/user/repo');
|
|
});
|
|
|
|
it('recognises ModelScope International as its own platform', () => {
|
|
const info = parseModelSourceUrl(
|
|
'https://www.modelscope.ai/models/referall13/EM1/files'
|
|
);
|
|
expect(info.platform).toBe('modelscope-ai');
|
|
expect(info.groupPrefix).toBe('msai');
|
|
expect(info.sourceId).toBe('referall13/EM1');
|
|
expect(info.url).toBe('https://www.modelscope.ai/models/referall13/EM1');
|
|
});
|
|
|
|
it('recognises TensorArt URLs and keeps only the numeric id', () => {
|
|
const info = parseModelSourceUrl(
|
|
'https://tensor.art/models/827823520299086029/Vivid-Impressions-Storybook-Sstyle-V1.0'
|
|
);
|
|
expect(info.platform).toBe('tensorart');
|
|
expect(info.sourceId).toBe('827823520299086029');
|
|
expect(info.url).toBe('https://tensor.art/models/827823520299086029');
|
|
});
|
|
|
|
it('rejects unsupported URLs', () => {
|
|
expect(parseModelSourceUrl('https://example.com/x')).toBeNull();
|
|
expect(parseModelSourceUrl('')).toBeNull();
|
|
expect(parseModelSourceUrl(null)).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('getModelSourceInfo', () => {
|
|
it('falls back to the legacy hf_url field', () => {
|
|
const info = getModelSourceInfo({ hf_url: 'https://huggingface.co/user/repo' });
|
|
expect(info.platform).toBe('huggingface');
|
|
expect(info.sourceId).toBe('user/repo');
|
|
});
|
|
|
|
it('prefers the canonical source fields', () => {
|
|
const info = getModelSourceInfo({
|
|
source_platform: 'modelscope',
|
|
source_url: 'https://modelscope.cn/models/user/repo',
|
|
hf_url: 'https://huggingface.co/old/repo',
|
|
});
|
|
expect(info.platform).toBe('modelscope');
|
|
});
|
|
|
|
it('returns null when there is no source', () => {
|
|
expect(getModelSourceInfo({})).toBeNull();
|
|
expect(getModelSourceInfo({ hf_url: '' })).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('getModelSourceUrl', () => {
|
|
it('reads source_url then hf_url', () => {
|
|
expect(getModelSourceUrl({ source_url: 'https://a.example/1' })).toBe('https://a.example/1');
|
|
expect(getModelSourceUrl({ hf_url: 'https://huggingface.co/u/r' })).toBe(
|
|
'https://huggingface.co/u/r'
|
|
);
|
|
expect(getModelSourceUrl({})).toBe('');
|
|
});
|
|
});
|
|
|
|
describe('getModelSourceGroupKey', () => {
|
|
it('matches the backend group-key shapes', () => {
|
|
// TensorArt's numeric id already identifies a single model.
|
|
expect(getModelSourceGroupKey({ source_url: 'https://tensor.art/models/123' })).toBe(
|
|
'ta:123'
|
|
);
|
|
// ModelScope groups by the site-native published-model id.
|
|
expect(
|
|
getModelSourceGroupKey({
|
|
source_url: 'https://modelscope.cn/models/u/r',
|
|
source_model_id: '555',
|
|
})
|
|
).toBe('ms:555');
|
|
expect(
|
|
getModelSourceGroupKey({
|
|
source_url: 'https://www.modelscope.ai/models/u/r',
|
|
source_model_id: '678',
|
|
})
|
|
).toBe('msai:678');
|
|
});
|
|
|
|
it('returns an empty string for sources without a model identity', () => {
|
|
// Hugging Face repos are not a model identity: never grouped.
|
|
expect(getModelSourceGroupKey({ hf_url: 'https://huggingface.co/u/r' })).toBe('');
|
|
// Unenriched ModelScope models stay standalone rather than collapsing
|
|
// a whole collection repo into one group.
|
|
expect(
|
|
getModelSourceGroupKey({ source_url: 'https://modelscope.cn/models/u/r' })
|
|
).toBe('');
|
|
expect(
|
|
getModelSourceGroupKey({
|
|
source_url: 'https://modelscope.cn/models/u/r',
|
|
source_model_id: ' ',
|
|
})
|
|
).toBe('');
|
|
});
|
|
|
|
it('returns an empty string without a source', () => {
|
|
expect(getModelSourceGroupKey({})).toBe('');
|
|
});
|
|
});
|
|
|
|
describe('canEnrichModelSource', () => {
|
|
it('allows Hugging Face and ModelScope', () => {
|
|
expect(canEnrichModelSource({ hf_url: 'https://huggingface.co/u/r' })).toBe(true);
|
|
expect(
|
|
canEnrichModelSource({ source_url: 'https://modelscope.cn/models/u/r' })
|
|
).toBe(true);
|
|
});
|
|
|
|
it('disallows TensorArt and unlinked models', () => {
|
|
expect(canEnrichModelSource({ source_url: 'https://tensor.art/models/123' })).toBe(false);
|
|
expect(canEnrichModelSource({})).toBe(false);
|
|
});
|
|
});
|
|
|
|
describe('getModelSourceViewTitle', () => {
|
|
it('uses the branded label for non-HF sources', () => {
|
|
expect(getModelSourceViewTitle(getModelSource('modelscope'))).toBe('View on ModelScope');
|
|
expect(getModelSourceViewTitle(getModelSource('tensorart'))).toBe('View on TensorArt');
|
|
});
|
|
|
|
it('keeps the historical Hugging Face title', () => {
|
|
expect(getModelSourceViewTitle(getModelSource('huggingface'))).toBe(
|
|
'View on Hugging Face'
|
|
);
|
|
});
|
|
});
|
|
|
|
describe('parseModelSourceGroupKey', () => {
|
|
it('parses every external group-key prefix', () => {
|
|
expect(parseModelSourceGroupKey('hf:user/repo')).toEqual({
|
|
platform: 'huggingface',
|
|
label: 'Hugging Face',
|
|
sourceId: 'user/repo',
|
|
});
|
|
expect(parseModelSourceGroupKey('ms:user/repo').platform).toBe('modelscope');
|
|
expect(parseModelSourceGroupKey('ta:123').platform).toBe('tensorart');
|
|
});
|
|
|
|
it('rejects numeric CivitAI model ids and unknown prefixes', () => {
|
|
expect(parseModelSourceGroupKey(222)).toBeNull();
|
|
expect(parseModelSourceGroupKey('222')).toBeNull();
|
|
expect(parseModelSourceGroupKey('unknown:1')).toBeNull();
|
|
expect(parseModelSourceGroupKey('')).toBeNull();
|
|
expect(parseModelSourceGroupKey(null)).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('openModelSource', () => {
|
|
it('opens the URL in a new tab', () => {
|
|
const openSpy = vi.spyOn(window, 'open').mockImplementation(() => {});
|
|
openModelSource('https://modelscope.cn/models/u/r');
|
|
expect(openSpy).toHaveBeenCalledWith(
|
|
'https://modelscope.cn/models/u/r',
|
|
'_blank',
|
|
'noopener,noreferrer'
|
|
);
|
|
openSpy.mockRestore();
|
|
});
|
|
|
|
it('does nothing without a URL', () => {
|
|
const openSpy = vi.spyOn(window, 'open').mockImplementation(() => {});
|
|
openModelSource('');
|
|
expect(openSpy).not.toHaveBeenCalled();
|
|
openSpy.mockRestore();
|
|
});
|
|
});
|
|
});
|