Skip to content

Updating bookingSteps.rb and search feature#20

Open
annel-paca-tx wants to merge 2 commits into
tx-auto-mobile:masterfrom
annel-paca-tx:recentSearch
Open

Updating bookingSteps.rb and search feature#20
annel-paca-tx wants to merge 2 commits into
tx-auto-mobile:masterfrom
annel-paca-tx:recentSearch

Conversation

@annel-paca-tx
Copy link
Copy Markdown
Contributor

No description provided.

@auto-mobile-tx
Copy link
Copy Markdown

Hola Annel,
Por favor corregir.
Igual que en anterior commit, después de esto :
And I fill in enter destination input with "Little Havana"
no se puede hacer ninguno de los pasos indicados sin haber seleccionado un resultado de la lista. Antes de "select a date" deberías "tap on result from the results list".

@annel-paca-tx
Copy link
Copy Markdown
Contributor Author

Hi Carlos.
La observacion es correcta.
La seleccion de la lista se la esta realizando en el .rb file que esta incluida en ese paso:

And(/^I fill in enter destination input with "(._)"$/) do |arg|
xpath("//android.widget.EditText[@resource-id="com.booking:id/disam_search"]").type "#{arg}"
sleep(3.seconds)
_xpath("//android.widget.LinearLayout[@index="0"]").click*
end

Si ven conveniente podemos separar este, como otro paso.
De todos modos aclarar el feature subido de search.feature y su repectivo bookingSteps.rb file, los valide previamente, los mismos estan corriendo as Passed.

@auto-mobile-tx
Copy link
Copy Markdown

Creo que separarlo es mejor, será mas claro, y mas versátil, esto nos permitiría seleccionar otros resultados de la lista si es necesario:
And I type destination "Little Havana" in enter destination search field
And I open the first result from destination search

@annel-paca-tx
Copy link
Copy Markdown
Contributor Author

Correcto.
Me parece bien.

@auto-mobile-tx
Copy link
Copy Markdown

Por favor:

  1. Cambiar la palabra "choice" (selección) por "choose" (seleccionar).
  2. Renombrar el archivo "bookingSteps.rb" a "search.rb"
  3. No veo los cambios solicitados en el anterior comentario, pero bueno, cuando haya que implementar se apreciara la necesidad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants