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

test/e2e #64

Merged
merged 44 commits into from
Nov 5, 2024
Merged

test/e2e #64

merged 44 commits into from
Nov 5, 2024

Conversation

alemidev
Copy link
Member

add end-to-end tests and relevant fixtures, to leverage codemp server and test both

@alemidev alemidev added bug Something isn't working as intended ci This issue is related to the CI system core This issue is related to the core Rust library tweak Code quality labels Oct 30, 2024
@alemidev alemidev marked this pull request as draft October 30, 2024 12:55
alemidev and others added 15 commits October 30, 2024 14:22
@alemidev
fix: more time in tryRecv to compensate longer RTT
e0b919b
@alemidev
ci(test): only run build tests after functional
752a682 
so that functional tests run sooner
@alemidev
test: show for which test workspace was
cd4e09c 
basically fixtures leak workspaces when errors occur, so to better debug
what is happening every test now names its workspace. this is tedious
and should probably be removed eventually but for now it helps a ton
@zaaarf
test: buffer fixture
111020a
@ghyatzo
test: workspace creation and lookup
7a535d4
@ghyatzo
fmt: formatter shenanigans
dc627dc
@ghyatzo
chore: implement from ConnectionError and from RemoteError for Assert…
44e5e39 
…ionError
@ghyatzo
test: can't create workspace more than once
9ff9a47
@ghyatzo
test: attaching to workspaces and making them active
ccb5406
@ghyatzo
test: attaching to a non existing workspace is an error
fdd2726
@ghyatzo
test: leaving workspaces, attaching after leaving
781fd9b
@ghyatzo
test: delete empty workspace
9d0c961
@ghyatzo
test: deleting twice or non existing is an error
35b9b12
@ghyatzo
test(WIP): added skeleton for testing of behaviour when deleting work…
84bfc45 
…space with people attached.

Need to know what is the expected behaviour that we want.
@ghyatzo
test: inviting users to one's workspace
7f17d1f
ghyatzo and others added 11 commits November 3, 2024 17:26
@ghyatzo
test: moved around previously existing tests. and other formatter she…
3773ebf 
…nanigans
@ghyatzo
fix(test): add more clear explainations of what went wrong.
c0bff45
@zaaarf
docs(java): specify the new behaviour of leaveWorkspace
518143d
@zaaarf
tests: cleanup redundant code
c08e209
@zaaarf
tests: actually delete the buffer in the buffer fixture
c1c9dea
@zaaarf
docs: better wording in detach_buffer docs
888f7fd
@alemidev @ghyatzo
chore: removed need of adding type hints
ee2ced5 
Co-authored-by: cschen <cschen@codemp.dev>
@alemidev @ghyatzo
test: better assert_or_err messages
c42b091 
Co-authored-by: cschen <cschen@codemp.dev>
@alemidev @ghyatzo @zaaarf
fix: cleanup for buffer fixture
300f662 
idk who made this?? it was already fixed in dev env, uncommitted

Co-authored-by: cschen <cschen@codemp.dev>
Co-authored-by: zaaarf <me@zaaarf.foo>
@alemidev
feat(test): added fixture macro
3f0b04a
@alemidev @ghyatzo
test: improved tests cleanup and logic
2155f0d 
Co-authored-by: cschen <cschen@codemp.dev>
@alemidev alemidev marked this pull request as ready for review November 4, 2024 23:45
@zaaarf zaaarf merged commit 1a2542a into dev Nov 5, 2024
16 checks passed
@zaaarf zaaarf deleted the test/e2e branch November 5, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zaaarf zaaarf zaaarf approved these changes

Assignees

@alemidev alemidev

@zaaarf zaaarf

Labels
bug Something isn't working as intended ci This issue is related to the CI system core This issue is related to the core Rust library tweak Code quality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alemidev @zaaarf @ghyatzo