Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

tests: Improve tests workflows #156

Merged
merged 18 commits into from
May 10, 2024

Conversation

frascuchon
Copy link
Member

@frascuchon frascuchon commented May 10, 2024

This PR reviews and improves some tests-related aspects:

  • Separate unit and integration tests
  • Waiting for argilla quickstart is up and running (CI)
  • Run unit and integration tests separately (CI)
  • Fix failing tests
  • Define a common client fixture for all integration tests (see tests/integratio/conftest.py)

After this merge, incoming PRs should pass the tests to be merged :-)

@frascuchon frascuchon changed the title tests: separate unit and integration tests [WIP] tests: separate unit and integration tests May 10, 2024
@frascuchon frascuchon force-pushed the tests/separate-unit-and-integration-tests branch from 4d4c698 to 3615d94 Compare May 10, 2024 09:04
@frascuchon frascuchon changed the title [WIP] tests: separate unit and integration tests [WIP] tests: Improve tests workflows May 10, 2024
@frascuchon frascuchon force-pushed the tests/separate-unit-and-integration-tests branch from 3615d94 to 109e223 Compare May 10, 2024 10:19
* refactor(Dataset): Review dataset resource class

- `model.workspace_id` is used if present
- `name` getter/setter and remove _sync call
- `status` and `id` args removed from init since are read-only

* tests: add more tests
…m:argilla-io/argilla-python into tests/separate-unit-and-integration-tests
@frascuchon frascuchon changed the title [WIP] tests: Improve tests workflows tests: Improve tests workflows May 10, 2024
@frascuchon frascuchon merged commit 8049888 into main May 10, 2024
5 of 6 checks passed
@frascuchon frascuchon deleted the tests/separate-unit-and-integration-tests branch May 10, 2024 11:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant