►
Description
Result: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/33189#note_377446537
B
B
Mr.
snack
managed
to
do
a
mess
here,
don't
you
wait
when
you
move
desktop
around
and
you
end
up
splitting
stacking
three
four
pieces
like
three
three
windows
on
the
same?
Oh
okay,
so
that's
the
plan
share
Facebook.
So,
as
a
matter
of
fact,
I'm
working
on
you
might
a
changes.
A
lot
and
a
lot
was
creature
that
then
you
can
maintain
the
noise
and
out
of
me
because
they
need
to
be
precise.
B
The
you
need
to
have
the
correct
date,
blah
blah
blah
and
so
I
thought
we
have
our
crappy
but
a
test
that
they
are
actually
loading.
The
page
all
right.
Why
can't
they
take
a
screenshot?
They
take
a
speech
at
where
they
hail,
so
no
takers
featured
when
I
asked
you
and
then
I
told
that
we
have
fixtures
for
the
front
end
right
where
we
specify
the
path
on
the
title
and
if
we
generate
a
Jason,
so
I
put
the
two
things
together
and
basically
at
this.
B
So
this
is
the
simplex
that
he
will
go
and
ride
in.
So
this
is
every
eight
block.
It's
in
screenshot,
right,
yeah
and
with
two
two
different
men.
You
just
need
to
use
spectral
data
if
you
want
so
that
you
can
zoom
down
to
an
element,
and
otherwise
you
take
this
the
full
page,
and
so
this
is
a
proof
of
concept
a
mile-
and
this
is
our
screenshot
taken
right
and
my
dad
very
cold
right
and-
and
it's
also
very
good,
because
you
can
control
the
kind
of
you
don't
meter.
B
K
and
stuff,
because
you
can
use
so-
and
you
know
you
code
it
once
it
probably
it's
a
little
bit
of
overwrite,
but
that
is
done
it.
You
will
keep
running
every
time.
The
UI
updates
you
Ronnie,
and
the
thing
that
is
missing
here
is
to
attach
the
rate
common
that
also
run
PNG
quaint,
but
will
like
compress
them.
D
B
C
C
B
Argument
so
this
that's
true,
I
I
did
involve
the
technical
writer
team
before
even
like
thinking
about
and-
and
they
are
on
board
of
this.
As
long
as
then,
we
don't
use
this
tool
as
an
excuse
to
add
the
useless
screenshots
to
the
documentation
right,
because
the
idea
is
that
they
should
be
kept
to
a
minimum,
and
it
doesn't
matter
one
nor
316.
Once
it's
done
is
done
so
I
see
it,
but
is
it
she.
A
C
A
He
may
be
aware
of,
or
or
Achilles
he
may
be
aware
of.
Caching
issues
with
if
we
overwrite
images
and
that
might
be
with
Jose
is
touching
on
to
why
we.
Why?
Because,
technically
these
documentation
are
hosted
on
one
site
and
you
can
go
back
to
different
miles
versions,
and
so
that
might
be
why
you
do
need
to
prefix
it.
So
I
would
try
to
get
someone.
That's
really
aware
of
the
caching
behavior
of
multiple
versions
and
see
if
this
would
cause
an
issue
of
we
can.
We
can
end
the
milestone
yeah.
B
Otherwise
I
would
have
finalized
it
right
yeah.
So
you
see,
this
is
a
is
not
a
feature
test,
so
I
didn't
call
it
spec,
okay,
because
it's
not
a
spec,
it's
something
that
is
using
the
spec,
but
if
I
move
it
out
from
the
feature
folder,
this
thing
completely
stopped
working
because
our
spec
slash,
capybara
I,
don't
know
all
the
magic
that
is
going
on
here.
It's
doing
stuff,
folder
based
so
as.
C
B
B
C
Gonna
probably
have
to
create
a
new
helper
for
that,
because
you
use
introspect
helper
and
the
spec
helper.
It
has
all
the
sort
of
code
that
capybara
needs
to
run
and
that
a
specific
folder.
So
if
we
take
a
look
at
spec
fold,
the
spec
helper
inside
the
spec
helper
I
think
there's
another
call
to
another
file.
B
C
D
C
B
C
He
might
probably
guide
you
in
and
I
get
in
the
right
path
to
actually
create
a
file
specific
to
the
screenshots,
because
this
uses
a
whole
lot
of
dependencies,
like
logging
active
record
helpers,
and
that
involves
database
and
data.
As
you
know,
that
database
can
be
like
a
real
bottleneck
when
it
comes.
B
C
A
Right
I
think,
there's
I
think
it's
from
what
I
see
I
see
two
things
like
that
could
be
really
helpful.
Improving
the
usability
of
this,
and
one
is
because
database
is
slow
rather
than
having
one
speck
per
screenshot.
Let
me
do
one
big
walkthrough
of
and
then
save
a
screenshot
at
different
points.
That
could
be
really
really
helpful
because
when.
A
Because
we
clear
we
clean
everything
out,
we
build
everything
back
again
and
for
this
kind
of
thing,
if
that's
not
needed,
like
that's
needed
for
tests
or
just
want
to
generate
screenshots,
and
so
let's
do
it.
We
have
just
one
big
walkthrough
of
like
okay,
I
want
to
get
screenshot
here
and
then
here
and
then
here
that
could
be.
That
can
help
what's
the
speed
of
it
and
then
yeah.
If
we
can
move
it
outside
the
features,
folder
that'd
be
great.
A
B
A
The
test:
do
you,
do
you
think
that
do
you
think
that
would
be
helpful,
or
do
you
think
that
having
one
per
image
is
the
right
way
to
go
like
there's
a
little
bit
of
a
trade-off
here,
but
I
feel
like
in
this
context,
it
might
be
nicer
to
have
one
big
walk
through
yeah.
D
B
I
told
it
was
a
neat
idea
to
have
it
the
title
of
this,
so
that
you
can
easily
pinpoint-
and
you
know
this,
take
what
I
think
the
container
register
on
takes
40
seconds
will
spring
up,
considering
the
fact
that
we
don't
also
need
to
like
a
few
million
screen
shot
going
on
another.
Another
thing
is:
can
we
make
this
an
optional
effort
that
attaches
to
your
feature
test
that
you
will
produce
the
screen
shot
direct
in
the
future
test,
but
I,
don't
know,
I'm
afraid
that
too
much
magic
is
too
much
magic.
Well,.
A
Our
future
tests
do
screenshots
when
they
fail,
like
it,
so
that's
kind
of
when
it's
helpful,
but
you
know
it's
also
helpful
when
they
don't
fail.
If
something
looks
really
wonky.
This
is
really
interesting.
This
is
really
interesting.
I
kind
of
wanted
to
like
so
we're
gonna
manually
run
these
right.
A
Yeah
right
right,
I
could
see
again
so
super
helpful,
just
something
to
just
be
in
mind
since
since
CI
doesn't
run
it
like
it's
very
possible
that
these
things
get
out
of
style,
I
mean
like
that
they
fail
and
then
the
next
time
we
run
it's
like.
Oh
I
got
a
someone
introduced
to
broke
breaking
change
to
the
application
flow
and
now
I
gotta
try
to
fix
it
or
whatever,
like
that's
kind
of
that,
could
be
the
downside
to
this,
but
automating
and
automating
almost
anything
can
be
helpful.
So
I
see
this
is
still
helpful.
A
B
C
D
B
C
So,
basically,
setting
that
property
there
is
telling
which
type
of
spec
you're
running,
which
folder
you
found
it
housing.
If
this
is
it
so
what
we
need
to
do
and
what
could
probably
view
but
I
mean
this
is
create
anywhere.
We
could
probably
create
a
new
spec
helper,
and
that
is
by
helper.
Here
we
can
just
probably
I,
don't
know
so
wait.
C
A
C
Basically,
how
our
spec
works
is
that
if
you
say
whatever,
if
you
set,
which
type
of
test
you're
running,
even
though
the
file
it
is
not
exactly
in
the
folder
you
want,
it
will
run
us
that
a
specific
title,
a
type
of
test.
So
in
this
case,
since
it's
a
picture
test
technically
your
screenshot
helper,
you
need
to
add
type.
You
know
type.
What
type
your
feet.
A
A
Yeah,
if
you
scroll
down
so
that's
the
canonical
structure
automatically
adding
data
go
back
up
that
one
set
type
for
serializers
directory.
So
if
you
want
to
set
metadata
for
a
custom
directory
that
doesn't
follow
the
canonical
structure
above
do
the
following
and
then
I
think
and
then
I
think
at
the
end
of
it,
you
still
do
in
first
spec
type
from
location,
so
we
can
specify
this
directory.
Has
this
special
meaning
I
think.
B
A
Do
it's
right
there
define
derived
metadata
with
EE
is
in
there.
Yes,
so
we're
gonna
do
kind
of
the
same
thing.
I.
B
A
A
C
A
Do
not
override
type
if
it's
already
set?
Yes,
okay,
so
this
is
for
all
underscore
spec
stuff
we
can
manually
set
the
type
it's
allowing
us
to
do
some
of
that
fancy
stuff,
but
since
ours
is
not
underscore
spec
we're
not
getting
any
of
these
benefits
like
where
you
can
do
features
we
need
to,
though
I
would
say,
let's
config
it's
a
feature
spec
from
here.
Let's
try
not
to
touch
anything
else.
Yeah
I
would
just
do
in
brand
new
one.
A
config
defined
derives
metadata.
A
C
C
B
Going
to
route
any
copy,
this
I'm
horrible
with
rejects
in
JavaScript
imagining
Ruby
I.
C
D
B
A
B
A
A
B
A
B
C
B
C
So
it
because
what
I'm
seeing
like
on
the
currently
defined
the
right
method,
that
method
it
says,
method
and
it
says
level
so
I'm
thinking
we're,
probably
not
overriding
the
correct
type,
I'm,
not
sure
if
it's
a
different
property
or
something
but
I,
think
where
it
says
meta
data
type.
We
should
probably
do
better
that
I
feature
their
information.
C
C
Features
so
instead
of
that
saying
for
this
particular
spec
file,
if
it's
resides
in
the
features
folder,
it
will
just
say
you
know
what
this
this
type
is
features
I'm.
Sorry,
if
this
recital
features
just
single
eyes
that
feature
in
add
that
is
the
metadata
for
this.
So
yeah,
it's
tricky
missing
this
quality
level.
I,
don't
know
what
quality
level
is
or.
A
Can
we
add
a
key?
We
add
some
some
logs
here
to
see?
What's
actually
getting
hit
or
not
yes,
I
would
I,
don't
know
a
binding
pride
does
as
much
as
yeah.
We
could
do
that.
I
was
thinking.
We
could
do
some
some
puts
in.
A
A
C
A
B
A
A
D
A
A
B
A
B
B
C
C
C
B
A
A
A
B
B
B
B
B
C
A
Okay,
I
think
looking
around
at
some
of
our
pre-existing
specs
I
think
we
might
have
I
think
our
arts
are
spec
might
have
some
issues.
I
think
we
need
to
start
with
our
spec
describe
I'm,
seeing
a
lot
of
specs
start
that
way
really.
But
this
is.
A
A
B
B
C
C
C
B
B
B
C
A
A
That's
a
good
idea,
that's
a
good
point.
Yeah.
C
A
B
A
B
A
B
B
A
C
A
B
A
B
A
Well,
I
think,
like
I,
I
run
a
slant
and
prettier
on
save,
but
I
wouldn't
do
that
with
Ruby
and
it's
probably
cuz
I
don't
write
a
lot
of
Ruby
yeah.
A
B
A
B
A
B
B
B
A
A
B
Yeah
no,
but
he
was
saying
just
to
do
visual
revision
right
to
like
a
visual
regression
I
compared
the
peaks
of
difference
between
e2
picture
and
this
kind
of
stuff
yeah.
So
this
is
what
yeah
I'm
sorry
again
for
our
library.
It
makes
a
lot
of
sense
for
an
application.
Visual
regression,
you
maybe
overshoot
it
maybe
like
important
to
do
in
few
parts
like
very
selected.
B
C
B
Another
nice
step,
but
it
will
also
means
that
every
admirer
needs
to
have
one
block
like
this
coded
right.
So
I
don't
know
let
my
TAS.
Let's
look
at
this
running
as
an
option
so
because
now
the
workflow
is
you
want
to
make
a
Doc's.
Is
that
you
fire
the
GDK
you
navigate
manually?
You
take
a
screenshot.
You
then
run
PNG
cranes
and
then
change
their
keying
and
updated,
and
this
is
a
long
team
what
it
now
we
have
replacing.
All
of
this
thing.
A
B
A
A
Even
though
keeping
these
things
up
to
date
is
a
cost.
It
might
actually
end
up
be
less
of
a
cost
if
you
could
catch
things,
so
I
think
we
just
have
to
do
it
really
smart,
but
it's
cool
that
we
have
this
here.
I
like
us
a
lot
so
I
have
another
question
for
you.
You
push
all
this
up.
What's
to
prevent
a
community
contributor
from
saying:
oh
I
need
to
make
a
you
know:
Doc's
update
here's,
the
new
screenshot
they
generated
it
manually
and
they
over
wrote
whatever
they
did
and
they
didn't
use.
A
B
B
I
think
this
should
be
a
process
why
it's
like
we
put
this
on
the
dogs
and
then,
whenever
attack,
if
the
tech,
with
the
credit
team
like
this
solution,
they
are
gonna,
be
the
ones
like
us,
Montaigne
you're
like
to
stooling
on
radio
and
say
eh
I've
seen
a
deteriorated
creature
manually.
You
know
you
have
half
of
the
performance
bar
Tartine
and.
C
D
B
A
B
A
C
Could
probably
do
what
the
SRE
team
bus
for
the
what's
a
call
for
the
girlfriend
exports?
They
use
JSON
net
to
generate
their
own
graph
and
dashboards
and
Jason
made
20
runs.
It
creates
a
comment
at
the
top
of
each
graph
on
a
file
that
says:
hey,
do
not
do
this
manually
if
you
change
this
rerun
the
script.
So
all
this
changes
are
reflected
directly
in
the
fauna,
so
we
can
do
when
you.