Skip to content

Add confirmation button to share input #7196

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.

Sign up for GitHub

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

Merged
MorrisJobke merged 17 commits into from
Mar 21, 2018
Merged

Add confirmation button to share input #7196

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ce7775a
Replace information icon with confirmation button in share input
jancborchardt Nov 16, 2017
6fef01c
Adjust search term to test
danxuliu Mar 16, 2018
375eab9
Add tests for emails and circles already shared with
danxuliu Mar 16, 2018
8af9c55
Add tests for exact search results already shared with
danxuliu Mar 16, 2018
3980364
Add autocompletion tests for each type of share
danxuliu Mar 16, 2018
d606219
Extract code to get suggestions to its own method
danxuliu Mar 16, 2018
fcef15a
Move stub setup outside the test method
danxuliu Mar 19, 2018
ed1452d
Use "showTemporary" instead of explicitly hiding the notification
danxuliu Mar 19, 2018
1c44051
Show an error when getting the suggestions succeeds with failure content
danxuliu Mar 19, 2018
89b0e34
Extract code to filter suggestions to its own function
danxuliu Mar 19, 2018
5e2a8cc
Return also exact matches besides all suggestions
danxuliu Mar 19, 2018
9371b61
Add a share when clicking on the confirm button
danxuliu Mar 19, 2018
10a4f8e
Confirm a share also by pressing enter on the input field
danxuliu Mar 20, 2018
6eb5cc5
Reuse last suggestions if the same parameters are used
danxuliu Mar 20, 2018
9a0fbe3
Discard cached suggestions when adding a share
danxuliu Mar 21, 2018
a2c52cd
Extract code to restore the UI after confirming a share
danxuliu Mar 20, 2018
203bf51
Keep showing the working icon while there are pending operations
danxuliu Mar 20, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Move stub setup outside the test method 
Stubs should be restored outside the test method in which they are used
to ensure that they are properly restored no matter the result of the
test (for example, if an exception is thrown).

Besides that, this will make possible to reuse the stub in other sibling
tests without having to explicitly setup it in them.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
  • Loading branch information
@danxuliu
danxuliu committed Mar 20, 2018
commit fcef15af801b4e88f857b5621ad028abdeebec28
15 changes: 11 additions & 4 deletions core/js/tests/specs/sharedialogviewSpec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ describe('OC.Share.ShareDialogView', function() {
var saveLinkShareStub;

var fetchStub;
var notificationStub;

var configModel;
var shareModel;
Expand Down Expand Up @@ -473,6 +472,16 @@ describe('OC.Share.ShareDialogView', function() {
});
});
describe('autocompletion of users', function() {
var showNotificationStub;

beforeEach(function() {
showNotificationStub = sinon.stub(OC.Notification, 'show');
});

afterEach(function() {
showNotificationStub.restore();
});

describe('triggers autocomplete display and focus with data when ajax search succeeds', function () {
it('users', function () {
dialog.render();
Expand Down Expand Up @@ -1474,12 +1483,10 @@ describe('OC.Share.ShareDialogView', function() {
});

it('throws a notification when the ajax search lookup fails', function () {
notificationStub = sinon.stub(OC.Notification, 'show');
dialog.render();
dialog.autocompleteHandler({term: 'bob'}, sinon.stub());
fakeServer.requests[0].respond(500);
expect(notificationStub.calledOnce).toEqual(true);
notificationStub.restore();
expect(showNotificationStub.calledOnce).toEqual(true);
});

describe('renders the autocomplete elements', function() {
Expand Down