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

Fix Credentials Manager Test #325

Merged
merged 6 commits into from
Nov 25, 2019
Merged

Fix Credentials Manager Test #325

merged 6 commits into from
Nov 25, 2019

Conversation

cocojoe
Copy link
Member

@cocojoe cocojoe commented Nov 24, 2019

Update CI

Changes

Thought these changes had already been applied to the expired token PR, something that would have been caught in a working codecov :(

References

Please include relevant links supporting this change such as a:

  • support ticket
  • community post
  • StackOverflow post
  • support forum thread

Please note any links that are not publicly accessible.

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

  • This change adds unit test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

@cocojoe cocojoe requested a review from a team November 24, 2019 14:56
@cocojoe cocojoe added this to the vNext milestone Nov 24, 2019
damieng
damieng previously approved these changes Nov 24, 2019
@cocojoe
Copy link
Member Author

cocojoe commented Nov 24, 2019

@damieng if you can re-review please, was working on CI. At least it's finally working on the codecov side, https://codecov.io/gh/auth0/Auth0.swift/tree/e15ec4701c0c4602424ee4a916f89c049196a69e

Just not reporting back.

@auth0 auth0 deleted a comment from crs2020 Nov 25, 2019
@cocojoe cocojoe merged commit f5f735f into master Nov 25, 2019
@cocojoe cocojoe deleted the fixed-credentialsmanager branch November 25, 2019 11:08
@cocojoe cocojoe mentioned this pull request Nov 27, 2019
sam-w pushed a commit to LoungeBuddy/Auth0.swift that referenced this pull request Mar 11, 2020
* Fix Credentials Manager Tests
* CI Tweaks
* Move to Slather for CodeCov conversion
sam-w added a commit to LoungeBuddy/Auth0.swift that referenced this pull request Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants