BigW Consortium Gitlab
Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
G
gitlab-ce
Project
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
Registry
Registry
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Commits
Issue Boards
Open sidebar
Forest Godfrey
gitlab-ce
Commits
86dc5985
Commit
86dc5985
authored
Jan 29, 2018
by
Filipa Lacerda
Browse files
Options
Browse Files
Download
Plain Diff
Merge branch 'axios-post-edit-blob' into 'master'
Replace $.post in edit blob with axios See merge request gitlab-org/gitlab-ce!16693
parents
44edd111
f7811b4b
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
29 additions
and
8 deletions
+29
-8
edit_blob.js
app/assets/javascripts/blob_edit/edit_blob.js
+10
-5
edit_spec.rb
spec/features/projects/blobs/edit_spec.rb
+19
-3
No files found.
app/assets/javascripts/blob_edit/edit_blob.js
View file @
86dc5985
/* global ace */
import
axios
from
'~/lib/utils/axios_utils'
;
import
createFlash
from
'~/flash'
;
import
{
__
}
from
'~/locale'
;
import
TemplateSelectorMediator
from
'../blob/file_template_mediator'
;
export
default
class
EditBlob
{
...
...
@@ -56,12 +59,14 @@ export default class EditBlob {
if
(
paneId
===
'#preview'
)
{
this
.
$toggleButton
.
hide
();
return
$
.
post
(
currentLink
.
data
(
'preview-url'
),
{
axios
.
post
(
currentLink
.
data
(
'preview-url'
),
{
content
:
this
.
editor
.
getValue
(),
},
(
response
)
=>
{
currentPane
.
empty
().
append
(
response
);
return
currentPane
.
renderGFM
();
});
})
.
then
(({
data
})
=>
{
currentPane
.
empty
().
append
(
data
);
currentPane
.
renderGFM
();
})
.
catch
(()
=>
createFlash
(
__
(
'An error occurred previewing the blob'
)));
}
this
.
$toggleButton
.
show
();
...
...
spec/features/projects/blobs/edit_spec.rb
View file @
86dc5985
...
...
@@ -17,12 +17,15 @@ feature 'Editing file blob', :js do
sign_in
(
user
)
end
def
edit_and_commit
def
edit_and_commit
(
commit_changes:
true
)
wait_for_requests
find
(
'.js-edit-blob'
).
click
find
(
'#editor'
)
execute_script
(
'ace.edit("editor").setValue("class NextFeature\nend\n")'
)
click_button
'Commit changes'
if
commit_changes
click_button
'Commit changes'
end
end
context
'from MR diff'
do
...
...
@@ -39,13 +42,26 @@ feature 'Editing file blob', :js do
context
'from blob file path'
do
before
do
visit
project_blob_path
(
project
,
tree_join
(
branch
,
file_path
))
edit_and_commit
end
it
'updates content'
do
edit_and_commit
expect
(
page
).
to
have_content
'successfully committed'
expect
(
page
).
to
have_content
'NextFeature'
end
it
'previews content'
do
edit_and_commit
(
commit_changes:
false
)
click_link
'Preview changes'
wait_for_requests
old_line_count
=
page
.
all
(
'.line_holder.old'
).
size
new_line_count
=
page
.
all
(
'.line_holder.new'
).
size
expect
(
old_line_count
).
to
be
>
0
expect
(
new_line_count
).
to
be
>
0
end
end
end
...
...
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