neonctl 2.22.2 → 2.23.1

package/commands/set_context.test.js

@@ -1,159 +0,0 @@
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { rmSync, writeFileSync, readFileSync } from 'node:fs';
-import { describe, expect } from 'vitest';
-import { test as originalTest } from '../test_utils/fixtures';
-const CONTEXT_FILE = join(tmpdir(), `neon_${Date.now()}`);
-const test = originalTest.extend({
-    // eslint-disable-next-line no-empty-pattern
-    cleanupFile: async ({}, use) => {
-        let writtenFilename;
-        await use((name) => (writtenFilename = name));
-        if (writtenFilename) {
-            rmSync(writtenFilename);
-        }
-    },
-    writeFile: async ({ cleanupFile }, use) => {
-        await use((name, content) => {
-            writeFileSync(name, JSON.stringify(content));
-            cleanupFile(name);
-        });
-    },
-    readFile: async ({ cleanupFile }, use) => {
-        await use((name) => {
-            const content = readFileSync(name, 'utf-8');
-            cleanupFile(name);
-            return content;
-        });
-    },
-});
-describe('set_context', () => {
-    describe('should set the context to project', () => {
-        test('set-context', async ({ testCliCommand, readFile }) => {
-            await testCliCommand([
-                'set-context',
-                '--project-id',
-                'test',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-            expect(readFile(CONTEXT_FILE)).toMatchSnapshot();
-        });
-        test('list branches selecting project from the context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(CONTEXT_FILE, {
-                projectId: 'test',
-            });
-            await testCliCommand([
-                'branches',
-                'list',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-        });
-        const overrideContextFile = join(tmpdir(), `neon_override_ctx_${Date.now()}`);
-        test('get project id overrides context set project', async ({ testCliCommand, writeFile, }) => {
-            writeFile(overrideContextFile, {
-                projectId: 'new-project id',
-            });
-            await testCliCommand([
-                'project',
-                'get',
-                'project-id-123',
-                '--context-file',
-                overrideContextFile,
-            ]);
-        });
-        test('set the branchId and projectId is from context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(overrideContextFile, {
-                projectId: 'test',
-                branchId: 'test_branch',
-            });
-            await testCliCommand([
-                'databases',
-                'list',
-                '--context-file',
-                overrideContextFile,
-            ]);
-        });
-        test('set the branchId and projectId is from context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(overrideContextFile, {
-                projectId: 'project-id-123',
-                branchId: 'test_branch',
-            });
-            await testCliCommand([
-                'databases',
-                'list',
-                '--project-id',
-                'test',
-                '--context-file',
-                overrideContextFile,
-            ], {
-                code: 1,
-                stderr: 'ERROR: Not Found',
-            });
-        });
-    });
-    describe('should set the context to organization', () => {
-        test('set-context', async ({ testCliCommand, readFile }) => {
-            await testCliCommand([
-                'set-context',
-                '--org-id',
-                'org-2',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-            expect(readFile(CONTEXT_FILE)).toMatchSnapshot();
-        });
-        test('list projects selecting organization from the context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(CONTEXT_FILE, {
-                orgId: 'org-2',
-            });
-            await testCliCommand([
-                'projects',
-                'list',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-        });
-        test('list projects with explicit org id overrides context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(CONTEXT_FILE, {
-                orgId: 'org-2',
-            });
-            await testCliCommand([
-                'project',
-                'list',
-                '--org-id',
-                'org-3',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-        });
-        test('create projects selecting organization from the context', async ({ testCliCommand, writeFile, }) => {
-            writeFile(CONTEXT_FILE, {
-                orgId: 'org-2',
-            });
-            await testCliCommand([
-                'projects',
-                'create',
-                '--name',
-                'test_project',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-        });
-    });
-    describe('can set the context to project and organization at the same time', () => {
-        test('set-context', async ({ testCliCommand, readFile }) => {
-            await testCliCommand([
-                'set-context',
-                '--project-id',
-                'test_project',
-                '--org-id',
-                'org-2',
-                '--context-file',
-                CONTEXT_FILE,
-            ]);
-            expect(readFile(CONTEXT_FILE)).toMatchSnapshot();
-        });
-    });
-});

package/commands/vpc_endpoints.test.js

@@ -1,69 +0,0 @@
-import { describe } from 'vitest';
-import { test } from '../test_utils/fixtures';
-describe('vpc-endpoints', () => {
-    test('list org VPC endpoints', async ({ testCliCommand }) => {
-        await testCliCommand(['vpc', 'endpoint', 'list', '--org-id', '1', '--region-id', 'test'], { mockDir: 'single_org' });
-    });
-    test('list org VPC endpoints implicit org', async ({ testCliCommand }) => {
-        await testCliCommand(['vpc', 'endpoint', 'list', '--region-id', 'test'], {
-            mockDir: 'single_org',
-        });
-    });
-    test('update org VPC endpoint', async ({ testCliCommand }) => {
-        await testCliCommand([
-            'vpc',
-            'endpoint',
-            'update',
-            'vpc-test',
-            '--label',
-            'newlabel',
-            '--region-id',
-            'test',
-        ], { mockDir: 'single_org' });
-    });
-    test('delete org VPC endpoint', async ({ testCliCommand }) => {
-        await testCliCommand(['vpc', 'endpoint', 'remove', 'vpc-test', '--region-id', 'test'], { mockDir: 'single_org' });
-    });
-    test('get org VPC endpoint status', async ({ testCliCommand }) => {
-        await testCliCommand(['vpc', 'endpoint', 'status', 'vpc-test', '--region-id', 'test'], { mockDir: 'single_org' });
-    });
-    test('set org VPC endpoint in azure', async ({ testCliCommand }) => {
-        await testCliCommand([
-            'vpc',
-            'endpoint',
-            'update',
-            'vpc-test',
-            '--label',
-            'newlabel',
-            '--region-id',
-            'azure-test',
-        ], {
-            mockDir: 'single_org',
-            stderr: 'INFO: VPC endpoint configuration is not supported for Azure regions',
-        });
-    });
-    test('list project VPC endpoint restrictions', async ({ testCliCommand }) => {
-        await testCliCommand([
-            'vpc',
-            'project',
-            'list',
-            '--region-id',
-            'test',
-            '--project-id',
-            'test-project-123456',
-        ], { mockDir: 'single_org' });
-    });
-    test('list project VPC endpoint restrictions with implicit project', async ({ testCliCommand, }) => {
-        await testCliCommand(['vpc', 'project', 'list', '--region-id', 'test'], {
-            mockDir: 'single_org',
-        });
-    });
-    test('set project VPC endpoint restrictions', async ({ testCliCommand }) => {
-        await testCliCommand(['vpc', 'project', 'update', 'vpc-test', '--label', 'newlabel'], { mockDir: 'single_org' });
-    });
-    test('delete project VPC endpoint restrictions', async ({ testCliCommand, }) => {
-        await testCliCommand(['vpc', 'project', 'remove', 'vpc-test'], {
-            mockDir: 'single_org',
-        });
-    });
-});

package/context.test.js

@@ -1,119 +0,0 @@
-import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync, } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { afterEach, beforeEach, describe, expect, test } from 'vitest';
-import { applyContext, currentContextFile, ensureGitignored, } from './context.js';
-describe('currentContextFile', () => {
-    let workspace;
-    beforeEach(() => {
-        workspace = mkdtempSync(join(tmpdir(), 'neonctl-ctx-'));
-    });
-    afterEach(() => {
-        rmSync(workspace, { recursive: true, force: true });
-    });
-    test('defaults to <cwd>/.neon when no .neon exists anywhere upward', () => {
-        const sub = join(workspace, 'sub');
-        mkdirSync(sub);
-        expect(currentContextFile(sub)).toBe(join(sub, '.neon'));
-    });
-    test('walks up to an existing .neon in a parent directory', () => {
-        writeFileSync(join(workspace, '.neon'), JSON.stringify({ projectId: 'parent-project' }));
-        const sub = join(workspace, 'nested', 'deeper');
-        mkdirSync(sub, { recursive: true });
-        expect(currentContextFile(sub)).toBe(join(workspace, '.neon'));
-    });
-    test('does NOT walk up for unrelated project markers (package.json, .git)', () => {
-        // Regression: previously `currentContextFile` treated `package.json` and
-        // `.git` as project markers and walked up to them, which made
-        // `neonctl link` from a fresh sub-directory inside an existing repo land
-        // its `.neon` at the parent repo's root instead of the cwd.
-        writeFileSync(join(workspace, 'package.json'), '{}');
-        mkdirSync(join(workspace, '.git'));
-        const sub = join(workspace, 'fresh-sub');
-        mkdirSync(sub);
-        expect(currentContextFile(sub)).toBe(join(sub, '.neon'));
-    });
-});
-describe('ensureGitignored', () => {
-    let workspace;
-    beforeEach(() => {
-        workspace = mkdtempSync(join(tmpdir(), 'neonctl-gi-'));
-    });
-    afterEach(() => {
-        rmSync(workspace, { recursive: true, force: true });
-    });
-    test('creates a .gitignore listing .neon when none exists', () => {
-        const contextFile = join(workspace, '.neon');
-        ensureGitignored(contextFile);
-        const gitignore = readFileSync(join(workspace, '.gitignore'), 'utf-8');
-        expect(gitignore).toBe('.neon\n');
-    });
-    test('appends .neon to an existing .gitignore that does not have it', () => {
-        const gi = join(workspace, '.gitignore');
-        writeFileSync(gi, 'node_modules\ndist\n');
-        ensureGitignored(join(workspace, '.neon'));
-        expect(readFileSync(gi, 'utf-8')).toBe('node_modules\ndist\n.neon\n');
-    });
-    test('does not duplicate .neon when already present', () => {
-        const gi = join(workspace, '.gitignore');
-        writeFileSync(gi, 'node_modules\n.neon\ndist\n');
-        ensureGitignored(join(workspace, '.neon'));
-        expect(readFileSync(gi, 'utf-8')).toBe('node_modules\n.neon\ndist\n');
-    });
-    test('tolerates a .gitignore that has no trailing newline', () => {
-        const gi = join(workspace, '.gitignore');
-        writeFileSync(gi, 'node_modules');
-        ensureGitignored(join(workspace, '.neon'));
-        expect(readFileSync(gi, 'utf-8')).toBe('node_modules\n.neon\n');
-    });
-    test('treats surrounding whitespace as part of the line', () => {
-        const gi = join(workspace, '.gitignore');
-        // Trailing spaces around the entry should still count as a match.
-        writeFileSync(gi, '  .neon  \n');
-        ensureGitignored(join(workspace, '.neon'));
-        expect(readFileSync(gi, 'utf-8')).toBe('  .neon  \n');
-    });
-    test('does NOT match partial entries like *.neon or foo/.neon', () => {
-        const gi = join(workspace, '.gitignore');
-        writeFileSync(gi, '*.neon\nfoo/.neon\n');
-        ensureGitignored(join(workspace, '.neon'));
-        expect(readFileSync(gi, 'utf-8')).toBe('*.neon\nfoo/.neon\n.neon\n');
-    });
-});
-describe('applyContext', () => {
-    let workspace;
-    beforeEach(() => {
-        workspace = mkdtempSync(join(tmpdir(), 'neonctl-apply-'));
-    });
-    afterEach(() => {
-        rmSync(workspace, { recursive: true, force: true });
-    });
-    test('scaffolds .gitignore only when the context file is created', () => {
-        const file = join(workspace, '.neon');
-        applyContext(file, {
-            orgId: 'org-x',
-            projectId: 'proj-y',
-            branchId: 'br-z',
-        });
-        expect(JSON.parse(readFileSync(file, 'utf-8'))).toEqual({
-            orgId: 'org-x',
-            projectId: 'proj-y',
-            branchId: 'br-z',
-        });
-        expect(readFileSync(join(workspace, '.gitignore'), 'utf-8')).toBe('.neon\n');
-    });
-    test('does NOT re-add .neon to .gitignore on updates to an existing file', () => {
-        const file = join(workspace, '.neon');
-        // First write creates the file and scaffolds .gitignore.
-        applyContext(file, { projectId: 'proj-y', branchId: 'br-1' });
-        // The user deliberately un-ignores .neon (e.g. to commit shared context).
-        writeFileSync(join(workspace, '.gitignore'), 'node_modules\n');
-        // A subsequent update must NOT re-add the entry.
-        applyContext(file, { projectId: 'proj-y', branchId: 'br-2' });
-        expect(JSON.parse(readFileSync(file, 'utf-8'))).toEqual({
-            projectId: 'proj-y',
-            branchId: 'br-2',
-        });
-        expect(readFileSync(join(workspace, '.gitignore'), 'utf-8')).toBe('node_modules\n');
-    });
-});

package/env.test.js

@@ -1,55 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { getGithubEnvVars } from './env';
-describe('getGithubEnvVars', () => {
-    it('success all keys', () => {
-        const env = {
-            GITHUB_ACTION_PATH: '1',
-            GITHUB_REPOSITORY: '2',
-            GITHUB_RUN_ID: '3',
-            GITHUB_RUN_NUMBER: '4',
-            GITHUB_SERVER_URL: '5',
-            GITHUB_WORKFLOW_REF: '6',
-            RUNNER_ARCH: '7',
-            RUNNER_ENVIRONMENT: '8',
-            RUNNER_OS: '9',
-            unrelated: 'unrelated',
-        };
-        const ret = {
-            GITHUB_ACTION_PATH: '1',
-            GITHUB_REPOSITORY: '2',
-            GITHUB_RUN_ID: '3',
-            GITHUB_RUN_NUMBER: '4',
-            GITHUB_SERVER_URL: '5',
-            GITHUB_WORKFLOW_REF: '6',
-            RUNNER_ARCH: '7',
-            RUNNER_ENVIRONMENT: '8',
-            RUNNER_OS: '9',
-        };
-        expect(getGithubEnvVars(env)).toEqual(ret);
-    });
-    it('empty all keys', () => {
-        expect(getGithubEnvVars({})).toEqual({});
-    });
-    it('action path', () => {
-        expect(getGithubEnvVars({
-            GITHUB_ACTION_PATH: '/home/runner/work/_actions/neondatabase/create-branch-action/v5',
-        })).toEqual({
-            GITHUB_ACTION_PATH: 'neondatabase/create-branch-action/v5',
-        });
-        expect(getGithubEnvVars({
-            GITHUB_ACTION_PATH: '/home/runner/actions-runner/_work/actions/neondatabase/create-branch-action/v5',
-        })).toEqual({
-            GITHUB_ACTION_PATH: 'neondatabase/create-branch-action/v5',
-        });
-        expect(getGithubEnvVars({
-            GITHUB_ACTION_PATH: 'C:\\b\\_actions\\neondatabase\\create-branch-action\\v5',
-        })).toEqual({
-            GITHUB_ACTION_PATH: 'C:\\b\\_actions\\neondatabase\\create-branch-action\\v5',
-        });
-        expect(getGithubEnvVars({
-            GITHUB_ACTION_PATH: '/home/runner/work/app/app/./.github/actions/custom-action',
-        })).toEqual({
-            GITHUB_ACTION_PATH: 'custom-action',
-        });
-    });
-});

package/utils/formats.test.js

@@ -1,32 +0,0 @@
-import { test, describe, expect } from 'vitest';
-import { looksLikeBranchId, looksLikeLSN, looksLikeTimestamp } from './formats';
-describe('branch formats', () => {
-    test('branch name', () => {
-        expect(looksLikeBranchId('master')).toBe(false);
-    });
-    test('initial short', () => {
-        expect(looksLikeBranchId('br-flower-sunshine-123456')).toBe(true);
-    });
-    test('update 1, longer version', () => {
-        expect(looksLikeBranchId('br-flower-sunshine-12345678')).toBe(true);
-    });
-    test('update 2, includes region', () => {
-        expect(looksLikeBranchId('br-bold-recipe-a13oexw7')).toBe(true);
-    });
-});
-describe('timestamp formats', () => {
-    test('valid', () => {
-        expect(looksLikeTimestamp('2021-03-13T19:47:33.000Z')).toBe(true);
-    });
-    test('invalid', () => {
-        expect(looksLikeTimestamp('branch_name')).toBe(false);
-    });
-});
-describe('LSN formats', () => {
-    test('valid', () => {
-        expect(looksLikeLSN('0/1F56000')).toBe(true);
-    });
-    test('invalid', () => {
-        expect(looksLikeLSN('branch_name')).toBe(false);
-    });
-});

package/writer.test.js

@@ -1,104 +0,0 @@
-import { PassThrough } from 'node:stream';
-import { describe, it, expect } from 'vitest';
-import { writer } from './writer.js';
-const getMockWritable = () => {
-    const chunks = [];
-    const stream = new PassThrough();
-    stream.on('data', (chunk) => {
-        chunks.push(chunk.toString());
-    });
-    return {
-        stream,
-        getData: () => {
-            return chunks.join('');
-        },
-    };
-};
-describe('writer', () => {
-    describe('outputs yaml', () => {
-        it('outputs single data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'yaml', out: stream });
-            out.end({ foo: 'bar' }, { fields: ['foo'] });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs single data with title', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'yaml', out: stream });
-            out.end({ foo: 'bar' }, { fields: ['foo'], title: 'baz' });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs multiple data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'yaml', out: stream });
-            out
-                .write({ foo: 'bar' }, { fields: ['foo'], title: 'T1' })
-                .write({ baz: 'xyz' }, { fields: ['baz'], title: 'T2' })
-                .end();
-            expect(getData()).toMatchSnapshot();
-        });
-    });
-    describe('outputs json', () => {
-        it('outputs single data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'json', out: stream });
-            out.end({ foo: 'bar' }, { fields: ['foo'] });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs single data with title', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'json', out: stream });
-            out.end({ foo: 'bar' }, { fields: ['foo'], title: 'baz' });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs multiple data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'json', out: stream });
-            out
-                .write({ foo: 'bar' }, { fields: ['foo'], title: 'T1' })
-                .write({ baz: 'xyz' }, { fields: ['baz'], title: 'T2' })
-                .end();
-            expect(getData()).toMatchSnapshot();
-        });
-    });
-    describe('outputs table', () => {
-        it('outputs single data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'table', out: stream });
-            out.end({ foo: 'bar', extra: 'extra' }, { fields: ['foo'] });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs single data with title', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'table', out: stream });
-            out.end({ foo: 'bar', extra: 'extra' }, { fields: ['foo'], title: 'baz' });
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs multiple data', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({ output: 'table', out: stream });
-            out
-                .write({ foo: 'bar', extra: 'extra' }, { fields: ['foo'], title: 'T1' })
-                .write({ baz: 'xyz', extra: 'extra' }, { fields: ['baz'], title: 'T2' })
-                .end();
-            expect(getData()).toMatchSnapshot();
-        });
-        it('outputs table with custom renderer', () => {
-            const { stream, getData } = getMockWritable();
-            const out = writer({
-                output: 'table',
-                out: stream,
-            });
-            out
-                .write({ foo: 'bar' }, {
-                fields: ['foo'],
-                title: 'T1',
-                renderColumns: {
-                    foo: ({ foo }) => `Here is: ${foo}`,
-                },
-            })
-                .end();
-            expect(getData()).toMatchSnapshot();
-        });
-    });
-});