npm-update-package 0.31.3 → 0.33.0

package/src/git/GitRepository.ts

@@ -1,57 +0,0 @@
-import parse from 'parse-github-url'
-
-export class GitRepository {
-  readonly host: string
-  readonly owner: string
-  readonly name: string
-
-  private constructor ({
-    host,
-    owner,
-    name
-  }: {
-    host: string
-    owner: string
-    name: string
-  }) {
-    this.host = host
-    this.owner = owner
-    this.name = name
-  }
-
-  static of (url: string): GitRepository {
-    const parsed = parse(url)
-
-    if (parsed === null) {
-      throw new Error(`Failed to parse url. url=${url}`)
-    }
-
-    const {
-      host,
-      owner,
-      name
-    } = parsed
-
-    if (host === null || owner === null || name === null) {
-      throw new Error(`Failed to parse url. url=${url}`)
-    }
-
-    return new GitRepository({
-      host,
-      owner,
-      name
-    })
-  }
-
-  get apiEndPoint (): string {
-    if (this.isGitHubDotCom) {
-      return 'https://api.github.com'
-    } else {
-      return `https://${this.host}/api/v3`
-    }
-  }
-
-  get isGitHubDotCom (): boolean {
-    return this.host === 'github.com'
-  }
-}

package/src/git/createBranchName.test.ts

@@ -1,14 +0,0 @@
-import { SemVer } from '../semver'
-import { createBranchName } from './createBranchName'
-
-describe('createBranchName', () => {
-  it('returns branch name', () => {
-    const actual = createBranchName({
-      name: '@npm-update-package/example',
-      currentVersion: SemVer.of('1.0.0'),
-      newVersion: SemVer.of('1.2.3'),
-      level: 'major'
-    })
-    expect(actual).toBe('npm-update-package/@npm-update-package/example/v1.2.3')
-  })
-})

package/src/git/createBranchName.ts

@@ -1,7 +0,0 @@
-import type { OutdatedPackage } from '../core'
-
-export const createBranchName = (outdatedPackage: OutdatedPackage): string => {
-  const packageName = outdatedPackage.name
-  const newVersion = outdatedPackage.newVersion.version
-  return `npm-update-package/${packageName}/v${newVersion}`
-}

package/src/git/index.ts

@@ -1,4 +0,0 @@
-export * from './CommitMessageCreator'
-export * from './createBranchName'
-export * from './Git'
-export * from './GitRepository'

package/src/github/GitHub.ts

@@ -1,198 +0,0 @@
-import type {
-  Octokit,
-  RestEndpointMethodTypes
-} from '@octokit/rest'
-import type { ValuesType } from 'utility-types'
-
-export type Branch = ValuesType<RestEndpointMethodTypes['repos']['listBranches']['response']['data']>
-export type CreatedPullRequest = RestEndpointMethodTypes['pulls']['create']['response']['data']
-export type Label = RestEndpointMethodTypes['issues']['getLabel']['response']['data']
-export type PullRequest = ValuesType<RestEndpointMethodTypes['pulls']['list']['response']['data']>
-export type Repository = RestEndpointMethodTypes['repos']['get']['response']['data']
-
-// TODO: add test
-export class GitHub {
-  constructor (private readonly octokit: Octokit) {}
-
-  async addLabels ({
-    owner,
-    repo,
-    issueNumber,
-    labels
-  }: {
-    owner: string
-    repo: string
-    issueNumber: number
-    labels: string[]
-  }): Promise<void> {
-    await this.octokit.issues.addLabels({
-      owner,
-      repo,
-      issue_number: issueNumber,
-      labels
-    })
-  }
-
-  async closePullRequest ({
-    owner,
-    repo,
-    pullNumber
-  }: {
-    owner: string
-    repo: string
-    pullNumber: number
-  }): Promise<void> {
-    await this.octokit.pulls.update({
-      owner,
-      repo,
-      pull_number: pullNumber,
-      state: 'closed'
-    })
-  }
-
-  async createLabel ({
-    owner,
-    repo,
-    name,
-    description,
-    color
-  }: {
-    owner: string
-    repo: string
-    name: string
-    description?: string
-    color?: string
-  }): Promise<void> {
-    await this.octokit.issues.createLabel({
-      owner,
-      repo,
-      name,
-      description,
-      color
-    })
-  }
-
-  async createPullRequest ({
-    owner,
-    repo,
-    baseBranch,
-    headBranch,
-    title,
-    body
-  }: {
-    owner: string
-    repo: string
-    baseBranch: string
-    headBranch: string
-    title: string
-    body: string
-  }): Promise<CreatedPullRequest> {
-    const { data } = await this.octokit.pulls.create({
-      owner,
-      repo,
-      base: baseBranch,
-      head: headBranch,
-      title,
-      body
-    })
-    return data
-  }
-
-  async deleteBranch ({
-    owner,
-    repo,
-    branch
-  }: {
-    owner: string
-    repo: string
-    branch: string
-  }): Promise<void> {
-    await this.octokit.git.deleteRef({
-      owner,
-      repo,
-      ref: `heads/${branch}`
-    })
-  }
-
-  // TODO: fetch all branches with page option
-  async fetchBranches ({
-    owner,
-    repo
-  }: {
-    owner: string
-    repo: string
-  }): Promise<Branch[]> {
-    const { data } = await this.octokit.repos.listBranches({
-      owner,
-      repo,
-      per_page: 100
-    })
-    return data
-  }
-
-  async fetchLabel ({
-    owner,
-    repo,
-    name
-  }: {
-    owner: string
-    repo: string
-    name: string
-  }): Promise<Label> {
-    const { data } = await this.octokit.issues.getLabel({
-      owner,
-      repo,
-      name
-    })
-    return data
-  }
-
-  // TODO: fetch all pull requests with page option
-  async fetchPullRequests ({
-    owner,
-    repo
-  }: {
-    owner: string
-    repo: string
-  }): Promise<PullRequest[]> {
-    const { data } = await this.octokit.pulls.list({
-      owner,
-      repo,
-      per_page: 100
-    })
-    return data
-  }
-
-  async fetchRepository ({
-    owner,
-    repo
-  }: {
-    owner: string
-    repo: string
-  }): Promise<Repository> {
-    const { data } = await this.octokit.repos.get({
-      owner,
-      repo
-    })
-    return data
-  }
-
-  async requestReviewers ({
-    owner,
-    repo,
-    pullNumber,
-    reviewers
-  }: {
-    owner: string
-    repo: string
-    pullNumber: number
-    reviewers: string[]
-  }): Promise<void> {
-    await this.octokit.pulls.requestReviewers({
-      owner,
-      repo,
-      pull_number: pullNumber,
-      reviewers
-    })
-  }
-}

package/src/github/branch/finder/BranchFinder.test.ts

@@ -1,21 +0,0 @@
-import type { Branch } from '../../GitHub'
-import { BranchFinder } from './BranchFinder'
-
-describe('BranchFinder', () => {
-  describe('findByName', () => {
-    const branch = {
-      name: 'test_branch_name'
-    } as unknown as Branch
-    const branchFinder = new BranchFinder([branch])
-
-    it('returns branch if exists', () => {
-      const actual = branchFinder.findByName('test_branch_name')
-      expect(actual).toBe(branch)
-    })
-
-    it('returns undefined if does not exists', () => {
-      const actual = branchFinder.findByName('not_test_branch_name')
-      expect(actual).toBeUndefined()
-    })
-  })
-})

package/src/github/branch/finder/BranchFinder.ts

@@ -1,9 +0,0 @@
-import type { Branch } from '../../GitHub'
-
-export class BranchFinder {
-  constructor (private readonly branches: Branch[]) {}
-
-  findByName (branchName: string): Branch | undefined {
-    return this.branches.find(({ name }) => name === branchName)
-  }
-}

package/src/github/branch/finder/index.ts

@@ -1 +0,0 @@
-export * from './BranchFinder'

package/src/github/branch/index.ts

@@ -1 +0,0 @@
-export * from './finder'

package/src/github/createGitHub.test.ts

@@ -1,16 +0,0 @@
-import { GitRepository } from '../git'
-import { createGitHub } from './createGitHub'
-import { GitHub } from './GitHub'
-
-describe('createGitHub', () => {
-  it('returns new GitHub instance', () => {
-    const repository = GitRepository.of('https://github.com/npm-update-package/npm-update-package.git')
-    const token = 'token'
-    const github = createGitHub({
-      repository,
-      token
-    })
-
-    expect(github).toBeInstanceOf(GitHub)
-  })
-})

package/src/github/createGitHub.ts

@@ -1,17 +0,0 @@
-import type { GitRepository } from '../git'
-import { createOctokit } from './createOctokit'
-import { GitHub } from './GitHub'
-
-export const createGitHub = ({
-  repository,
-  token
-}: {
-  repository: GitRepository
-  token: string
-}): GitHub => {
-  const octokit = createOctokit({
-    repository,
-    token
-  })
-  return new GitHub(octokit)
-}

package/src/github/createOctokit.test.ts

@@ -1,27 +0,0 @@
-import { Octokit } from '@octokit/rest'
-import { GitRepository } from '../git'
-import { createOctokit } from './createOctokit'
-
-describe('createOctokit', () => {
-  const token = 'token'
-
-  it('returns new Octokit instance for GitHub.com if repository is GitHub.com', () => {
-    const repository = GitRepository.of('https://github.com/npm-update-package/npm-update-package.git')
-    const octokit = createOctokit({
-      repository,
-      token
-    })
-
-    expect(octokit).toBeInstanceOf(Octokit)
-  })
-
-  it('returns new Octokit instance for GitHub Enterprise if repository is not GitHub.com', () => {
-    const repository = GitRepository.of('https://git.example.com/npm-update-package/npm-update-package.git')
-    const octokit = createOctokit({
-      repository,
-      token
-    })
-
-    expect(octokit).toBeInstanceOf(Octokit)
-  })
-})

package/src/github/createOctokit.ts

@@ -1,27 +0,0 @@
-import { Octokit } from '@octokit/rest'
-import pkg from '../../package.json'
-import type { GitRepository } from '../git'
-
-export const createOctokit = ({
-  repository,
-  token
-}: {
-  repository: GitRepository
-  token: string
-}): Octokit => {
-  const auth = `token ${token}`
-  const userAgent = `${pkg.name}/${pkg.version}`
-
-  if (repository.isGitHubDotCom) {
-    return new Octokit({
-      auth,
-      userAgent
-    })
-  } else {
-    return new Octokit({
-      auth,
-      userAgent,
-      baseUrl: repository.apiEndPoint
-    })
-  }
-}

package/src/github/errors/NotFoundError.ts

@@ -1,11 +0,0 @@
-import { RequestError } from '@octokit/request-error'
-import { StatusCodes } from 'http-status-codes'
-
-export interface NotFoundError extends RequestError {
-  status: StatusCodes.NOT_FOUND
-}
-
-// TODO: Add test
-export const isNotFoundError = (value: unknown): value is NotFoundError => {
-  return value instanceof RequestError && value.status === StatusCodes.NOT_FOUND
-}

package/src/github/errors/index.ts

@@ -1 +0,0 @@
-export * from './NotFoundError'

package/src/github/index.ts

@@ -1,5 +0,0 @@
-export * from './branch'
-export * from './createGitHub'
-export * from './errors'
-export * from './label'
-export * from './pull-request'

package/src/github/label/creator/LabelCreator.ts

@@ -1,71 +0,0 @@
-import type { GitRepository } from '../../../git'
-import type { Logger } from '../../../logger'
-import { isNotFoundError } from '../../errors'
-import type {
-  GitHub,
-  Label
-} from '../../GitHub'
-
-// TODO: Add test
-export class LabelCreator {
-  private readonly github: GitHub
-  private readonly gitRepo: GitRepository
-  private readonly logger: Logger
-
-  constructor ({
-    github,
-    gitRepo,
-    logger
-  }: {
-    github: GitHub
-    gitRepo: GitRepository
-    logger: Logger
-  }) {
-    this.github = github
-    this.gitRepo = gitRepo
-    this.logger = logger
-  }
-
-  async create ({
-    name,
-    description,
-    color
-  }: {
-    name: string
-    description?: string
-    color?: string
-  }): Promise<void> {
-    const label = await this.fetchLabel(name)
-
-    if (label !== undefined) {
-      this.logger.info(`Skip creating ${name} label because it already exists.`)
-      return
-    }
-
-    await this.github.createLabel({
-      owner: this.gitRepo.owner,
-      repo: this.gitRepo.name,
-      name,
-      description,
-      color
-    })
-    this.logger.info(`${name} label has created.`)
-  }
-
-  private async fetchLabel (name: string): Promise<Label | undefined> {
-    try {
-      return await this.github.fetchLabel({
-        owner: this.gitRepo.owner,
-        repo: this.gitRepo.name,
-        name
-      })
-    } catch (e) {
-      if (isNotFoundError(e)) {
-        this.logger.warn(e)
-        return undefined
-      } else {
-        throw e
-      }
-    }
-  }
-}

package/src/github/label/creator/index.ts

@@ -1 +0,0 @@
-export * from './LabelCreator'

package/src/github/label/index.ts

@@ -1 +0,0 @@
-export * from './creator'

package/src/github/pull-request/closer/PullRequestCloser.test.ts

@@ -1,53 +0,0 @@
-import type {
-  GitHub,
-  PullRequest
-} from '../../GitHub'
-import { PullRequestCloser } from './PullRequestCloser'
-
-describe('PullRequestCloser', () => {
-  describe('close', () => {
-    const githubClosePullRequestMock = jest.fn()
-    const githubDeleteBranchMock = jest.fn()
-
-    const github = {
-      closePullRequest: githubClosePullRequestMock,
-      deleteBranch: githubDeleteBranchMock
-    } as unknown as GitHub
-
-    afterEach(() => {
-      githubClosePullRequestMock.mockReset()
-      githubDeleteBranchMock.mockReset()
-    })
-
-    it('closes pull request', async () => {
-      const pullRequest = {
-        number: 1,
-        head: {
-          ref: 'new-topic'
-        },
-        base: {
-          repo: {
-            name: 'npm-update-package',
-            owner: {
-              login: 'npm-update-package'
-            }
-          }
-        }
-      } as unknown as PullRequest
-
-      const pullRequestCreator = new PullRequestCloser(github)
-      await pullRequestCreator.close(pullRequest)
-
-      expect(githubClosePullRequestMock).toBeCalledWith({
-        owner: pullRequest.base.repo.owner.login,
-        repo: pullRequest.base.repo.name,
-        pullNumber: pullRequest.number
-      })
-      expect(githubDeleteBranchMock).toBeCalledWith({
-        owner: pullRequest.base.repo.owner.login,
-        repo: pullRequest.base.repo.name,
-        branch: pullRequest.head.ref
-      })
-    })
-  })
-})

package/src/github/pull-request/closer/PullRequestCloser.ts

@@ -1,21 +0,0 @@
-import type {
-  GitHub,
-  PullRequest
-} from '../../GitHub'
-
-export class PullRequestCloser {
-  constructor (private readonly github: GitHub) {}
-
-  async close (pullRequest: PullRequest): Promise<void> {
-    await this.github.closePullRequest({
-      owner: pullRequest.base.repo.owner.login,
-      repo: pullRequest.base.repo.name,
-      pullNumber: pullRequest.number
-    })
-    await this.github.deleteBranch({
-      owner: pullRequest.base.repo.owner.login,
-      repo: pullRequest.base.repo.name,
-      branch: pullRequest.head.ref
-    })
-  }
-}

package/src/github/pull-request/closer/index.ts

@@ -1 +0,0 @@
-export * from './PullRequestCloser'

package/src/github/pull-request/creator/PullRequestCreator.test.ts

@@ -1,98 +0,0 @@
-import type { OutdatedPackage } from '../../../core'
-import type { GitRepository } from '../../../git'
-import {
-  createLogger,
-  LogLevel
-} from '../../../logger'
-import type {
-  GitHub,
-  Repository as GitHubRepository
-} from '../../GitHub'
-import * as createPullRequestBodyModule from './createPullRequestBody'
-import { PullRequestCreator } from './PullRequestCreator'
-import type { PullRequestTitleCreator } from './PullRequestTitleCreator'
-
-describe('PullRequestCreator', () => {
-  describe('create', () => {
-    const pullRequestTitleCreatorCreateMock = jest.fn()
-    const githubCreatePullRequestMock = jest.fn()
-    const githubAddLabelsMock = jest.fn()
-    const githubRequestReviewersMock = jest.fn()
-    const createPullRequestBodySpy = jest.spyOn(createPullRequestBodyModule, 'createPullRequestBody')
-
-    const github = {
-      createPullRequest: githubCreatePullRequestMock,
-      addLabels: githubAddLabelsMock,
-      requestReviewers: githubRequestReviewersMock
-    } as unknown as GitHub
-    const gitRepo = {
-      owner: 'repository owner',
-      name: 'repository name'
-    } as unknown as GitRepository
-    const githubRepo = {
-      default_branch: 'master'
-    } as unknown as GitHubRepository
-    const pullRequestTitleCreator = {
-      create: pullRequestTitleCreatorCreateMock
-    } as unknown as PullRequestTitleCreator
-    const logger = createLogger(LogLevel.Off)
-    const outdatedPackage = {} as unknown as OutdatedPackage
-    const branchName = 'branch name'
-
-    afterEach(() => {
-      pullRequestTitleCreatorCreateMock.mockReset()
-      githubCreatePullRequestMock.mockReset()
-      githubAddLabelsMock.mockReset()
-      githubRequestReviewersMock.mockReset()
-      createPullRequestBodySpy.mockReset()
-    })
-
-    it('creates pull request', async () => {
-      const title = 'pull request title'
-      pullRequestTitleCreatorCreateMock.mockReturnValue(title)
-      const body = 'pull request body'
-      createPullRequestBodySpy.mockReturnValue(body)
-      const pullRequest = {
-        number: 1
-      }
-      githubCreatePullRequestMock.mockResolvedValue(pullRequest)
-
-      const reviewers = ['npm-update-package']
-      const pullRequestCreator = new PullRequestCreator({
-        github,
-        gitRepo,
-        githubRepo,
-        pullRequestTitleCreator,
-        logger,
-        reviewers
-      })
-      await pullRequestCreator.create({
-        outdatedPackage,
-        branchName
-      })
-
-      expect(pullRequestTitleCreatorCreateMock).toBeCalledWith(outdatedPackage)
-      expect(createPullRequestBodySpy).toBeCalledWith(outdatedPackage)
-      expect(githubCreatePullRequestMock).toBeCalledWith({
-        owner: gitRepo.owner,
-        repo: gitRepo.name,
-        baseBranch: githubRepo.default_branch,
-        headBranch: branchName,
-        title,
-        body
-      })
-      expect(githubAddLabelsMock).toBeCalledWith({
-        owner: gitRepo.owner,
-        repo: gitRepo.name,
-        issueNumber: pullRequest.number,
-        labels: ['npm-update-package']
-      })
-      expect(githubRequestReviewersMock).toBeCalledWith({
-        owner: gitRepo.owner,
-        repo: gitRepo.name,
-        pullNumber: pullRequest.number,
-        reviewers
-      })
-    })
-  })
-})