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

claim file: move data structs and handlers to the CNF repo #1609

Closed
wants to merge 2 commits into from

Conversation

jmontesi
Copy link
Contributor

@jmontesi jmontesi commented Nov 13, 2023

The claim file used for presenting the results of the CNF Certification Suite as well as some other information related to the CNF under test, the test environment or the configuration is an asset that belongs to the CNF Certification Suite and that no other tool is generating, that's why it might be more appropriate that the data structs and functions to handle it are placed in the CNF Certification Suite repository.

In this PR, the whole pkg/claim package has been moved from test-network-function-claim to this repo's pkg directory. As a result, the claimFormat field of the claim file has been deleted, as now the claim file version will be the same as the CNF Certification Suite version.

This PR makes the QE repo ready to work with the claim package in its new location.

@@ -1 +1,230 @@
This is a placeholder for the results.html file that will be downloaded from https://github.com/test-network-function/parser when this project is built
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this got accidentally built.

@jmontesi jmontesi closed this Nov 13, 2023
@jmontesi
Copy link
Contributor Author

Closed in favor of PR#1613.

@sebrandon1 sebrandon1 deleted the port_test_network_function_claim branch November 13, 2023 18:19
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.

2 participants