Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

Add consumer side of WIT+Auth contract tests. #2393

Merged
merged 31 commits into from
Feb 21, 2019

Conversation

pmacik
Copy link
Contributor

@pmacik pmacik commented Jan 14, 2019

This PR adds consumer side of the contract tests for the WIT service consuming Auth service API endpoints.

CICO job: openshiftio/openshiftio-cico-jobs#940

The provider side (Auth service) of the contract tests is covered by fabric8-services/fabric8-auth#711

WorkItem: https://openshift.io/openshiftio/Openshift_io/plan/detail/1329

@pmacik pmacik requested a review from kwk as a code owner January 14, 2019 12:16
@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

1 similar comment
@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@pmacik pmacik changed the title WIP: Add contract tests. WIP: Add consumer side of WIT+Auth contract tests. Jan 14, 2019
@pmacik pmacik force-pushed the contract-tests-poc branch from 2465e9f to d02c0ad Compare January 17, 2019 14:12
@alien-ike alien-ike changed the title WIP: Add consumer side of WIT+Auth contract tests. Add consumer side of WIT+Auth contract tests. Jan 18, 2019
test/contracts/consumer/fabric8auth/api_status.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/api_token.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/api_status.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/consumer_test.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/consumer_test.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/consumer_test.go Outdated Show resolved Hide resolved
test/contracts/consumer/fabric8auth/consumer_test.go Outdated Show resolved Hide resolved
@pmacik pmacik force-pushed the contract-tests-poc branch from e497bd3 to f9d4840 Compare January 22, 2019 11:21
@pmacik pmacik changed the title Add consumer side of WIT+Auth contract tests. WIP: Add consumer side of WIT+Auth contract tests. Jan 22, 2019
@pmacik pmacik force-pushed the contract-tests-poc branch 2 times, most recently from 4242601 to f3e28d7 Compare January 23, 2019 16:36
@pmacik pmacik changed the title WIP: Add consumer side of WIT+Auth contract tests. Add consumer side of WIT+Auth contract tests. Jan 24, 2019
@pmacik pmacik force-pushed the contract-tests-poc branch from f3e28d7 to 0383ca6 Compare January 24, 2019 13:09
Copy link
Collaborator

@kwk kwk left a comment

Choose a reason for hiding this comment

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

LGTM Thanks for your patience

@kwk
Copy link
Collaborator

kwk commented Jan 30, 2019

[test]

Copy link
Collaborator

@kwk kwk left a comment

Choose a reason for hiding this comment

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

LGTM

@kwk
Copy link
Collaborator

kwk commented Feb 4, 2019

[test]

Gopkg.lock Show resolved Hide resolved
Gopkg.lock Show resolved Hide resolved
@pmacik
Copy link
Contributor Author

pmacik commented Feb 18, 2019

[test]

@pmacik
Copy link
Contributor Author

pmacik commented Feb 19, 2019

[test]

@codecov
Copy link

codecov bot commented Feb 19, 2019

Codecov Report

Merging #2393 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2393      +/-   ##
==========================================
- Coverage   67.73%   67.67%   -0.07%     
==========================================
  Files         183      183              
  Lines       18123    18123              
==========================================
- Hits        12276    12265      -11     
- Misses       4664     4675      +11     
  Partials     1183     1183
Impacted Files Coverage Δ
remoteworkitem/github.go 77.14% <0%> (-22.86%) ⬇️
remoteworkitem/scheduler.go 53.65% <0%> (-7.32%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf91483...f8189b4. Read the comment docs.

@kwk kwk merged commit 54759c8 into fabric8-services:master Feb 21, 2019
@pmacik pmacik deleted the contract-tests-poc branch March 5, 2019 11:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants