opencommit icon indicating copy to clipboard operation
opencommit copied to clipboard

Add E2E Testing for OpenCommit CLI in GitHub Actions

Open malpou opened this issue 2 years ago • 10 comments

This draft PR introduces end-to-end (E2E) tests for the OpenCommit CLI tool. The main goal of these tests is to simulate user interactions and ensure the tool functions as expected.

Key Points:

  • Tests are designed to be executed in GitHub Actions.
  • These tests will be triggered on all new pull requests to ensure robustness before merging.

Currently facing some challenges with the tests. Looking forward to discussing and resolving them with @di-sukharev.

malpou avatar Sep 08 '23 10:09 malpou

@malpou it's an honour to be part of the team with you <3

let me know what help you need and i will do my best to assist

di-sukharev avatar Sep 09 '23 07:09 di-sukharev

You too, it's a fun project you have made!

I've started getting the tests to work locally (they still have some issues), but the blocker now is getting them to run consistent in the actions. Currently I'm just getting a timeout on them, so something is hanging 🤷‍♂️

What are your thought of adding the --test flag that will just give us a generic test commit message instead of calling the API?

malpou avatar Sep 09 '23 07:09 malpou

@malpou

why do you think we need --test flag? i guess mocking openAI api isnt working in GitHub action, right?

di-sukharev avatar Sep 10 '23 13:09 di-sukharev

Stale pull request message

github-actions[bot] avatar Oct 20 '23 21:10 github-actions[bot]

@malpou hey brother, any blockers here i can help with?

di-sukharev avatar Oct 23 '23 11:10 di-sukharev

Stale pull request message

github-actions[bot] avatar Dec 02 '23 21:12 github-actions[bot]

@malpou im finally back, do you still want to finish this PR?

di-sukharev avatar Feb 28 '24 05:02 di-sukharev

@malpou <3 ping me when you have time, tests is all we need :) please pull the latest dev to solve conflicts

di-sukharev avatar Mar 03 '24 11:03 di-sukharev

Hi @di-sukharev I'm busy with other things at the moment so won't have time to finish this right now

malpou avatar Mar 03 '24 11:03 malpou

no worries man, thank you for all the contributions <3 <3 <3

di-sukharev avatar Mar 03 '24 12:03 di-sukharev