Project

Profile

Help

Test #4123

Story #4687: As a user, I can use either Python or Ruby bindings to interact with Pulp

Test the generated API binds

Added by kersom 12 months ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
High
Category:
-
Sprint/Milestone:
-
Version:
Platform Release:
Tags:
Sprint:
Sprint 51

Description

  • Evaluate the best way to test the generated API binds.
    One possible solution is to use .json file generated in a certain date, and use to compare with the new one.
  • See: generarte-bindings

Related issues

Related to Pulp - Issue #3965: Created Resource schema generated incorrectly MODIFIED Actions
Related to Pulp - Issue #3964: OpenAPI schema requires Progress Report suffix to be 1 char or longer MODIFIED Actions
Related to Pulp - Issue #3850: Return relative URIs (e.g. /pulp/api/v3/repositories/1/) in the API MODIFIED Actions
Related to Pulp - Issue #4509: API docs returning different values CLOSED - NOTABUG Actions
Related to Pulp - Task #4104: Add index and headings to REST API docs MODIFIED Actions

Associated revisions

Revision ce33164d View on GitHub
Added by kersom 7 months ago

Add test to compare API bindings

Use the generated OpenAPI schema in json format generated on
April/01/2019 as base to compare with generated on the fly API schemas.
Main goal is to detect changes in the API schema.

In case of intentional changes, the `stored_open_api.json` has to be
updated.

https://pulp.plan.io/issues/4123
closes #4123

Revision dd90a811 View on GitHub
Added by dkliban@redhat.com 6 months ago

Problem: bindings break all the time

Solution: build and test bindings continuosly

This patch adds another job to the Travis build matrix. This new build installs pulpcore and pulp_file
and then builds a client library for each. The client libraries are built using pulp-swagger-codegen,
a thin wrapper around swagger-codegen-cli. After building the bindings, this job installs them and runs
a script that performs the following:

- Create a File Remote
- Create a Repository
- Sync a Repository
- Monitor the sync task
- Create an artifact from a local file
- Create a FileContent from the artifact
- Add the new FileContent to a repository version
- Monitor the repo version creation task
- Create a FilePublisher
- Create a publication from the latest version of the repository
- Monitor the publish task
- Create a distribution pointint to the publication

fixes: #4123
https://pulp.plan.io/issues/4123

History

#1 Updated by kersom 12 months ago

  • Related to Issue #3965: Created Resource schema generated incorrectly added

#2 Updated by kersom 12 months ago

  • Related to Issue #3964: OpenAPI schema requires Progress Report suffix to be 1 char or longer added

#3 Updated by kersom 12 months ago

  • Related to Issue #3850: Return relative URIs (e.g. /pulp/api/v3/repositories/1/) in the API added

#4 Updated by kersom 10 months ago

  • Description updated (diff)

#5 Updated by kersom 8 months ago

  • Priority changed from Normal to High

#6 Updated by kersom 8 months ago

  • Assignee set to kersom

#7 Updated by kersom 8 months ago

  • Status changed from NEW to ASSIGNED

#8 Updated by kersom 8 months ago

  • Related to Issue #4509: API docs returning different values added

#9 Updated by kersom 7 months ago

  • Status changed from ASSIGNED to MODIFIED

#10 Updated by kersom 7 months ago

  • Related to Task #4104: Add index and headings to REST API docs added

#11 Updated by dkliban@redhat.com 6 months ago

  • Status changed from MODIFIED to ASSIGNED
  • Assignee changed from kersom to dkliban@redhat.com

I'll add a test of the bindings using https://github.com/pulp/pulp-swagger-codegen

#12 Updated by dkliban@redhat.com 6 months ago

  • Status changed from ASSIGNED to POST
  • Sprint set to Sprint 51

#13 Updated by dkliban@redhat.com 6 months ago

  • Status changed from POST to MODIFIED

#14 Updated by dkliban@redhat.com 6 months ago

  • Parent task set to #4687

#15 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF