Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
5e679331
Commit
5e679331
authored
Aug 29, 2015
by
muhammad-ammar
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
fix flaky
TNL-3166
parent
072a66e1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
9 additions
and
10 deletions
+9
-10
cms/static/js/spec/views/pages/library_users_spec.js
+9
-10
No files found.
cms/static/js/spec/views/pages/library_users_spec.js
View file @
5e679331
...
...
@@ -87,27 +87,26 @@ function ($, AjaxHelpers, ViewHelpers, ManageUsersFactory, ViewUtils) {
expect
(
requests
.
length
).
toEqual
(
0
);
});
// TODO: This test has become flaky. See TNL-3166
xit
(
"displays an error when the user has already been added"
,
function
()
{
it
(
"displays an error when the user has already been added"
,
function
()
{
var
requests
=
AjaxHelpers
.
requests
(
this
);
var
promptSpy
=
ViewHelpers
.
createPromptSpy
();
$
(
'.create-user-button'
).
click
();
$
(
'.user-email-input'
).
val
(
'honor@example.com'
);
var
warningPromptSelector
=
'.wrapper-prompt.is-shown .prompt.warning'
;
expect
(
$
(
warningPromptSelector
).
length
).
toEqual
(
0
);
$
(
'.form-create.create-user .action-primary'
).
click
();
expect
(
$
(
warningPromptSelector
).
length
).
toEqual
(
1
);
expect
(
$
(
warningPromptSelector
)).
toContainText
(
'Already a library team member'
);
ViewHelpers
.
verifyPromptShowing
(
promptSpy
,
'Already a library team member'
);
expect
(
requests
.
length
).
toEqual
(
0
);
});
// TODO: This test has become flaky. See TNL-3166
x
it
(
"can remove a user's permission to access the library"
,
function
()
{
it
(
"can remove a user's permission to access the library"
,
function
()
{
var
requests
=
AjaxHelpers
.
requests
(
this
);
var
promptSpy
=
ViewHelpers
.
createPromptSpy
();
var
reloadSpy
=
spyOn
(
ViewUtils
,
'reload'
);
var
email
=
"honor@example.com"
;
$
(
'.user-item[data-email="'
+
email
+
'"] .action-delete .delete'
).
click
();
expect
(
$
(
'.wrapper-prompt.is-shown .prompt.warning'
).
length
).
toEqual
(
1
);
$
(
'.wrapper-prompt.is-shown .action-primary'
).
click
();
ViewHelpers
.
verifyPromptShowing
(
promptSpy
,
'Are you sure?'
);
ViewHelpers
.
confirmPrompt
(
promptSpy
);
ViewHelpers
.
verifyPromptHidden
(
promptSpy
);
AjaxHelpers
.
expectJsonRequest
(
requests
,
'DELETE'
,
getUrl
(
email
),
{
role
:
null
});
AjaxHelpers
.
respondWithJson
(
requests
,
{
'result'
:
'ok'
});
expect
(
reloadSpy
).
toHaveBeenCalled
();
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment