Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add integration test to CI/CD #12

Merged
merged 18 commits into from
Jan 12, 2025
Merged

Add integration test to CI/CD #12

merged 18 commits into from
Jan 12, 2025

Conversation

kkew3
Copy link
Owner

@kkew3 kkew3 commented Jan 12, 2025

Overview:

  • Drop property-based integration tests and use manually written tests instead.
  • Add to CI/CD.

kkew3 added 18 commits January 11, 2025 22:17
@kkew3
Drop property-based random integration tests
bf8ed5f 
I make this decision based on the following reasons:

- The distribution of vim buffer and keystroke sequence is too complicated and cannot be explored efficiently.
- Shrinkage to minimum working examples almost never works due to complex correlation between random variables.
- Moreover, it's quite hard to write assertions without considering in a case-by-case manner.

From now on, integration tests will be written by hand, and the correctness of the tests where the buffer involves only ASCII characters will again be verified automatically by junegunn/vader.vim.
@kkew3
Add test runner, and convert existing test cases to the new format
a0bb993
@kkew3
Manage integration test dependencies using poetry
a1c98ff
@kkew3
Update venv location
426857a
@kkew3
Remove requirements.txt, since we already have pyproject.toml
8e91259
@kkew3
Relax VIM_BIN_NAME to allow path to vim
29e6791
@kkew3
Ignore env files
840946b
@kkew3
Ignore run test script; instead, instruct how to run test in README
b0cd29d 
This is because how to run tests is quite personalized.
@kkew3
Add integration test to github workflow cicd
8c18e8d
@kkew3
Update test vimrc to accommodate to python3 in github CI/CD
244e0fd
@kkew3
Debugging github workflow cicd
3b4cf9f
@kkew3
Use VIM_BUNDLE_PATH in test vimrc; also remove debugging line from gi…
6f0917f 
…thub workflow cicd
@kkew3
Add build before integration test in github workflow cicd; debugging …
99fdcb5 
…github workflow cicd
@kkew3
Build vim with +python3 in github workflow cicd; also remove debuggin…
c72ae60 
…g line from github workflow cicd
@kkew3
Place checkout action to the first step in github workflow cicd
6dc620e 
Otherwise actions/setup-python@v5 refuses to work.
@kkew3
Some tweaks on compiling vim
dc46d3d 
Because previous commit fails on Vim 8.2.
@kkew3
Add vim to PATH if cache hits; debugging github workflow cicd
1c7d7de
@kkew3
Tweaking around compiling vim 8.2 with +python3; also remove debuggin…
bfb002e 
…g line from github workflow cicd
@kkew3 kkew3 merged commit 6e7ba78 into rust Jan 12, 2025
6 checks passed
@kkew3 kkew3 deleted the feat/integration-test branch January 12, 2025 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kkew3