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
Adjust specs to work with new lib interface #1351
Adjust specs to work with new lib interface #1351
Conversation
extend, forEach, merge and trim are replaced by lodash
test/specs/basicAuth.spec.js
Outdated
it('should accept HTTP Basic auth with username/password', function (done) { | ||
axios.get('/foo', { | ||
auth: { | ||
username: 'Aladdin', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't Alibaba
who said that 🙈
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hahahaha, I just copy & 🍝 it, but yeah you're right 😂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, after we Aladin
--> Alibaba
Umbrella: #1333
As later on documented in the Changelog:
btoa
Polyfill anymoreaxios()
interface will be removed,axios.request()
and the method shorthand methods are the way to gotest/specs/utils
:trim
,merge
,forEach
&extend
are removed, they will be superseded by their lodash equivalents