Add tests on PR

[jsfour]

When there is a GitHub pull request approved we need to grab the files changed within the PR and generate (or update) a jest test file for the updates in the PR.

The entry point for this would be:
https://github.com/octaviuslabs/walter/blob/main/src/index.ts

[jsfour]

Thoughts on the best approach here @imwalterbot

[imwalterbot]

Thoughts on the best approach here @imwalterbot

Queued for processing...

[imwalterbot]

Thoughts on the best approach here @imwalterbot

Processing this now

[imwalterbot]

Based on the provided context, I suggest the following approach to generate or update a Jest test file for the updates in the PR when a GitHub pull request is approved:

1. Create a new event handler for the pull_request_review event.
2. Check if the action in the event payload is submitted and the review state is approved.
3. Use the GitHub API to get the list of files changed in the PR.
4. For each changed file, check if it has a corresponding Jest test file. If not, create one; otherwise, update the existing test file.
5. Commit the changes to the repository.

Here's the pseudocode for the suggested approach:

function handlePullRequestReview(event) {
  if (event.payload.action == 'submitted' && event.payload.review.state == 'approved') {
    const pullRequest = event.payload.pull_request;
    const changedFiles = getChangedFiles(pullRequest);

    for (const file of changedFiles) {
      const testFile = getTestFile(file);

      if (testFile.exists()) {
        updateTestFile(testFile, file);
      } else {
        createTestFile(testFile, file);
      }
    }

    commitChanges(pullRequest);
  }
}

function getChangedFiles(pullRequest) {
  // Use GitHub API to fetch the list of files changed in the PR
}

function getTestFile(file) {
  // Determine the Jest test file path based on the file path
}

function updateTestFile(testFile, file) {
  // Update the existing test file based on the changes in the file
}

function createTestFile(testFile, file) {
  // Create a new Jest test file for the given file
}

function commitChanges(pullRequest) {
  // Commit the changes to the repository
}

// Add the new event handler to the webhooks
webhooks.on("pull_request_review", handlePullRequestReview);

Don't forget to import the necessary dependencies and API clients to interact with the GitHub API.

[jsfour]

Can you split this into 2 classes. One that does the test file creation. And another that does the getting from GitHub?

[imwalterbot]

Can you split this into 2 classes. One that does the test file creation. And another that does the getting from GitHub?

Queued for processing...

[imwalterbot]

Can you split this into 2 classes. One that does the test file creation. And another that does the getting from GitHub?

Processing this now

[imwalterbot]

Based on the context provided, I have split the functionality into two classes: GitHubService and TestFileService. Here's the pseudocode for the updated solution:

class GitHubService {
  function getChangedFiles(pullRequest) {
    // Use GitHub API to fetch the list of files changed in the PR
  }

  function commitChanges(pullRequest) {
    // Commit the changes to the repository
  }
}

class TestFileService {
  function getTestFile(file) {
    // Determine the Jest test file path based on the file path
  }

  function updateTestFile(testFile, file) {
    // Update the existing test file based on the changes in the file
  }

  function createTestFile(testFile, file) {
    // Create a new Jest test file for the given file
  }
}

function handlePullRequestReview(event) {
  if (event.payload.action == 'submitted' && event.payload.review.state == 'approved') {
    const pullRequest = event.payload.pull_request;
    const gitHubService = new GitHubService();
    const changedFiles = gitHubService.getChangedFiles(pullRequest);
    const testFileService = new TestFileService();

    for (const file of changedFiles) {
      const testFile = testFileService.getTestFile(file);

      if (testFile.exists()) {
        testFileService.updateTestFile(testFile, file);
      } else {
        testFileService.createTestFile(testFile, file);
      }
    }

    gitHubService.commitChanges(pullRequest);
  }
}

// Add the new event handler to the webhooks
webhooks.on("pull_request_review", handlePullRequestReview);

In this solution, GitHubService is responsible for handling GitHub-related operations, such as fetching the list of changed files and committing changes. TestFileService is responsible for creating and updating Jest test files. The handlePullRequestReview function now uses instances of these classes to perform the necessary operations.

[jsfour]

@imwalterbot APPROVED extend add the pull request service to this file: https://github.com/octaviuslabs/walter/blob/main/src/gh.ts

We will handle TestFileService and handlePullRequestReview separately.