mirror of
https://github.com/renovatebot/renovate.git
synced 2026-11-01 10:09:50 +00:00
762ac01133
* test(manager): replace file snapshots with explicit assertions * test(manager): replace inline snapshots with explicit assertions * test(manager): assert on relevant sections instead of full bodies * test(manager): drop assertions implied by full equality * test(manager): address review feedback * test(manager): assert depType and versioning in the regex matcher * test(bundler): replace file snapshots with explicit assertions * test(bundler): replace inline snapshots with explicit assertions * test(bundler): drop assertions implied by full equality * test(bundler): address review feedback * test(npm): replace file snapshots with explicit assertions * test(npm): replace inline snapshots with explicit assertions * test(platform): replace file snapshots with explicit assertions * test(platform): replace inline snapshots with explicit assertions * test(platform): assert on relevant sections instead of full bodies * test(platform): address review feedback * test(platform): use resolves instead of awaiting inside expect
324 lines
10 KiB
TypeScript
324 lines
10 KiB
TypeScript
import { Readable } from 'node:stream';
|
|
import type { GitPullRequest } from 'azure-devops-node-api/interfaces/GitInterfaces.js';
|
|
import { buildTestJwt } from '~test/jwt-util.ts';
|
|
import { partial } from '~test/util.ts';
|
|
import { streamToString } from '../../../util/streams.ts';
|
|
import {
|
|
getBranchNameWithoutRefsheadsPrefix,
|
|
getGitStatusContextCombinedName,
|
|
getGitStatusContextFromCombinedName,
|
|
getProjectAndRepo,
|
|
getRenovatePRFormat,
|
|
getRepoByName,
|
|
getStorageExtraCloneOpts,
|
|
getWorkItemTitle,
|
|
max4000Chars,
|
|
} from './util.ts';
|
|
|
|
describe('modules/platform/azure/util', () => {
|
|
describe('getGitStatusContextCombinedName', () => {
|
|
it('should return undefined if null context passed', () => {
|
|
const contextName = getGitStatusContextCombinedName(null);
|
|
expect(contextName).toBeUndefined();
|
|
});
|
|
|
|
it('should combine valid genre and name with slash', () => {
|
|
const contextName = getGitStatusContextCombinedName({
|
|
genre: 'my-genre',
|
|
name: 'status-name',
|
|
});
|
|
expect(contextName).toMatch('my-genre/status-name');
|
|
});
|
|
|
|
it('should combine valid empty genre and name without a slash', () => {
|
|
const contextName = getGitStatusContextCombinedName({
|
|
genre: undefined,
|
|
name: 'status-name',
|
|
});
|
|
expect(contextName).toMatch('status-name');
|
|
});
|
|
});
|
|
|
|
describe('getGitStatusContextFromCombinedName', () => {
|
|
it('should return undefined if null context passed', () => {
|
|
const context = getGitStatusContextFromCombinedName(null);
|
|
expect(context).toBeUndefined();
|
|
});
|
|
|
|
it('should parse valid genre and name with slash', () => {
|
|
const context = getGitStatusContextFromCombinedName(
|
|
'my-genre/status-name',
|
|
);
|
|
expect(context).toEqual({
|
|
genre: 'my-genre',
|
|
name: 'status-name',
|
|
});
|
|
});
|
|
|
|
it('should parse valid genre and name with multiple slashes', () => {
|
|
const context = getGitStatusContextFromCombinedName(
|
|
'my-genre/sub-genre/status-name',
|
|
);
|
|
expect(context).toEqual({
|
|
genre: 'my-genre/sub-genre',
|
|
name: 'status-name',
|
|
});
|
|
});
|
|
|
|
it('should parse valid empty genre and name without a slash', () => {
|
|
const context = getGitStatusContextFromCombinedName('status-name');
|
|
expect(context).toEqual({
|
|
genre: undefined,
|
|
name: 'status-name',
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('getBranchNameWithoutRefsheadsPrefix', () => {
|
|
it('should be renamed', () => {
|
|
const res = getBranchNameWithoutRefsheadsPrefix('refs/heads/testBB');
|
|
expect(res).toBe(`testBB`);
|
|
});
|
|
|
|
it('should log error and return undefined', () => {
|
|
const res = getBranchNameWithoutRefsheadsPrefix(undefined);
|
|
expect(res).toBeUndefined();
|
|
});
|
|
|
|
it('should return the input', () => {
|
|
const res = getBranchNameWithoutRefsheadsPrefix('testBB');
|
|
expect(res).toBe('testBB');
|
|
});
|
|
});
|
|
|
|
describe('getRenovatePRFormat', () => {
|
|
it('should be formated (closed)', () => {
|
|
const res = getRenovatePRFormat(partial<GitPullRequest>({ status: 2 }));
|
|
expect(res).toMatchObject({
|
|
state: 'closed',
|
|
status: 2,
|
|
});
|
|
});
|
|
|
|
it('should be formated (closed v2)', () => {
|
|
const res = getRenovatePRFormat(partial<GitPullRequest>({ status: 3 }));
|
|
expect(res).toMatchObject({
|
|
state: 'merged',
|
|
status: 3,
|
|
});
|
|
});
|
|
|
|
it('should be formated (not closed)', () => {
|
|
const res = getRenovatePRFormat(partial<GitPullRequest>({ status: 1 }));
|
|
expect(res).toMatchObject({
|
|
state: 'open',
|
|
status: 1,
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('streamToString', () => {
|
|
it('converts Readable stream to string', async () => {
|
|
const res = await streamToString(Readable.from('foobar'));
|
|
expect(res).toBe('foobar');
|
|
});
|
|
|
|
it('handles error', async () => {
|
|
const stream = Readable.from('foobar');
|
|
const res = streamToString(stream);
|
|
stream.destroy(new Error('some unknown error'));
|
|
await expect(res).rejects.toThrow('some unknown error');
|
|
});
|
|
});
|
|
|
|
describe('getStorageExtraCloneOpts', () => {
|
|
it('should configure basic auth', () => {
|
|
const res = getStorageExtraCloneOpts({
|
|
username: 'user',
|
|
password: 'pass',
|
|
});
|
|
expect(res).toEqual({
|
|
'-c': 'http.extraHeader=AUTHORIZATION: basic dXNlcjpwYXNz',
|
|
});
|
|
});
|
|
|
|
it('should configure personal access token', () => {
|
|
const res = getStorageExtraCloneOpts({
|
|
token: '123456789012345678901234567890123456789012345678test',
|
|
});
|
|
expect(res).toEqual({
|
|
'-c': 'http.extraHeader=AUTHORIZATION: basic OjEyMzQ1Njc4OTAxMjM0NTY3ODkwMTIzNDU2Nzg5MDEyMzQ1Njc4OTAxMjM0NTY3OHRlc3Q=',
|
|
});
|
|
});
|
|
|
|
it('should configure non-JWT token as PAT', () => {
|
|
const res = getStorageExtraCloneOpts({ token: 'token' });
|
|
expect(res['-c']).toContain('AUTHORIZATION: basic');
|
|
});
|
|
|
|
it('should use bearer when token is a JWT', () => {
|
|
const token = buildTestJwt(
|
|
{ typ: 'JWT', alg: 'RS256' },
|
|
{ aud: '499b84ac', sub: 'test', exp: 9999999999 },
|
|
'fake-sig',
|
|
);
|
|
const res = getStorageExtraCloneOpts({ token });
|
|
expect(res['-c']).toContain('AUTHORIZATION: bearer');
|
|
expect(res['-c']).toContain(token);
|
|
});
|
|
|
|
it('should use basic for a 52-char PAT', () => {
|
|
const token = '1234567890123456789012345678901234567890123456789012';
|
|
const res = getStorageExtraCloneOpts({ token });
|
|
expect(res['-c']).toContain('AUTHORIZATION: basic');
|
|
expect(res['-c']).not.toContain('bearer');
|
|
});
|
|
|
|
it('should use basic for a short opaque token', () => {
|
|
const res = getStorageExtraCloneOpts({ token: 'short-token' });
|
|
expect(res['-c']).toContain('AUTHORIZATION: basic');
|
|
expect(res['-c']).not.toContain('bearer');
|
|
});
|
|
|
|
it('should use basic for an invalid JWT-like token', () => {
|
|
const res = getStorageExtraCloneOpts({ token: 'not.valid.jwt' });
|
|
expect(res['-c']).toContain('AUTHORIZATION: basic');
|
|
expect(res['-c']).not.toContain('bearer');
|
|
});
|
|
});
|
|
|
|
describe('max4000Chars', () => {
|
|
it('should be the same', () => {
|
|
const res = max4000Chars('Hello');
|
|
expect(res).toBe('Hello');
|
|
});
|
|
|
|
it('should be truncated', () => {
|
|
let str = '';
|
|
for (let i = 0; i < 5000; i += 1) {
|
|
str += 'a';
|
|
}
|
|
const res = max4000Chars(str);
|
|
expect(res).toHaveLength(3999);
|
|
});
|
|
});
|
|
|
|
describe('getProjectAndRepo', () => {
|
|
it('should return the object with same strings', () => {
|
|
const res = getProjectAndRepo('myRepoName');
|
|
expect(res).toEqual({
|
|
project: 'myRepoName',
|
|
repo: 'myRepoName',
|
|
});
|
|
});
|
|
|
|
it('should return the object with project and repo', () => {
|
|
const res = getProjectAndRepo('prjName/myRepoName');
|
|
expect(res).toEqual({
|
|
project: 'prjName',
|
|
repo: 'myRepoName',
|
|
});
|
|
});
|
|
|
|
it('should return an error', () => {
|
|
expect(() => getProjectAndRepo('prjName/myRepoName/blalba')).toThrow(
|
|
Error(
|
|
`Azure repository can be only structured this way : 'repository' or 'projectName/repository'!`,
|
|
),
|
|
);
|
|
});
|
|
});
|
|
|
|
describe('getRepoByName', () => {
|
|
it('returns null when repos array is empty', () => {
|
|
expect(getRepoByName('foo/bar', [])).toBeNull();
|
|
expect(getRepoByName('foo/bar', undefined)).toBeNull();
|
|
expect(getRepoByName('foo/bar', null)).toBeNull();
|
|
});
|
|
|
|
it('returns null when repo is not found', () => {
|
|
expect(
|
|
getRepoByName('foo/foo', [{ name: 'bar', project: { name: 'bar' } }]),
|
|
).toBeNull();
|
|
});
|
|
|
|
it('finds repo', () => {
|
|
expect(
|
|
getRepoByName('foo/bar', [
|
|
{ id: '1', name: 'baz', project: { name: 'qux' } },
|
|
null,
|
|
undefined,
|
|
{ id: '2', name: 'bar' },
|
|
{ id: '3', name: 'bar', project: { name: 'foo' } },
|
|
{ id: '4', name: 'bar', project: { name: 'foo' } },
|
|
]),
|
|
).toMatchObject({ id: '3' });
|
|
});
|
|
|
|
it('supports shorthand names', () => {
|
|
expect(
|
|
getRepoByName('foo', [
|
|
{ id: '1', name: 'bar', project: { name: 'bar' } },
|
|
{ id: '2', name: 'foo', project: { name: 'foo' } },
|
|
]),
|
|
).toMatchObject({ id: '2' });
|
|
});
|
|
|
|
it('is case-independent', () => {
|
|
const repos = [
|
|
{ id: '1', name: 'FOO', project: { name: 'FOO' } },
|
|
{ id: '2', name: 'foo', project: { name: 'foo' } },
|
|
];
|
|
expect(getRepoByName('FOO/foo', repos)).toMatchObject({ id: '1' });
|
|
expect(getRepoByName('foo/FOO', repos)).toMatchObject({ id: '1' });
|
|
expect(getRepoByName('foo/foo', repos)).toMatchObject({ id: '1' });
|
|
});
|
|
|
|
it('throws when repo name is invalid', () => {
|
|
// TODO: better error handling #22198
|
|
expect(() => getRepoByName(undefined as never, [])).toThrow(
|
|
"Cannot read properties of undefined (reading 'split')",
|
|
);
|
|
expect(() => getRepoByName(null as never, [])).toThrow(
|
|
"Cannot read properties of null (reading 'split')",
|
|
);
|
|
expect(() => getRepoByName('foo/bar/baz', [])).toThrow(
|
|
'Azure repository can be only structured this way',
|
|
);
|
|
});
|
|
});
|
|
it('returns the raw title if not a dependency dashboard', () => {
|
|
expect(getWorkItemTitle('Some Issue', 'project/repo')).toBe('Some Issue');
|
|
});
|
|
|
|
it('returns the raw title if dependency dashboard and already prefixed', () => {
|
|
expect(
|
|
getWorkItemTitle('[repo] Dependency Dashboard', 'project/repo'),
|
|
).toBe('[repo] Dependency Dashboard');
|
|
});
|
|
|
|
it('prefixes dependency dashboard title with repo name if not already prefixed', () => {
|
|
expect(getWorkItemTitle('Dependency Dashboard', 'project/repo')).toBe(
|
|
'[repo] Dependency Dashboard',
|
|
);
|
|
});
|
|
|
|
it('handles repository names with slashes', () => {
|
|
expect(getWorkItemTitle('Dependency Dashboard', 'org/project/repo')).toBe(
|
|
'[repo] Dependency Dashboard',
|
|
);
|
|
});
|
|
|
|
it('does not double prefix if repo name is already present', () => {
|
|
expect(
|
|
getWorkItemTitle('[repo] Dependency Dashboard', 'org/project/repo'),
|
|
).toBe('[repo] Dependency Dashboard');
|
|
});
|
|
|
|
it('handles titles with apostrophes', () => {
|
|
expect(getWorkItemTitle("Dependency Dashboard's", 'project/repo')).toBe(
|
|
"[repo] Dependency Dashboard's",
|
|
);
|
|
});
|
|
});
|