-
Notifications
You must be signed in to change notification settings - Fork 7
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
Dev/sessiontest #1460
Dev/sessiontest #1460
Conversation
we'll wanna rebase this guy on top of develop so that it doesn't include changes from #1452 (properly implement sessionTrackingEnabled) |
4c7a0d9
to
64abe0f
Compare
package-lock.json
Outdated
"commander": "^2.7.1", | ||
"lodash.get": "^4.0.0", | ||
"lodash.isequal": "^4.0.0", | ||
"validator": "^8.0.0" |
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.
This seems to be causing a build error. There's no ,
after this line, so npm
fails ti parse the package-lock
tests/core/core.js
Outdated
@@ -3,6 +3,11 @@ import SearchConfig from '../../src/core/models/searchconfig'; | |||
import Storage from '../../src/core/storage/storage'; | |||
import StorageKeys from '../../src/core/storage/storagekeys'; | |||
|
|||
jest.mock('../../src/core/utils/uuid'); | |||
const foo = require('../../src/core/utils/uuid'); |
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.
does importing generateUUID like
const { generateUUID } = require('../../src/core/utils/uuid')
work? If that doesn't work we should rename foo
to uuid
tests/core/core.js
Outdated
@@ -41,6 +46,44 @@ describe('Search requests are created properly', () => { | |||
}); | |||
}); | |||
|
|||
describe('Search requests are created properly', () => { |
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.
We shouldn't have two describe
blocks with the same name. We'd either put these tests in the block above or give this describe
block another name.
tests/core/core.js
Outdated
); | ||
}); | ||
|
||
it('sessionId is passed in universal search', () => { |
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.
Here and below, we're verifying that sessionId
is not passed in search
6af0336
to
dc8379a
Compare
added sessionId testing
J=SLAP-1409
TEST=automatic
ran tests and they work as expected.