caikit / caikit/caikit

temp_dpool fixture should be global

Open
#333 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
114
Forks
70
PR merge metrics
No merged PRs in 30d

Description

## Describe the bug

Right now, the `temp_dpool` fixture is limited to `test_dataobject`, which means any other test file that wants to create `dataobjects` needs to maintain unique names for their classes otherwise they will face name collisions.

If the `temp_dpool` fixture is made global, then any test that wants to test using a `dataobject` can just use that fixture.

## Platform

Please provide details about the environment you are using, including the following:

- Interpreter version: 0.9.x
- Library version: 3.9

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.