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.

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

EXSWHTEC-224 - Test cases ID clean up and documentation for Graph Management #95

Open
wants to merge 141 commits into
base: develop
Choose a base branch
from

Conversation

milos-mozetic
Copy link
Contributor

gargrahul and others added 30 commits October 26, 2022 03:59
Change-Id: I66f0c09e9c7405ec7430b1883e0e89542fdb87a0
Change-Id: I212b82b1b3a78a368b85ea64e338371a34b405f9
Change-Id: Ib455f72b5be77e1a81137d15c07ea41161b16a3e
… definitions

Customized Doxygen configuration named DoxyfileTests is added, as well as separate header file
used for definition of test groups.
Change-Id: Ief96e274f4143e80ceb3e40f04d38ae217777583
…etCaptureInfo_v2 and hipStreamIsCapturing

- Refactor existing tests by including more catch2 features
- Reduce code line numbers by using helper guard classes
- Add some positive and negative tests
- Add doxygen test descriptions
…ph*MemcpyNode[From|To]Symbol family of APIs

- Generalize from symbol positive test to work for different types
- Implement negative parameter tests
- Extract common code for to/from copy
- Reorganize files
- Tidy up hipGraphAddKernelNode tests
- Tidy up hipGraphKernelNodeGetParams tests
- Tidy up hipGraphKernelNodeSetParams tests
- Tidy up hipGraphExecKernelNodeSetParams tests.
- Tidy up hipGraphAddHostNode tests
- Tidy up hipGraphHostNodeGetParams tests
- Tidy up hipGraphHostNodeSetParams tests
- Tidy up hipGraphExecHostNodeSetParams tests.
- Test setting the access policy window with
hipGraphKernelNodeSetAttribute
- Test setting cooperative attribute with hipGraphKernelNodeSetAttribute
- Negative parameter tests for hipGraphKernelNodeSetAttribute
- Negative parameter tests for hipGraphKernelNodeGetAttribute.
milos-mozetic and others added 15 commits December 21, 2022 08:38
…vic/hip-tests into doxygen_graph_management_documentation
…hip-tests into doxygen_graph_management_documentation
…c/hip-tests into doxygen_graph_management_documentation
…za-halilcevic/hip-tests into doxygen_graph_management_documentation
…vic/hip-tests into doxygen_graph_management_documentation
…-tests into doxygen_graph_management_documentation
…za-halilcevic/hip-tests into doxygen_graph_management_documentation
…a-halilcevic/hip-tests into doxygen_graph_management_documentation
…tests into doxygen_graph_management_documentation
…c/hip-tests into doxygen_graph_management_documentation
…vic/hip-tests into doxygen_graph_management_documentation
…irza-halilcevic/hip-tests into doxygen_graph_management_documentation
@rorake
Copy link
Contributor

rorake commented Jun 21, 2023

@milos-mozetic
Will you please resolve the merge conflicts for this PR?

@rakesroy
Copy link
Contributor

rakesroy commented Jul 7, 2023

@milos-mozetic
Merge conflicts again occurred due to recent pushes to develop branch.
Will you please resolve them?

@milos-mozetic
Copy link
Contributor Author

@rakesroy: merge conflict resolved within commit bd81e7b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants