chore: removing scaffolder tests for v1
Signed-off-by: blam <ben@blam.sh>
This commit is contained in:
@@ -64,120 +64,6 @@ const mockUrlReader = UrlReaders.default({
|
||||
config: new ConfigReader({}),
|
||||
});
|
||||
|
||||
describe('createRouter - working directory', () => {
|
||||
const mockPrepare = jest.fn();
|
||||
const mockPreparers = new Preparers();
|
||||
|
||||
beforeAll(() => {
|
||||
const mockPreparer = {
|
||||
prepare: mockPrepare,
|
||||
};
|
||||
mockPreparers.register('dev.azure.com', mockPreparer);
|
||||
});
|
||||
|
||||
beforeEach(() => {
|
||||
jest.resetAllMocks();
|
||||
});
|
||||
|
||||
const workDirConfig = (path: string) => ({
|
||||
backend: {
|
||||
workingDirectory: path,
|
||||
},
|
||||
});
|
||||
|
||||
const template = {
|
||||
apiVersion: 'backstage.io/v1alpha1',
|
||||
kind: 'Template',
|
||||
metadata: {
|
||||
annotations: {
|
||||
'backstage.io/managed-by-location': 'url:https://dev.azure.com',
|
||||
},
|
||||
},
|
||||
spec: {
|
||||
owner: 'template@backstage.io',
|
||||
path: '.',
|
||||
schema: {},
|
||||
},
|
||||
};
|
||||
|
||||
it('should throw an error when working directory does not exist or is not writable', async () => {
|
||||
mockAccess.mockImplementation(() => {
|
||||
throw new Error('access error');
|
||||
});
|
||||
|
||||
await expect(
|
||||
createRouter({
|
||||
logger: getVoidLogger(),
|
||||
preparers: new Preparers(),
|
||||
templaters: new Templaters(),
|
||||
publishers: new Publishers(),
|
||||
config: new ConfigReader(workDirConfig('/path')),
|
||||
database: createDatabase(),
|
||||
catalogClient: createCatalogClient([template]),
|
||||
reader: mockUrlReader,
|
||||
}),
|
||||
).rejects.toThrow('access error');
|
||||
});
|
||||
|
||||
it('should use the working directory when configured', async () => {
|
||||
const router = await createRouter({
|
||||
logger: getVoidLogger(),
|
||||
preparers: mockPreparers,
|
||||
templaters: new Templaters(),
|
||||
publishers: new Publishers(),
|
||||
config: new ConfigReader(workDirConfig('/path')),
|
||||
database: createDatabase(),
|
||||
catalogClient: createCatalogClient([template]),
|
||||
reader: mockUrlReader,
|
||||
});
|
||||
|
||||
const app = express().use(router);
|
||||
await request(app)
|
||||
.post('/v1/jobs')
|
||||
.send({
|
||||
templateName: '',
|
||||
values: {
|
||||
storePath: 'https://github.com/backstage/good',
|
||||
},
|
||||
});
|
||||
|
||||
expect(mockPrepare).toBeCalledWith({
|
||||
logger: expect.anything(),
|
||||
workspacePath: expect.stringContaining('path'),
|
||||
url: expect.anything(),
|
||||
});
|
||||
});
|
||||
|
||||
it('should not pass along anything when no working directory is configured', async () => {
|
||||
const router = await createRouter({
|
||||
logger: getVoidLogger(),
|
||||
preparers: mockPreparers,
|
||||
templaters: new Templaters(),
|
||||
publishers: new Publishers(),
|
||||
config: new ConfigReader({}),
|
||||
database: createDatabase(),
|
||||
catalogClient: createCatalogClient([template]),
|
||||
reader: mockUrlReader,
|
||||
});
|
||||
|
||||
const app = express().use(router);
|
||||
await request(app)
|
||||
.post('/v1/jobs')
|
||||
.send({
|
||||
templateName: '',
|
||||
values: {
|
||||
storePath: 'https://github.com/backstage/goodrepo',
|
||||
},
|
||||
});
|
||||
|
||||
expect(mockPrepare).toBeCalledWith({
|
||||
logger: expect.anything(),
|
||||
workspacePath: expect.anything(),
|
||||
url: expect.anything(),
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('createRouter', () => {
|
||||
let app: express.Express;
|
||||
const template = {
|
||||
@@ -239,21 +125,6 @@ describe('createRouter', () => {
|
||||
jest.resetAllMocks();
|
||||
});
|
||||
|
||||
describe('POST /v1/jobs', () => {
|
||||
it('rejects template values which do not match the template schema definition', async () => {
|
||||
const response = await request(app)
|
||||
.post('/v1/jobs')
|
||||
.send({
|
||||
templateName: '',
|
||||
values: {
|
||||
storePath: 'https://github.com/backstage/backstage',
|
||||
},
|
||||
});
|
||||
|
||||
expect(response.status).toEqual(400);
|
||||
});
|
||||
});
|
||||
|
||||
describe('GET /v2/actions', () => {
|
||||
it('lists available actions', async () => {
|
||||
const response = await request(app).get('/v2/actions').send();
|
||||
|
||||
Reference in New Issue
Block a user