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

Added ability to set cassette options [Tests Required] #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pnicholls
Copy link

The options added so far allow a user to select what attributes a request should be matched on.

Request matching options include URL, Path, HTTPMethod and HTTPBody.

I have added one test but this really needs more, I will add more once I find some time.

// MARK: - Functions

func interactionForRequest(request: NSURLRequest) -> Interaction? {
for interaction in interactions {
let interactionRequest = interaction.request

Copy link
Author

@pnicholls pnicholls Jul 8, 2016

Choose a reason for hiding this comment

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

The below feels pretty ugly, anyone have a better approach?

@pnicholls
Copy link
Author

pnicholls commented Jul 8, 2016

Build seems to be broken upstream? Seems to be broken just on CI.

@pnicholls pnicholls changed the title Added ability to set cassette request options [Tests Required] Added ability to set cassette options [Tests Required] Jul 8, 2016
The options added so far allow a user to select what attributes a request should be matched on.

Request matching options include URL, Path, HTTPMethod and HTTPBody.
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.

1 participant