facebookresearch / facebookresearch/fairo

[polymetis] Simplified integration tests

Open
#930 0 comments 0 reactions 1 assignee Claimed by @1heart View on GitHub
Dominant language
Jupyter Notebook
Stars
929
Forks
123
PR merge metrics
No merged PRs in 30d

Description

## Type of Issue

Select the type of issue:
- [ ] Bug report (to report a bug)
- [ ] Feature request (to request an additional feature)
- [ ] Tracker (I am just using this as a tracker)
- [x] Refactor request
- [ ] Documentation Ask

## Description

In #926, I split integration tests into 3 shell scripts. Currently, it's a bit awkward because process lifecycle (server, robot client, user client) is being managed by shell scripts; after fbrp integration, we should revisit this.

[Some ideas](https://github.com/facebookresearch/fairo/pull/926#discussion_r777861745) for removing tech debt.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.