Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Update Braintree creds for test #4616

Merged
merged 3 commits into from
Sep 10, 2017
Merged

Update Braintree creds for test #4616

merged 3 commits into from
Sep 10, 2017

Conversation

chadwhitacre
Copy link
Contributor

@chadwhitacre chadwhitacre commented Sep 10, 2017

#4614#4598

I need to add a custom field to our Braintree sandbox account as part of #4613, but I don't have access to do that. In the interest of time, here is a PR that rerecords all of our Braintree fixture against an account that I have access to.

  • change base to master

@chadwhitacre
Copy link
Contributor Author

Verbose log in #4615.

@chadwhitacre chadwhitacre mentioned this pull request Sep 10, 2017
@chadwhitacre chadwhitacre changed the base branch from master to selenium-workaround September 10, 2017 01:27
@chadwhitacre
Copy link
Contributor Author

Hmm ... probs w/ TestPages?

@chadwhitacre
Copy link
Contributor Author

Alright, so what's it doing there?

@chadwhitacre
Copy link
Contributor Author

Two of the tests are trying to add calls to the cassette.

test_on_the_fence_can_browse
test_new_participant_can_browse

@chadwhitacre
Copy link
Contributor Author

Back from #4615 ... redid rerecord w/ blanket rerecord on TestPages. Was 0a7213e.

For whatever reason they are different now when I browse to find them in
the sandbox admin. This also folds back to one set of creds for both dev
and test. We split test in 6d8b0a1 but
I'm not sure why (convenience at the time?).
Ahead of rerecording them using new API creds.
Redoes all of TestPages, including non-Braintree calls.
@chadwhitacre chadwhitacre changed the base branch from selenium-workaround to master September 10, 2017 08:50
@chadwhitacre
Copy link
Contributor Author

Rebased on master post-#4614 and base changed, waiting for Travis ...

@chadwhitacre chadwhitacre merged commit d57585d into master Sep 10, 2017
@chadwhitacre
Copy link
Contributor Author

🚨 Self-merge license 🚨

@chadwhitacre chadwhitacre deleted the update-braintree-creds branch September 10, 2017 16:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant