PointCloudLibrary / PointCloudLibrary/pcl

[GPU Test] Move tests from GPU module to Test module

Open
#2,678 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

kind: todo module: gpu module: test
Dominant language
C++
Stars
11.1k
Forks
4.7k
Avg merge
4d 10h
Merged PRs (30d)
6

Description

During coding on #2677 I found some old tests, which are not enabled. They are not located within pcl/tests. Instead they are located within gpu directory pcl/*/test/. What should happen with them?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the old tests under pcl/*/test/ and compare them with the enabled test layout in pcl/tests; read the context from issue #2677. Determine which tests belong in the Test module and confirm that the resulting tests are discovered and enabled. Done means the relevant old tests have a clear, agreed location and are covered by the test setup.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.